Clarify testing standard related to caplog assertions in AGENTS.md (#69803)

This commit is contained in:
Amogh Desai authored and GitHub committed 2026-07-13 14:09:30 +05:30
1 parent 5d1e204379
commit fd42d8694e
2 files changed
+2 -2

No files matched your search

+1 -1
View File
@@ -158,7 +158,7 @@ reported as such are described in "What is NOT considered a security vulnerabili
- Use `@pytest.mark.db_test` for tests that require database access.
- Test fixtures: `devel-common/src/tests_common/pytest_plugin.py`.
- Test location mirrors source: `airflow/cli/cli_parser.py` → `tests/cli/test_cli_parser.py`.
- Do not use `caplog` in tests, prefer checking logic and not log output.
- Do not assert on raw log text (`caplog.text` for example), these are legacy string matching APIs planned for removal. Structured assertions via `caplog` (which resolves to `cap_structlog` under structlog) are fine and preferred: `"event name" in caplog` or `{"event": ..., "field": ...} in caplog`.
## Output conventions
+1 -1
View File
@@ -1,3 +1,3 @@
# Generated by dev/skill-evals/eval.py — do not edit or resolve conflicts by hand.
# Run `prek run run-skill-eval --hook-stage manual --all-files` to regenerate.
f4a71898d35b99e6c42ce756a71fa24ba243ada0611ed55966fce25711ac8c02
9901af13862556bf571a3db5950f1bd32b1cbb79e8e18d3eb41e4fc3827788ce