Why
Every test for internal/azure runs against a hand-written fake (fakeClient) implementing the Client interface. Nothing in this package has ever talked to a real Azure Key Vault. The rollback logic in particular makes non-trivial assumptions about version-listing behavior that deserve a real check.
What to verify against a real Azure Key Vault
Write path (KeyVault.WriteMany)
Rollback (KeyVault.Rollback)
Reference parsing
Auth
Scope
Needs a real (disposable) Azure Key Vault and, ideally, access to a non-commercial Azure cloud instance to check the sovereign-cloud hostname patterns — if that's not available, at least confirm commercial Azure end-to-end and flag the others as still-unverified.
Why
Every test for
internal/azureruns against a hand-written fake (fakeClient) implementing theClientinterface. Nothing in this package has ever talked to a real Azure Key Vault. The rollback logic in particular makes non-trivial assumptions about version-listing behavior that deserve a real check.What to verify against a real Azure Key Vault
Write path (
KeyVault.WriteMany)ContentType,Tags,SecretAttributespreserved from the existing version) — confirm Key Vault actually returns these fields the wayazsecrets.GetSecretResponseis assumed to shape them.statusCode(err, 404)) creates the secret correctly.Rollback (
KeyVault.Rollback)ListSecretVersions+ sort-by-Created-time-then-version — confirm real version metadata is always fully populated (ID,ID.Version(),Attributes.Created). The code deliberately fails closed (InvalidState) on any version missing this data rather than guessing at ordering — verify that's a rare/never case in practice, not something that'll routinely block rollback on real vaults (e.g. after a soft-delete/recover cycle, or a version created through the Azure portal rather than the API).Reference parsing
canonicalVaultHost— confirm real vault hostnames actually match for all four supported clouds (*.vault.azure.net,*.vault.usgovcloudapi.net,*.vault.azure.cn,*.vault.microsoftazure.de), ideally against a real vault in at least the Gov or China cloud if accessible, not just commercial Azure.Auth
azidentity.NewDefaultAzureCredential(nil)resolution against a real managed identity,az loginsession, and service-principal environment-variable combination.Scope
Needs a real (disposable) Azure Key Vault and, ideally, access to a non-commercial Azure cloud instance to check the sovereign-cloud hostname patterns — if that's not available, at least confirm commercial Azure end-to-end and flag the others as still-unverified.