molcay commented on PR #74758: URL: https://github.com/apache/airflow/pull/74758#issuecomment-6254868953
> Welcome to Apache Airflow and thanks for your contribution! Please note that the unit tests on operators links don't really test a real behavior, but rather duplicate the entries from the data structures and assert on them - which isn't that useful. I'm closing this PR, but please don't be discouraged because of that: > > 1. You're more than welcome to create another one! > 2. I'll create a PR to distinguish real missing unit tests from cases like this one. > > CC: @VladaZakharova @MaksYermak @molcay - I'd appreciate your attention to this matter :) I was also thinking the same at the beginning and then I though that maybe it make sense have this tests to have a bit less items in the `OVERLOOKED_TESTS` variable in `airflow-core/tests/unit/always/test_project_structure.py` file. So, I gave "LGTM" 😅 If the community does not see any real outcome for this; I think we can keep those values in this variable. Thank you for the heads-up! -- 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]
