Skip to content

Dev#267

Open
mmcdermott wants to merge 21 commits intomainfrom
dev
Open

Dev#267
mmcdermott wants to merge 21 commits intomainfrom
dev

Conversation

@mmcdermott
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Nov 16, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
15 1 14 0
View the top 1 failed test(s) by shortest run time
tests.conftest::tests.conftest.get_and_validate_cache_settings
Stack Traces | 0.002s run time
186         >>> with TemporaryDirectory() as temp_dir:
187         ...     cache_dir = Path(temp_dir) / "cache_dir"
188         ...     cache_dir.mkdir()
189         ...     out = get_and_validate_cache_settings(MockRequest(cache_dir, ["d1", "d2", "all"], [], ["m1"]))
190         ...     assert out[0] == cache_dir, f"Expected {cache_dir}, got {out[0]}"
191         ...     assert len(out) == 2, f"Expected 2, got {len(out)}"
192         ...     assert out[1] == (set(DATASETS), set(), {"m1"}), (
193         ...         f"Want ({set(DATASETS)}, [], {{'m1'}}) got {out[1]}"
194         ...     )
195         >>> get_and_validate_cache_settings(MockRequest(None, ["d1", "d2", "all"], [], ["m1"]))
Differences (unified diff with -expected +actual):
    @@ -1,3 +1,8 @@
     Traceback (most recent call last):
    -    ...
    -_pytest.config.exceptions.UsageError: Persistent cache directory must be set if any cache options are!
    +  File ".../hostedtoolcache/Python/3.11.14.../x64/lib/python3.11/doctest.py", line 1355, in __run
    +    exec(compile(example.source, filename, "single",
    +  File "<doctest tests.conftest.get_and_validate_cache_settings[5]>", line 1, in <module>
    +    get_and_validate_cache_settings(MockRequest(None, ["d1", "d2", "all"], [], ["m1"]))
    +  File ".../MEDS-DEV/tests/conftest.py", line 211, in get_and_validate_cache_settings
    +    raise pytest.UsageError("Persistent cache directory must be set if any cache options are!")
    +pytest.UsageError: Persistent cache directory must be set if any cache options are!

.../MEDS-DEV/tests/conftest.py:195: DocTestFailure

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant