remove test-result, and fix bug in smoke test

Author: sgillen

.github/workflows/debian-packages.yml

@@ -142,11 +142,13 @@ jobs:
           curl -sSL https://raw.githubusercontent.com/${{ github.repository }}/${{ github.sha }}/r2s_gw/requirements.txt > /tmp/requirements.txt
           python3 -m pip install -r /tmp/requirements.txt
         fi
+      shell: bash
 
     - name: Test r2s_gw execution
       run: |
         source /opt/ros/${{ matrix.ros_distro }}/setup.bash
         timeout 10s bash -lc "script -qfec 'ros2 run r2s_gw r2s_gw' /dev/null <<< \$'q'" || true
+      shell: bash
 
     - name: Test greenwave_monitor execution
       run: |
@@ -156,4 +158,5 @@ jobs:
         ros2 node list | tee /tmp/nodes.txt
         grep -q greenwave_monitor /tmp/nodes.txt
         kill -TERM "$(cat /tmp/gwm.pid)" || true
-        wait "$(cat /tmp/gwm.pid)" || true
+        wait "$(cat /tmp/gwm.pid)" || true
+      shell: bash

.github/workflows/ros-tests.yml

@@ -73,12 +73,4 @@ jobs:
       with:
         name: test-results-${{ matrix.ros_distro }}
         path: build/*/test_results/**/*.xml
-        retention-days: 7
-
-    - name: Publish test results
-      uses: EnricoMi/publish-unit-test-result-action@v2
-      if: always()
-      with:
-        files: build/*/test_results/*/*.xml
-        check_name: "Test Results (${{ matrix.ros_distro }})"
-        report_individual_runs: true
+        retention-days: 7