[ ] ~Figure out a better way to watch the vm.set eventual consistency~ Going to hold off on this for now. It seems that the current_operations field could be of use here
[x] make testacc passes
[x] Update documentation
[x] Add validation for high_availability (should only allow specific string values).
[x] Created #42 because I realized that the terraform provider does not correctly show jsonrpc errors to the user
# Test output
ddelnano@ddelnano-desktop:~/go/src/github.com/ddelnano/terraform-provider-xenorchestra$ make testacc
TF_ACC=1 go test ./... -v
? github.com/ddelnano/terraform-provider-xenorchestra [no test files]
=== RUN TestGetPIFByDevice
--- PASS: TestGetPIFByDevice (0.75s)
=== RUN TestPoolCompare
--- PASS: TestPoolCompare (0.00s)
=== RUN TestGetPoolByName
--- PASS: TestGetPoolByName (0.15s)
=== RUN TestStorageRepositoryCompare
--- PASS: TestStorageRepositoryCompare (0.00s)
=== RUN TestGetStorageRepositoryByType
--- PASS: TestGetStorageRepositoryByType (0.14s)
=== RUN TestGetTemplate
--- PASS: TestGetTemplate (0.21s)
=== RUN TestThatUnmarshalingWorks
--- PASS: TestThatUnmarshalingWorks (0.00s)
=== RUN TestUnmarshalingVmObject
--- PASS: TestUnmarshalingVmObject (0.00s)
PASS
ok github.com/ddelnano/terraform-provider-xenorchestra/client 1.256s
=== RUN TestAccXenorchestraDataSource_pool
--- PASS: TestAccXenorchestraDataSource_pool (1.13s)
=== RUN TestAccXenorchestraDataSource_pif
--- PASS: TestAccXenorchestraDataSource_pif (0.65s)
=== RUN TestAccXenorchestraDataSource_storageRepository
--- PASS: TestAccXenorchestraDataSource_storageRepository (0.68s)
=== RUN TestAccXenorchestraDataSource_storageRepositoryWithPoolId
--- PASS: TestAccXenorchestraDataSource_storageRepositoryWithPoolId (0.66s)
=== RUN TestAccXenorchestraDataSource_storageRepositoryWithNonExistantPoolId
--- PASS: TestAccXenorchestraDataSource_storageRepositoryWithNonExistantPoolId (0.80s)
=== RUN TestAccXenorchestraDataSource_template
--- PASS: TestAccXenorchestraDataSource_template (0.74s)
=== RUN TestAccXenorchestraCloudConfig_readAfterDelete
--- PASS: TestAccXenorchestraCloudConfig_readAfterDelete (1.69s)
=== RUN TestAccXenorchestraCloudConfig_create
--- PASS: TestAccXenorchestraCloudConfig_create (0.91s)
=== RUN TestAccXenorchestraCloudConfig_updateName
--- PASS: TestAccXenorchestraCloudConfig_updateName (1.51s)
=== RUN TestAccXenorchestraCloudConfig_updateTemplate
--- PASS: TestAccXenorchestraCloudConfig_updateTemplate (1.53s)
=== RUN TestAccXenorchestraCloudConfig_import
--- PASS: TestAccXenorchestraCloudConfig_import (1.07s)
=== RUN TestAccXenorchestraVm_create
--- PASS: TestAccXenorchestraVm_create (125.60s)
=== RUN TestAccXenorchestraVm_updatesWithoutReboot
--- PASS: TestAccXenorchestraVm_updatesWithoutReboot (182.81s)
PASS
ok github.com/ddelnano/terraform-provider-xenorchestra/xoa 319.809s
This is to address #35
Todo
current_operations
field could be of use heremake testacc
passeshigh_availability
(should only allow specific string values).