From fabcb8eeebebcf430f0f4f65b061d485d5c17dac Mon Sep 17 00:00:00 2001 From: Jim Date: Wed, 21 Jun 2023 09:38:47 -0400 Subject: [PATCH] fix: update recursive auth-method listing test (#3348) Recently, we made the initial dev ldap auth-method active-public, so it now shows up in listings. This simply fixes the tests to reflect that change. --- internal/tests/cluster/recursive_anon_listing_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/internal/tests/cluster/recursive_anon_listing_test.go b/internal/tests/cluster/recursive_anon_listing_test.go index 04378f3fd7..a6830fa48c 100644 --- a/internal/tests/cluster/recursive_anon_listing_test.go +++ b/internal/tests/cluster/recursive_anon_listing_test.go @@ -32,18 +32,18 @@ func TestListAnonymousRecursing(t *testing.T) { require.NoError(err) require.NotNil(am) - // We expect to see all three with the normal token + // We expect to see all four with the normal token l, err := amClient.List(tc.Context(), scope.Global.String(), amapi.WithRecursive(true)) require.NoError(err) require.NotNil(l) require.Len(l.GetItems(), 4) - // Originally we also expect to see all three as anon user + // Originally we also expect to see all four as anon user amClient.ApiClient().SetToken("") l, err = amClient.List(tc.Context(), scope.Global.String(), amapi.WithRecursive(true)) require.NoError(err) require.NotNil(l) - require.Len(l.GetItems(), 3) + require.Len(l.GetItems(), 4) // Find the global roles and delete them rl, err := rolesClient.List(tc.Context(), scope.Global.String())