From 89923ebe7090038c57ee3ad23eca86858c9c2eca Mon Sep 17 00:00:00 2001 From: kolaente Date: Fri, 20 Mar 2026 10:43:18 +0100 Subject: [PATCH] fix: update test expectations for new disabled user fixture - TestListUsers expects 17 users (was 16) - TestCleanupOldTokens expects 3 old tokens deleted (was 2) --- pkg/models/user_list_test.go | 4 ++-- pkg/user/user_test.go | 7 ++++--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/pkg/models/user_list_test.go b/pkg/models/user_list_test.go index cc2301dd5..616818b7c 100644 --- a/pkg/models/user_list_test.go +++ b/pkg/models/user_list_test.go @@ -58,7 +58,7 @@ func TestListUsers(t *testing.T) { all, err := user.ListAllUsers(s) require.NoError(t, err) - assert.Len(t, all, 16) + assert.Len(t, all, 17) }) t.Run("no search term", func(t *testing.T) { db.LoadAndAssertFixtures(t) @@ -171,7 +171,7 @@ func TestListUsers(t *testing.T) { MatchFuzzily: true, }) require.NoError(t, err) - assert.Len(t, all, 16) + assert.Len(t, all, 17) }) // External team discoverability bypass tests diff --git a/pkg/user/user_test.go b/pkg/user/user_test.go index 7f0ab5cb6..6bfb3f771 100644 --- a/pkg/user/user_test.go +++ b/pkg/user/user_test.go @@ -517,9 +517,10 @@ func TestCleanupOldTokens(t *testing.T) { deleted, err := CleanupOldTokens(s) require.NoError(t, err) - // Fixtures have two old tokens that should be cleaned up: - // id=1 (kind=1, TokenPasswordReset, created 2021) and id=4 (kind=3, TokenAccountDeletion, created 2021) - assert.Equal(t, int64(2), deleted) + // Fixtures have three old tokens that should be cleaned up: + // id=1 (kind=1, TokenPasswordReset, created 2021), id=4 (kind=3, TokenAccountDeletion, created 2021), + // and id=5 (kind=1, TokenPasswordReset for disabled user, created 2024) + assert.Equal(t, int64(3), deleted) err = s.Commit() require.NoError(t, err)