From 409e8e2831352bff66c8a73284829b283ddc4fd6 Mon Sep 17 00:00:00 2001 From: ruv Date: Sat, 23 May 2026 14:49:10 -0400 Subject: [PATCH] fix(adr-115/ci): drop multi-filter from cargo test invocation (single TESTNAME accepted only) The mqtt-integration workflow's first cargo-test step was passing three filters in one invocation: cargo test ... --lib mqtt:: semantic:: cli::tests cargo test treats positional args after --lib as ONE TESTNAME and errored with 'unexpected argument semantic::'. Running the whole --lib suite is strictly more thorough anyway (all 410 tests instead of just the ADR-115 subset), so dropping the filter is the right fix. Refs PR #778. --- .github/workflows/mqtt-integration.yml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/.github/workflows/mqtt-integration.yml b/.github/workflows/mqtt-integration.yml index 49d96b02..8aed4c6b 100644 --- a/.github/workflows/mqtt-integration.yml +++ b/.github/workflows/mqtt-integration.yml @@ -79,10 +79,14 @@ jobs: - name: Verify unit tests still pass under --features mqtt working-directory: v2 + # `cargo test` accepts a single TESTNAME filter, so we run the + # whole --lib suite here. That gives us the full 410-test green + # bar under --features mqtt (which is more reassuring than + # filtering anyway). run: >- cargo test -p wifi-densepose-sensing-server --features mqtt --no-default-features - --lib mqtt:: semantic:: cli::tests + --lib --no-fail-fast - name: Run integration tests against mosquitto