From 0fc5cd5a3372a3819b9282f97996d330b8d29d61 Mon Sep 17 00:00:00 2001 From: Timothy Messier Date: Fri, 1 Apr 2022 08:57:21 -0400 Subject: [PATCH] test(account): Use require to prevent panic on test failure If these assertions failed, then next assersions would result in a panic when accessing the slice. --- internal/tests/api/accounts/account_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/tests/api/accounts/account_test.go b/internal/tests/api/accounts/account_test.go index ee84c2704f..671d0af8c8 100644 --- a/internal/tests/api/accounts/account_test.go +++ b/internal/tests/api/accounts/account_test.go @@ -63,7 +63,7 @@ func TestListPassword(t *testing.T) { ulResult, err = accountClient.List(tc.Context(), am.Id, accounts.WithFilter(fmt.Sprintf(`"/item/attributes/login_name"==%q`, filterItem.Attributes["login_name"]))) require.NoError(err) - assert.Len(ulResult.Items, 1) + require.Len(ulResult.Items, 1) assert.Equal(filterItem.Id, ulResult.Items[0].Id) } @@ -120,7 +120,7 @@ func TestListOidc(t *testing.T) { ulResult, err = accountClient.List(tc.Context(), am.Id, accounts.WithFilter(fmt.Sprintf(`"/item/attributes/subject"==%q`, filterItem.Attributes["subject"]))) require.NoError(err) - assert.Len(ulResult.Items, 1) + require.Len(ulResult.Items, 1) assert.Equal(filterItem.Id, ulResult.Items[0].Id) }