[cicd] pipeline and timeout

FrancoisCarouge · Jan 18, 2025 · df2fc6a · df2fc6a
1 parent c8de63e
commit df2fc6a
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 14 deletions.
diff --git a/.github/workflows/deploy_test_coverage_coveralls.yml b/.github/workflows/deploy_test_coverage_coveralls.yml
@@ -43,7 +43,7 @@ jobs:
         lcov --rc branch_coverage=1 --gcov-tool gcov-14 --remove base.info '*/test/*' --output-file base.info
         lcov --rc branch_coverage=1 --gcov-tool gcov-14 --remove base.info '/usr/*' --output-file base.info
     - name: 'Test'
-      run: ctest --test-dir 'build' --tests-regex 'kalman' --verbose --parallel 4
+      run: ctest --test-dir 'build' --timeout 2 --tests-regex 'kalman' --verbose --parallel 4
     - name: 'Coverage: Test'
       run: |
         lcov --rc branch_coverage=1 --gcov-tool gcov-14 --capture --directory . --output-file test.info

diff --git a/.github/workflows/memory_valgrind.yml b/.github/workflows/memory_valgrind.yml
@@ -32,4 +32,4 @@ jobs:
     - name: 'Build'
       run: cmake --build 'build' --verbose --parallel 4
     - name: 'Test'
-      run: ctest --test-dir 'build' --tests-regex 'kalman' --verbose --parallel 4
+      run: ctest --test-dir 'build' --timeout 5 --tests-regex 'kalman' --verbose --parallel 4
diff --git a/.github/workflows/pipeline.yml b/.github/workflows/pipeline.yml
@@ -19,8 +19,8 @@ jobs:
         include:
         - { os: 'ubuntu-24.04', cxx: 'clang++-18', cc: 'clang-18', generator: 'Unix Makefiles', packages: 'clang-18'  }
         - { os: 'ubuntu-24.04', cxx: 'g++-14', cc: 'gcc-14', generator: 'Unix Makefiles', packages: 'g++-14'  }
-        - { os: 'windows-latest', cxx: 'cl', cc: 'cl', generator: 'Ninja', config: 'Debug' }
-        - { os: 'windows-latest', cxx: 'cl', cc: 'cl', generator: 'Ninja', config: 'Release' }
+        - { os: 'windows-2022', cxx: 'cl', cc: 'cl', generator: 'Visual Studio 17 2022', config: 'Debug' }
+        - { os: 'windows-2022', cxx: 'cl', cc: 'cl', generator: 'Visual Studio 17 2022', config: 'Release' }
     name: '${{ matrix.os }} / ${{ matrix.cxx }} / ${{ matrix.generator }} / ${{ matrix.config }}'
     runs-on: '${{ matrix.os }}'
     steps:
@@ -30,8 +30,6 @@ jobs:
         egress-policy: audit
     - name: 'Checkout'
       uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
-    - name: 'Install: MSVC'
-      uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756 # v1.13.0
     - name: 'Configure'
       env:
         CXX: '${{ matrix.cxx }}'
@@ -40,10 +38,8 @@ jobs:
     - name: 'Build'
       run: cmake --build 'build' --config '${{ matrix.config }}' --verbose --parallel 4
     - name: 'Test'
-      run: ctest --test-dir 'build' --tests-regex 'kalman' --verbose --parallel 4
+      run: ctest --test-dir 'build' --build-config '${{ matrix.config }}' --timeout 2 --tests-regex 'kalman' --verbose --parallel 4
     - name: 'Install'
-      run: cmake --install 'build' --prefix 'install' --verbose
+      run: cmake --install 'build' --config '${{ matrix.config }}' --prefix 'install' --verbose
     - name: 'Package'
-      run: cmake --build 'build' --target 'package' --verbose --parallel 4
-    - name: 'Package Source'
-      run: cmake --build 'build' --target 'package_source' --verbose --parallel 4
+      run: cmake --build 'build' --target 'package' --config '${{ matrix.config }}' --verbose --parallel 4
diff --git a/.github/workflows/sanitizer.yml b/.github/workflows/sanitizer.yml
@@ -48,4 +48,4 @@ jobs:
     - name: 'Build'
       run: cmake --build 'build' --verbose --parallel 4
     - name: 'Test'
-      run: ${{ matrix.sanitizer.run_flags }} ctest --test-dir 'build' --tests-regex 'kalman_(test|sample)' --verbose --parallel 4
+      run: ${{ matrix.sanitizer.run_flags }} ctest --test-dir 'build' --timeout 5 --tests-regex 'kalman_(test|sample)' --verbose --parallel 4
diff --git a/test/linalg_identity_default.cpp b/test/linalg_identity_default.cpp
@@ -46,7 +46,7 @@ namespace {
 //! @test Verifies the 1x1 identity matrix deduced default value is a unit
 //! double.
 [[maybe_unused]] auto test{[] {
-  auto i{identity<>};
+  [[maybe_unused]] auto i{identity<>};
 
   assert(i == 1.0);
   static_assert(std::is_same_v<decltype(i), double>);

diff --git a/test/linalg_zero_default.cpp b/test/linalg_zero_default.cpp
@@ -45,7 +45,7 @@ namespace fcarouge::test {
 namespace {
 //! @test Verifies the 1x1 zero matrix deduced default value is a null double.
 [[maybe_unused]] auto test{[] {
-  auto z{zero<>};
+  [[maybe_unused]] auto z{zero<>};
 
   assert(z == 0.0);
   static_assert(std::is_same_v<decltype(z), double>);