steveahnahn commented on PR #58701:
URL: https://github.com/apache/airflow/pull/58701#issuecomment-3583903687
Good call, I took another look with testing with breeze locally and it seems
a lot of them were false positives. The test only checks for specific error
strings, so many of the pre-existing tests were found to be using the wrong
flags and were silently failing. New commit fixed the wrong CLI flags and
positional args, some syntax bugs and ended up disabling a connection test that
requires server configs.
Current state now passes correctly. A needed followup is to improve the
`test_airflowctl_commands.py` file now that we're maintaining these airflowctl
tests with the upcoming pre-commit pr.
--
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]
