Skip to content

Feature/ci pipeline#1

Merged
karakayahuseyin merged 7 commits into
mainfrom
feature/ci-pipeline
Jun 7, 2026
Merged

Feature/ci pipeline#1
karakayahuseyin merged 7 commits into
mainfrom
feature/ci-pipeline

Conversation

@karakayahuseyin

Copy link
Copy Markdown
Contributor

This pull request introduces a comprehensive C++ and Python testing framework for the project, integrates it into CI, and adds robust unit tests for core C++ modules. The main highlights are the addition of a GitHub Actions CI workflow, C++ unit test targets and test runner, and improved Python test scripts for dialect parsing and generator validation.

Continuous Integration & Build System Integration

  • Added a GitHub Actions workflow (.github/workflows/ci.yml) to automate build, Python and C++ unit tests, and dialect validation on push and pull requests.
  • Updated CMake configuration to enable CTest and add a C++ test runner executable (test_runner) that runs all C++ unit tests. [1] [2]

C++ Unit Tests

  • Added C++ unit tests for the Router service (bridge/test/test_router.cc) covering subscription, message routing, filter logic, and cleanup.
  • Added C++ unit tests for the MessageConverter (bridge/test/test_converter.cc) to verify round-trip conversion between MAVLink and Protobuf messages.
  • Created a test runner (bridge/test/test_runner.cc) to orchestrate and report results for all C++ unit tests.

Python Test Improvements

  • Improved dialect path detection in Python test scripts (generator/test/test_parser.py, generator/test/test_generator.py) to support both third_party/mavlink and mavlink source directories. [1] [2]
  • Updated generator test to use get_flattened_dialect for more robust dialect loading.

Dependency Updates

  • Added nlohmann-json3-dev to system dependencies in both the setup script and CI workflow to support C++ JSON functionality. [1] [2]

@karakayahuseyin karakayahuseyin self-assigned this Jun 7, 2026
@karakayahuseyin karakayahuseyin added the enhancement New feature or request label Jun 7, 2026
@karakayahuseyin karakayahuseyin merged commit 9c4e749 into main Jun 7, 2026
2 checks passed
@karakayahuseyin karakayahuseyin deleted the feature/ci-pipeline branch June 7, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant