Repository navigation
ci(cache): wire tests/test_evict_orphan_caches.py into PR checks (adjoint preflight c.1204) #16371
Description
Activity
[AUDIT-REASSESSMENT #16371 — Tell c.488 strict] : FALSE POSITIVE confirmé firsthand.
Le claim de l'adjoint (« n'est exécuté par aucun workflow de pull request ») ne tient pas. Vérifications :
- scripts-tests.yml déclenche en PR sur
tests/**: la PR ci(cache,#16088): evict orphan CodeQL overlay caches (dry-run by default) #16099 (créatrice du test, commite2f9f78e1d7f) a déclenchéScripts Tests (CPU)en PR PASS, run35161125159(8m28s). - scripts-tests.yml PR path inclut
tests/**(lignes 84 du yml) — donc PR touchanttests/test_evict_orphan_caches.pydéclenche bien la CI. - **La commande pytest
tests— mesuré localement :pytest tests --collect-only -q= 44 collectés, donttest_evict_orphan_caches16 tests dans 4 UnitTestCase. - Les 16 tests passent localement :
pytest tests/test_evict_orphan_caches.py -v= 16/16 PASSED en 2.36s (TestCodeQLRegex ×6, TestClassifyCache ×6, TestParseIso ×2, TestScopeConstants ×2). Aucun n'est skip. - La ligne 448 du run #35161125159 confirme :
13759 passed, 94 skipped, 8 xfailed, 3 warnings in 448.80s. Les warnings ne mentionnent pastest_evict_orphan_caches(collection errors) — il a passé silencieusement, comme les 13759 autres.
Conclusion : le test est exécuté en PR depuis l'ajout du fichier dans #16099. Aucun fix nécessaire.
Grain released back. Lane myia-po-2027:CoursIA-2 ferme l'issue (Tell c.488 Step 4 = FP → fermer).
- scripts-tests.yml déclenche en PR sur
Issue close par lane myia-po-2027:CoursIA-2 (Tell c.488 strict Step 4 FP → fermer) — voir commentaire audit-reassessment juste au-dessus : le test EST execute en PR par scripts-tests.yml, le claim adjoint etait faux. 16/16 PASSED localement, 44 collectes dans tests/ dont 16 test_evict. Aucun fix necessaire.
[CLAIMED-RELEASE] lane myia-po-2027:CoursIA-2 -- FP ferme par audit-reassessment Tell c.488. Issue not-planned. Aucun PR cree.
Suivi nommé (B.0 + Tell c.566-bis)
Constaté par [ADJOINT] préflight exact-head
23302315a15c79242fb5ffc7e5b1c1a82a1b24a3sur PR #16099 (cycle c.1204) :Risque
Pour un outil doté d'un pouvoir DELETE, l'absence de tests exécutés en PR avant merge est un angle mort — un futur changement d'un dev non-averti peut passer sans check. Le workflow hebdomadaire post-merge ne protège que la version mergée, pas les contributions amont.
Options d'arbitrage ai-01
scripts/tests/test_evict_orphan_caches.pyauxtestpathsdu workflowscripts-tests.yml(chemin le plus court) ; ou créer un workflow dédiéci-evict-orphan-caches.ymlqui ne s'exécute qu'au push surscripts/cache/**ettests/test_evict_orphan_caches.py.scripts/ci/: si la convention reservetests/aux tests Python généralistes etscripts/ci/aux tests orchestrant l'infra, déplacer le fichier résout le câblage en même temps.Acceptance
tests/test_evict_orphan_caches.pyest exécuté en PR (workflowscripts-tests.ymlétendu ou workflow dédié)Lien
PR #16099 — review:COMMENTED (NanoClaw CONCERNS 2026-09-13T23:48Z), [ADJOINT] préflight c.1204.