appoptics / terraform-provider-appoptics

Apache License 2.0
4 stars 2 forks source link

GS-485: Get Services working and fix Alerts tests #18

Closed solarchad closed 4 years ago

solarchad commented 4 years ago

This changes the service resource to use the new SDK/client. It fixes alerts too so that it uses the AlertRequest struct (which allows services to be in []int form instead of []*Service form).

All service tests (and now alert tests) are passing (see below).

go test -v                                    
=== RUN   TestProvider
--- PASS: TestProvider (0.00s)
=== RUN   TestProvider_impl
--- PASS: TestProvider_impl (0.00s)
=== RUN   TestAccAppOpticsAlertMinimal
--- PASS: TestAccAppOpticsAlertMinimal (0.82s)
=== RUN   TestAccAppOpticsAlertBasic
--- PASS: TestAccAppOpticsAlertBasic (0.68s)
=== RUN   TestAccAppOpticsAlertFull
--- PASS: TestAccAppOpticsAlertFull (2.53s)
=== RUN   TestAccAppOpticsAlertUpdated
--- PASS: TestAccAppOpticsAlertUpdated (9.42s)
=== RUN   TestAccAppOpticsAlertRename
--- PASS: TestAccAppOpticsAlertRename (9.61s)
=== RUN   TestAccAppOpticsAlertFullUpdate
--- PASS: TestAccAppOpticsAlertFullUpdate (3.88s)
=== RUN   TestAccAppOpticsServiceBasic
--- PASS: TestAccAppOpticsServiceBasic (3.12s)
=== RUN   TestAccAppOpticsServiceUpdated
--- PASS: TestAccAppOpticsServiceUpdated (11.83s)