Skip to content

Commit c0a2c09

Browse files
committed
test: Add comprehensive tests for Datadog API Key environment variable support
- Add testApiKeyFromConfigurationProperty: Verify API key resolution from config - Add testApiKeyFromEnvironmentVariable: Test environment variable fallback - Add testConfigurationPropertyTakesPrecedenceOverEnvironmentVariable: Ensure config takes priority - Add testEmptyConfigurationPropertyFallsBackToEnvironmentVariable: Test empty config fallback - Use reflection to test private getApiKey() method directly - Avoid API validation issues in tests using environment variable manipulation
1 parent 9d35157 commit c0a2c09

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

flink-metrics/flink-metrics-datadog/src/test/java/org/apache/flink/metrics/datadog/DatadogHttpReporterFactoryTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,7 @@ void testEmptyConfigurationPropertyFallsBackToEnvironmentVariable() throws Excep
9191
newEnv.put("DATADOG_API_KEY", envApiKey);
9292
CommonTestUtils.setEnv(newEnv, true);
9393

94-
final String resolvedApiKey =
95-
getApiKeyViaReflection(createConfig("apikey", ""));
94+
final String resolvedApiKey = getApiKeyViaReflection(createConfig("apikey", ""));
9695
assertThat(resolvedApiKey).isEqualTo(envApiKey);
9796
} finally {
9897
CommonTestUtils.setEnv(originalEnv, true);

0 commit comments

Comments
 (0)