hankehly commented on code in PR #22872:
URL: https://github.com/apache/airflow/pull/22872#discussion_r846582439
##########
tests/secrets/test_local_filesystem.py:
##########
@@ -122,8 +122,9 @@ def test_missing_file(self, mock_exists):
)
def test_yaml_file_should_load_variables(self, file_content,
expected_variables):
with mock_local_file(file_content):
- variables = local_filesystem.load_variables('a.yaml')
- assert expected_variables == variables
+ vars_yaml = local_filesystem.load_variables('a.yaml')
+ vars_yml = local_filesystem.load_variables('a.yml')
+ assert expected_variables == vars_yaml == vars_yml
Review Comment:
The `load_variables` function is currently only used in
`LocalFilesystemBackend`
--
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]
