fix: do not expose empty cipassword to API

[jay7x]

Fixes Telmate/terraform-provider-proxmox#1097

[Tinyblargon]

@jay7x thank you for your contribution, could you add a test case to cement this behavior.

This is the test I'm referring to:

proxmox-api-go/proxmox/config_qemu_test.go

Line 16 in 032717b

func Test_ConfigQemu_mapToAPI(t *testing.T) {

Can you add the test case in the following category:

proxmox-api-go/proxmox/config_qemu_test.go

Line 431 in 032717b

{category: `CloudInit`, // Create CloudInit no need for update as update and create behave the same. will be changed in the future

Looking at it it's probably only a create case. If you want me to add the test case that's also possible.

[jay7x]

I'll try to add it this weekend :)

[jay7x]

Later is better than never! I added a case when cipassword should be deleted when set to an empty value. Not really sure it is meaningful at all, but it behaves the same as ciuser now.
@Tinyblargon let me know if you'd prefer to drop this latest behavior change or split the PR.

[Tinyblargon]

@jay7x I'll try to review it tomorrow. Setting the password to an empty string in the GUI also deletes it. Therefore, how you implement it is the correct behavior.

[Tinyblargon]

@jay7x everything looks good.