update kafka and api-jey tests

confluentinc · Sep 10, 2021 · 9876553 · 9876553
1 parent 1900962
commit 9876553
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/internal/cmd/api-key/command.go b/internal/cmd/api-key/command.go
@@ -216,7 +216,11 @@ func (c *command) list(cmd *cobra.Command, _ []string) error {
 			return errors.New(errors.BadServiceAccountIDErrorMsg)
 		}
 		userIdMap := c.mapResourceIdToUserId(allUsers)
-		userId = userIdMap[serviceAccountID]
+		var ok bool
+		userId, ok = userIdMap[serviceAccountID]
+		if !ok {
+			return errors.NewErrorWithSuggestions(fmt.Sprintf(errors.ServiceAccountNotFoundErrorMsg, serviceAccountID), errors.ServiceAccountNotFoundSuggestions)
+		}
 	}
 
 	currentUser, err := cmd.Flags().GetBool("current-user")

diff --git a/test/fixtures/output/kafka/kafka-acls-list-json.golden b/test/fixtures/output/kafka/kafka-acls-list-json.golden
@@ -3,7 +3,7 @@
     "operation": "READ",
     "pattern_type": "LITERAL",
     "permission": "ALLOW",
-    "principal": "User:sa-123",
+    "principal": "User:sa-12345",
     "resource_name": "test-topic",
     "resource_type": "TOPIC"
   }

diff --git a/test/fixtures/output/kafka/kafka-acls-list-yaml.golden b/test/fixtures/output/kafka/kafka-acls-list-yaml.golden
@@ -1,6 +1,6 @@
 - operation: READ
   pattern_type: LITERAL
   permission: ALLOW
-  principal: User:sa-123
+  principal: User:sa-12345
   resource_name: test-topic
   resource_type: TOPIC