Skip to content

Commit d435823

Browse files
committed
fix
1 parent 03bb602 commit d435823

File tree

1 file changed

+10
-11
lines changed

1 file changed

+10
-11
lines changed

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

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +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.assertThat;
11-
import static org.assertj.core.api.Assertions.assertThatCode;
129
import static org.assertj.core.api.Assertions.assertThatThrownBy;
1310
import static org.mockito.ArgumentMatchers.any;
1411
import static org.mockito.Mockito.doReturn;
@@ -96,21 +93,23 @@ void configFile_fileNotFound() {
9693
assertThatThrownBy(
9794
() ->
9895
AutoConfiguredOpenTelemetrySdk.builder()
99-
.addPropertiesSupplier(() -> singletonMap("otel.config.file", "foo"))
100-
.addPropertiesSupplier(
101-
() -> singletonMap("otel.experimental.config.file", "foo"))
102-
.addPropertiesSupplier(() -> singletonMap("otel.sdk.disabled", "true"))
96+
.setConfig(
97+
DefaultConfigProperties.createFromMap(
98+
Collections.singletonMap("otel.experimental.config.file", "foo")))
10399
.build())
104100
.isInstanceOf(ConfigurationException.class)
105101
.hasMessageContaining("Configuration file not found");
106102

107-
assertThatCode(
103+
assertThatThrownBy(
108104
() ->
109105
AutoConfiguredOpenTelemetrySdk.builder()
110-
.addPropertiesSupplier(() -> singletonMap("otel.experimental.config.file", ""))
111-
.addPropertiesSupplier(() -> singletonMap("otel.sdk.disabled", "true"))
106+
.setConfig(
107+
DefaultConfigProperties.createFromMap(
108+
Collections.singletonMap("otel.experimental.config.file", "")))
112109
.build())
113-
.doesNotThrowAnyException();
110+
.hasMessageContaining(
111+
"otel.metrics.exporter set to \"otlp\" but opentelemetry-exporter-otlp not found on "
112+
+ "classpath. Make sure to add it as a dependency.");
114113
}
115114

116115
@Test

0 commit comments

Comments
 (0)