Run integration tests from ci

.github/workflows/ci.yml

@@ -12,15 +12,15 @@ permissions:
 
 jobs:
   host_build_test_ubuntu:
-    name: Host Build & Test (ubuntu-latest)
-    runs-on: ubuntu-latest
+    name: Host Build & Test (ubuntu-24.04)
+    runs-on: ubuntu-24.04
     steps:
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
         with:
           persist-credentials: false
       - uses: hendrikmuhs/ccache-action@ed74d11c0b343532753ecead8a951bb09bb34bc9 # v1.2.14
         with:
-          key: ${{ github.job }}-ubuntu-latest
+          key: ${{ github.job }}-ubuntu-24.04
           variant: sccache
       - uses: seanmiddleditch/gha-setup-ninja@96bed6edff20d1dd61ecff9b75cc519d516e6401 # v5
       - uses: lukka/run-cmake@af1be47fd7c933593f687731bc6fdbee024d3ff4 # v10.8
@@ -45,7 +45,7 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [windows-latest, macos-latest]
+        os: [windows-latest]
     steps:
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
         with:
@@ -68,7 +68,7 @@ jobs:
       #    path: build/host/Testing/Temporary/
   embedded_build:
     name: Embedded Build
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     needs: [host_build_test_ubuntu]
     strategy:
       matrix:
@@ -116,3 +116,26 @@ jobs:
           configurePreset: ${{ matrix.target }}
           buildPreset: ${{ matrix.target }}-${{ matrix.configuration }}
           configurePresetAdditionalArgs: "['-DCMAKE_C_COMPILER_LAUNCHER=ccache', '-DCMAKE_CXX_COMPILER_LAUNCHER=ccache']"
+
+  integration_test:
+    name: Integration test
+    runs-on: ubuntu-24.04
+    needs: [embedded_build, host_build_test_ubuntu]
+    permissions:
+      packages: read
+    services:
+      flex:
+        image: ghcr.io/philips-software/amp-postmaster:pr-5
+        credentials:
+          username: ${{ github.actor }}
+          password: ${{ secrets.GITHUB_TOKEN }}
+    steps:
+      - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
+        with:
+          persist-credentials: false
+      - uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8
+        with:
+          name: hal_st
+      - run: tar -zxvf hal_st-*.tar.gz
+      - run: mkdir install && mv hal_st-*/* install/
+      - run: ls install/bin

.github/workflows/codeql.yml

@@ -0,0 +1,90 @@
+# For most projects, this workflow file will not need changing; you simply need
+# to commit it to your repository.
+#
+# You may wish to alter this file to override the set of languages analyzed,
+# or to provide custom queries or build logic.
+#
+# ******** NOTE ********
+# We have attempted to detect the languages in your repository. Please check
+# the `language` matrix defined below to confirm you have the correct set of
+# supported CodeQL languages.
+#
+name: "CodeQL Advanced"
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    types: [opened, synchronize, reopened]
+
+jobs:
+  analyze:
+    name: Analyze (${{ matrix.language }})
+    # Runner size impacts CodeQL analysis time. To learn more, please see:
+    #   - https://gh.io/recommended-hardware-resources-for-running-codeql
+    #   - https://gh.io/supported-runners-and-hardware-resources
+    #   - https://gh.io/using-larger-runners (GitHub.com only)
+    # Consider using larger runners or machines with greater resources for possible analysis time improvements.
+    runs-on: ${{ (matrix.language == 'swift' && 'macos-latest') || 'ubuntu-24.04' }}
+    permissions:
+      # required for all workflows
+      security-events: write
+
+      # required to fetch internal or private CodeQL packs
+      packages: read
+
+      # only required for workflows in private repositories
+      actions: read
+      contents: read
+
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+        - language: c-cpp
+          build-mode: autobuild
+        # CodeQL supports the following values keywords for 'language': 'c-cpp', 'csharp', 'go', 'java-kotlin', 'javascript-typescript', 'python', 'ruby', 'swift'
+        # Use `c-cpp` to analyze code written in C, C++ or both
+        # Use 'java-kotlin' to analyze code written in Java, Kotlin or both
+        # Use 'javascript-typescript' to analyze code written in JavaScript, TypeScript or both
+        # To learn more about changing the languages that are analyzed or customizing the build mode for your analysis,
+        # see https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/customizing-your-advanced-setup-for-code-scanning.
+        # If you are analyzing a compiled language, you can modify the 'build-mode' for that language to customize how
+        # your codebase is analyzed, see https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/codeql-code-scanning-for-compiled-languages
+    steps:
+    - name: Checkout repository
+      uses: actions/checkout@v4
+
+    # Initializes the CodeQL tools for scanning.
+    - name: Initialize CodeQL
+      uses: github/codeql-action/init@v3
+      with:
+        languages: ${{ matrix.language }}
+        build-mode: ${{ matrix.build-mode }}
+        # If you wish to specify custom queries, you can do so here or in a config file.
+        # By default, queries listed here will override any specified in a config file.
+        # Prefix the list here with "+" to use these queries and those in the config file.
+
+        # For more details on CodeQL's query packs, refer to: https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
+        # queries: security-extended,security-and-quality
+
+    # If the analyze step fails for one of the languages you are analyzing with
+    # "We were unable to automatically build your code", modify the matrix above
+    # to set the build mode to "manual" for that language. Then modify this step
+    # to build your code.
+    # ℹ️ Command-line programs to run using the OS shell.
+    # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
+    - if: matrix.build-mode == 'manual'
+      shell: bash
+      run: |
+        echo 'If you are using a "manual" build mode for one or more of the' \
+          'languages you are analyzing, replace this with the commands to build' \
+          'your code, for example:'
+        echo '  make bootstrap'
+        echo '  make release'
+        exit 1
+
+    - name: Perform CodeQL Analysis
+      uses: github/codeql-action/analyze@v3
+      with:
+        category: "/language:${{matrix.language}}"

.github/workflows/linting-formatting.yml

@@ -16,7 +16,7 @@ concurrency:
 jobs:
   linter:
     name: Linter
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     permissions:
       contents: read
       pull-requests: write

CMakeLists.txt

@@ -27,7 +27,7 @@ if (HALST_STANDALONE)
         FetchContent_Declare(
             cucumber-cpp-runner
             GIT_REPOSITORY https://github.com/philips-software/amp-cucumber-cpp-runner.git
-            GIT_TAG        70c424e53f532f520d96a303b6586272d3efc5f5 # unreleased
+            GIT_TAG        145ef9eda8198ba080f610859e24d998cde83b0e # unreleased
         )
 
         FetchContent_MakeAvailable(cucumber-cpp-runner)
@@ -64,6 +64,38 @@ if (HALST_STANDALONE)
     emil_folderize_all_targets()
 endif()
 
+if (HALST_STANDALONE AND NOT CMAKE_CROSSCOMPILING)
+    function(generate_export_targets name)
+        install(EXPORT halst${name}Targets
+            FILE halst${name}Targets.cmake
+            DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/halst
+        )
+    endfunction()
+
+    foreach(target IN ITEMS IntegrationTest)
+        generate_export_targets(${target})
+    endforeach()
+
+    write_basic_package_version_file(
+        "${CMAKE_CURRENT_BINARY_DIR}/cmake/${PROJECT_NAME}ConfigVersion.cmake"
+        COMPATIBILITY SameMajorVersion
+        # When cross-compiling for a 32-bit architecture and re-using host tooling from a 64-bit architecture
+        # ARCH_INDEPENDENT is necessary here. See: https://cmake.org/cmake/help/latest/module/CMakePackageConfigHelpers.html.
+        ARCH_INDEPENDENT
+    )
+
+    configure_package_config_file(${CMAKE_CURRENT_SOURCE_DIR}/cmake/Config.cmake.in
+        "${CMAKE_CURRENT_BINARY_DIR}/cmake/${PROJECT_NAME}Config.cmake"
+        INSTALL_DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}
+    )
+
+    install(FILES
+        "${CMAKE_CURRENT_BINARY_DIR}/cmake/${PROJECT_NAME}Config.cmake"
+        "${CMAKE_CURRENT_BINARY_DIR}/cmake/${PROJECT_NAME}ConfigVersion.cmake"
+        DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}
+    )
+endif()
+
 set(CPACK_GENERATOR "ZIP;TGZ")
 set(CPACK_SOURCE_IGNORE_FILES ".vs/;.git/;build/")
 set(CPACK_PACKAGE_NAME "${PROJECT_NAME}")

cmake/Config.cmake.in

@@ -0,0 +1,5 @@
+@PACKAGE_INIT@
+
+include("${CMAKE_CURRENT_LIST_DIR}/halstIntegrationTestTargets.cmake")
+
+check_required_components(IntegrationTest)

integration_test/runner/CMakeLists.txt

@@ -1,5 +1,6 @@
 add_executable(integration_test.runner)
-emil_build_for(integration_test.runner HOST Windows)
+emil_build_for(integration_test.runner HOST All PREREQUISITE_BOOL HALST_STANDALONE)
+emil_install(integration_test.runner EXPORT halstIntegrationTestTargets DESTINATION bin)
 
 target_sources(integration_test.runner PRIVATE
     FixtureEcho.cpp
@@ -13,7 +14,7 @@ target_sources(integration_test.runner PRIVATE
 
 target_link_libraries(integration_test.runner PRIVATE
     args
-    cucumber-cpp-runner
+    cucumber_cpp.runner
     integration_test.logic
     hal.generic
     gtest

integration_test/runner/FixtureEcho.hpp

@@ -25,6 +25,8 @@ namespace main_
     struct FixtureEchoSerial
         : FixtureEchoSerialBase
     {
+        using FixtureEchoSerialBase::FixtureEchoSerialBase;
+
         main_::EchoOnSesame<256> echoOnSesame{ bufferedSerial, serializerFactory };
 
         services::Echo& echo{ echoOnSesame.echo };
@@ -33,6 +35,8 @@ namespace main_
     struct FixtureTracingEchoSerial
         : FixtureEchoSerialBase
     {
+        using FixtureEchoSerialBase::FixtureEchoSerialBase;
+
         main_::TracingEchoOnSesame<256> echoOnSesame{ bufferedSerial, serializerFactory, services::GlobalTracer() };
 
         services::Echo& echo{ echoOnSesame.echo };

integration_test/runner/Hooks.cpp

@@ -1,4 +1,4 @@
-#include "cucumber-cpp/Hooks.hpp"
+#include "cucumber_cpp/library/Hooks.hpp"
 #include "args.hxx"
 #include "generated/echo/Testing.pb.hpp"
 #include "hal/generic/SynchronousRandomDataGeneratorGeneric.hpp"

integration_test/runner/StepsGpio.cpp

@@ -1,4 +1,4 @@
-#include "cucumber-cpp/Steps.hpp"
+#include "cucumber_cpp/library/Steps.hpp"
 #include "generated/echo/Testing.pb.hpp"
 #include "integration_test/logic/Tested.hpp"
 #include "integration_test/runner/Waiting.hpp"

integration_test/runner/StepsUart.cpp

@@ -1,4 +1,4 @@
-#include "cucumber-cpp/Steps.hpp"
+#include "cucumber_cpp/library/Steps.hpp"
 #include "generated/echo/Testing.pb.hpp"
 #include "integration_test/logic/Tested.hpp"
 #include "integration_test/runner/Waiting.hpp"

integration_test/runner/Waiting.hpp

@@ -1,7 +1,7 @@
 #ifndef INTEGRATION_TEST_WAITING_HPP
 #define INTEGRATION_TEST_WAITING_HPP
 
-#include "cucumber-cpp/Context.hpp"
+#include "cucumber_cpp/library/Context.hpp"
 #include "infra/timer/Timer.hpp"
 
 namespace infra