From 79d73f77f5f54a7de4bccf793ca6ac45655c9f65 Mon Sep 17 00:00:00 2001 From: Kyle Carberry Date: Tue, 28 May 2024 11:45:41 -0500 Subject: [PATCH] chore: skip Azure `TestExpiresSoon` (#13385) Adds some context to the test skip so it can be removed or enabled in the future. --- coderd/azureidentity/azureidentity_test.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/coderd/azureidentity/azureidentity_test.go b/coderd/azureidentity/azureidentity_test.go index 854f95a6e4..bd55ae2538 100644 --- a/coderd/azureidentity/azureidentity_test.go +++ b/coderd/azureidentity/azureidentity_test.go @@ -64,6 +64,10 @@ func TestValidate(t *testing.T) { func TestExpiresSoon(t *testing.T) { t.Parallel() + // TODO (@kylecarbs): It's unknown why Microsoft does not have new certificates live... + // The certificate is automatically fetched if it's not found in our database, + // so in a worst-case scenario expired certificates will only impact 100% airgapped users. + t.Skip() const threshold = 1 for _, c := range azureidentity.Certificates {