WIP: --changed

Author: LarsAsplund

vunit/ui/__init__.py

@@ -173,8 +173,10 @@ def __init__(
         self._builtins = Builtins(self, self._vhdl_standard, simulator_class)
 
         self._dependency_graph = None
-        self._include_in_test_pattern: Optional[List[Union[str, Path]]] = []
-        self._exclude_from_test_pattern: Optional[List[Union[str, Path]]] = []
+        self._include_in_test_pattern: Optional[List[Union[str, Path]]] = None
+        self._exclude_from_test_pattern: Optional[List[Union[str, Path]]] = None
+        self._latest_dependency_updates = None
+        self._test_history = None
 
     def _create_database(self):
         """
@@ -750,8 +752,8 @@ def _main(self, post_run):
         """
         Base vunit main function without performing exit
         """
-        if self._include_in_test_pattern or self._exclude_from_test_pattern:
-            self._update_test_filter(self._include_in_test_pattern, self._exclude_from_test_pattern)
+        if self._include_in_test_pattern or self._exclude_from_test_pattern or self._args.changed:
+            self._update_test_filter(self._include_in_test_pattern, self._exclude_from_test_pattern, self._args.changed)
 
         if self._args.export_json is not None:
             return self._main_export_json(self._args.export_json)
@@ -768,7 +770,7 @@ def _main(self, post_run):
         all_ok = self._main_run(post_run)
         return all_ok
 
-    def _update_test_filter(self, include_dependencies=None, exclude_dependencies=None):
+    def _update_test_filter(self, include_dependencies=None, exclude_dependencies=None, only_tests_depending_on_changes=False):
         """
         Update test filter to reflect included and excluded testbenches
         """
@@ -802,15 +804,31 @@ def get_dependent_files(dependencies):
             return dependent_files
 
         dependent_files = get_dependent_files(include_dependencies) - get_dependent_files(exclude_dependencies)
-
-        # Extract testbenches from dependent files and create corresponding test patterns:
-        # lib_name.tb_name*
-        test_patterns = []
+        
+        # Extract testbenches from dependent files
+        dependent_testbench_files = {}
         for dependent_file in dependent_files:
             library_name = dependent_file.library.name
             for testbench in self._test_bench_list.get_test_benches_in_library(library_name):
                 if testbench.design_unit.source_file == dependent_file:
-                    test_patterns.append(f"{library_name}.{testbench.name}*")
+                    dependent_testbench_files[dependent_file] = testbench.name
+
+        # Remove testbenches not depending on changes if --change options has been set
+        # if only_tests_depending_on_changes:
+        #     if self._latest_dependency_updates is None:
+        #         self._latest_dependency_updates = self._get_latest_dependency_updates()
+
+        #     if self._test_history is None:
+        #         self._test_history = self._get_test_history(simulator_if=None)
+
+        #     for dependent_testbench_file in dependent_testbench_files:
+        #         latest_dependency_update = self._latest_dependency_updates[str(dependent_testbench_file.original_name)]
+
+        # Create test patterns for remaining testbenches: lib_name.tb_name*
+        test_patterns = []
+        for dependent_file, testbench_name in dependent_testbench_files.items():
+            library_name = dependent_file.library.name
+            test_patterns.append(f"{library_name}.{testbench_name}*")
 
         # Update test filter to match test patterns
         if isinstance(self._args.test_patterns, list):

vunit/vunit_cli.py

@@ -166,6 +166,13 @@ def _create_argument_parser(description=None, for_documentation=False):
         help="Output path for compilation and simulation artifacts",
     )
 
+    parser.add_argument(
+        "--changed",
+        action="store_true",
+        default=False,
+        help="Include only tests that depend on file changes since the last recorded test run",
+    )
+
     parser.add_argument(
         "--test-prio",
         choices=["opt", "ordered"],