Skip to content

Add envconfig support to test harness - #2998

Merged
THardy98 merged 8 commits into
mainfrom
feat/envconfig-harness
Sep 2, 2026
Merged

Add envconfig support to test harness#2998
THardy98 merged 8 commits into
mainfrom
feat/envconfig-harness

Conversation

@THardy98

@THardy98 THardy98 commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

What was changed?

The internal SDK test harness can now opt into standard Temporal client environment configuration with TEMPORAL_TEST_ENV_CONFIG_SERVER. When enabled, the existing harness loads ClientConfigProfile and uses its server address, namespace, API key, TLS settings, and gRPC metadata.

Envconfig mode connects to an externally managed server and skips the harness namespace-registration step. Existing in-memory, managed dev-server, virtual-thread, and legacy USE_EXTERNAL_SERVICE behavior remains unchanged. CONTRIBUTING.md documents the developer workflow.

Why?

SDK developers need to run the Java test suite against arbitrary Temporal deployments, including Temporal Cloud, without maintaining a parallel set of harness-specific client settings. Reusing envconfig lets the same TOML profiles and standard TEMPORAL_* variables work locally and in CI.

Breaking changes?

None. Envconfig mode is explicitly opt-in.

Server PR

None.

How was this tested?

  • Focused ExternalServiceTestConfiguratorTest coverage for both harness builders and required profile fields.
  • Full temporal-testing and temporal-sdk test tasks.
  • Envconfig-selected SignalTest.signalCompletedWorkflow against the managed dev server.
  • Namespace-registration guard with envconfig and the legacy external-service flag both enabled.

Documentation

CONTRIBUTING.md documents direct environment variables and TOML/profile-based configuration for the harness.

@THardy98
THardy98 force-pushed the feat/envconfig-harness branch from edd4c65 to 53769e5 Compare August 11, 2026 16:59
@THardy98
THardy98 marked this pull request as ready for review August 25, 2026 16:28
@THardy98
THardy98 requested a review from a team as a code owner August 25, 2026 16:28
@Quinn-With-Two-Ns Quinn-With-Two-Ns self-assigned this Aug 27, 2026
@THardy98
THardy98 enabled auto-merge (squash) September 2, 2026 12:52
@THardy98
THardy98 disabled auto-merge September 2, 2026 14:54
@THardy98
THardy98 merged commit 496ddc0 into main Sep 2, 2026
28 of 29 checks passed
@THardy98
THardy98 deleted the feat/envconfig-harness branch September 2, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants