From 8a64e5d868d22ef7d10c9f2782644215e2748e59 Mon Sep 17 00:00:00 2001 From: Matthias Bisping Date: Mon, 13 Jun 2022 10:36:26 +0200 Subject: [PATCH] narrowed down the pytest bug: n_items interacts with s3_backend: when n_items has more than one entry, s3_backend must not be the last decorator --- test/integration_tests/serve_test.py | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/test/integration_tests/serve_test.py b/test/integration_tests/serve_test.py index 7ed8b6f..ee2b2bf 100644 --- a/test/integration_tests/serve_test.py +++ b/test/integration_tests/serve_test.py @@ -19,13 +19,6 @@ from pyinfra.visitor import get_object_descriptor, QueueVisitor from test.utils.input import pair_data_with_queue_message -@pytest.mark.parametrize( - "s3_backend", - [ - "minio", - "aws", - ], -) @pytest.mark.parametrize( "batched", [ @@ -47,6 +40,15 @@ from test.utils.input import pair_data_with_queue_message True, ], ) +# NOTE: There is a bug in pytest, which leads this test to fail, when n_items has more than one entry and s3_backend is +# the last decorator. Yes, really. +@pytest.mark.parametrize( + "s3_backend", + [ + "minio", + "aws", + ], +) @pytest.mark.parametrize( "n_items", [ @@ -84,7 +86,7 @@ from test.utils.input import pair_data_with_queue_message @pytest.mark.parametrize( "components_type", [ - "test", + # "test", "real", ], )