Merge pull request #408 from souradeep-das/souradeep/add_tests_dir

feat: add tests_dir to config

Author: Otto-AA

e2e_projects/config/config_pkg/math.py

@@ -11,4 +11,7 @@ def call_depth_four():
     return call_depth_five() - 1
 
 def call_depth_five():
-    return 5
+    return 5
+
+def func_with_no_tests():
+    return 420

e2e_projects/config/pyproject.toml

@@ -28,5 +28,4 @@ paths_to_mutate = [ "config_pkg/" ]
 do_not_mutate = [ "*ignore*" ]
 also_copy = [ "data" ]
 max_stack_depth=8 # Includes frames by mutmut, see https://github.com/boxed/mutmut/issues/378
-# NOTE: as of writing, the following options are not implemented
-# tests_dir = [ "my_tests/" ]
+tests_dir = [ "tests/main/" ]

e2e_projects/config/tests/ignored/test_ignored.py

@@ -0,0 +1,4 @@
+from config_pkg.math import func_with_no_tests
+
+def test_func_with_no_tests():
+    assert func_with_no_tests() == 420

e2e_projects/config/tests/test_main.py renamed to e2e_projects/config/tests/main/test_main.py

@@ -400,16 +400,34 @@ def pytest_runtest_makereport(self, item, call):
 
         stats_collector = StatsCollector()
 
+        pytest_args = ['-x', '-q']
+        if tests:
+            pytest_args += list(tests)
+        else:
+            tests_dir = mutmut.config.tests_dir
+            if tests_dir:
+                pytest_args += tests_dir
         with change_cwd('mutants'):
-            return int(self.execute_pytest(['-x', '-q'] + list(tests), plugins=[stats_collector]))
+            return int(self.execute_pytest(pytest_args, plugins=[stats_collector]))
 
     def run_tests(self, *, mutant_name, tests):
+        pytest_args = ['-x', '-q']
+        if tests:
+            pytest_args += list(tests)
+        else:
+            tests_dir = mutmut.config.tests_dir
+            if tests_dir:
+                pytest_args += tests_dir
         with change_cwd('mutants'):
-            return int(self.execute_pytest(['-x', '-q'] + list(tests)))
+            return int(self.execute_pytest(pytest_args))
 
     def run_forced_fail(self):
+        pytest_args = ['-x', '-q']
+        tests_dir = mutmut.config.tests_dir
+        if tests_dir:
+            pytest_args += tests_dir
         with change_cwd('mutants'):
-            return int(self.execute_pytest(['-x', '-q']))
+            return int(self.execute_pytest(pytest_args))
 
     def list_all_tests(self):
         class TestsCollector:
@@ -418,8 +436,13 @@ def pytest_collection_modifyitems(self, items):
 
         collector = TestsCollector()
 
+        tests_dir = mutmut.config.tests_dir
+        pytest_args = ['-x', '-q', '--collect-only']
+        if tests_dir:
+            pytest_args += tests_dir
+
         with change_cwd('mutants'):
-            exit_code = int(self.execute_pytest(['-x', '-q', '--collect-only'], plugins=[collector]))
+            exit_code = int(self.execute_pytest(pytest_args, plugins=[collector]))
             if exit_code != 0:
                 raise CollectTestsFailedException()
 
@@ -623,6 +646,7 @@ class Config:
     max_stack_depth: int
     debug: bool
     paths_to_mutate: List[Path]
+    tests_dir: List[str] = None
 
     def should_ignore_for_mutation(self, path):
         if not str(path).endswith('.py'):
@@ -701,7 +725,8 @@ def load_config():
         paths_to_mutate=[
             Path(y)
             for y in s('paths_to_mutate', [])
-        ] or guess_paths_to_mutate()
+        ] or guess_paths_to_mutate(),
+        tests_dir=s('tests_dir', []),
     )
 
 

mutmut/__main__.py

@@ -12,6 +12,7 @@
     "config_pkg.math.x_call_depth_three__mutmut_2": 1,
     "config_pkg.math.x_call_depth_four__mutmut_1": 33,
     "config_pkg.math.x_call_depth_four__mutmut_2": 33,
-    "config_pkg.math.x_call_depth_five__mutmut_1": 33
+    "config_pkg.math.x_call_depth_five__mutmut_1": 33,
+    "config_pkg.math.x_func_with_no_tests__mutmut_1": 33
   }
 }