Add test explicitly demonstrating MDC behavior

typelevel · Jun 25, 2024 · 97ad066 · 97ad066
1 parent a808c2c
commit 97ad066
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/slf4j/src/test/scala/org/typelevel/log4cats/slf4j/internal/Slf4jLoggerInternalSuite.scala b/slf4j/src/test/scala/org/typelevel/log4cats/slf4j/internal/Slf4jLoggerInternalSuite.scala
@@ -141,6 +141,19 @@ class Slf4jLoggerInternalSuite extends CatsEffectSuite {
         )
   }
 
+  testLoggerFixture().test("Slf4jLoggerInternal ignores(?) values already in the MDC") { testLogger =>
+    IO.delay(MDC.put("baz", "bar")) >>
+      Slf4jLogger.getLoggerFromSlf4j[IO](testLogger).info(Map("foo" -> "bar"))("A log went here") >>
+      IO(testLogger.logs())
+        .map(toDeferredLogs)
+        .assertEquals(
+          List(
+            DeferredLogMessage.info(Map("foo" -> "bar"), none, () => "A log went here")
+          ),
+          clue("Context should not include baz->bar")
+        )
+  }
+
   testLoggerFixture(
     traceEnabled = false,
     debugEnabled = false,