Add comments back
Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/65706c0e Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/65706c0e Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/65706c0e Branch: refs/heads/master Commit: 65706c0e4b7d08c430e62d8cf5cc7dc4b4f77368 Parents: bb8f2ee Author: Mikael Ståldal <[email protected]> Authored: Sat Jul 8 21:38:14 2017 +0200 Committer: Mikael Ståldal <[email protected]> Committed: Sat Jul 8 21:38:14 2017 +0200 ---------------------------------------------------------------------- .../org/apache/logging/log4j/ThreadContextUtilityClass.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/65706c0e/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java ---------------------------------------------------------------------- diff --git a/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java b/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java index 21a9951..a5603c5 100644 --- a/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java +++ b/log4j-api/src/test/java/org/apache/logging/log4j/ThreadContextUtilityClass.java @@ -53,15 +53,20 @@ public class ThreadContextUtilityClass { ThreadContext.clearMap(); final Map<String, String> map1 = ThreadContext.getContext(); assertTrue(map1.isEmpty()); - map1.put("K", "val"); + map1.put("K", "val"); // no error assertEquals("val", map1.get("K")); + + // adding to copy does not affect thread context map assertTrue(ThreadContext.getContext().isEmpty()); + ThreadContext.put("key", "val2"); final Map<String, String> map2 = ThreadContext.getContext(); assertEquals(1, map2.size()); assertEquals("val2", map2.get("key")); - map2.put("K", "val"); + map2.put("K", "val"); // no error assertEquals("val", map2.get("K")); + + // first copy is not affected assertNotSame(map1, map2); assertEquals(1, map1.size()); }
