cleanup

Author: paula-stacho

package-lock.json

@@ -97,6 +97,7 @@
     "@mongodb-js/prettier-config-compass": "^1.2.2",
     "@mongodb-js/testing-library-compass": "^1.2.2",
     "@mongodb-js/tsconfig-compass": "^1.2.2",
+    "mongodb": "^6.12.0",
     "@types/chai": "^4.2.21",
     "@types/chai-dom": "^0.0.10",
     "@types/mocha": "^9.0.0",

packages/compass-components/package.json

@@ -1,5 +1,4 @@
 import React, { useCallback, useEffect, useRef, useState } from 'react';
-import type { MongoServerError } from 'mongodb';
 import type HadronDocument from 'hadron-document';
 import { Element } from 'hadron-document';
 import { Button } from '../leafygreen';
@@ -89,13 +88,17 @@ function useHadronDocumentStatus(
   const invalidElementsRef = useRef(new Set());
 
   const updateStatus = useCallback(
-    (newStatus: Status, error: Error | MongoServerError | null = null) => {
+    (
+      newStatus: Status,
+      error: Error | null = null,
+      errorDetails?: Record<string, unknown>
+    ) => {
       setStatus(newStatus);
       setError(
         error
           ? {
               message: error?.message,
-              details: (error as MongoServerError).errInfo,
+              details: errorDetails,
             }
           : null
       );
@@ -142,17 +145,23 @@ function useHadronDocumentStatus(
     const onUpdateSuccess = () => {
       updateStatus('UpdateSuccess');
     };
-    const onUpdateError = (err: Error | MongoServerError) => {
-      updateStatus('UpdateError', err);
+    const onUpdateError = (
+      err: Error,
+      errorDetails?: Record<string, unknown>
+    ) => {
+      updateStatus('UpdateError', err, errorDetails);
     };
     const onRemoveStart = () => {
       updateStatus('DeleteStart');
     };
     const onRemoveSuccess = () => {
       updateStatus('DeleteSuccess');
     };
-    const onRemoveError = (err: Error | MongoServerError) => {
-      updateStatus('DeleteError', err);
+    const onRemoveError = (
+      err: Error,
+      errorDetails?: Record<string, unknown>
+    ) => {
+      updateStatus('DeleteError', err, errorDetails);
     };
 
     doc.on(Element.Events.Added, onUpdate);

packages/compass-components/src/components/document-list/document-edit-actions-footer.tsx

@@ -20,7 +20,6 @@ import {
 import { context as testRunnerContext } from '../helpers/test-runner-context';
 import type { ChainablePromiseElement } from 'webdriverio';
 import { tryToInsertDocument } from '../helpers/commands/try-to-insert-document';
-import { isClickable } from 'webdriverio/build/commands/element';
 
 const { expect } = chai;
 
@@ -738,7 +737,7 @@ FindIterable<Document> result = collection.find(filter);`);
         await browser.runFindOperation('Documents', '{ "phone": 12345 }');
       });
 
-      it.only('shows error info when editing via list view', async function () {
+      it('shows error info when editing via list view', async function () {
         const document = browser.$(Selectors.DocumentListEntry);
         await document.waitForDisplayed();
 
@@ -751,11 +750,6 @@ FindIterable<Document> result = collection.find(filter);`);
         const deleteBtn = browser.$(
           `${Selectors.HadronDocumentElement}:last-child ${Selectors.HadronDocumentRemoveElement}`
         );
-        console.log({
-          isDisplayed: await deleteBtn.isDisplayed(),
-          isClickable: await deleteBtn.isClickable(),
-          isEnabled: await deleteBtn.isEnabled(),
-        });
 
         await deleteBtn.waitForDisplayed();
         console.log('I am displayed');

packages/compass-e2e-tests/tests/collection-documents-tab.test.ts

@@ -12,6 +12,7 @@ import type { BSONArray, BSONObject, BSONValue } from './utils';
 import { objectToIdiomaticEJSON } from './utils';
 import type { HadronEJSONOptions } from './utils';
 import { DocumentEvents } from '.';
+import type { MongoServerError } from 'mongodb';
 
 /**
  * The event constant.
@@ -478,7 +479,7 @@ export class Document extends EventEmitter {
   }
 
   onUpdateError(error: Error) {
-    this.emit('update-error', error);
+    this.emit('update-error', error, (error as MongoServerError).errInfo);
   }
 
   markForDeletion() {
@@ -505,7 +506,7 @@ export class Document extends EventEmitter {
   }
 
   onRemoveError(error: Error) {
-    this.emit('remove-error', error);
+    this.emit('remove-error', error, (error as MongoServerError).errInfo);
   }
 
   setModifiedEJSONString(ejson: string | null) {