Skip to content

Commit 22fbcb7

Browse files
akarivclaude
andcommitted
CI: surface captured stdout for passing tests (-rA)
Several tests in test_quality_report.py are deliberately non-blocking reports (near-duplicate canonicals, naming-convention violations) that print findings without failing. pytest hides captured stdout for passing tests by default, which was silently swallowing these reports in CI - verified the first PR run showed 0 findings surfaced despite both report tests actually finding things to report. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent 3798313 commit 22fbcb7

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

.github/workflows/lamas-tests.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,4 +26,9 @@ jobs:
2626
run: pip install -e "Lamas/[dev]"
2727

2828
- name: Run pytest
29-
run: pytest Lamas/tests/ -v
29+
# -rA surfaces captured stdout for EVERY outcome, including PASSED - several tests in
30+
# test_quality_report.py (near-duplicate canonicals, naming-convention violations) are
31+
# deliberately non-blocking reports that print findings without failing; pytest hides
32+
# captured stdout for passing tests by default, which would otherwise silently swallow
33+
# them here.
34+
run: pytest Lamas/tests/ -v -rA

0 commit comments

Comments
 (0)