Add option to upload test artifact

Signed-off-by: Raul Sanchez-Mateos <[email protected]>

multiplatform/colcon_build_test/action.yml

@@ -29,6 +29,13 @@ inputs:
     required: false
     default: ''
 
+  upload_test_report_artifact:
+    description: >
+      Whether to upload the tests results as an artifact or not.
+    required: false
+    type: boolean
+    default: true
+
   cmake_build_type:
     description: 'Set the cmake build type'
     required: false
@@ -71,6 +78,7 @@ runs:
         workspace: ${{ inputs.workspace }}
         packages_names: ${{ inputs.packages_names }}
         test_report_artifact: ${{ inputs.test_report_artifact }}
+        upload_test_report_artifact: ${{ inputs.upload_test_report_artifact }}
         colcon_test_args: ${{ inputs.colcon_test_args }}
         ctest_args: ' ${{ inputs.ctest_args }}'
         workspace_dependencies: ${{ inputs.workspace_dependencies }}

multiplatform/colcon_test/action.yml

@@ -49,6 +49,14 @@ inputs:
     required: false
     default: ''
 
+  upload_test_report_artifact:
+    description: >
+      Whether to upload the tests results as an artifact or not.
+    required: false
+    type: boolean
+    default: true
+
+
 outputs:
   ctest_results_path:
     description: "Path to test results"
@@ -72,6 +80,7 @@ runs:
         workspace: ${{ inputs.workspace }}
         workspace_dependencies: ${{ inputs.workspace_dependencies }}
         test_report_artifact: ${{ inputs.test_report_artifact }}
+        upload_test_report_artifact: ${{ inputs.upload_test_report_artifact }}
 
     - name: Run in windows
       id: test_windows
@@ -87,3 +96,4 @@ runs:
         workspace: ${{ inputs.workspace }}
         workspace_dependencies: ${{ inputs.workspace_dependencies }}
         test_report_artifact: ${{ inputs.test_report_artifact }}
+        upload_test_report_artifact: ${{ inputs.upload_test_report_artifact }}

ubuntu/colcon_test/action.yml

@@ -49,6 +49,13 @@ inputs:
     required: false
     default: ''
 
+  upload_test_report_artifact:
+    description: >
+      Whether to upload the tests results as an artifact or not.
+    required: false
+    type: boolean
+    default: true
+
 outputs:
   ctest_results_path:
     description: "Path to test results"
@@ -136,7 +143,7 @@ runs:
 
     - name: Upload test report in JUnit format
       uses: eProsima/eProsima-CI/external/upload-artifact@main
-      if: ${{ ! cancelled() }}
+      if: ${{ inputs.upload_test_report_artifact && !cancelled() }}
       with:
           name: ${{ inputs.test_report_artifact || format('test_report_{0}_{1}_{2}', github.workflow, github.job, join(matrix.*, '_')) }}
           path: ${{ inputs.workspace }}/test_results

ubuntu/sphinx_docs/action.yml

@@ -57,6 +57,20 @@ inputs:
     type: string
     default: 'v2'
 
+  test_report_artifact:
+    description: >
+      Name of the artifact to be uploaded containing the test report in JUnit format. It shall be unique within the workflow run.
+      Default: test_report_<github.workflow>_<github.job>_<matrix.*>
+    required: false
+    default: ''
+
+  upload_test_report_artifact:
+    description: >
+      Whether to upload the tests results as an artifact or not.
+    required: false
+    type: boolean
+    default: true
+
   upload_artifact_name:
     description: >
       Name of the artifact to upload.
@@ -110,6 +124,8 @@ runs:
       with:
         packages_names: ${{ inputs.docs_subpackage_name }}
         workspace_dependencies: '${{ github.workspace }}/install'
+        test_report_artifact: ${{ inputs.test_report_artifact }}
+        upload_test_report_artifact: ${{ inputs.upload_test_report_artifact }}
 
     - name: Upload documentation
       uses: eProsima/eProsima-CI/external/upload-artifact@main

versions.md

@@ -45,6 +45,8 @@ The [Forthcoming](#forthcoming) section includes those features added in `main`
 
 The upcoming release will include the following **features**:
 
+- Add an option whether to upload the test aritfact or not.
+
 ## v0.31.0
 
 - Upgrade [upload-artifact](/external/upload-artifact/action) action version to ``v4.6.0``.

windows/colcon_test/action.yml

@@ -49,6 +49,14 @@ inputs:
     required: false
     default: ''
 
+  upload_test_report_artifact:
+    description: >
+      Whether to upload the tests results as an artifact or not.
+    required: false
+    type: boolean
+    default: true
+
+
 outputs:
   ctest_results_path:
     description: "Path to test results"
@@ -140,7 +148,7 @@ runs:
 
     - name: Upload test report in JUnit format
       uses: eProsima/eProsima-CI/external/upload-artifact@main
-      if: ${{ ! cancelled() }}
+      if: ${{ inputs.upload_test_report_artifact && !cancelled() }}
       with:
           name: ${{ inputs.test_report_artifact || format('test_report_{0}_{1}_{2}', github.workflow, github.job, join(matrix.*, '_')) }}
           path: "${{ inputs.workspace }}\\test_results"