🔧 (.github/workflows/pro-extension-test.yml): refactor test running s…

…cript to handle extra Maven arguments more efficiently and consistently
liquibase · Aug 27, 2024 · 294145d · 294145d
1 parent db6ef5c
commit 294145d
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/.github/workflows/pro-extension-test.yml b/.github/workflows/pro-extension-test.yml
@@ -300,12 +300,21 @@ jobs:
       - name: Run Tests
         if: ${{ !inputs.nightly }}
         shell: bash
-        run: mvn -B test -P '${{ inputs.mavenProfiles }}' ${${{ inputs.extraMavenArgs }}:+\"${{ inputs.extraMavenArgs }}\"}
-
+        run: |
+          if [ -n "${{ inputs.extraMavenArgs }}" ]; then
+            mvn -B test -P '${{ inputs.mavenProfiles }}' "${{ inputs.extraMavenArgs }}"
+          else
+            mvn -B test -P '${{ inputs.mavenProfiles }}'
+          fi
+  
       - name: Run Tests
         if: ${{ inputs.nightly }}
-        shell: bash
-        run: mvn -B test -P '${{ inputs.mavenProfiles }}' ${${{ inputs.extraMavenArgs }}:+\"${{ inputs.extraMavenArgs }}\"} "-Dliquibase.version=master-SNAPSHOT"
+        run: |
+          if [ -n "${{ inputs.extraMavenArgs }}" ]; then
+            mvn -B test -P '${{ inputs.mavenProfiles }}' "${{ inputs.extraMavenArgs }}" "-Dliquibase.version=master-SNAPSHOT"
+          else
+            mvn -B test -P '${{ inputs.mavenProfiles }}' "-Dliquibase.version=master-SNAPSHOT"
+          fi
 
       - name: Notify Slack on Build Failure
         if: ${{ failure() && inputs.nightly }}