[GitHub] [airflow] mik-laj edited a comment on issue #8390: Enforce system tests for Google operators

[GitBox <gi...@apache.org>]

mik-laj edited a comment on issue #8390:
URL: https://github.com/apache/airflow/pull/8390#issuecomment-616251216


   What do you think about checking on CI and then informing you that the test is missing, but then let the author of the contribution knowingly decide? I see on the list a few operators that have been prepared by the Polidea team, so they should have system tests. However, these tests do not exist by mistake. Instead of "enforce" it is just to inform. If we have a permanent list of missing tests, it will also be easier to involve new people to work on these tests, because it will be easier to find.
   ```
   'tests/providers/google/cloud/operators/test_cloud_build_system.py',
   'tests/providers/google/cloud/operators/test_datacatalog_system.py',
   'tests/providers/google/cloud/operators/test_datafusion_system.py',
   'tests/providers/google/cloud/operators/test_sheets_to_gcs_system.py',
   'tests/providers/google/cloud/operators/test_speech_to_text_system.py',
   'tests/providers/google/cloud/operators/test_text_to_speech_system.py',
   'tests/providers/google/cloud/operators/test_translate_speech_system.py',
   'tests/providers/google/suite/operators/test_gcs_to_gdrive_system.py',
   'tests/providers/google/suite/operators/test_gcs_to_sheets_system.py'
   ```
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org