getGpusUsableByYarn has the following issues:
- Duplicate GPU device definitions are not denied: This seems to be the biggest issue as it could increase the number of devices on the node if the device ID is defined 2 or more times.
- An empty-string is accepted, it works like the user would not want to use auto-discovery and haven't defined any GPU devices: This will result in an empty device list, but the empty-string check is never explicitly there in the code, so this behavior just coincidental.
- Number validation does not happen on GPU device IDs (separated by commas)
Many testcases are added as the coverage was already very low.