From 6b5808c0908c9e2dc1d039f3bbfa99eb1044de3c Mon Sep 17 00:00:00 2001
From: tdejoigny-ledger <thibault.de-joigny@ledger.fr>
Date: Mon, 29 Jul 2024 18:05:06 +0200
Subject: [PATCH] PR remarks fixes

---
 .../continuous-integration-workflow.yml       |  9 +++++++++
 .../reusable_ragger_tests_latest_speculos.yml | 19 +++++++++++--------
 2 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/continuous-integration-workflow.yml b/.github/workflows/continuous-integration-workflow.yml
index f5db54f7..47eb917b 100644
--- a/.github/workflows/continuous-integration-workflow.yml
+++ b/.github/workflows/continuous-integration-workflow.yml
@@ -140,6 +140,15 @@ jobs:
       test_dir: tests
       speculos_app_branch_name: ${{ github.ref }}
 
+  package_and_test_docker_for_nanos:
+    name: Build and test the Speculos docker for Nano S
+    uses: ./.github/workflows/reusable_ragger_tests_latest_speculos.yml
+    with:
+      app_repository: LedgerHQ/app-boilerplate
+      app_branch_name: nanos_baseline
+      test_dir: tests
+      speculos_app_branch_name: ${{ github.ref }}
+
   deploy_docker:
     name: Build and Upload the Speculos docker
     runs-on: ubuntu-latest
diff --git a/.github/workflows/reusable_ragger_tests_latest_speculos.yml b/.github/workflows/reusable_ragger_tests_latest_speculos.yml
index 8d236153..5922e79d 100644
--- a/.github/workflows/reusable_ragger_tests_latest_speculos.yml
+++ b/.github/workflows/reusable_ragger_tests_latest_speculos.yml
@@ -57,19 +57,22 @@ jobs:
         name: speculos_image
         path: /tmp/speculos_image.tar
 
+  call_get_app_metadata:
+    name: Retrieve application metadata
+    uses: LedgerHQ/ledger-app-workflows/.github/workflows/_get_app_metadata.yml@v1
+    with:
+      app_repository: ${{ inputs.app_repository }}
+      app_branch_name: ${{ inputs.app_branch_name }}
+
   ragger_tests:
     name: Functional tests with Ragger
     runs-on: ubuntu-latest
-    needs: [build_docker_image, build_application]
+    needs: [build_docker_image, build_application, call_get_app_metadata]
     strategy:
       fail-fast: false
       matrix:
-        include:
-          - device: nanos
-          - device: nanox
-          - device: nanosp
-          - device: stax
-          - device: flex
+        device: ${{ fromJSON(needs.call_get_app_metadata.outputs.compatible_devices) }}
+
     steps:
     - name: Clone
       uses: actions/checkout@v4
@@ -83,7 +86,7 @@ jobs:
       uses: actions/checkout@v4
       with:
           repository: ${{ inputs.app_repository }}
-          ref: ${{ matrix.device == 'nanos' && 'nanos_baseline' || inputs.app_branch_name  }}
+          ref: ${{ inputs.app_branch_name }}
           path: app
           submodules: recursive
           fetch-depth: 0