Skip to content

Commit 1fcf3a4

Browse files
committed
fix
1 parent 1978204 commit 1fcf3a4

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

sdk-extensions/autoconfigure/src/testIncubating/java/io/opentelemetry/sdk/autoconfigure/DeclarativeConfigurationTest.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@
66
package io.opentelemetry.sdk.autoconfigure;
77

88
import static io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions.assertThat;
9-
import static java.util.Collections.singletonMap;
10-
import static org.assertj.core.api.Assertions.assertThatCode;
119
import static org.assertj.core.api.Assertions.assertThatThrownBy;
1210
import static org.mockito.ArgumentMatchers.any;
1311
import static org.mockito.Mockito.doReturn;
@@ -94,21 +92,23 @@ void configFile_fileNotFound() {
9492
assertThatThrownBy(
9593
() ->
9694
AutoConfiguredOpenTelemetrySdk.builder()
97-
.addPropertiesSupplier(() -> singletonMap("otel.config.file", "foo"))
98-
.addPropertiesSupplier(
99-
() -> singletonMap("otel.experimental.config.file", "foo"))
100-
.addPropertiesSupplier(() -> singletonMap("otel.sdk.disabled", "true"))
95+
.setConfig(
96+
DefaultConfigProperties.createFromMap(
97+
Collections.singletonMap("otel.experimental.config.file", "foo")))
10198
.build())
10299
.isInstanceOf(ConfigurationException.class)
103100
.hasMessageContaining("Configuration file not found");
104101

105-
assertThatCode(
102+
assertThatThrownBy(
106103
() ->
107104
AutoConfiguredOpenTelemetrySdk.builder()
108-
.addPropertiesSupplier(() -> singletonMap("otel.experimental.config.file", ""))
109-
.addPropertiesSupplier(() -> singletonMap("otel.sdk.disabled", "true"))
105+
.setConfig(
106+
DefaultConfigProperties.createFromMap(
107+
Collections.singletonMap("otel.experimental.config.file", "")))
110108
.build())
111-
.doesNotThrowAnyException();
109+
.hasMessageContaining(
110+
"otel.metrics.exporter set to \"otlp\" but opentelemetry-exporter-otlp not found on "
111+
+ "classpath. Make sure to add it as a dependency.");
112112
}
113113

114114
@Test

0 commit comments

Comments
 (0)