pipelines work one way, other query types another

Author: lerouxb

packages/compass-saved-aggregations-queries/src/stores/delete-item.ts

@@ -55,10 +55,11 @@ export const confirmDeleteItem = (
       undefined // this event is connection scoped when triggered from the aggregation or query screen
     );
 
-    if (item.type === 'query') {
-      await queryStorage?.delete(item.id);
-    } else {
+    if (item.type === 'aggregation') {
       await pipelineStorage?.delete(item.id);
+    } else {
+      // query or updatemany
+      await queryStorage?.delete(item.id);
     }
 
     dispatch({ type: ActionTypes.DeleteItemConfirm, id: item.id });

packages/compass-saved-aggregations-queries/src/stores/edit-item.ts

@@ -87,13 +87,14 @@ export const updateItem =
       return;
     }
 
-    if (item.type === 'query') {
+    if (item.type === 'aggregation') {
+      await pipelineStorage?.updateAttributes(id, attributes);
+    } else {
+      // query or updatemany
       await queryStorage?.updateAttributes(id, {
         _name: attributes.name,
         _dateModified: new Date(),
       });
-    } else {
-      await pipelineStorage?.updateAttributes(id, attributes);
     }
 
     dispatch({

packages/compass-saved-aggregations-queries/src/stores/open-item.ts

@@ -572,7 +572,8 @@ export const openSelectedItem =
         await pipelineStorage?.updateAttributes(id, {
           namespace: newNamespace,
         });
-      } else if (selectedItem.type === 'query') {
+      } else {
+        // queries, bulk updates and everything else
         await queryStorage?.updateAttributes(id, { _ns: newNamespace });
       }
     }