[ci] Make sure CI fails if the tests fail

Fix the tests, they have been broken since we require CMake >= 3.10
jrl-umi3218 · Dec 6, 2023 · 6b33181 · 6b33181
1 parent 7602dd6
commit 6b33181
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 4 deletions.
diff --git a/.github/workflows/cmake.yml b/.github/workflows/cmake.yml
@@ -8,7 +8,7 @@ jobs:
     runs-on: [ubuntu-22.04]
 
     steps:
-    - uses: actions/checkout@v1
+    - uses: actions/checkout@v3
 
     - name: Run project tests
       run: |

diff --git a/.gitignore b/.gitignore
@@ -1,2 +1,4 @@
 hpp/idl/*.pyc
 .docs/build
+_unittests/build/
+_unittests/install/
diff --git a/_unittests/cpp/CMakeLists.txt b/_unittests/cpp/CMakeLists.txt
@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.2)
+cmake_minimum_required(VERSION 3.10)
 
 set(PROJECT_NAME jrl-cmakemodules-cpp)
 set(PROJECT_VERSION 0.0.0)

diff --git a/_unittests/dependency/CMakeLists.txt b/_unittests/dependency/CMakeLists.txt
@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.2)
+cmake_minimum_required(VERSION 3.10)
 
 set(PROJECT_NAME jrl-cmakemodules-dependency)
 set(PROJECT_VERSION 0.0.0)

diff --git a/_unittests/python/CMakeLists.txt b/_unittests/python/CMakeLists.txt
@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.1)
+cmake_minimum_required(VERSION 3.10)
 
 # These variables have to be defined before running SETUP_PROJECT
 set(PROJECT_NAME jrl-cmakemodules-python)

diff --git a/_unittests/run_unit_tests.sh b/_unittests/run_unit_tests.sh
@@ -1,5 +1,8 @@
 #!/bin/bash
 
+set -e
+set -x
+
 unittests="python cpp dependency catkin"
 
 # Code for running a specific unit test