Skip to content

Commit

Permalink
Use higher timeout for signal-specific timeout tests. (#3448)
Browse files Browse the repository at this point in the history
  • Loading branch information
Anuraag Agrawal authored Aug 17, 2021
1 parent 58c16a9 commit e868c82
Showing 1 changed file with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -133,18 +133,18 @@ void configureExportersGeneral() {
props.put("otel.exporter.otlp.endpoint", "https://localhost:" + server.httpsPort());
props.put("otel.exporter.otlp.certificate", certificate.certificateFile().getAbsolutePath());
props.put("otel.exporter.otlp.headers", "header-key=header-value");
props.put("otel.exporter.otlp.timeout", "5s");
props.put("otel.exporter.otlp.timeout", "15s");
ConfigProperties properties = DefaultConfigProperties.createForTest(props);
SpanExporter spanExporter = SpanExporterConfiguration.configureExporter("otlp", properties);
MetricExporter metricExporter =
MetricExporterConfiguration.configureOtlpMetrics(
properties, SdkMeterProvider.builder().build());

assertThat(spanExporter).extracting("timeoutNanos").isEqualTo(TimeUnit.SECONDS.toNanos(5));
assertThat(spanExporter).extracting("timeoutNanos").isEqualTo(TimeUnit.SECONDS.toNanos(15));
assertThat(
spanExporter
.export(Lists.newArrayList(generateFakeSpan()))
.join(10, TimeUnit.SECONDS)
.join(15, TimeUnit.SECONDS)
.isSuccess())
.isTrue();
assertThat(otlpTraceRequests).hasSize(1);
Expand All @@ -155,11 +155,11 @@ void configureExportersGeneral() {
":path", "/opentelemetry.proto.collector.trace.v1.TraceService/Export")
&& headers.contains("header-key", "header-value"));

assertThat(metricExporter).extracting("timeoutNanos").isEqualTo(TimeUnit.SECONDS.toNanos(5));
assertThat(metricExporter).extracting("timeoutNanos").isEqualTo(TimeUnit.SECONDS.toNanos(15));
assertThat(
metricExporter
.export(Lists.newArrayList(generateFakeMetric()))
.join(10, TimeUnit.SECONDS)
.join(15, TimeUnit.SECONDS)
.isSuccess())
.isTrue();
assertThat(otlpMetricsRequests).hasSize(1);
Expand All @@ -184,12 +184,12 @@ void configureSpanExporter() {
props.put(
"otel.exporter.otlp.traces.certificate", certificate.certificateFile().getAbsolutePath());
props.put("otel.exporter.otlp.traces.headers", "header-key=header-value");
props.put("otel.exporter.otlp.traces.timeout", "5s");
props.put("otel.exporter.otlp.traces.timeout", "15s");
SpanExporter spanExporter =
SpanExporterConfiguration.configureExporter(
"otlp", DefaultConfigProperties.createForTest(props));

assertThat(spanExporter).extracting("timeoutNanos").isEqualTo(TimeUnit.SECONDS.toNanos(5));
assertThat(spanExporter).extracting("timeoutNanos").isEqualTo(TimeUnit.SECONDS.toNanos(15));
assertThat(
spanExporter
.export(Lists.newArrayList(generateFakeSpan()))
Expand Down Expand Up @@ -218,16 +218,16 @@ public void configureMetricExporter() {
props.put(
"otel.exporter.otlp.metrics.certificate", certificate.certificateFile().getAbsolutePath());
props.put("otel.exporter.otlp.metrics.headers", "header-key=header-value");
props.put("otel.exporter.otlp.metrics.timeout", "5s");
props.put("otel.exporter.otlp.metrics.timeout", "15s");
MetricExporter metricExporter =
MetricExporterConfiguration.configureOtlpMetrics(
DefaultConfigProperties.createForTest(props), SdkMeterProvider.builder().build());

assertThat(metricExporter).extracting("timeoutNanos").isEqualTo(TimeUnit.SECONDS.toNanos(5));
assertThat(metricExporter).extracting("timeoutNanos").isEqualTo(TimeUnit.SECONDS.toNanos(15));
assertThat(
metricExporter
.export(Lists.newArrayList(generateFakeMetric()))
.join(10, TimeUnit.SECONDS)
.join(15, TimeUnit.SECONDS)
.isSuccess())
.isTrue();
assertThat(otlpMetricsRequests).hasSize(1);
Expand Down

0 comments on commit e868c82

Please sign in to comment.