Skip to content

[tooling] test_workflow_dispatch_revalidate.py : bloc __main__ appelle pytest.main sans import pytest (code mort trompeur) #13988

Closed

Description

@myia-ai-01

Releve par Hermes en review de #13917 (verdict LGTM avec 1 note mineure, non bloquant), issue de suivi ouverte avant le merge pour que la remarque ne meure pas avec la PR.

scripts/tests/test_workflow_dispatch_revalidate.py porte un bloc __main__ qui appelle pytest.main sans import pytest. Une execution directe echoue :

python scripts/tests/test_workflow_dispatch_revalidate.py
# -> ModuleNotFoundError: yaml   (en env sans pyyaml, arrive en premier)
# -> puis NameError: name 'pytest' is not defined

Aucun impact CI : le chemin canonique pytest scripts/tests/... fonctionne (2 passed in 0.94s, reproduit par le reviewer). C'est donc du code mort trompeur, pas un test casse : quelqu'un qui lance le fichier directement conclut a un test casse alors que la suite est verte.

Deux resolutions acceptables : retirer le bloc __main__, ou ajouter import pytest. La premiere est preferable si aucun autre test du repertoire ne porte ce bloc (a verifier — si le motif est repandu, l'aligner plutot que de le retirer isolement).

Le meme motif est a chercher ailleurs : grep -rn 'pytest.main' scripts/tests/ avant de conclure que c'est un cas isole.

See #13917, #13861.

Activity

  1. added a commit that references this issue on Sep 2, 2026
  2. added
    candidate-deliveredReferenced by a merged PR with no post-merge activity -- candidate for close triage (#10466)
    on Sep 2, 2026
  3. jsboige commented on Sep 2, 2026

    @jsboige
    Owner

    Closing LIVRÉ-urn (Tell c.835 procédure) — vérification first-hand 2026-09-02 par myia-po-2024:CoursIA-2 : PR #13998 MERGED 2026-09-02T08:35:53Z livre entièrement cette issue.

    Acceptance #13988 vérifiée :

    • test_workflow_dispatch_revalidate.py bloc main corrigé (import pytest ajouté)
    • Test exécutable maintenant via pytest

    Tell c.835 procédure : close légitime, substance vérifiable.

    See #13998 · Closes #13988

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    candidate-deliveredReferenced by a merged PR with no post-merge activity -- candidate for close triage (#10466)

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions