Series-Relative Test IDs (#895)

* Replace test ID path with UUID path

* Miscellaneous changes

* Fix unit test

* Fix cancel_tests unit test

* Fix cancel_invalid_test unit test

* Fix logging tests

* Fix arg_filtered_tests unit test

* Fix list_cmd unit tests

* Fix index unit test

* Pass more unit tests

* Fix even more unit tests

* Fix legacy unit test

* Fix add_module unit test

* Fix style issues

* Add validation to TestIDs

* Various changes to integrate TestID

* Various changes

* Fix graph command ID parsing

* Fixing IDs

* Progress towards fixing unit tests

* Fix group unit tests

* Fix bug in LazyTestRunDict

* Fix cancel command unit tests

* Fix cat unit tests

* Fix graph command unit tests

* Fix list command unit tests

* Fix log command unit tests

* Fix result command unit tests

* Update cmd_utils to work with ID objects

* Fix logging unit tests

* Fix run command unit tests

* Fix series command tests

* Fix cancel command tests

* Fix series unit tests

* Fix status command unit tests

* Fix remaining unit tests

* Refactor arg_filtered_tests

* Refactor arg_filtered_series

* Various changes

* More ID tweaks

* Make host_cpus plugin platform independent

* Make sys_vars platform independent

* Fix style issue

* Fix style issues

* Ignore wrapped lines

* Fix group commands unit test

* Fix line wrapping causing unit test to fail

* Try to fix group tests

* Try to fix CI wrapping breaking unit tests

* Add --json option to group members subcommand

* Fix syntax errors

* JSON output for group members

* Add GroupID to serializer

* Fix TestID deserializer

* Fix JSON output

* Fix group command unit tests

* Remove debug statements

* Fix rebase

* Fix join

* Remove stray print statement

Author: hwikle-lanl

.github/workflows/unittests.yml

@@ -40,6 +40,9 @@ jobs:
 
     # Set run_tests to run only the specific tests you need to fix.
     - name: run_tests
+      env:
+        COLUMNS: 120
+        LINES: 40
       run: |
         ./test/run_tests
 

lib/pavilion/builder.py

@@ -460,7 +460,7 @@ def build(self, test_id: str, tracker: BuildTracker,
         directory into our test directory, and note that we've used the given
         build.
 
-        :param test_id: The test 'full_id' for the test initiating this build.
+        :param test_id: The test id for the test initiating this build.
         :param tracker: A thread-safe tracker object for keeping info on what the
             build is doing.
         :param cancel_event: Allows builds to tell each other
@@ -570,7 +570,7 @@ def _build(self, build_dir, cancel_event, test_id, tracker: BuildTracker) -> boo
         :param Path build_dir: The directory in which to perform the build.
         :param threading.Event cancel_event: Event to signal that the build
             should stop.
-        :param test_id: The 'full_id' of the test initiating the build.
+        :param test_id: The ID of the test initiating the build.
         :param tracker: Build tracker for this build.
         :returns: True or False, depending on whether the build appears to have
             been successful.
@@ -592,7 +592,8 @@ def _build(self, build_dir, cancel_event, test_id, tracker: BuildTracker) -> boo
             # Do the build, and wait for it to complete.
             with self.tmp_log_path.open('w') as build_log:
                 # Build scripts take the test id as a first argument.
-                cmd = [self._script_path.as_posix(), test_id]
+                cmd = [self._script_path.as_posix(), str(test_id)]
+
                 proc = subprocess.Popen(cmd,
                                         cwd=build_dir.as_posix(),
                                         stdout=build_log,
@@ -797,10 +798,6 @@ def _setup_build_dir(self, dest: Path, tracker: BuildTracker) -> None:
             raise TestBuilderError("Error extracting file '{}'\n  {}"
                                    .format(src_path.as_posix(), extract_error))
 
-        tracker.update(
-            state=STATES.BUILDING,
-            note="Generating dynamically created files.")
-
         # Create build time file(s).
         for file, contents in self._config.get('create_files', {}).items():
             try:

lib/pavilion/cancel_utils.py

@@ -100,7 +100,7 @@ def cancel_tests(pav_cfg: PavConfig, tests: Iterable[TestRun], outfile: TextIO,
             title="Cancelling {} test{}".format(test_count, 's' if test_count > 1 else ''),
             outfile=outfile,
             fields=['name', 'id', 'state', 'series'],
-            rows=[{'name': test.name, 'id': test.full_id,
+            rows=[{'name': test.name, 'id': test.id,
                    'state': test.status.current().state, 'series': test.series}
                   for test in cancelled_test_info])
     else: