[Workspace] Fix: Show a error toast when workspace read only user delete saved objects

changelogs/fragments/6756.yml

@@ -0,0 +1,2 @@
+fix:
+- Show error toast when fail to delete saved objects ([#6756](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6756))

src/plugins/saved_objects_management/public/management_section/objects_table/saved_objects_table.test.tsx

@@ -613,6 +613,42 @@ describe('SavedObjectsTable', () => {
       );
       expect(component.state('selectedSavedObjects').length).toBe(0);
     });
+
+    it('should show error toast when failing to delete saved objects', async () => {
+      const mockSelectedSavedObjects = [
+        { id: '1', type: 'index-pattern' },
+      ] as SavedObjectWithMetadata[];
+
+      const mockSavedObjects = mockSelectedSavedObjects.map((obj) => ({
+        id: obj.id,
+        type: obj.type,
+        source: {},
+      }));
+
+      const mockSavedObjectsClient = {
+        ...defaultProps.savedObjectsClient,
+        bulkGet: jest.fn().mockImplementation(() => ({
+          savedObjects: mockSavedObjects,
+        })),
+        delete: jest.fn().mockImplementation(() => {
+          throw new Error('Unable to delete saved objects');
+        }),
+      };
+
+      const component = shallowRender({ savedObjectsClient: mockSavedObjectsClient });
+
+      // Ensure all promises resolve
+      await new Promise((resolve) => process.nextTick(resolve));
+      // Ensure the state changes are reflected
+      component.update();
+
+      // Set some as selected
+      component.instance().onSelectionChanged(mockSelectedSavedObjects);
+
+      await component.instance().delete();
+
+      expect(notifications.toasts.addDanger).toHaveBeenCalled();
+    });
   });
 
   describe('workspace filter', () => {

src/plugins/saved_objects_management/public/management_section/objects_table/saved_objects_table.tsx

@@ -589,7 +589,7 @@ export class SavedObjectsTable extends Component<SavedObjectsTableProps, SavedOb
   };
 
   delete = async () => {
-    const { savedObjectsClient } = this.props;
+    const { savedObjectsClient, notifications } = this.props;
     const { selectedSavedObjects, isDeleting } = this.state;
 
     if (isDeleting) {
@@ -603,22 +603,29 @@ export class SavedObjectsTable extends Component<SavedObjectsTableProps, SavedOb
       await this.props.indexPatterns.clearCache();
     }
 
-    const objects = await savedObjectsClient.bulkGet(selectedSavedObjects);
-    const deletes = objects.savedObjects.map((object) =>
-      savedObjectsClient.delete(object.type, object.id, { force: true })
-    );
-    await Promise.all(deletes);
-
-    // Unset this
-    this.setState({
-      selectedSavedObjects: [],
-    });
-
-    // Fetching all data
-    await this.fetchSavedObjects();
-    await this.fetchCounts();
-
-    // Allow the user to interact with the table once the saved objects have been re-fetched.
+    try {
+      const objects = await savedObjectsClient.bulkGet(selectedSavedObjects);
+      const deletes = objects.savedObjects.map((object) =>
+        savedObjectsClient.delete(object.type, object.id, { force: true })
+      );
+      await Promise.all(deletes);
+      // Unset this
+      this.setState({
+        selectedSavedObjects: [],
+      });
+      // Fetching all data
+      await this.fetchSavedObjects();
+      await this.fetchCounts();
+    } catch (error) {
+      notifications.toasts.addDanger({
+        title: i18n.translate(
+          'savedObjectsManagement.objectsTable.unableDeleteSavedObjectsNotificationMessage',
+          { defaultMessage: 'Unable to delete saved objects' }
+        ),
+        text: `${error}`,
+      });
+    }
+    // Allow the user to interact with the table once the saved objects have been re-fetched or failed to delete.
     this.setState({
       isShowingDeleteConfirmModal: false,
       isDeleting: false,