Skip to content

Conversation

@MKhayari
Copy link
Collaborator

@MKhayari MKhayari commented Jul 7, 2025

mise à jour du vocabulaire

@MKhayari MKhayari self-assigned this Jul 7, 2025
@MKhayari MKhayari added the enhancement New feature or request label Jul 7, 2025
@parmentf
Copy link
Contributor

parmentf commented Jul 9, 2025

J'ai failli oublier (et ça aurait été publier des services bancals): il faut aussi récupérer le fichier dvc.lock (dans ws-data).

@MKhayari
Copy link
Collaborator Author

MKhayari commented Jul 9, 2025

est-ce que je remplace le fichier ? et dcv.yaml ?

@parmentf
Copy link
Contributor

parmentf commented Jul 9, 2025

est-ce que je remplace le fichier ? et dcv.yaml ?

Oui, il faut remplacer le fichier dvc.lock.
Le dvc.yaml ne sert qu'à générer les fichiers databases (c'est une sorte de Makefile amélioré), il est propre au dépôt ws-data, donc pas besoin de le copier.

@MKhayari
Copy link
Collaborator Author

MKhayari commented Jul 9, 2025

OK. Merci

Copy link
Contributor

@parmentf parmentf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Les tests sont maintenant cassés.
image

image
Sans doute à cause de la mise à jour des données.
C'est sur la ligne 1013 de tests.hurl.

"rdf$resource": "https://rdf.insee.fr/sparql/describe?uri=http://id.insee.fr/geo/commune/5691555d-e924-48b3-a6ce-f91db0bb5a20&default-graph-uri=data"

On peut trouver la requête qui est testée, en remontant jusqu'au premier POST rencontré.
Il s'agit de corriger les lignes attendues en réponse (en se basant sur le message d'erreur, je pense).

sameAs INSEE replaced by exactMatch in D63
Copy link
Contributor

@parmentf parmentf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Y'a plus qu'à continuer à corriger les attendus des tests.

Une fois que ce sera fait, et que tous les tests passeront, on pourra s'attaquer aux nouvelles routes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super.

Mais comme les tests vont plus loin, on achoppe sur autre chose (en ligne 45 de tests.hurl):
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants