Fix permanent diff on vault_rabbitmq_secret_backend_role

[larrywax]

Description

This PR fixes #952
Add sorting for vhosts after read
Add test

Relates OR Closes #952

Checklist

• Added CHANGELOG entry (only for user-facing changes)
• Acceptance tests where run against all supported Vault Versions

Output from acceptance testing:

$ make test TESTARGS='-v -run=TestFlattenRabbitMQSecretBackendRoleVhost_Order'                                                        (orbstack)
==> Checking that code complies with gofmt requirements...
TF_ACC= VAULT_TOKEN= go test -v -run=TestFlattenRabbitMQSecretBackendRoleVhost_Order -timeout 10m ./...
?   	github.com/hashicorp/terraform-provider-vault	[no test files]
?   	github.com/hashicorp/terraform-provider-vault/cmd/coverage	[no test files]
?   	github.com/hashicorp/terraform-provider-vault/cmd/generate	[no test files]
testing: warning: no tests to run
PASS
ok  	github.com/hashicorp/terraform-provider-vault/codegen	(cached) [no tests to run]
?   	github.com/hashicorp/terraform-provider-vault/helper	[no test files]
?   	github.com/hashicorp/terraform-provider-vault/internal/consts	[no test files]
testing: warning: no tests to run
PASS
ok  	github.com/hashicorp/terraform-provider-vault/internal/identity/entity	(cached) [no tests to run]
?   	github.com/hashicorp/terraform-provider-vault/internal/identity/group	[no test files]
?   	github.com/hashicorp/terraform-provider-vault/internal/identity/mfa	[no test files]
?   	github.com/hashicorp/terraform-provider-vault/internal/pki	[no test files]
testing: warning: no tests to run
PASS
ok  	github.com/hashicorp/terraform-provider-vault/internal/provider	(cached) [no tests to run]
?   	github.com/hashicorp/terraform-provider-vault/internal/rotation	[no test files]
?   	github.com/hashicorp/terraform-provider-vault/internal/sync	[no test files]
?   	github.com/hashicorp/terraform-provider-vault/schema	[no test files]
testing: warning: no tests to run
PASS
ok  	github.com/hashicorp/terraform-provider-vault/testutil	(cached) [no tests to run]
testing: warning: no tests to run
PASS
ok  	github.com/hashicorp/terraform-provider-vault/util	(cached) [no tests to run]
testing: warning: no tests to run
PASS
ok  	github.com/hashicorp/terraform-provider-vault/util/mountutil	(cached) [no tests to run]
=== RUN   TestFlattenRabbitMQSecretBackendRoleVhost_Order
--- PASS: TestFlattenRabbitMQSecretBackendRoleVhost_Order (0.00s)
PASS
ok  	github.com/hashicorp/terraform-provider-vault/vault	2.700s

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

[hashicorp-cla-app]

All committers have signed the CLA.

[hashicorp-cla-app]

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

---

0 out of 2 committers have signed the CLA.

• Giuseppe D'Anna
• larrywax

---

Giuseppe D'Anna seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA.
If you have already a GitHub account, please add the email address used for this commit to your account.

_{Have you signed the CLA already but the status is still pending? Recheck it.}

[larrywax]

@AnPucel I've signed CLA but one of the message says the opposite. Is that a problem?

[Copilot]

Pull Request Overview

This PR fixes a permanent diff issue in the vault_rabbitmq_secret_backend_role resource by ensuring consistent ordering of vhosts. The fix prevents unnecessary state diffs when the order of vhosts changes between Terraform operations.

• Adds deterministic sorting to vhost processing in the flatten function
• Includes a unit test to verify the sorting behavior works correctly

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
vault/resource_rabbitmq_secret_backend_role.go	Modifies vhost flattening to sort keys before processing, ensuring consistent output order
vault/resource_rabbitmq_secret_backend_role_test.go	Adds unit test to verify vhosts are returned in alphabetical order

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}