[YUNIKORN-1520] TestApplyConf: poor unit test coverage

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.2.0
• Component/s: core - scheduler
• Labels: newbie pull-request-available
• Assignee: Hsuan Zong Wu
• Reporter: Wilfred Spiegelenburg
• Created: 28/Dec/22 05:10
• Updated: 07/Feb/23 22:33
• Resolved: 25/Jan/23 02:38

Description

The TestApplyConf does not cover the code in ApplyConf() as it always calls applyConf (the unlocked version).

The applyConf coverage is also minimal, it does not:

• cover any error cases
• cover dynamic to managed type change
• auto parent conversion based on children

Issue Links

• links to: GitHub Pull Request #495