potiuk commented on PR #30977:
URL: https://github.com/apache/airflow/pull/30977#issuecomment-1529419059
Just FYI (it's a good idea to add those BTW) - the reason why there are no
tests for provider's CLI is that they have higher level - say "integration" -
tests running in CI:
https://github.com/apache/airflow/blob/ced56ea5a783a2846094996503d84fc21a33d6ae/scripts/in_container/verify_providers.py#L811
- they don't test too much - they merely run all the cli commands for
providers and see that they are not failing. This happens after all the
providers have been build in main and installed on top of Airflow.
But yes. it is great to have Pytest tests to test them too.
And BTW. By checking it, I have found that the above tests have been
accidentally excluded in recent refactor - accidentally. Fix to it is coming:
https://github.com/apache/airflow/pull/30986
--
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]
