Abhishek-kumar-samsung commented on code in PR #29375:
URL: https://github.com/apache/airflow/pull/29375#discussion_r1103844764
##########
tests/always/test_project_structure.py:
##########
@@ -90,8 +89,10 @@ def test_providers_modules_should_have_tests(self):
missing_tests_files = expected_test_files -
expected_test_files.intersection(current_test_files)
- with self.subTest("Detect missing tests in providers module"):
+ try:
Review Comment:
I didn't knew if it can be done that way, i knew try catch way, and tried
testing by putting missing_tests_files as none, try except way was working fine
so i kept that.
And also as per understanding, try except was much easy to understand.
--
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]
