Skip to content

Add envconfig support to test harness - #2998

Open
THardy98 wants to merge 6 commits into
mainfrom
feat/envconfig-harness
Open

Add envconfig support to test harness#2998
THardy98 wants to merge 6 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
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.

1 participant