feat[#13]: Add integration tests to CI pipeline (#14)

* feat: add integration tests to CI pipeline

- Split CI test step into unit and integration tests
- Add python3-requests test dependency to package.xml
- Update README with instructions for running tests separately

* Improve CI workflow with ROS 2 best practices

- Add separate step for linting checks
- Add console_direct+ event handler for real-time output
- Show test results after each test category (linters, unit, integration)
- Add rosdep package caching to speed up builds
- Add timeout for integration tests (10 min) and overall job (60 min)
- Upload test artifacts for all runs
- Run CI on pushes to main branch in addition to PRs
- Update README with improved testing documentation

* Run full test suite once in CI

- drop the unused rosdep cache step
- run a single `colcon test` covering lint/unit/integration
- document the new workflow and artifact upload in README

Author: bburda

.github/workflows/ci.yml

@@ -3,10 +3,13 @@ name: CI
 on:
   pull_request:
     branches: [main]
+  push:
+    branches: [main]
 
 jobs:
   build-and-test:
     runs-on: ubuntu-24.04
+    timeout-minutes: 60
 
     steps:
       - name: Checkout repository
@@ -26,15 +29,27 @@ jobs:
       - name: Build packages
         run: |
           source /opt/ros/jazzy/setup.bash
-          colcon build --symlink-install --cmake-args -DCMAKE_BUILD_TYPE=Release
+          colcon build --symlink-install \
+            --cmake-args -DCMAKE_BUILD_TYPE=Release \
+            --event-handlers console_direct+
 
       - name: Run tests
+        timeout-minutes: 15
         run: |
           source /opt/ros/jazzy/setup.bash
           source install/setup.bash
-          colcon test --return-code-on-test-failure
+          colcon test --return-code-on-test-failure \
+            --event-handlers console_direct+
 
       - name: Show test results
         if: always()
-        run: |
-          colcon test-result --verbose
+        run: colcon test-result --verbose
+
+      - name: Upload test results
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: test-results
+          path: |
+            log/
+            build/*/test_results/

README.md

@@ -49,15 +49,48 @@ colcon build --symlink-install
 
 ### Testing
 
+Run all tests:
+
 ```bash
+source install/setup.bash
 colcon test
 colcon test-result --verbose
 ```
 
+Run linters:
+
+```bash
+source install/setup.bash
+colcon test --ctest-args -L linters
+colcon test-result --verbose
+```
+
+Run only unit tests (everything except integration):
+
+```bash
+source install/setup.bash
+colcon test --ctest-args -E test_integration
+colcon test-result --verbose
+```
+
+Run only integration tests:
+
+```bash
+source install/setup.bash
+colcon test --ctest-args -R test_integration
+colcon test-result --verbose
+```
+
 ### CI/CD
 
-All pull requests are automatically built and tested using GitHub Actions.
-The CI workflow runs on Ubuntu 24.04 with ROS 2 Jazzy.
+All pull requests and pushes to main are automatically built and tested using GitHub Actions.
+The CI workflow runs on Ubuntu 24.04 with ROS 2 Jazzy, executes a single `colcon test` to cover:
+
+- Code linting and formatting checks
+- Unit tests
+- Integration tests with demo automotive nodes
+
+After every run the workflow always calls `colcon test-result --verbose` and uploads the generated logs/results as artifacts for debugging.
 
 ## Contributing
 

src/ros2_medkit_gateway/package.xml

@@ -24,6 +24,7 @@
   <test_depend>ament_cmake_gtest</test_depend>
   <test_depend>ament_cmake_pytest</test_depend>
   <test_depend>launch_testing_ament_cmake</test_depend>
+  <test_depend>python3-requests</test_depend>
 
   <export>
     <build_type>ament_cmake</build_type>