Use set connection_url when available instead of value read for API response for MongoDB plugin

[marceloalmeida]

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Relates And Closes #215

Release note for CHANGELOG:

Use set connection_url when available instead of the value read for the API response for MongoDB

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccDatabaseSecretBackendConnection_mongodb'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v -run=TestAccDatabaseSecretBackendConnection_mongodb -timeout 120m
?   	github.com/terraform-providers/terraform-provider-vault	[no test files]
?   	github.com/terraform-providers/terraform-provider-vault/cmd/coverage	[no test files]
testing: warning: no tests to run
PASS
ok  	github.com/terraform-providers/terraform-provider-vault/util	(cached) [no tests to run]
=== RUN   TestAccDatabaseSecretBackendConnection_mongodb
--- PASS: TestAccDatabaseSecretBackendConnection_mongodb (5.88s)
PASS
ok  	github.com/terraform-providers/terraform-provider-vault/vault

[paulojmdias]

Awesome 👏

[tyrannosaurus-becks]

Hi @marceloalmeida , this PR states that it fixes #215 , but that issue is already closed by another PR. Can you elaborate on the issue you're fixing, with steps to reproduce it?

Also, if it's a permanent diff, will you look at #662 and see if it resolves it for you?

[marceloalmeida]

Hi @tyrannosaurus-becks, this PR states that fixes #215 because the PR #217 does not fix the issue for MongoDB.

Regarding #662 it resolves the issue that this PR also fixes.

[tyrannosaurus-becks]

Thanks for the clarification! I appreciate your work on this.