ecolabdata / ckanext-ecospheres

GNU Affero General Public License v3.0
3 stars 0 forks source link

Fusion des branches `develop` et `spatial` #2

Open alhyss opened 1 year ago

alhyss commented 1 year ago

Procédure suivie :

Le résultat de la fusion est donc l'actuelle branche spatial_for_merge.

Cf. issues #4, #5, #6 pour les interrogations subsistant suite à la fusion.

alhyss commented 1 year ago

J'ai supprimé le test test_vocabulary.test_parser.test_index, qui avait le mauvais goût de charger tous les vocabulaires.

Il aurait accessoirement dû s'appeler test_vocabulary.test_parser.test_loader, puisqu'il n'exécutait qu'une fonction, issue de ce module.

Ajouter des tests moins chronophages pour les modules vocabulary.index et vocabulary.loader conserve un intérêt, mais paraît moins pressant que la suppression de ce test.

alhyss commented 1 year ago

J'ai ajouté au module vocabulary.reader la classe qu'il contenait sur la branche spatial, permettant d'interroger les vocabulaires en JSON, sous le nom VocabularyJSONReader. Outre le fait qu'elle pourrait conserver une certaine utilité pour la maintenance, elle a aussi l'intérêt de présenter les fonctionnalités attendues du module dans le cadre du moissonnage INSPIRE.

Le module test_reader n'avait rien à voir sur les branches spatial et develop. La dernière version commune datait du temps où reader lisait des graphes RDF, beaucoup de choses ont été corrigées et ajoutées de deux côtés. Après examen, je pense que les tests de la branche spatial sont toujours pertinents, même si certains échouent aujourd'hui. Je les ai intégrés au module de la branche spatial-for-merge en ajoutant Spatial dans les noms de classes pour les distinguer de celles qui viennent de la branche develop. Il faudra refaire le point sur ce qui est à conserver ou non à l'occasion de la consolidation du module reader.

alhyss commented 1 year ago

Tous les tests des modules issus de spatial passent, ainsi que ceux qui valident la structure du schéma YAML - module tests.scheming.test_schemas. À date, je considère que tout est en ordre de mon côté sur la branche spatial_for_merge.