call all to.throw

Author: lerouxb

packages/compass-aggregations/src/components/add-stage/add-stage.spec.tsx

@@ -24,7 +24,7 @@ describe('AddStage', function () {
       const button = screen.getByTestId('add-stage-icon-button');
       expect(() => {
         within(button).getByText('Add stage');
-      }).to.throw;
+      }).to.throw();
     });
 
     it('calls onAddStage with index when clicked', function () {

packages/compass-aggregations/src/components/aggregation-side-panel/index.spec.tsx

@@ -50,7 +50,7 @@ describe('aggregation side panel', function () {
 
   it('renders a search input', async function () {
     await renderAggregationSidePanel();
-    expect(screen.getByRole('search')).to.not.throw;
+    expect(() => screen.getByRole('search')).to.not.throw();
   });
 
   it('renders all the usecases', async function () {
@@ -71,7 +71,7 @@ describe('aggregation side panel', function () {
         .getByTestId('side-panel-content')
         .querySelectorAll('[data-testid^="use-case-"]')
     ).to.have.lengthOf(1);
-    expect(screen.getByTestId('use-case-sort')).to.not.throw;
+    expect(() => screen.getByTestId('use-case-sort')).to.not.throw();
   });
 
   it('renders usecases filtered by search text matching the stage operator of the usecases', async function () {
@@ -83,7 +83,7 @@ describe('aggregation side panel', function () {
         .getByTestId('side-panel-content')
         .querySelectorAll('[data-testid^="use-case-"]')
     ).to.have.lengthOf(1);
-    expect(screen.getByTestId('use-case-lookup')).to.not.throw;
+    expect(() => screen.getByTestId('use-case-lookup')).to.not.throw();
 
     userEvent.clear(searchBox);
     userEvent.type(searchBox, '$lookup');
@@ -92,7 +92,7 @@ describe('aggregation side panel', function () {
         .getByTestId('side-panel-content')
         .querySelectorAll('[data-testid^="use-case-"]')
     ).to.have.lengthOf(1);
-    expect(screen.getByTestId('use-case-lookup')).to.not.throw;
+    expect(() => screen.getByTestId('use-case-lookup')).to.not.throw();
   });
 
   it('calls onSelectUseCase when a use case is clicked', async function () {

packages/compass-aggregations/src/components/aggregation-side-panel/stage-wizard-use-cases/group/group-with-subset.spec.tsx

@@ -103,7 +103,7 @@ describe('group with subset', function () {
         );
       });
       it('renders number of records input', function () {
-        expect(screen.getByTestId('number-of-records-input')).to.throw;
+        expect(() => screen.getByTestId('number-of-records-input')).to.throw();
       });
     });
 

packages/compass-aggregations/src/components/aggregation-side-panel/stage-wizard-use-cases/match/match-group-form.spec.tsx

@@ -104,7 +104,7 @@ describe('group', function () {
           ).to.exist;
           expect(() =>
             screen.getByTestId(TEST_IDS.removeGroupBtn(group.id))
-          ).to.throw;
+          ).to.throw();
         });
       });
 

packages/compass-aggregations/src/components/aggregation-side-panel/stage-wizard-use-cases/use-case-card.spec.tsx

@@ -23,7 +23,7 @@ describe('UseCaseCard', function () {
         onSelect={Sinon.spy()}
       />
     );
-    expect(screen.getByTestId(`use-case-${useCase.id}`)).to.not.throw;
+    expect(() => screen.getByTestId(`use-case-${useCase.id}`)).to.not.throw();
   });
 
   it('should call onSelect when a usecase is selected', function () {

packages/compass-aggregations/src/components/focus-mode/focus-mode-stage-editor.spec.tsx

@@ -22,10 +22,10 @@ describe('FocusMode', function () {
     await renderFocusModeStageEditor({ index: -1 });
     expect(() => {
       screen.getByTestId('stage-operator-combobox');
-    }).to.throw;
+    }).to.throw();
     expect(() => {
       screen.getByText(/open docs/i);
-    }).to.throw;
+    }).to.throw();
   });
 
   context('when operator is not defined', function () {
@@ -44,7 +44,7 @@ describe('FocusMode', function () {
     it('does not render docs link', function () {
       expect(() => {
         screen.getByText(/open docs/i);
-      }).to.throw;
+      }).to.throw();
     });
   });
 

packages/compass-aggregations/src/components/focus-mode/focus-mode.spec.tsx

@@ -22,7 +22,7 @@ describe('FocusMode', function () {
     await waitFor(() => {
       expect(() => {
         screen.getByTestId('focus-mode-modal');
-      }).to.throw;
+      }).to.throw();
     });
   });
 
@@ -44,6 +44,6 @@ describe('FocusMode', function () {
 
     expect(() => {
       screen.getByTestId('focus-mode-modal');
-    }).to.throw;
+    }).to.throw();
   });
 });

packages/compass-aggregations/src/components/pipeline-builder-workspace/index.spec.tsx

@@ -38,7 +38,7 @@ describe('PipelineBuilderWorkspace', function () {
     const container = screen.getByTestId('pipeline-builder-workspace');
     expect(() => {
       within(container).getByTestId('aggregation-side-panel');
-    }).to.not.throw;
+    }).to.not.throw();
   });
 
   it('does not render side panel when enabled in as text mode', async function () {
@@ -47,6 +47,6 @@ describe('PipelineBuilderWorkspace', function () {
     const container = screen.getByTestId('pipeline-builder-workspace');
     expect(() => {
       within(container).getByTestId('aggregation-side-panel');
-    }).to.throw;
+    }).to.throw();
   });
 });

packages/compass-aggregations/src/components/pipeline-builder-workspace/pipeline-as-text-workspace/index.spec.tsx

@@ -26,6 +26,6 @@ describe('PipelineAsTextWorkspace', function () {
     await renderPipelineAsTextWorkspace({ isAutoPreview: false });
     expect(() => {
       screen.getByTestId('pipeline-as-text-preview');
-    }).to.throw;
+    }).to.throw();
   });
 });

packages/compass-aggregations/src/components/pipeline-builder-workspace/pipeline-as-text-workspace/pipeline-preview.spec.tsx

@@ -113,11 +113,11 @@ describe('PipelinePreview', function () {
     // By default we don't expand nested props of a document
     expect(within(docList).getByText(/_id/)).to.exist;
     expect(within(docList).getByText(/score/)).to.exist;
-    expect(() => within(docList).getByText(/number/)).to.throw;
-    expect(() => within(docList).getByText(/another/)).to.throw;
-    expect(() => within(docList).getByText(/deep/)).to.throw;
-    expect(() => within(docList).getByText(/nested/)).to.throw;
-    expect(() => within(docList).getByText(/document/)).to.throw;
+    expect(() => within(docList).getByText(/number/)).to.throw();
+    expect(() => within(docList).getByText(/another/)).to.throw();
+    expect(() => within(docList).getByText(/deep/)).to.throw();
+    expect(() => within(docList).getByText(/nested/)).to.throw();
+    expect(() => within(docList).getByText(/document/)).to.throw();
 
     // Expand the whole document
     userEvent.click(
@@ -153,11 +153,11 @@ describe('PipelinePreview', function () {
 
     expect(within(docList).getByText(/_id/)).to.exist;
     expect(within(docList).getByText(/score/)).to.exist;
-    expect(() => within(docList).getByText(/number/)).to.throw;
-    expect(() => within(docList).getByText(/another/)).to.throw;
-    expect(() => within(docList).getByText(/deep/)).to.throw;
-    expect(() => within(docList).getByText(/nested/)).to.throw;
-    expect(() => within(docList).getByText(/document/)).to.throw;
+    expect(() => within(docList).getByText(/number/)).to.throw();
+    expect(() => within(docList).getByText(/another/)).to.throw();
+    expect(() => within(docList).getByText(/deep/)).to.throw();
+    expect(() => within(docList).getByText(/nested/)).to.throw();
+    expect(() => within(docList).getByText(/document/)).to.throw();
   });
 
   it('renders output stage preview', async function () {