Some updates, temp disable other Actions

.github/workflows/compile-all-batteries.yml

@@ -2,11 +2,7 @@
 name: Compile All Batteries
 
 # Here we tell GitHub when to run the workflow.
-on:
+on: [push, pull_request]
-  # The workflow is run when a commit is pushed or for a
-  # Pull Request.
-  - push
-  - pull_request
 
 # This is the list of jobs that will be run concurrently.
 # Since we use a build matrix, the actual number of jobs

.github/workflows/compile-all-inverters.yml

@@ -2,11 +2,7 @@
 name: Compile All Inverters
 
 # Here we tell GitHub when to run the workflow.
-on:
+on: [push, pull_request]
-  # The workflow is run when a commit is pushed or for a
-  # Pull Request.
-  - push
-  - pull_request
 
 # This is the list of jobs that will be run concurrently.
 # Since we use a build matrix, the actual number of jobs

.github/workflows/run-pre-commit.yml

@@ -2,9 +2,7 @@
 
 name: Run pre-commit
 
-on:
+on: [push, pull_request]
-  - push
-  - pull_request
 
 jobs:
   pre-commit:

.github/workflows/unit-tests.yml

@@ -1,6 +1,6 @@
 name: Run Unit Tests
 
-on: [push]
+on: [push, pull_request]
 
 jobs:
   build:
@@ -19,5 +19,6 @@ jobs:
 
     - name: Run unit tests
       run: |
+        set -e  # Exit immediately on non-zero exit code
         cd build/test
         find . -type f -executable -exec {} \;

Software/src/devboard/utils/CMakeLists.txt

@@ -0,0 +1 @@
+# add_library(utils_library events.cpp)

test/utils/timer_test.cpp

@@ -9,24 +9,32 @@
 
 TEST(timer_test) {
   unsigned long test_interval = 10;
+  bool result;
 
-  testlib_millis = 0;
+  testlib_millis = 30;
   MyTimer timer(test_interval);
-  ASSERT_EQ(timer.elapsed(), false);
+  result = timer.elapsed();
+  ASSERT_EQ(result, false);
 
-  testlib_millis = test_interval - 1;
+  testlib_millis += test_interval - 1;
-  ASSERT_EQ(timer.elapsed(), false);
+  result = timer.elapsed();
+  ASSERT_EQ(result, false);
 
-  testlib_millis = test_interval;
+  testlib_millis += 1;
-  ASSERT_EQ(timer.elapsed(), true);
+  result = timer.elapsed();
-  ASSERT_EQ(timer.elapsed(), false);
+  ASSERT_EQ(result, true);
+  result = timer.elapsed();
+  ASSERT_EQ(result, false);
 
-  testlib_millis = 2 * test_interval - 1;
+  testlib_millis += test_interval - 1;
-  ASSERT_EQ(timer.elapsed(), false);
+  result = timer.elapsed();
+  ASSERT_EQ(result, false);
 
-  testlib_millis = 2 * test_interval;
+  testlib_millis += 1;
-  ASSERT_EQ(timer.elapsed(), true);
+  result = timer.elapsed();
-  ASSERT_EQ(timer.elapsed(), false);
+  ASSERT_EQ(result, true);
+  result = timer.elapsed();
+  ASSERT_EQ(result, true);  // Injected fault to catch unit test errors
 }
 
 TEST_MAIN();