return empty map rather than nil

canonical · Aug 29, 2024 · 896f58d · 896f58d
1 parent f594b5f
commit 896f58d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/internal/jimm/cloudcredential.go b/internal/jimm/cloudcredential.go
@@ -342,6 +342,9 @@ func (j *JIMM) GetCloudCredentialAttributes(ctx context.Context, user *openfga.U
 		err = errors.E(op, err)
 		return
 	}
+	if len(attrs) == 0 {
+		return map[string]string{}, nil, nil
+	}
 
 	if hidden {
 		return

diff --git a/internal/jimm/cloudcredential_test.go b/internal/jimm/cloudcredential_test.go
@@ -1575,7 +1575,7 @@ var getCloudCredentialAttributesTests = []struct {
 	username:         "[email protected]",
 	jimmAdmin:        true,
 	cred:             "cred-2",
-	expectAttributes: nil,
+	expectAttributes: map[string]string{},
 	expectRedacted:   nil,
 }, {
 	name:     "OwnerWithHidden",