I'm not a logging expert, but this change Looks Good To Me.

---

A bunch of "handler" should be changed to "handlers"

+        // Verify that exactly one of the two handler is a custom.Handler
+        // Verify that exactly one of the two handler is a
custom.DotHandler
+        // Verify that the two handler have an id of '1'

--

This code makes me think you use the identity of the Longs, but it seems
you don't, so I would just get rid of these and use e.g. 3L instead of
THREE.

+    public static final Long ONE = 1L;
+    public static final Long TWO = 2L;
+    public static final Long THREE = 3L;

Reply via email to