You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/12/06 21:41:04 UTC

[GitHub] [airflow] IAL32 commented on a diff in pull request #28145: Migrate amazon provider transfer tests from `unittests` to `pytest`

IAL32 commented on code in PR #28145:
URL: https://github.com/apache/airflow/pull/28145#discussion_r1041487038


##########
tests/providers/amazon/aws/transfers/test_dynamodb_to_s3.py:
##########
@@ -18,13 +18,12 @@
 from __future__ import annotations
 
 import json
-import unittest
 from unittest.mock import MagicMock, patch
 
 from airflow.providers.amazon.aws.transfers.dynamodb_to_s3 import DynamoDBToS3Operator
 
 
-class DynamodbToS3Test(unittest.TestCase):
+class DynamodbToS3Test:
     def setUp(self):

Review Comment:
   I made the change anyways, wouldn't want that to bite us in the back later on ahah



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