[YARN-11038] Fix testQueueSubmitWithACL* tests in TestAppManager

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.4.0
• Component/s: yarn
• Labels: pull-request-available
• Hadoop Flags: Reviewed
• Assignee: Tamas Domok
• Reporter: Tamas Domok
• Created: 07/Dec/21 14:03
• Updated: 08/Dec/21 15:42
• Resolved: 08/Dec/21 15:41

Description

These two tests doesn't test anything:

 - testQueueSubmitWithACLsEnabledWithQueueMapping

 - testQueueSubmitWithACLsEnabledWithQueueMappingForAutoCreatedQueue

 

Issues:

 - no assert if the exception is not thrown

 - the configuration doesn't even loaded (csConf is not used for the mockRM)

 - the placement manager did not even match the test scenario

 - the successful submit was not asserted either

 

Issue Links

• links to: GitHub Pull Request #3759