remove redundant

Author: trask

instrumentation/logback/logback-appender-1.0/library/src/test/java/io/opentelemetry/instrumentation/logback/appender/v1_0/AbstractOpenTelemetryAppenderTest.java

@@ -6,39 +6,21 @@
 package io.opentelemetry.instrumentation.logback.appender.v1_0;
 
 import static io.opentelemetry.instrumentation.testing.junit.code.SemconvCodeStabilityUtil.codeAttributesLogCount;
-import static io.opentelemetry.instrumentation.testing.junit.code.SemconvCodeStabilityUtil.codeFileAndLineAssertions;
-import static io.opentelemetry.instrumentation.testing.junit.code.SemconvCodeStabilityUtil.codeFunctionAssertions;
-import static io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions.assertThat;
 import static io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions.equalTo;
-import static io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions.satisfies;
-import static io.opentelemetry.semconv.ExceptionAttributes.EXCEPTION_MESSAGE;
-import static io.opentelemetry.semconv.ExceptionAttributes.EXCEPTION_STACKTRACE;
-import static io.opentelemetry.semconv.ExceptionAttributes.EXCEPTION_TYPE;
 
 import ch.qos.logback.classic.LoggerContext;
 import ch.qos.logback.core.ContextBase;
 import io.opentelemetry.api.common.AttributeKey;
-import io.opentelemetry.api.logs.Severity;
 import io.opentelemetry.instrumentation.testing.junit.InstrumentationExtension;
 import io.opentelemetry.sdk.common.InstrumentationScopeInfo;
-import io.opentelemetry.sdk.logs.data.LogRecordData;
 import io.opentelemetry.sdk.resources.Resource;
-import io.opentelemetry.sdk.testing.assertj.AttributeAssertion;
 import java.lang.reflect.Field;
 import java.lang.reflect.Method;
-import java.time.Instant;
-import java.util.Collections;
-import java.util.List;
 import java.util.Map;
-import java.util.concurrent.TimeUnit;
-import org.assertj.core.api.AssertAccess;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import org.slf4j.MDC;
-import org.slf4j.Marker;
-import org.slf4j.MarkerFactory;
 
 abstract class AbstractOpenTelemetryAppenderTest {
 
@@ -80,91 +62,6 @@ static void resetLoggerContext() {
 
   protected abstract InstrumentationExtension getTesting();
 
-  @Test
-  void logNoSpan() {
-    logger.info("log message 1");
-
-    executeAfterLogsExecution();
-
-    getTesting()
-        .waitAndAssertLogRecords(
-            logRecord ->
-                logRecord
-                    .hasResource(resource)
-                    .hasInstrumentationScope(instrumentationScopeInfo)
-                    .hasBody("log message 1")
-                    .hasTotalAttributeCount(codeAttributesLogCount()));
-  }
-
-  @Test
-  void logWithExtras() {
-    Instant start = Instant.now();
-    String markerName = "aMarker";
-    Marker marker = MarkerFactory.getMarker(markerName);
-    logger.info(marker, "log message 1", new IllegalStateException("Error!"));
-
-    executeAfterLogsExecution();
-
-    List<AttributeAssertion> assertions =
-        codeFunctionAssertions(AbstractOpenTelemetryAppenderTest.class, "logWithExtras");
-    assertions.addAll(
-        codeFileAndLineAssertions(
-            AbstractOpenTelemetryAppenderTest.class.getSimpleName() + ".java"));
-    assertions.add(equalTo(EXCEPTION_TYPE, IllegalStateException.class.getName()));
-    assertions.add(equalTo(EXCEPTION_MESSAGE, "Error!"));
-    assertions.add(
-        satisfies(EXCEPTION_STACKTRACE, stackTrace -> stackTrace.contains("logWithExtras")));
-    assertions.add(
-        equalTo(
-            AttributeKey.stringArrayKey("logback.marker"), Collections.singletonList(markerName)));
-
-    Instant now = Instant.now();
-    getTesting()
-        .waitAndAssertLogRecords(
-            logRecord -> {
-              logRecord
-                  .hasResource(resource)
-                  .hasInstrumentationScope(instrumentationScopeInfo)
-                  .hasBody("log message 1")
-                  .hasSeverity(Severity.INFO)
-                  .hasSeverityText("INFO")
-                  .hasAttributesSatisfyingExactly(assertions);
-
-              LogRecordData logRecordData = AssertAccess.getActual(logRecord);
-              assertThat(logRecordData.getTimestampEpochNanos())
-                  .isGreaterThanOrEqualTo(TimeUnit.MILLISECONDS.toNanos(start.toEpochMilli()))
-                  .isLessThanOrEqualTo(
-                      TimeUnit.SECONDS.toNanos(now.getEpochSecond()) + now.getNano());
-            });
-  }
-
-  @Test
-  void logContextData() {
-    MDC.put("key1", "val1");
-    MDC.put("key2", "val2");
-    MDC.put("event.name", "MyEventName");
-    try {
-      logger.info("log message 1");
-    } finally {
-      MDC.clear();
-    }
-
-    executeAfterLogsExecution();
-
-    getTesting()
-        .waitAndAssertLogRecords(
-            logRecord ->
-                logRecord
-                    .hasResource(resource)
-                    .hasInstrumentationScope(instrumentationScopeInfo)
-                    .hasBody("log message 1")
-                    .hasTotalAttributeCount(2 + codeAttributesLogCount()) // code attributes
-                    .hasEventName("MyEventName")
-                    .hasAttributesSatisfying(
-                        equalTo(AttributeKey.stringKey("key1"), "val1"),
-                        equalTo(AttributeKey.stringKey("key2"), "val2")));
-  }
-
   @Test
   void logLoggerContext() {
     LoggerContext loggerContext = (LoggerContext) LoggerFactory.getILoggerFactory();