tests/client: fix pytest mark skipif syntax (#2019)

xrmx · web-flow · commit 6eca84684e52 · 2024-04-02T10:00:41.000+02:00
For some reason tests passed with this but now are failing because
pytest test does not recognize skipIf.
diff --git a/tests/client/client_tests.py b/tests/client/client_tests.py
@@ -233,7 +233,7 @@ def test_custom_transport(elasticapm_client):
     assert isinstance(elasticapm_client._transport, DummyTransport)
 
 
-@pytest.mark.skipIf(simplejson_dumps is None)
+@pytest.mark.skipif(simplejson_dumps is None, reason="no test without simplejson")
 @pytest.mark.parametrize(
     "elasticapm_client", [{"transport_json_serializer": "elasticapm.utils.simplejson_encoder.dumps"}], indirect=True
 )

Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,7 @@ def test_custom_transport(elasticapm_client):`
`233`	`233`	`assert isinstance(elasticapm_client._transport, DummyTransport)`
`234`	`234`
`235`	`235`
`236`		`-@pytest.mark.skipIf(simplejson_dumps is None)`
	`236`	`+@pytest.mark.skipif(simplejson_dumps is None, reason="no test without simplejson")`
`237`	`237`	`@pytest.mark.parametrize(`
`238`	`238`	`"elasticapm_client", [{"transport_json_serializer": "elasticapm.utils.simplejson_encoder.dumps"}], indirect=True`
`239`	`239`	`)`