fix: enforce no nesting, adjsut enzyme test and move setup to testing-library

Author: gagik

configs/testing-library-compass/src/index.tsx

@@ -45,7 +45,10 @@ import {
   ReadOnlyPreferenceAccess,
 } from 'compass-preferences-model/provider';
 import { TelemetryProvider } from '@mongodb-js/compass-telemetry/provider';
-import { CompassComponentsProvider } from '@mongodb-js/compass-components';
+import {
+  CompassComponentsProvider,
+  ContextMenuProvider,
+} from '@mongodb-js/compass-components';
 import {
   TestEnvCurrentConnectionContext,
   ConnectionInfoProvider,
@@ -349,41 +352,43 @@ function createWrapper(
             <PreferencesProvider value={wrapperState.preferences}>
               <LoggerProvider value={logger}>
                 <TelemetryProvider options={telemetryOptions}>
-                  <ConnectionStorageProvider
-                    value={wrapperState.connectionStorage}
-                  >
-                    <ConnectFnProvider connect={wrapperState.connect}>
-                      <CompassConnections
-                        appName={options.appName ?? 'TEST'}
-                        onFailToLoadConnections={
-                          options.onFailToLoadConnections ??
-                          (() => {
-                            // noop
-                          })
-                        }
-                        onExtraConnectionDataRequest={
-                          options.onExtraConnectionDataRequest ??
-                          (() => {
-                            return Promise.resolve([{}, null] as [any, null]);
-                          })
-                        }
-                        onAutoconnectInfoRequest={
-                          options.onAutoconnectInfoRequest
-                        }
-                        preloadStorageConnectionInfos={connections}
-                      >
-                        <StoreGetter>
-                          <TestEnvCurrentConnectionContext.Provider
-                            value={TEST_ENV_CURRENT_CONNECTION}
-                          >
-                            <TestingLibraryWrapper {...props}>
-                              {children}
-                            </TestingLibraryWrapper>
-                          </TestEnvCurrentConnectionContext.Provider>
-                        </StoreGetter>
-                      </CompassConnections>
-                    </ConnectFnProvider>
-                  </ConnectionStorageProvider>
+                  <ContextMenuProvider>
+                    <ConnectionStorageProvider
+                      value={wrapperState.connectionStorage}
+                    >
+                      <ConnectFnProvider connect={wrapperState.connect}>
+                        <CompassConnections
+                          appName={options.appName ?? 'TEST'}
+                          onFailToLoadConnections={
+                            options.onFailToLoadConnections ??
+                            (() => {
+                              // noop
+                            })
+                          }
+                          onExtraConnectionDataRequest={
+                            options.onExtraConnectionDataRequest ??
+                            (() => {
+                              return Promise.resolve([{}, null] as [any, null]);
+                            })
+                          }
+                          onAutoconnectInfoRequest={
+                            options.onAutoconnectInfoRequest
+                          }
+                          preloadStorageConnectionInfos={connections}
+                        >
+                          <StoreGetter>
+                            <TestEnvCurrentConnectionContext.Provider
+                              value={TEST_ENV_CURRENT_CONNECTION}
+                            >
+                              <TestingLibraryWrapper {...props}>
+                                {children}
+                              </TestingLibraryWrapper>
+                            </TestEnvCurrentConnectionContext.Provider>
+                          </StoreGetter>
+                        </CompassConnections>
+                      </ConnectFnProvider>
+                    </ConnectionStorageProvider>
+                  </ContextMenuProvider>
                 </TelemetryProvider>
               </LoggerProvider>
             </PreferencesProvider>

packages/compass-crud/src/components/document-list-view.spec.tsx

@@ -1,28 +1,32 @@
 import React from 'react';
 import { mount } from 'enzyme';
+import type { ReactWrapper } from 'enzyme';
 import HadronDocument from 'hadron-document';
 import { expect } from 'chai';
-import sinon from 'sinon';
 
 import DocumentListView from './document-list-view';
+import { ContextMenuProvider } from '@mongodb-js/compass-components';
 
 describe('<DocumentListView />', function () {
   describe('#render', function () {
     context('when the documents have objects for ids', function () {
       const docs = [{ _id: { name: 'test-1' } }, { _id: { name: 'test-2' } }];
       const hadronDocs = docs.map((doc) => new HadronDocument(doc));
-      const component = mount(
-        <DocumentListView
-          docs={hadronDocs}
-          isEditable={false}
-          isTimeSeries={false}
-          copyToClipboard={sinon.spy()}
-          removeDocument={sinon.spy()}
-          replaceDocument={sinon.spy()}
-          updateDocument={sinon.spy()}
-          openInsertDocumentDialog={sinon.spy()}
-        />
-      );
+      let component: ReactWrapper;
+      beforeEach(function () {
+        component = mount(
+          <DocumentListView
+            docs={hadronDocs}
+            isEditable={false}
+            isTimeSeries={false}
+          />,
+          { wrappingComponent: ContextMenuProvider }
+        );
+      });
+
+      afterEach(function () {
+        component?.unmount();
+      });
 
       it('renders all the documents', function () {
         const wrapper = component.find('[data-testid="readonly-document"]');