From ae37684a79f6930ec6cb6d4852b5d5874c35d23e Mon Sep 17 00:00:00 2001 From: Christian Richter Date: Wed, 5 Nov 2025 16:25:23 +0100 Subject: [PATCH] fix GetUserTest without tennantId Signed-off-by: Christian Richter --- services/graph/pkg/identity/cache/cache_test.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/services/graph/pkg/identity/cache/cache_test.go b/services/graph/pkg/identity/cache/cache_test.go index 10c76756dc..5cf238a536 100644 --- a/services/graph/pkg/identity/cache/cache_test.go +++ b/services/graph/pkg/identity/cache/cache_test.go @@ -39,7 +39,7 @@ var _ = Describe("Cache", func() { }) Describe("GetUser", func() { - FIt("should return no error", func() { + It("should return no error", func() { alan := &cs3User.User{ Id: &cs3User.UserId{ OpaqueId: "alan", @@ -54,7 +54,7 @@ var _ = Describe("Cache", func() { ru, err := idc.GetUser(ctx, "", "alan") Expect(err).To(BeNil()) Expect(ru).ToNot(BeNil()) - Expect(ru.GetId()).To(Equal(alan.GetId())) + Expect(ru.GetId()).To(Equal(alan.GetId().GetOpaqueId())) Expect(ru.GetDisplayName()).To(Equal(alan.GetDisplayName())) }) @@ -81,7 +81,9 @@ var _ = Describe("Cache", func() { DisplayName: "Alan", } // Persist the user to the cache for 1 hour - idc.users.Set(alan.GetId().GetTenantId()+"|"+alan.GetId().GetOpaqueId(), alan, 3600) + cu := idc.users.Set(alan.GetId().GetTenantId()+"|"+alan.GetId().GetOpaqueId(), alan, 3600) + // Test if element has been persisted in the cache + Expect(cu.Value().GetId().GetOpaqueId()).To(Equal(alan.GetId().GetOpaqueId())) ru, err := idc.GetUser(ctx, "1234", "alan") Expect(err).To(BeNil()) Expect(ru.GetDisplayName()).To(Equal(alan.GetDisplayName()))