nspcc-dev/neofs-testcases

Error messages in lock tests are not descriptive

Ouverte

#759 ouverte le 27 mars 2024

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Python (18 forks)auto 404
I4S4U3buggood first issue

Métriques du dépôt

Stars
 (4 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

tests.object.test_object_lock.TestObjectLockWithGrpc#test_expired_object_should_be_deleted_after_locks_are_expired tests.

Current Behavior

Got Failed: DID NOT RAISE <class 'Exception'> message if an object is not deleted.

Expected Behavior

Some understandable message about object not deleted after some time.

Possible Solution

Do some assert with a message. It is possible in pytest?

Steps to Reproduce

See https://rest.fs.neo.org/HXSaMJXk2g8C14ht8HSi7BBaiYZ1HeWh2xnWPGQCg4H6/1441-1711487340/index.html#suites/295ed2e000f8fa6f3ade59cc13b98615/3f2b8cbfd27db081/

Guide contributeur

Direction de recherche
Localisez le test échoué dans le dépôt, spécifiquement le test `test expired object should be deleted after locks are expired` dans la classe `tests.object.test object lock.TestObjectLockWithGrpc`. Remplacez l'attente d'exception actuelle par une assertion incluant un message descriptif, par exemple en utilisant `pytest.fail` ou `assert` avec un message. Assurez vous que le test produit une sortie significative en cas d'échec.
Stack technique
python
Domaine
testingbackend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Pythonpytest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90