utkarsharma2 commented on a change in pull request #9907:
URL: https://github.com/apache/airflow/pull/9907#discussion_r478774597
##########
File path: tests/cli/commands/test_connection_command.py
##########
@@ -321,3 +330,87 @@ def test_cli_delete_invalid_connection(self):
# Check deletion attempt stdout
self.assertIn("\tDid not find a connection with `conn_id`=fake",
stdout)
+
+
+class TestCliImportConnections(unittest.TestCase):
Review comment:
@mik-laj, I can think of a few scenarios which are already handled:
> 1. Missing file scenario is handled by cli_parser.py
> 2. Unsupported file format is handled in local_filesystems.py
> 3. Parsing error is handled in local_filesystems.py
> 4. Wrong parameter for --conflict-disposition is handled by cli_parser.py
Can you provide an example?
----------------------------------------------------------------
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:
[email protected]
