-
Notifications
You must be signed in to change notification settings - Fork 186
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: Allow registering custom logger handlers (#1156)
- Loading branch information
Showing
2 changed files
with
74 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,54 @@ | ||
package com.box.sdk; | ||
|
||
import static org.hamcrest.CoreMatchers.is; | ||
import static org.hamcrest.MatcherAssert.assertThat; | ||
import static org.hamcrest.Matchers.empty; | ||
|
||
import java.util.ArrayList; | ||
import java.util.List; | ||
import java.util.logging.Handler; | ||
import java.util.logging.LogRecord; | ||
import org.junit.Test; | ||
|
||
public class BoxLoggerTest { | ||
@Test | ||
public void canAddHandler() { | ||
HandlerForTests handler = new HandlerForTests(); | ||
BoxLogger boxLogger = BoxLogger.defaultLogger(); | ||
|
||
boxLogger.addHandler(handler); | ||
boxLogger.info("Test"); | ||
|
||
LogRecord logRecord = handler.logEntries.get(0); | ||
assertThat(logRecord.getMessage(), is("Test")); | ||
} | ||
|
||
@Test | ||
public void canRemoveHandler() { | ||
HandlerForTests handler = new HandlerForTests(); | ||
BoxLogger boxLogger = BoxLogger.defaultLogger(); | ||
|
||
boxLogger.addHandler(handler); | ||
boxLogger.removeHandler(handler); | ||
boxLogger.info("Test"); | ||
|
||
assertThat(handler.logEntries, empty()); | ||
} | ||
|
||
private static final class HandlerForTests extends Handler { | ||
List<LogRecord> logEntries = new ArrayList<>(); | ||
|
||
@Override | ||
public void publish(LogRecord record) { | ||
logEntries.add(record); | ||
} | ||
|
||
@Override | ||
public void flush() { | ||
} | ||
|
||
@Override | ||
public void close() throws SecurityException { | ||
} | ||
} | ||
} |