Skip to content

Commit 54e59f5

Browse files
committed
Fix latest spec compatibility job
The newly introduced cucumber based integration tests do not use the standard test harness and therefore do not support the listing of test cases as part of the cargo nextest run. The test execution has therefore been adapted to use the standard cargo test command instead of cargo nextest.
1 parent 88fa72e commit 54e59f5

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

.github/workflows/latest-up-spec-compatibility.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ jobs:
6363
file-patterns: "${{ env.OFT_FILE_PATTERNS }}"
6464
tags: "${{ env.OFT_TAGS_}}"
6565

66-
# now try to build and run the tests which may fail if incomaptible changes
66+
# now try to build and run the tests which may fail if incompatible changes
6767
# have been introduced into the uProtocol Core API
6868
- uses: dtolnay/rust-toolchain@master
6969
with:
@@ -72,10 +72,10 @@ jobs:
7272
- uses: taiki-e/install-action@nextest
7373
- name: Run tests
7474
run: |
75-
# Using nextest because it's faster than built-in test
76-
cargo nextest run --all-features
77-
# but it cannot execute doc tests
78-
cargo test --doc --all-features
75+
# We need to use standard "cargo test" here because nextest cannot execute
76+
# cucumber based integration tests which do not use the standard test harness.
77+
cargo test --all-features --all-targets
78+
cargo test --all-features --doc
7979
8080
# This step will only be run if the tests in the previous step have succeeded.
8181
# In that case, we use the exit code produced by the OFT run as the job's

0 commit comments

Comments
 (0)