fix failed test case

Signed-off-by: Hailong Cui <ihailong@amazon.com>
ruanyl · Mar 1, 2024 · f54318b · f54318b
1 parent c5e3ab2
commit f54318b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/...d_objects_management/public/management_section/objects_table/saved_objects_table.test.tsx b/...d_objects_management/public/management_section/objects_table/saved_objects_table.test.tsx
@@ -648,7 +648,7 @@ describe('SavedObjectsTable', () => {
       expect(component.state('selectedSavedObjects').length).toBe(0);
     });
 
-    it('show workspace filter when workspace on and not in a workspace', async () => {
+    it('show workspace filter when workspace turn on and not in any workspace', async () => {
       const applications = applicationServiceMock.createStartContract();
       applications.capabilities = {
         navLinks: {},
@@ -675,6 +675,8 @@ describe('SavedObjectsTable', () => {
         },
       ];
       workspaces.workspaceList$.next(workspaceList);
+      workspaces.currentWorkspaceId$.next('');
+      workspaces.currentWorkspace$.next(null);
 
       const component = shallowRender({ applications, workspaces });
 
@@ -693,7 +695,7 @@ describe('SavedObjectsTable', () => {
       expect(filters[1].options[1].value).toBe('bar');
     });
 
-    it('show workspace filter when workspace on and in a workspace', async () => {
+    it('show workspace filter when workspace turn on and enter a workspace', async () => {
       const applications = applicationServiceMock.createStartContract();
       applications.capabilities = {
         navLinks: {},

diff --git a/.../saved_objects_management/public/management_section/objects_table/saved_objects_table.tsx b/.../saved_objects_management/public/management_section/objects_table/saved_objects_table.tsx
@@ -202,7 +202,7 @@ export class SavedObjectsTable extends Component<SavedObjectsTableProps, SavedOb
 
   private get wsNameIdLookup() {
     const { availableWorkspaces } = this.state;
-    //  Assumption: workspace name is unique across the system
+    // Assumption: workspace name is unique across the system
     return availableWorkspaces?.reduce((map, ws) => {
       return map.set(ws.name, ws.id);
     }, new Map<string, string>());