Ajouter une logique de test

[emilienschultz]

No description provided.

[emilienschultz]

ajout d'un fichier test_pyshs avec les premiers tests. @Carreau est-ce que la logique est bonne ?

[Carreau]

Oui, test_pyshs.py c'est bon, mais tu l'as appelé tests_pyshs.py (pluriel).

J'ai corrigé, etcorrigé les fixtures aussi, plus les type(...) == ... en isinstance(..., ...).

Notons que pour lancer les tests, il faut juste faire pytest dans un shell, plutot que utiliser if __name__ == "__main__" pour multiple raisons.

[Carreau]

I would suggest for you to setup https://app.codecov.io/gh/emilienschultz/pyshs-bib ( I can't do it for you it's your account), it should give you a badge that tell you the percentage of the library that is tested.

I've setup github action to automatically tests the library on each pull request.

[emilienschultz]

I moved the tests in a test file : does this have a consequences on the automatization of the test ?
(where do you configure the test on github)
Thanks !!

[Carreau]

dans .github/workflows/tests.yml, mais ça devrais être bon, il va trouver les tests tout seul.

…

On Sun, Jul 10, 2022 at 7:33 PM Émilien Schultz ***@***.***> wrote: I moved the tests in a test file : does this have a consequences on the automatization of the test ? (where do you configure the test on github) Thanks !! — Reply to this email directly, view it on GitHub <#27 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACR5T4AXSBXQQ2VU5NFAIDVTMCORANCNFSM53EZ564Q> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[emilienschultz]

@Carreau je ne sais pas ce que j'ai cassé mais les tests échouent (ils passent sur ma machine). Une idée ?

[emilienschultz]

Nevermind, corrigé, mais je ne comprends pas pourquoi pytest passait en local vu qu'il y avait une erreur.