[YUNIKORN-2089] Move usedResource type and tests to their own files

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.5.0
• Component/s: core - common
• Labels: newbie pull-request-available
• Target Version: 1.5.0
• Assignee: Vinayak Hegde
• Reporter: Wilfred Spiegelenburg
• Created: 30/Oct/23 06:31
• Updated: 20/Mar/24 14:32
• Resolved: 27/Nov/23 15:43

Description

The UsedResource type should not be part of the resources.go file.

All declarations and functions that have a UsedResource as receiver or return value value should be moved. Unused functions should be removed.

Unneeded exports on functions / types should be removed. If exported the comments on the functions and types need to comply with standard go rules.

All remaining functions should get tests to make sure they are working correctly.

Additionally, implement Stringer interface so that zap.Any() calls can be replaced with zap.Stringer() to avoid reflection.

Issue Links

• links to: GitHub Pull Request #713