refine tests

Author: trask

sdk/logs/src/test/java/io/opentelemetry/sdk/logs/SdkLogRecordBuilderTest.java

@@ -124,75 +124,20 @@ void emit_NoFields() {
   }
 
   @Test
-  void emit_WithMinimumSeverityConfiguration() {
-    LoggerConfig config =
-        LoggerConfig.builder().setMinimumSeverity(Severity.INFO.getSeverityNumber()).build();
-    SdkLogger logger = new SdkLogger(loggerSharedState, SCOPE_INFO, config);
-    builder = new SdkLogRecordBuilder(loggerSharedState, SCOPE_INFO, logger);
-
-    builder.setBody("too-low").setSeverity(Severity.DEBUG).emit();
-    assertThat(emittedLog.get()).isNull();
-
-    builder.setBody("allowed").setSeverity(Severity.INFO).emit();
-    assertThat(emittedLog.get().toLogRecordData()).hasBody("allowed");
-  }
-
-  @Test
-  void emit_DropsUnsampledTraceWhenTraceBased() {
-    LoggerConfig config = LoggerConfig.builder().setTraceBased(true).build();
-    SdkLogger logger = new SdkLogger(loggerSharedState, SCOPE_INFO, config);
-    builder = new SdkLogRecordBuilder(loggerSharedState, SCOPE_INFO, logger);
-
-    SpanContext unsampledSpanContext =
-        SpanContext.create(
-            "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa",
-            "bbbbbbbbbbbbbbbb",
-            TraceFlags.getDefault(),
-            TraceState.getDefault());
-    builder
-        .setBody("unsampled")
-        .setContext(Span.wrap(unsampledSpanContext).storeInContext(Context.root()))
-        .emit();
-    assertThat(emittedLog.get()).isNull();
-
-    SpanContext sampledSpanContext =
-        SpanContext.create(
-            "cccccccccccccccccccccccccccccccc",
-            "dddddddddddddddd",
-            TraceFlags.getSampled(),
-            TraceState.getDefault());
-    builder
-        .setBody("sampled")
-        .setContext(Span.wrap(sampledSpanContext).storeInContext(Context.root()))
-        .emit();
-    assertThat(emittedLog.get().toLogRecordData())
-        .hasSpanContext(sampledSpanContext)
-        .hasBody("sampled");
-  }
-
-  @Test
-  void emit_AllowsUndefinedSeverityWithMinimumSeverity() {
+  void emit_RespectsLoggerIsEnabled() {
+    // Test that emit() respects the logger's isEnabled() check
     LoggerConfig config =
         LoggerConfig.builder().setMinimumSeverity(Severity.WARN.getSeverityNumber()).build();
     SdkLogger logger = new SdkLogger(loggerSharedState, SCOPE_INFO, config);
     builder = new SdkLogRecordBuilder(loggerSharedState, SCOPE_INFO, logger);
 
-    // Undefined severity should bypass the minimum severity filter
-    builder.setBody("undefined-severity").setSeverity(Severity.UNDEFINED_SEVERITY_NUMBER).emit();
-    assertThat(emittedLog.get().toLogRecordData())
-        .hasBody("undefined-severity")
-        .hasSeverity(Severity.UNDEFINED_SEVERITY_NUMBER);
-  }
-
-  @Test
-  void emit_AllowsNoTraceContextWithTraceBased() {
-    LoggerConfig config = LoggerConfig.builder().setTraceBased(true).build();
-    SdkLogger logger = new SdkLogger(loggerSharedState, SCOPE_INFO, config);
-    builder = new SdkLogRecordBuilder(loggerSharedState, SCOPE_INFO, logger);
+    // Below minimum severity - should be dropped
+    builder.setBody("dropped").setSeverity(Severity.INFO).emit();
+    assertThat(emittedLog.get()).isNull();
 
-    // No trace context should bypass the trace-based filter
-    builder.setBody("no-trace-context").emit();
-    assertThat(emittedLog.get().toLogRecordData()).hasBody("no-trace-context");
+    // At minimum severity - should be emitted
+    builder.setBody("emitted").setSeverity(Severity.WARN).emit();
+    assertThat(emittedLog.get().toLogRecordData()).hasBody("emitted");
   }
 
   @Test

sdk/logs/src/test/java/io/opentelemetry/sdk/logs/SdkLoggerTest.java

@@ -141,20 +141,7 @@ void logRecordBuilder_AfterShutdown() {
   }
 
   @Test
-  void updateEnabled() {
-    LogRecordProcessor logRecordProcessor = mock(LogRecordProcessor.class);
-    SdkLoggerProvider loggerProvider =
-        SdkLoggerProvider.builder().addLogRecordProcessor(logRecordProcessor).build();
-    SdkLogger logger = (SdkLogger) loggerProvider.get("test");
-
-    logger.updateLoggerConfig(LoggerConfig.disabled());
-    assertThat(logger.isEnabled(Severity.UNDEFINED_SEVERITY_NUMBER, Context.current())).isFalse();
-    logger.updateLoggerConfig(LoggerConfig.enabled());
-    assertThat(logger.isEnabled(Severity.UNDEFINED_SEVERITY_NUMBER, Context.current())).isTrue();
-  }
-
-  @Test
-  void updateLoggerConfig_UpdatesAllFields() {
+  void updateLoggerConfig() {
     LogRecordProcessor logRecordProcessor = mock(LogRecordProcessor.class);
     SdkLoggerProvider loggerProvider =
         SdkLoggerProvider.builder().addLogRecordProcessor(logRecordProcessor).build();
@@ -237,44 +224,31 @@ void isEnabled_TraceBased() {
   }
 
   @Test
-  void isEnabled_MinimumSeverityAndTraceBased() {
+  void isEnabled_AllowsUndefinedSeverityWithMinimumSeverity() {
     LogRecordProcessor logRecordProcessor = mock(LogRecordProcessor.class);
     SdkLoggerProvider loggerProvider =
         SdkLoggerProvider.builder().addLogRecordProcessor(logRecordProcessor).build();
     SdkLogger logger = (SdkLogger) loggerProvider.get("test");
 
     LoggerConfig config =
-        LoggerConfig.builder()
-            .setMinimumSeverity(Severity.WARN.getSeverityNumber())
-            .setTraceBased(true)
-            .build();
+        LoggerConfig.builder().setMinimumSeverity(Severity.WARN.getSeverityNumber()).build();
     logger.updateLoggerConfig(config);
 
-    SpanContext sampledSpanContext =
-        SpanContext.create(
-            "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa",
-            "bbbbbbbbbbbbbbbb",
-            TraceFlags.getSampled(),
-            TraceState.getDefault());
-    Context sampledContext = Span.wrap(sampledSpanContext).storeInContext(Context.root());
-
-    SpanContext unsampledSpanContext =
-        SpanContext.create(
-            "cccccccccccccccccccccccccccccccc",
-            "dddddddddddddddd",
-            TraceFlags.getDefault(),
-            TraceState.getDefault());
-    Context unsampledContext = Span.wrap(unsampledSpanContext).storeInContext(Context.root());
+    // Undefined severity should bypass the minimum severity filter
+    assertThat(logger.isEnabled(Severity.UNDEFINED_SEVERITY_NUMBER, Context.current())).isTrue();
+  }
 
-    // Below minimum severity in sampled trace - should be disabled
-    assertThat(logger.isEnabled(Severity.INFO, sampledContext)).isFalse();
+  @Test
+  void isEnabled_AllowsNoTraceContextWithTraceBased() {
+    LogRecordProcessor logRecordProcessor = mock(LogRecordProcessor.class);
+    SdkLoggerProvider loggerProvider =
+        SdkLoggerProvider.builder().addLogRecordProcessor(logRecordProcessor).build();
+    SdkLogger logger = (SdkLogger) loggerProvider.get("test");
 
-    // At or above minimum severity in sampled trace - should be enabled
-    assertThat(logger.isEnabled(Severity.WARN, sampledContext)).isTrue();
-    assertThat(logger.isEnabled(Severity.ERROR, sampledContext)).isTrue();
+    LoggerConfig config = LoggerConfig.builder().setTraceBased(true).build();
+    logger.updateLoggerConfig(config);
 
-    // At or above minimum severity in unsampled trace - should be disabled
-    assertThat(logger.isEnabled(Severity.WARN, unsampledContext)).isFalse();
-    assertThat(logger.isEnabled(Severity.ERROR, unsampledContext)).isFalse();
+    // No trace context should bypass the trace-based filter
+    assertThat(logger.isEnabled(Severity.INFO, Context.current())).isTrue();
   }
 }