Dev-iL commented on code in PR #59785:
URL: https://github.com/apache/airflow/pull/59785#discussion_r2648942157
##########
airflow-core/tests/unit/cli/commands/test_dag_command.py:
##########
@@ -225,7 +225,7 @@ def test_cli_report(self, stdout_capture):
dag_command.dag_report(args)
out = temp_stdout.getvalue()
- assert "airflow/example_dags/example_complex.py" in out
Review Comment:
Claude says:
> **The test change is necessary and appropriate.** The test's purpose is to
verify the CLI report command works, not to test path formatting. The old
assertion `"airflow/example_dags/example_complex.py"` was coincidentally
passing because it's a substring of the full absolute path - it wasn't
intentionally testing a specific path format. The weaker assertion
`"example_complex.py"` is sufficient for the test's actual purpose.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
