Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fusion des branches develop et spatial #2

Open
alhyss opened this issue Nov 17, 2022 · 3 comments
Open

Fusion des branches develop et spatial #2

alhyss opened this issue Nov 17, 2022 · 3 comments
Assignees
Milestone

Comments

@alhyss
Copy link
Contributor

alhyss commented Nov 17, 2022

Procédure suivie :

  • Création d'une branche develop_for_merge basée sur develop.
  • Sur develop_for_merge, suppression du répertoire vocabularies. J'ai finalement également retiré le fichier vocabularies/adminstrations_type_referentiel_codage.json, car il s'avère qu'on retrouve les mêmes données en dur via vocabulary.reader.VocabularyReader.TYPE_ADMINISTRATION.
  • Sur develop_for_merge, suppression des préfixes test_ pour tous les sous-répertoires de tests.
  • Création d'une branche spatial_for_merge basée sur spatial. J'ai pris cette branche comme base pour m'assurer de bien retrouver l'historique de tous les commits.
  • Merge de develop_for_merge sur spatial_for_merge. J'ai eu des conflits à gérer sur quatre fichiers : le schéma YAML, le dev-requirements, le module reader et son module de test test_reader. Comme prévu, pour les deux derniers j'ai simplement repris les versions de la branche develop_for_merge. Pour les deux premiers, j'ai géré à la main les conflits identifiés (trivial pour le dev-requirements, j'ai juste tout mis).

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
Copy link
Contributor Author

alhyss commented Nov 18, 2022

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
Copy link
Contributor Author

alhyss commented Nov 18, 2022

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 alhyss self-assigned this Nov 18, 2022
@alhyss
Copy link
Contributor Author

alhyss commented Nov 18, 2022

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.

@alhyss alhyss added this to the fusion milestone Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant