[YARN-2799] cleanup TestLogAggregationService based on the change in YARN-90

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.7.0
• Component/s: test
• Hadoop Flags: Reviewed
• Assignee: Zhihai Xu
• Reporter: Zhihai Xu
• Created: 02/Nov/14 21:40
• Updated: 25/Oct/19 20:25
• Resolved: 20/Feb/15 17:31

Description

cleanup TestLogAggregationService based on the change in YARN-90.
The following code is added to setup in YARN-90,

    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.

Issue Links

• relates to: YARN-90 NodeManager should identify failed disks becoming good again