Merge pull request #783 from hpc/run-results-error

Run results error

.github/workflows/unittests.yml

@@ -55,7 +55,7 @@ jobs:
 
     - name: logs
       if: always()
-      uses: actions/upload-artifact@v1
+      uses: actions/upload-artifact@v4
       with:
         name: test_output
         path: test/working_dir.zip
@@ -191,7 +191,7 @@ jobs:
 
     - name: logs
       if: failure()
-      uses: actions/upload-artifact@v1
+      uses: actions/upload-artifact@v4
       with:
         name: test_output_2.6
         path: test/working_dir.zip
@@ -247,7 +247,7 @@ jobs:
 
     - name: logs
       if: failure()
-      uses: actions/upload-artifact@v1
+      uses: actions/upload-artifact@v4
       with:
         name: test_output_2.10
         path: test/working_dir.zip

lib/pavilion/commands/_run.py

@@ -234,9 +234,12 @@ def _run(self, test: TestRun):
         except TestRunError as err:
             # An unexpected TestRunError
             test.status.set(STATES.RUN_ERROR, err)
+            return
         except TimeoutError:
             # This is expected
-            pass
+            test.status.set(STATES.RUN_ERROR,
+                            f"Timed out waiting for test {test.name} to complete.")
+            return
         except Exception:
             # Some other unexpected exception.
             test.status.set(

lib/pavilion/schedulers/plugins/flux.py

@@ -1,4 +1,5 @@
 # pylint: disable=too-many-lines
+# pylint: disable=no-self-use
 """The Flux Framework Scheduler Plugin."""
 
 import os

test/requirements.txt

@@ -1,2 +1,2 @@
-pylint
+pylint==2.13.9
 matplotlib > 3

test/tests/utils_tests.py

@@ -66,7 +66,7 @@ def test_owner(self):
         # Try to set the permissions of the file to an unknown user.
         proc = sp.Popen(['sudo', '-n', 'chown', '12341', path.as_posix()],
                         stdout=sp.PIPE, stderr=sp.PIPE, stdin=sp.PIPE)
-        if proc.wait(1) == 0:
+        if proc.wait(2) == 0:
             self.assertEqual(utils.owner(path), "<unknown user '12341'>")
 
     def test_relative_to(self):