cleanup TestLogAggregationService based on the change in
The following code is added to setup in
dispatcher = createDispatcher(); appEventHandler = mock(EventHandler.class); dispatcher.register(ApplicationEventType.class, appEventHandler);
In this case, we should remove all these code from each test function to avoid duplicate code.
Same for dispatcher.stop() which is in tearDown,
we can remove dispatcher.stop() from from each test function also because it will always be called from tearDown for each test.
- relates to
YARN-90 NodeManager should identify failed disks becoming good again