Skip to content

Run Cloud-eligible integration tests with filtering - #1518

Draft
THardy98 wants to merge 3 commits into
mainfrom
feat/cloud-test-filtering
Draft

Run Cloud-eligible integration tests with filtering#1518
THardy98 wants to merge 3 commits into
mainfrom
feat/cloud-test-filtering

Conversation

@THardy98

@THardy98 THardy98 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

What was changed

Integration-test modules now declare Cloud exclusions where they are composed with
cloud_test_module_exclusion!; individual tests in mixed modules use
#[cloud_test_exclusion(...)]. Both forms take a fixed reason and an optional note. Tests without
an exclusion are Cloud-eligible by default.

When the integration runner receives --cloud, it compiles only eligible tests, prints their
inventory, and fails if none remain. Normal integration runs and Clippy continue to compile the
complete suite. The trusted Cloud job runs the selection through the envconfig-backed client and
isolated namespace added in #1483.

Why?

Cloud policy should be visible where test suites are maintained without depending on generated
libtest names. Module-level exclusions keep the classification compact: complete suites need one
declaration, while mixed suites can still exclude a specific test.

Checklist

  1. Part of Run integration tests against cloud features#851.

  2. How was this tested:

    • cargo +1.94 test -q -p temporalio-macros
    • cargo +1.94 integ-test -s external --cloud -- --list selects the existing timer test.
    • cargo +1.94 lint
    • cargo +1.94 test-lint
    • actionlint -ignore 'SC2086' .github/workflows/per-pr.yml
  3. Any docs updates needed?

    Yes. The README documents module- and function-level Cloud exclusions and inventory listing.

@THardy98
THardy98 force-pushed the feat/envconfig-harness branch from 086a427 to b735380 Compare August 24, 2026 18:51
@THardy98
THardy98 force-pushed the feat/cloud-test-filtering branch 2 times, most recently from 56f8638 to 1bb491c Compare August 24, 2026 19:04
@THardy98
THardy98 force-pushed the feat/envconfig-harness branch from b735380 to e19bfdd Compare August 25, 2026 13:42
@THardy98
THardy98 force-pushed the feat/cloud-test-filtering branch from 1bb491c to da76966 Compare August 25, 2026 13:43
@THardy98
THardy98 force-pushed the feat/envconfig-harness branch from e19bfdd to c7823e9 Compare August 25, 2026 14:43
@THardy98
THardy98 force-pushed the feat/cloud-test-filtering branch from da76966 to 4fa0651 Compare August 25, 2026 14:43
@THardy98
THardy98 force-pushed the feat/envconfig-harness branch from c7823e9 to 1c0f8aa Compare August 25, 2026 14:57
@THardy98
THardy98 force-pushed the feat/cloud-test-filtering branch 5 times, most recently from a1352d2 to 495c5e7 Compare August 26, 2026 15:13
Base automatically changed from feat/envconfig-harness to main August 26, 2026 18:25
@THardy98
THardy98 force-pushed the feat/cloud-test-filtering branch from 495c5e7 to d5cc2f0 Compare September 2, 2026 16:12
@THardy98 THardy98 added the skip-changelog PR changes do not require changelog updates label Sep 2, 2026
@THardy98
THardy98 marked this pull request as ready for review September 2, 2026 16:37
@THardy98
THardy98 requested a review from a team as a code owner September 2, 2026 16:37
@THardy98
THardy98 marked this pull request as draft September 2, 2026 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog PR changes do not require changelog updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant