fix: run all tests when no tags provided and option is not set (#21)

co-authored by: @Stefanhg

Co-authored-by: Sergio Castillo <sergio.lara@elementsinteractive.es>

pytest_tagging/selector.py

@@ -39,7 +39,7 @@ def get_items_to_run(self, tags_to_run, tags_to_exclude, items):
         """
         selected_items = set()
         deselected_items = set()
-        if self.config.tags == [] or (self.config.tags is not None and not tags_to_run):
+        if is_tags_empty(self.config.tags) or (is_tags_option_provided(self.config.tags) and not tags_to_run):
             # No tags match the conditions to run
             # or we just passed an empty `--tags` options to see them all.
             deselected_items.update(items)
@@ -50,8 +50,10 @@ def get_items_to_run(self, tags_to_run, tags_to_exclude, items):
                 # Excluding tags takes precendence ove any selection.
                 if test_tags & tags_to_exclude:
                     deselected_items.add(item)
-                elif (self.config.operand is OperandChoices.OR and test_tags & tags_to_run) or (
-                    self.config.operand is OperandChoices.AND and tags_to_run <= test_tags
+                elif (
+                    not is_tags_option_provided(self.config.tags)
+                    or (self.config.operand is OperandChoices.OR and test_tags & tags_to_run)
+                    or (self.config.operand is OperandChoices.AND and tags_to_run <= test_tags)
                 ):
                     selected_items.add(item)
                 else:
@@ -61,3 +63,11 @@ def get_items_to_run(self, tags_to_run, tags_to_exclude, items):
 
 def get_tags_from_item(item) -> set[str]:
     return set(item.get_closest_marker("tags").args) if item.get_closest_marker("tags") else set()
+
+
+def is_tags_empty(tags: list[str] | None) -> bool:
+    return tags == []
+
+
+def is_tags_option_provided(tags: list[str] | None) -> bool:
+    return tags is not None

tests/test_plugin.py

@@ -8,6 +8,13 @@
 from pytest_tagging.utils import TagCounterThreadSafe
 
 
+def test_no_tags(pytester):
+    """Test that with no tags specified a test is still executed"""
+    pytester.makepyfile("def test_pass(): pass")
+    res = pytester.runpytest()
+    res.assert_outcomes(passed=1)
+
+
 @pytest.mark.parametrize(
     "option, expected_counter",
     (