Skip to content

Commit 477be22

Browse files
vdusekclaude
andcommitted
fix: access fixture param via callspec to fix skip logic
The `request.param` attribute is only available inside parameterized fixtures, not in the test function itself where `request` is a `TopRequest`. Use `request.node.callspec.params` instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 80762c7 commit 477be22

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

tests/integration/test_request_queue.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -681,7 +681,8 @@ async def test_request_deduplication_edge_cases(
681681
request_queue_apify: RequestQueue, request: pytest.FixtureRequest
682682
) -> None:
683683
"""Test edge cases in request deduplication."""
684-
if request.param == 'shared':
684+
rq_access_mode = request.node.callspec.params.get('request_queue_apify')
685+
if rq_access_mode == 'shared':
685686
pytest.skip('Test is flaky, see https://github.com/apify/apify-sdk-python/issues/786')
686687

687688
rq = request_queue_apify

0 commit comments

Comments
 (0)