Skip to content

Add DirCon loopback integration test and CI workflow - #4397

Closed
cagnulein wants to merge 6 commits into
claude/wahoo-protocol-broadcasting-01RkFxRkDM83upZVo1fcTEprfrom
codex/test-patch-with-two-qz-instances
Closed

Add DirCon loopback integration test and CI workflow#4397
cagnulein wants to merge 6 commits into
claude/wahoo-protocol-broadcasting-01RkFxRkDM83upZVo1fcTEprfrom
codex/test-patch-with-two-qz-instances

Conversation

@cagnulein

Copy link
Copy Markdown
Owner

Motivation

  • Add an integration test to validate DirCon server/client loopback behavior and telemetry delivery to prevent regressions in DirCon connectivity.

Description

  • Add tst/Devices/dirconloopbacktestsuite.cpp and tst/Devices/dirconloopbacktestsuite.h which start a local fakebike DirCon server and a wahoodirconbike client, then assert successful handshake and non-zero telemetry (watts/cadence/speed).
  • Update tst/qdomyos-zwift-tests.pro to include the new test source and header in the test target.
  • Add a GitHub Actions workflow .github/workflows/dircon-loopback.yml that installs Qt dependencies, builds the static library and test binary, and runs the DirCon loopback test on ubuntu-24.04.

Testing

  • CI job DirCon loopback integration test is configured to run ./qdomyos-zwift-tests --gtest_filter=DirconLoopbackTestSuite.* after building the code and dependencies.
  • No automated test run results are included in this change; the workflow is added and ready to execute in CI.

Codex Task

@cagnulein
cagnulein marked this pull request as ready for review March 9, 2026 06:40
@cagnulein cagnulein closed this Mar 12, 2026
@cagnulein
cagnulein deleted the codex/test-patch-with-two-qz-instances branch March 12, 2026 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant