You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Taragolis (via GitHub)" <gi...@apache.org> on 2023/02/26 16:34:33 UTC

[GitHub] [airflow] Taragolis commented on a diff in pull request #29775: Migrate tests in google/cloud/operators from unittest to pytest

Taragolis commented on code in PR #29775:
URL: https://github.com/apache/airflow/pull/29775#discussion_r1118115259


##########
tests/providers/google/cloud/operators/test_bigquery_dts.py:
##########
@@ -51,7 +50,7 @@
 )
 
 
-class BigQueryCreateDataTransferOperatorTestCase(unittest.TestCase):
+class BigQueryCreateDataTransferOperatorTestCase:

Review Comment:
   ```suggestion
   class TestBigQueryCreateDataTransferOperator:
   ```



##########
tests/providers/google/cloud/operators/test_bigquery_dts.py:
##########
@@ -78,7 +77,7 @@ def test_execute(self, mock_hook):
         assert "access_key_id" not in return_value.get("params", {})
 
 
-class BigQueryDeleteDataTransferConfigOperatorTestCase(unittest.TestCase):
+class BigQueryDeleteDataTransferConfigOperatorTestCase:

Review Comment:
   ```suggestion
   class TestBigQueryDeleteDataTransferConfigOperator:
   ```



##########
tests/providers/google/cloud/operators/test_bigquery_dts.py:
##########
@@ -94,7 +93,7 @@ def test_execute(self, mock_hook):
         )
 
 
-class BigQueryDataTransferServiceStartTransferRunsOperatorTestCase(unittest.TestCase):
+class BigQueryDataTransferServiceStartTransferRunsOperatorTestCase:

Review Comment:
   ```suggestion
   class TestBigQueryDataTransferServiceStartTransferRunsOperator:
   ```



-- 
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: commits-unsubscribe@airflow.apache.org

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