aptible / terraform-provider-aptible

The official Terraform provider for Aptible Deploy
https://registry.terraform.io/providers/aptible/aptible/latest
10 stars 13 forks source link

Fix connection_url acceptance tests #29

Closed robertfairhead closed 4 years ago

robertfairhead commented 4 years ago

When I changed the connection_url to default_connection_url to allow for finding multiple credentials, I did not update all of the tests. This fixes those tests to refer to the correct value.

Also, I updated replicas to use the new field name for consistency. I did not update them to return multiple credentials yet. I plan to update replicas to do that once we have the "real" fix to correct the generated client.

robertfairhead commented 4 years ago

I'm running the DB acceptance tests against my sandbox to confirm this fix. I'll open for full review once I've confirmed they work.