Development #63192
Retours pylint - réactiver les tests
Début:
25 mars 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
4 tests à reprendre qui contenaient des assertions non testées, mais qui maintenant échouent, à corriger:
tests/test_orange.py:@pytest.mark.xfail(run=False) tests/test_orange.py-def test_send_msg(app, connector): -- tests/test_iparapheur.py:@pytest.mark.xfail(run=False) tests/test_iparapheur.py-@mock.patch('passerelle.utils.Request.get') -- tests/test_astregs.py:@pytest.mark.xfail(run=False) tests/test_astregs.py-@mock.patch('passerelle.utils.Request.get', side_effect=search_wsdl_side_effect) -- tests/test_api.py:@pytest.mark.xfail(run=False) tests/test_api.py-@mock.patch('passerelle.sms.models.SMSResource.send_job')
Fichiers
Demandes liées
Révisions associées
tests: re-enable a test on job api (#63192)
Historique
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Lié à Development #62099: Faire planter le build jenkins sur une erreur pylint ajouté
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an
- Fichier 0002-tests-re-enable-a-test-on-job-api-63192.patch 0002-tests-re-enable-a-test-on-job-api-63192.patch ajouté
- Fichier 0001-tests-re-enable-a-test-on-orange-connector-63192.patch 0001-tests-re-enable-a-test-on-orange-connector-63192.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Nicolas Roche (absent jusqu'au 3 avril)
- Patch proposed changé de Non à Oui
Il n'y a plus que 2 tests concernés :
$ git grep '@pytest.mark.xfail(run=False)' tests/test_api.py:@pytest.mark.xfail(run=False) tests/test_orange.py:@pytest.mark.xfail(run=False)
0001:
Les numéros de téléphones sont triés depuis #39650 via send()
.
Pour faire simple, j'ai triés les numéros utilisés dans les tests (qui ne passent pas tous par send).
0002:
Le test était inversé.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Pour info, on peut faire moins verbeux/plus lisible que freezer.move_to('2022-01-01T12:00:00Z')
en tapant juste freezer.move_to('2022-01-01 12:00')
.
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
(remarque prise en compte)
commit 88abc88953905087962f852accf23c5cb1342347 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Wed Sep 7 16:27:33 2022 +0200 tests: re-enable a test on job api (#63192) commit 686db642f5de7c37159310c6c25c40f378622f80 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Wed Sep 7 16:18:29 2022 +0200 tests: re-enable a test on orange connector (#63192)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
tests: re-enable a test on orange connector (#63192)