mirror of
https://github.com/tiennm99/java-design-patterns.git
synced 2026-05-23 20:26:50 +00:00
docs: update event aggregator docs
This commit is contained in:
@@ -35,7 +35,6 @@ class AppTest {
|
||||
|
||||
/**
|
||||
* Issue: Add at least one assertion to this test case.
|
||||
*
|
||||
* Solution: Inserted assertion to check whether the execution of the main method in {@link App#main(String[])}
|
||||
* throws an exception.
|
||||
*/
|
||||
|
||||
@@ -67,14 +67,14 @@ class KingJoffreyTest {
|
||||
assertEquals(i, appender.getLogSize());
|
||||
var event = Event.values()[i];
|
||||
kingJoffrey.onEvent(event);
|
||||
final var expectedMessage = "Received event from the King's Hand: " + event.toString();
|
||||
final var expectedMessage = "Received event from the King's Hand: " + event;
|
||||
assertEquals(expectedMessage, appender.getLastMessage());
|
||||
assertEquals(i + 1, appender.getLogSize());
|
||||
});
|
||||
|
||||
}
|
||||
|
||||
private class InMemoryAppender extends AppenderBase<ILoggingEvent> {
|
||||
private static class InMemoryAppender extends AppenderBase<ILoggingEvent> {
|
||||
private final List<ILoggingEvent> log = new LinkedList<>();
|
||||
|
||||
public InMemoryAppender(Class<?> clazz) {
|
||||
|
||||
@@ -49,11 +49,11 @@ class KingsHandTest extends EventEmitterTest<KingsHand> {
|
||||
|
||||
/**
|
||||
* The {@link KingsHand} is both an {@link EventEmitter} as an {@link EventObserver} so verify if
|
||||
* every event received is passed up to it's superior, in most cases {@link KingJoffrey} but now
|
||||
* every event received is passed up to its superior, in most cases {@link KingJoffrey} but now
|
||||
* just a mocked observer.
|
||||
*/
|
||||
@Test
|
||||
void testPassThrough() throws Exception {
|
||||
void testPassThrough() {
|
||||
final var observer = mock(EventObserver.class);
|
||||
final var kingsHand = new KingsHand();
|
||||
kingsHand.registerObserver(observer, Event.STARK_SIGHTED);
|
||||
|
||||
Reference in New Issue
Block a user