[YARN-10256] Refactor TestContainerSchedulerQueuing.testContainerUpdateExecTypeGuaranteedToOpportunistic

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.2.2, 3.4.0, 3.3.1
• Labels: refactoring unit-test
• Assignee: Ahmed Hussein
• Reporter: Ahmed Hussein
• Created: 30/Apr/20 21:27
• Updated: 10/Jun/21 08:14
• Resolved: 04/May/20 16:33

Description

In 3.x, TestContainerSchedulerQueuing.testContainerUpdateExecTypeGuaranteedToOpportunistic has redundant assertions. Since the UT throws timeout exception, GenericTestsUtils.waitFor() guarantees that the predicate is met successfully. Otherwise, the UT would throw a timeout exception.
The redundant loop causes confusion in understanding the test unit and may increase the possibility of failure in case the container terminates