New-style Path | str union doesn't seem to work in fixture
#1242
-
|
I have a fixture that uses Recently I updated Ruff to replace the old-style Am I missing something? Here's the fixture: @pytest.fixture
def mock_non_audio(audio_files: AudioFileTest, monkeypatch) -> Callable[..., Path]:
"""Fixture providing mocked non-audio file functionality."""
monkeypatch.setattr(AudioFile, "load", audio_files.load)
def create_non_audio_file(
path: Union[Path, str], # This Union is critical for pyfakefs!
size: int = 4000,
) -> Path:
"""Create and configure a mock non-audio file."""
assert audio_files._fs is not None
return audio_files._fs.create_file(path, st_size=size)
return create_non_audio_fileAnd when switched to a |
Beta Was this translation helpful? Give feedback.
Answered by
mrbean-bremen
Nov 11, 2025
Replies: 4 comments 3 replies
-
|
Thanks - that's very strange, and currently I have no idea what could cause that. |
Beta Was this translation helpful? Give feedback.
2 replies
-
|
I did not succeed to reproduce this so far, but I don't know what you are actually doing in this code. Can you please provide a reproducible example, and/or the definition of EDIT: I succeeded in creating another error: |
Beta Was this translation helpful? Give feedback.
-
|
Using |
Beta Was this translation helpful? Give feedback.
-
|
I am able to create a standalone test that reproduces the TypeError you got (unsupported operand) , but I haven't been able to get my original problem reproduced in a standalone test (yet?). I was also able to make the standalone work with |
Beta Was this translation helpful? Give feedback.
1 reply
-
|
I guess that your problem is indeed based on that reproducible problem, which has somehow to do with the wrapped function (it only appears with the type hint in the wrapped function in the fixture). I can reproduce it this way: from pathlib import Path
import pytest
@pytest.fixture
def wrapper(fs):
def wrapped(path: Path | str):
pass
return wrapped
def test_something(wrapper):
passUsing PathLike = Path | str
@pytest.fixture
def wrapper(fs):
def wrapped(path: PathLike):
pass
...I will try to understand and fix the problem - right now I'm quite stumped. If I can't fix it, I will at least add this to the troubleshooting guide. |
Beta Was this translation helpful? Give feedback.
-
|
I still did not really understand the problem, but I added a workaround that should fix it. You can check if the main branch works for you. |
Beta Was this translation helpful? Give feedback.
0 replies
Answer selected by
pdc1
-
|
I patched my install with the updated |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I still did not really understand the problem, but I added a workaround that should fix it. You can check if the main branch works for you.