test: Add test for membership selector when project not found

techmatters · Oct 12, 2023 · c86fb67 · c86fb67
1 parent b3ba407
commit c86fb67
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/selectors.test.ts b/src/selectors.test.ts
@@ -28,7 +28,7 @@ const generateUser = () => {
 };
 
 const generateProject = (
-  memberships: ProjectMembership[],
+  memberships: ProjectMembership[] = [],
   privacy?: ProjectPrivacy,
 ): Project => {
   const id = uuidv4();
@@ -107,3 +107,12 @@ test('can select memberships of specific project', () => {
   );
   expect(memberships).toStrictEqual([[membershipB, user]]);
 });
+
+test('not found project returns empty membership', () => {
+  const store = createStore(initState([], []));
+  const memberships = selectProjectMembershipsWithUsers(
+    store.getState(),
+    'badid',
+  );
+  expect(memberships).toStrictEqual([]);
+});
diff --git a/src/selectors.ts b/src/selectors.ts
@@ -5,7 +5,7 @@ import { type SharedState } from 'terraso-client-shared/store/store';
 import { ProjectMembership } from './project/projectSlice';
 
 const selectProjectMemberships = (state: SharedState, projectId: string) =>
-  state.project.projects[projectId].memberships;
+  state.project.projects[projectId]?.memberships ?? [];
 
 const selectUsers = (state: SharedState) => state.account.users;