Fix null role response from users fetch (#871) (#872)

gravitational · Jun 7, 2022 · bbb35a3 · bbb35a3
1 parent d4e2a99
commit bbb35a3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/packages/teleport/src/services/user/makeUser.ts b/packages/teleport/src/services/user/makeUser.ts
@@ -1,5 +1,5 @@
 /**
- * Copyright 2020 Gravitational, Inc.
+ * Copyright 2020-2022 Gravitational, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -21,7 +21,7 @@ export default function makeUser(json): User {
   const [name, roles, authType] = at(json, ['name', 'roles', 'authType']);
   return {
     name,
-    roles,
+    roles: roles || [],
     authType: authType === 'local' ? 'teleport local user' : authType,
     isLocal: authType === 'local',
   };

diff --git a/packages/teleport/src/services/user/user.test.ts b/packages/teleport/src/services/user/user.test.ts
@@ -1,5 +1,5 @@
 /**
- * Copyright 2020 Gravitational, Inc.
+ * Copyright 2020-2022 Gravitational, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -170,9 +170,15 @@ test('undefined values in context response gives proper default values', async (
   });
 });
 
-test('fetch users, null response gives empty array', async () => {
+test('fetch users, null response values gives empty array', async () => {
   jest.spyOn(api, 'get').mockResolvedValue(null);
-
-  const response = await user.fetchUsers();
+  let response = await user.fetchUsers();
   expect(response).toStrictEqual([]);
+
+  jest.spyOn(api, 'get').mockResolvedValue([{ name: '', authType: '' }]);
+
+  response = await user.fetchUsers();
+  expect(response).toStrictEqual([
+    { authType: '', isLocal: false, name: '', roles: [] },
+  ]);
 });