Bug fix: `ConfigQemu`. `Node`, `Pool`, `VmID`

Tinyblargon commented 1 year ago

Noticed that #255 broke ConfigQemu. Node, Pool, VmID

Work Done:

Fixed bug a752d2201e949d272f94a2a1b5ddf81122330223
Added unit test 761523a724cda6c3e7aa31f658498799071258d9
Changed structure of existing unit test to allow for easier extension f5eb43dfade238b7836d26096ee2d6ce88d6e57d. The change was nesting all existing Disk tests into an array item so each top level parameter in ConfigQemu has it's own array entry. This way the item containing the Disk tasks can be collapsed making the test more readable.

mleone87 commented 11 months ago

@Tinyblargon cant merge this PR, too much mess in the resolve for test file, can you have a look?

Tinyblargon commented 11 months ago

I'll look into it. It's probably easier to redo the commit than merge master back into it.

Telmate / proxmox-api-go