From 19cfa2b3a760e9c0cfa271c01429413fe175308b Mon Sep 17 00:00:00 2001 From: Rafaela Soares Date: Mon, 9 Sep 2024 18:56:20 +0100 Subject: [PATCH] fix test --- pkg/banneduser/banneduser_test.go | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/pkg/banneduser/banneduser_test.go b/pkg/banneduser/banneduser_test.go index 5565d8e0..01647013 100644 --- a/pkg/banneduser/banneduser_test.go +++ b/pkg/banneduser/banneduser_test.go @@ -41,7 +41,7 @@ func TestNewBannedUser(t *testing.T) { name: "userSignup with email hash label", userSignup: userSignup1, bannedBy: "admin", - banReason: "ban reason", + banReason: "ban reason 1", wantError: false, wantErrorMsg: "", expectedBannedUser: &toolchainv1alpha1.BannedUser{ @@ -54,7 +54,8 @@ func TestNewBannedUser(t *testing.T) { }, }, Spec: toolchainv1alpha1.BannedUserSpec{ - Email: userSignup1.Spec.IdentityClaims.Email, + Email: userSignup1.Spec.IdentityClaims.Email, + Reason: "ban reason 1", }, }, }, @@ -62,7 +63,7 @@ func TestNewBannedUser(t *testing.T) { name: "userSignup without email hash label and phone hash label", userSignup: userSignup2, bannedBy: "admin", - banReason: "ban reason", + banReason: "ban reason 2", wantError: true, wantErrorMsg: fmt.Sprintf("the UserSignup %s doesn't have the label '%s' set", userSignup2.Name, toolchainv1alpha1.UserSignupUserEmailHashLabelKey), expectedBannedUser: nil, @@ -71,7 +72,7 @@ func TestNewBannedUser(t *testing.T) { name: "userSignup with email hash label and phone hash label", userSignup: userSignup3, bannedBy: "admin", - banReason: "ban reason", + banReason: "ban reason 3", wantError: false, wantErrorMsg: "", expectedBannedUser: &toolchainv1alpha1.BannedUser{ @@ -85,7 +86,8 @@ func TestNewBannedUser(t *testing.T) { }, }, Spec: toolchainv1alpha1.BannedUserSpec{ - Email: userSignup3.Spec.IdentityClaims.Email, + Email: userSignup3.Spec.IdentityClaims.Email, + Reason: "ban reason 3", }, }, },