Skip to content

Commit 2f06d1c

Browse files
committed
separate test bridge for agent
1 parent 357da77 commit 2f06d1c

File tree

6 files changed

+25
-26
lines changed

6 files changed

+25
-26
lines changed

testing/agent-exporter/build.gradle.kts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ dependencies {
1515
compileOnly(project(":javaagent-extension-api"))
1616
compileOnly(project(":javaagent-bootstrap"))
1717
compileOnly(project(":javaagent-tooling"))
18-
implementation(project(":testing-common"))
1918

2019
implementation("io.opentelemetry:opentelemetry-exporter-otlp-common")
20+
compileOnly("io.opentelemetry:opentelemetry-api-incubator")
2121
compileOnly("io.opentelemetry:opentelemetry-sdk-extension-autoconfigure")
2222
}

testing/agent-exporter/src/main/java/io/opentelemetry/javaagent/testing/exporter/AgentTestingExporterFactory.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@
55

66
package io.opentelemetry.javaagent.testing.exporter;
77

8-
import io.opentelemetry.instrumentation.testing.provider.TestBridgeLogRecordExporterComponentProvider;
9-
import io.opentelemetry.instrumentation.testing.provider.TestBridgeMetricExporterComponentProvider;
10-
import io.opentelemetry.instrumentation.testing.provider.TestBridgeSpanExporterComponentProvider;
8+
import io.opentelemetry.javaagent.testing.provider.TestBridgeLogRecordExporterComponentProvider;
9+
import io.opentelemetry.javaagent.testing.provider.TestBridgeMetricExporterComponentProvider;
10+
import io.opentelemetry.javaagent.testing.provider.TestBridgeSpanExporterComponentProvider;
1111
import java.util.List;
1212
import java.util.concurrent.TimeUnit;
1313

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,19 @@
33
* SPDX-License-Identifier: Apache-2.0
44
*/
55

6-
package io.opentelemetry.instrumentation.testing.provider;
6+
package io.opentelemetry.javaagent.testing.provider;
77

88
import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
99
import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
1010
import io.opentelemetry.sdk.logs.export.LogRecordExporter;
11-
import org.slf4j.Logger;
12-
import org.slf4j.LoggerFactory;
11+
import java.util.logging.Level;
12+
import java.util.logging.Logger;
1313

1414
public class TestBridgeLogRecordExporterComponentProvider
1515
implements ComponentProvider<LogRecordExporter> {
1616

1717
private static final Logger logger =
18-
LoggerFactory.getLogger(TestBridgeLogRecordExporterComponentProvider.class);
18+
Logger.getLogger(TestBridgeLogRecordExporterComponentProvider.class.getName());
1919

2020
private static LogRecordExporter logRecordExporter;
2121

@@ -35,9 +35,8 @@ public LogRecordExporter create(DeclarativeConfigProperties config) {
3535
}
3636

3737
public static void setLogRecordExporter(LogRecordExporter logRecordExporter) {
38-
logger.info(
39-
"Setting TestLogRecordExporterComponentProvider logRecord exporter to {}",
40-
logRecordExporter.getClass().getName());
38+
logger.log(
39+
Level.INFO, "Setting logRecord exporter to {0}", logRecordExporter.getClass().getName());
4140
TestBridgeLogRecordExporterComponentProvider.logRecordExporter = logRecordExporter;
4241
}
4342
}
Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,19 @@
33
* SPDX-License-Identifier: Apache-2.0
44
*/
55

6-
package io.opentelemetry.instrumentation.testing.provider;
6+
package io.opentelemetry.javaagent.testing.provider;
77

88
import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
99
import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
1010
import io.opentelemetry.sdk.metrics.export.MetricExporter;
11-
import org.slf4j.Logger;
12-
import org.slf4j.LoggerFactory;
11+
import java.util.logging.Level;
12+
import java.util.logging.Logger;
1313

1414
public class TestBridgeMetricExporterComponentProvider
1515
implements ComponentProvider<MetricExporter> {
1616

1717
private static final Logger logger =
18-
LoggerFactory.getLogger(TestBridgeMetricExporterComponentProvider.class);
18+
Logger.getLogger(TestBridgeMetricExporterComponentProvider.class.getName());
1919

2020
private static MetricExporter metricExporter;
2121

@@ -35,9 +35,8 @@ public MetricExporter create(DeclarativeConfigProperties config) {
3535
}
3636

3737
public static void setMetricExporter(MetricExporter metricExporter) {
38-
logger.info(
39-
"Setting TestMetricExporterComponentProvider metric exporter to {}",
40-
metricExporter.getClass().getName());
38+
logger.log(
39+
Level.INFO, "Setting logRecord exporter to {0}", metricExporter.getClass().getName());
4140
TestBridgeMetricExporterComponentProvider.metricExporter = metricExporter;
4241
}
4342
}
Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,18 @@
33
* SPDX-License-Identifier: Apache-2.0
44
*/
55

6-
package io.opentelemetry.instrumentation.testing.provider;
6+
package io.opentelemetry.javaagent.testing.provider;
77

88
import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
99
import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
1010
import io.opentelemetry.sdk.trace.export.SpanExporter;
11-
import org.slf4j.Logger;
12-
import org.slf4j.LoggerFactory;
11+
import java.util.logging.Level;
12+
import java.util.logging.Logger;
1313

14-
// todo should be inherited from testing-common - not sure why it is not
1514
public class TestBridgeSpanExporterComponentProvider implements ComponentProvider<SpanExporter> {
1615

1716
private static final Logger logger =
18-
LoggerFactory.getLogger(TestBridgeSpanExporterComponentProvider.class);
17+
Logger.getLogger(TestBridgeSpanExporterComponentProvider.class.getName());
1918

2019
private static SpanExporter spanExporter;
2120

@@ -35,9 +34,7 @@ public SpanExporter create(DeclarativeConfigProperties config) {
3534
}
3635

3736
public static void setSpanExporter(SpanExporter spanExporter) {
38-
logger.info(
39-
"Setting TestSpanExporterComponentProvider span exporter to {}",
40-
spanExporter.getClass().getName());
37+
logger.log(Level.INFO, "Setting span exporter to {0}", spanExporter.getClass().getName());
4138
TestBridgeSpanExporterComponentProvider.spanExporter = spanExporter;
4239
}
4340
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
io.opentelemetry.javaagent.testing.provider.TestBridgeSpanExporterComponentProvider
2+
io.opentelemetry.javaagent.testing.provider.TestBridgeMetricExporterComponentProvider
3+
io.opentelemetry.javaagent.testing.provider.TestBridgeLogRecordExporterComponentProvider
4+

0 commit comments

Comments
 (0)