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

Exécute tous les tests avec Pytest #1313

Merged
merged 8 commits into from
Apr 19, 2019
Merged

Exécute tous les tests avec Pytest #1313

merged 8 commits into from
Apr 19, 2019

Conversation

Morendil
Copy link
Contributor

@Morendil Morendil commented Apr 19, 2019

  • Amélioration technique.
  • Détails :
    • Adapte les tests de France à l'utilisation de Pytest
    • Déclare le paquet compatible avec Core 34

@Morendil
Copy link
Contributor Author

@fpagnoux Je ne sais pas si j'ai la bonne interprétation de Semver: pour moi openfisca/openfisca-core#867 est breaking et va nous faire passer à Core 34 car elle exige au moins une mise à jour de tests dans France; par contre France reste compatible avec les anciennes versions de Core et sa propre mise à jour ne me semble pas nécessairement breaking pour les réutilisateurs. Tu valides ?

@fpagnoux
Copy link
Member

fpagnoux commented Apr 19, 2019

Bonne question 🤔

Mais en regardant de plus près, je n'ai pas l'impression que cette PR soit vraiment une adaptation requise par openfisca/openfisca-core#867.

Que je sois sur le master de Core ou la branche pytest, les tests que tu as modifiés, sur le master de france, sont ignorés par pytest, qui ne gère plus les yield depuis la version 4. Sur la CI, on peut également constater que ces tests ne sont déjà plus exécutés J'ai donc l'impression que les modifs de cette PR sont plus un fixup du passage à Pytest qu'une conséquence de la PR sur core.

openfisca/openfisca-core#867 change l'implémentation de openfisca test pour se baser sur pytest, mais a priori cette PR n'implique pas de changement sur les tests YAML, et je crois que rien n'empêcherait un ré-utilisateur de continuer à écrire des tests avec nose dans son country package.

La suppression de la function generate_tests pourrait par contre plaider pour un breaking change, vu qu'elle était documentée, même si très certainement jamais utilisée ailleurs qu'en interne à Core.

@fpagnoux
Copy link
Member

openfisca/openfisca-core#867 a été mergée en tant que 34.0.0

@fpagnoux fpagnoux changed the title Adapte France à Core v34 Exécute tous les tests avec Pytest Apr 19, 2019
@fpagnoux fpagnoux merged commit 9856280 into master Apr 19, 2019
@fpagnoux fpagnoux deleted the adapt-tests branch April 19, 2019 19:41
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

Successfully merging this pull request may close these issues.

2 participants