fixup

Author: jorgenmk

.github/workflows/build-and-target-test.yml

@@ -3,11 +3,6 @@ name: Build and Test
 on:
   workflow_dispatch:
     inputs:
-      pytest_marker:
-        description: Pytest marker selection
-        type: string
-        required: false
-        default: '-m "not slow"'
       build_all:
         description: Build all configurations
         type: boolean
@@ -27,7 +22,7 @@ jobs:
     uses: ./.github/workflows/build.yml
     secrets: inherit
     with:
-      build_all: ${{ inputs.build_all }}
+      build_all: ${{ inputs.build_all || false }}
   test:
     permissions:
       actions: read
@@ -39,4 +34,4 @@ jobs:
     with:
       artifact_fw_version: ${{ needs.build.outputs.version }}
       artifact_run_id: ${{ needs.build.outputs.run_id }}
-      pytest_marker: ${{ inputs.pytest_marker }}
+      test_all: ${{ inputs.build-all || false }}

.github/workflows/build.yml

@@ -169,6 +169,7 @@ jobs:
 
       # Asset Tracker Template debug firmware build
       - name: Build thingy91x debug firmware
+        if: ${{ inputs.build_all }}
         uses: ./asset-tracker-template/.github/actions/build-step
         with:
           memfault_project_key: ${{ secrets.MEMFAULT_PROJECT_KEY }}

.github/workflows/target-test.yml

@@ -9,10 +9,10 @@ on:
       artifact_fw_version:
         type: string
         required: true
-      pytest_marker:
-        type: string
+      test_all:
+        type: boolean
         required: false
-        default: -m "not slow"
+        default: false
       pytest_path:
         type: string
         required: false
@@ -33,10 +33,11 @@ on:
         description: The run ID of the workflow to fetch artifacts from
         type: string
         required: true
-      pytest_marker:
-        description: The pytest marker to run
-        type: string
+      test_all:
+        description: Run all tests
+        type: boolean
         required: false
+        default: false
       pytest_path:
         description: Select test execution path
         type: string
@@ -57,15 +58,18 @@ jobs:
     steps:
       - id: set-matrix
         run: |
-          if [[ ${{ github.event_name == 'schedule' }} == true ]]; then
+          if [[ ${{ github.event_name == 'schedule' }} ]]; then
+            marker=''
+            matrix='["thingy91x","nrf9151dk","ppk_thingy91x"]'
+          elif [[ ${{ inputs.test_all }} ]]; then
             marker=''
             matrix='["thingy91x","nrf9151dk","ppk_thingy91x"]'
-          elif [[ ${{ github.event_name == 'push' }} == true ]]; then
+          elif [[ ${{ github.event_name == 'push' }} ]]; then
             marker='-m "not slow"'
             matrix='["thingy91x"]'
           else
-            marker='${{ inputs.pytest_marker }}'
-            matrix=${{ inputs.devices }}
+            marker='-m "not slow"'
+            matrix='["thingy91x"]'
           fi
 
           echo Matrix as string: $matrix
@@ -128,7 +132,7 @@ jobs:
           nrfutil install trace
 
       - name: Upload symbol file to Memfault
-        if: ${{ matrix.device == 'thingy91x' }}
+        if: ${{ inputs.test_all }}
         working-directory: asset-tracker-template/tests/on_target/artifacts
         run: |
           memfault \
@@ -151,14 +155,19 @@ jobs:
         working-directory: asset-tracker-template/tests/on_target
         run: |
           mkdir -p results
+          if [[ ${{ inputs.build_all }} ]]; then
+            pytest_marker=""
+          else
+            pytest_marker='-m "not slow"'
+          fi
           if [[ '${{ matrix.device }}' == 'ppk_thingy91x' ]]; then
             # For PPK device, only run tests/test_ppk
-            pytest -v ${{ needs.set-matrix.outputs.pytest_marker }} \
+            pytest -v $pytest_marker \
               --junit-xml=results/test-results.xml \
               --html=results/test-results.html --self-contained-html \
               tests/test_ppk
           else
-            pytest -v ${{ needs.set-matrix.outputs.pytest_marker }} \
+            pytest -v $pytest_marker \
               --junit-xml=results/test-results.xml \
               --html=results/test-results.html --self-contained-html \
               --ignore=tests/test_ppk \

tests/on_target/tests/test_functional/test_memfault.py

@@ -88,6 +88,7 @@ def timestamp(event):
         event["captured_date"], "%Y-%m-%dT%H:%M:%S.%f%z"
     )
 
+@pytest.mark.slow
 def test_memfault(dut_board, debug_hex_file):
     # Save timestamp of latest coredump
     coredumps = get_latest_coredump_traces(IMEI)

tests/on_target/tests/test_functional/test_patched.py

@@ -4,16 +4,16 @@
 ##########################################################################################
 
 import os
-from utils.flash_tools import flash_device, reset_device
-import sys
 import pytest
 from utils.flash_tools import flash_device, reset_device
+import sys
 
 sys.path.append(os.getcwd())
 from utils.logger import get_logger
 
 logger = get_logger()
 
+@pytest.mark.slow
 def test_patched_firmware(dut_board, hex_file_patched):
     """Test the patched firmware with magnetometer functionality."""
     # Only run this test for thingy91x devices