Merge pull request #1219 from m-novikov/m-novikov-pytest-markers

Add test markers to allow more granularity selecting tests

Author: seandstewart

CHANGES/1219.misc

@@ -0,0 +1 @@
+Add pytest markers for parsers and connection types

pyproject.toml

@@ -7,3 +7,11 @@ issue_format = "#{issue}"
 underlines = ["", ""]
 template = ".towncrier.md.jinja"
 title_format = "## {version} - ({project_date})"
+
+[tool.pytest.ini_options]
+markers = [
+    "hiredis_parser: mark a test as using hiredis parser",
+    "python_parser: mark a test as using python parser",
+    "connection_pool: mark a test as using connection_pool client",
+    "single_connection: mark a test as using single connection client",
+]

tests/conftest.py

@@ -136,27 +136,39 @@ def skip_unless_arch_bits(arch_bits: int) -> _TestDecorator:
 
 @pytest.fixture(
     params=[
-        (True, PythonParser),
-        (False, PythonParser),
+        pytest.param(
+            (True, PythonParser),
+            marks=[pytest.mark.python_parser, pytest.mark.single_connection],
+            id="single-connection-python-parser",
+        ),
+        pytest.param(
+            (False, PythonParser),
+            marks=[pytest.mark.python_parser, pytest.mark.connection_pool],
+            id="pool-python-parser",
+        ),
         pytest.param(
             (True, HiredisParser),
-            marks=pytest.mark.skipif(
-                not HIREDIS_AVAILABLE, reason="hiredis is not installed"
-            ),
+            marks=[
+                pytest.mark.skipif(
+                    not HIREDIS_AVAILABLE, reason="hiredis is not installed"
+                ),
+                pytest.mark.hiredis_parser,
+                pytest.mark.single_connection,
+            ],
+            id="single-connection-hiredis",
         ),
         pytest.param(
             (False, HiredisParser),
-            marks=pytest.mark.skipif(
-                not HIREDIS_AVAILABLE, reason="hiredis is not installed"
-            ),
+            marks=[
+                pytest.mark.skipif(
+                    not HIREDIS_AVAILABLE, reason="hiredis is not installed"
+                ),
+                pytest.mark.hiredis_parser,
+                pytest.mark.connection_pool,
+            ],
+            id="pool-hiredis",
         ),
     ],
-    ids=[
-        "single-python-parser",
-        "pool-python-parser",
-        "single-hiredis",
-        "pool-hiredis",
-    ],
 )
 def create_redis(request, event_loop):
     """Wrapper around aioredis.create_redis."""