Skip to content

Commit 6b19f0c

Browse files
authored
Update assertion for testing locally (#2113)
<!-- Thanks for opening a pull request! --> <!-- In the case this PR will resolve an issue, please replace ${GITHUB_ISSUE_ID} below with the actual Github issue id. --> <!-- Closes #${GITHUB_ISSUE_ID} --> # Rationale for this change `make test` fails locally since `Config().get_known_catalogs()` also reads my local `~/.pyiceberg.yaml` Follow up to #2088 # Are these changes tested? # Are there any user-facing changes? <!-- In the case of user-facing changes, please add the changelog label. -->
1 parent 62a95cf commit 6b19f0c

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

tests/utils/test_config.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,9 @@ def test_fix_nested_objects_from_environment_variables() -> None:
5959
@mock.patch.dict(os.environ, EXAMPLE_ENV)
6060
@mock.patch.dict(os.environ, {"PYICEBERG_CATALOG__DEVELOPMENT__URI": "https://dev.service.io/api"})
6161
def test_list_all_known_catalogs() -> None:
62-
assert Config().get_known_catalogs() == [
63-
"production",
64-
"development",
65-
]
62+
catalogs = Config().get_known_catalogs()
63+
assert "production" in catalogs
64+
assert "development" in catalogs
6665

6766

6867
def test_from_configuration_files(tmp_path_factory: pytest.TempPathFactory) -> None:

0 commit comments

Comments
 (0)