store/keychain: fix test on list credentials

Signed-off-by: Alano Terblanche <18033717+Benehiko@users.noreply.github.com>

Author: Benehiko

store/keychain/keychain_test.go

@@ -2,7 +2,6 @@ package keychain
 
 import (
 	"errors"
-	"slices"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -111,10 +110,16 @@ func TestKeychain(t *testing.T) {
 			"com.test.test/test/bob": {
 				Username: "bob",
 				Password: "bob-password",
+				Attributes: map[string]string{
+					"color": "green",
+				},
 			},
 			"com.test.test/test/jeff": {
 				Username: "jeff",
 				Password: "jeff-password",
+				Attributes: map[string]string{
+					"game": "elden ring",
+				},
 			},
 			"com.test.test/test/pete": {
 				Username: "pete",
@@ -134,18 +139,29 @@ func TestKeychain(t *testing.T) {
 		require.NoError(t, err)
 		require.Len(t, secrets, 3)
 
-		expected := make([]store.ID, 0, 3)
-		for k := range moreCreds {
-			expected = append(expected, k)
+		actual := make(map[store.ID]*mocks.MockCredential)
+		for k, v := range secrets {
+			actual[k] = v.(*mocks.MockCredential)
 		}
-		slices.Sort(expected)
 
-		actual := make([]store.ID, 0, 3)
-		for key := range secrets {
-			actual = append(actual, key)
+		expected := moreCreds
+		for k, v := range expected {
+			// listing credentials from the store won't retrieve the actual
+			// credentials, only the metadata.
+			// That is why we set username and password to empty
+			v.Username = ""
+			v.Password = ""
+
+			// the store sets some attributes internally, which we need to setup
+			// here on our expected map.
+			if v.Attributes == nil {
+				v.Attributes = make(map[string]string)
+			}
+			v.Attributes["id"] = k.String()
+			v.Attributes["service:group"] = "com.test.test"
+			v.Attributes["service:name"] = "test"
 		}
-		slices.Sort(actual)
-		require.Equal(t, expected, actual)
+		assert.EqualValues(t, expected, actual)
 	})
 
 	t.Run("filter credentials", func(t *testing.T) {