From fc514e61fc48d961fa2c930f57b879a4e50d257e Mon Sep 17 00:00:00 2001
From: Brett Nicholas <7547222+bigbrett@users.noreply.github.com>
Date: Fri, 6 Dec 2024 14:05:13 -0700
Subject: [PATCH] add test for wolfHSM ml-dsa to CI

---
 .github/workflows/test-wolfhsm-simulator.yml | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/test-wolfhsm-simulator.yml b/.github/workflows/test-wolfhsm-simulator.yml
index ac4349bb7..5a7db2ab1 100644
--- a/.github/workflows/test-wolfhsm-simulator.yml
+++ b/.github/workflows/test-wolfhsm-simulator.yml
@@ -9,6 +9,17 @@ on:
 
 jobs:
   wolfhsm_simulator_test:
+    # Matrix strategy runs all steps below for each config specified.
+    # This allows testing multiple configurations without duplicating the workflow.
+    strategy:
+      matrix:
+        config:
+          - name: "Standard wolfHSM"
+            file: "config/examples/sim-wolfHSM.config"
+          - name: "wolfHSM ML-DSA"
+            file: "config/examples/sim-wolfHSM-mldsa.config"
+      fail-fast: false
+    
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
@@ -25,9 +36,9 @@ jobs:
         run: |
           make distclean
 
-      - name: Select config (wolfHSM simulator)
+      - name: Select config (${{ matrix.config.name }})
         run: |
-          cp config/examples/sim-wolfHSM.config .config
+          cp ${{ matrix.config.file }} .config
 
       - name: Build tools
         run: |
@@ -62,5 +73,6 @@ jobs:
 
       # Kill the server if it is still running
       - name: Kill POSIX TCP server
+        if: always()
         run: |
-          kill $TCP_SERVER_PID
+          kill $TCP_SERVER_PID || true