Tree item abort

e2e/tests/functional/tree.e2e.spec.js

@@ -174,6 +174,44 @@ test.describe('Main Tree', () => {
       ]);
     });
   });
+  test('Opening and closing an item before the request has been fulfilled will abort the request @couchdb', async ({
+    page,
+    openmctConfig
+  }) => {
+    const { myItemsFolderName } = openmctConfig;
+    let requestWasAborted = false;
+
+    page.on('requestfailed', (request) => {
+      // check if the request was aborted
+      if (request.failure().errorText === 'net::ERR_ABORTED') {
+        requestWasAborted = true;
+      }
+    });
+
+    await createDomainObjectWithDefaults(page, {
+      type: 'Folder',
+      name: 'Foo'
+    });
+
+    // Intercept and delay request
+    const delayInMs = 500;
+
+    await page.route('**', async (route, request) => {
+      await new Promise((resolve) => setTimeout(resolve, delayInMs));
+      route.continue();
+    });
+
+    // Quickly Expand/close the root folder
+    const mainTree = page.getByRole('tree', {
+      name: 'Main Tree'
+    });
+    const treeItem = mainTree.getByRole('treeitem', {
+      myItemsFolderName
+    });
+    await treeItem.locator('.c-disclosure-triangle').dblclick({ delay: 400 });
+
+    expect(requestWasAborted).toBe(true);
+  });
 });
 
 /**

src/plugins/persistence/couch/CouchObjectProvider.js

@@ -223,13 +223,17 @@ class CouchObjectProvider {
       // Network error, CouchDB unreachable.
       if (response === null) {
         this.indicator.setIndicatorToState(DISCONNECTED);
-        console.error(error.message);
+
+        if (error.name !== 'AbortError') {
+          console.error(error.message);
+        }
+
         throw new Error(`CouchDB Error - No response"`);
       } else {
         if (body?.model && isNotebookOrAnnotationType(body.model)) {
           // warn since we handle conflicts for notebooks
           console.warn(error.message);
-        } else {
+        } else if (error.name !== 'AbortError') {
           console.error(error.message);
         }
 

src/ui/layout/mct-tree.vue

@@ -326,12 +326,13 @@ export default {
     },
     async openTreeItem(parentItem) {
       const parentPath = parentItem.navigationPath;
+      const abortSignal = this.startItemLoad(parentPath);
 
-      this.startItemLoad(parentPath);
       // pass in abort signal when functional
       const childrenItems = await this.loadAndBuildTreeItemsFor(
         parentItem.object.identifier,
-        parentItem.objectPath
+        parentItem.objectPath,
+        abortSignal
       );
       const parentIndex = this.treeItems.indexOf(parentItem);