Repository navigation
docs(tck): refresh the 0.6.0 conformance results - #74
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Refreshes the Sparkplug TCK conformance results for the 0.6.0 release candidate, rerun against the merged release docs (
masterat #73).edgesuite: 6 of 7 tests pass.MultipleBrokerTestnow runs with the second broker available (inver on port 1884) and records TIMEOUT with 27 passed, 2 failed:operational-behavior-primary-application-state-with-multiple-servers-walkoperational-behavior-edge-node-birth-sequence-waitThe previously not-executed single-server assertion now passes; both failures follow from the missing broker failover.
hostsuite: 6 of 6 pass, includingSendCommandTest(36 passed, 1 not executed) andMultipleBrokerTest(7 passed, 1 not executed).tck/CONFORMANCE.md: updated the results table, documented the two failing edge assertions and the timeout, and explained that the single-server assertion now passes because the second broker is provided (--second-broker); added a note for theTIMEOUTresult meaning.CHANGELOG.md: known-limitation wording now saysedge:MultipleBrokerTestfails its failover assertions.Phase 2 verification
cargo update: no package changescargo fmt --all --check: passcargo clippy --workspace --all-targets -- -D warnings: passcargo test --workspace: pass (0 failures)RUSTDOCFLAGS=-Dwarnings cargo doc --lib --no-deps --all-features: passcargo +1.85.0 test --workspace: passcargo package --listfor all 7 publishable crates: passTCK runs
./target/debug/tck-runner --suite edge --docker-image srad-tck \ --node-cmd ./target/debug/examples/tck-node --timeout 120 ./target/debug/tck-runner --suite host --docker-image srad-tck \ --node-cmd ./target/debug/examples/tck-host \ --host-app-id foo --client-id client --console-reply PASS --timeout 120Results and per-test logs:
tck/results/1793820997(edge) andtck/results/1793821208(host).Remaining known gap
edge:MultipleBrokerTeststill times out becausesrad-eonhas no broker failover. This is documented as a known limitation and does not block 0.6.0.