tests and update message

Author: paula-stacho

packages/compass-schema/src/components/schema-toolbar/schema-toolbar.spec.tsx

@@ -40,7 +40,7 @@ describe('SchemaToolbar', function () {
       <MockQueryBarPlugin {...(queryBarProps as any)}>
         <SchemaToolbar
           analysisState="complete"
-          errorMessage={''}
+          error={undefined}
           isOutdated={false}
           onAnalyzeSchemaClicked={() => {}}
           onResetClicked={() => {}}
@@ -60,23 +60,38 @@ describe('SchemaToolbar', function () {
     sinon.restore();
   });
 
-  it("renders errors when they're passed", function () {
-    renderSchemaToolbar({
-      analysisState: 'error',
-      errorMessage: 'test error msg',
-    });
-
-    expect(screen.getByText(testErrorMessage)).to.be.visible;
-    expect(screen.getByTestId('schema-toolbar-error-message')).to.be.visible;
-  });
+  describe('errors', function () {
+    it('renders general error', function () {
+      renderSchemaToolbar({
+        analysisState: 'initial',
+        error: {
+          errorType: 'GENERAL',
+          errorMessage: 'test error msg',
+        },
+      });
 
-  it('does not render errors when the analysis state is not error', function () {
-    renderSchemaToolbar({
-      errorMessage: 'test error msg',
+      expect(screen.getByText(testErrorMessage)).to.be.visible;
+      expect(screen.getByTestId('schema-toolbar-error-message')).to.be.visible;
     });
 
-    expect(screen.queryByText(testErrorMessage)).to.not.exist;
-    expect(screen.queryByTestId('schema-toolbar-error-message')).to.not.exist;
+    it('renders complexity abort error', function () {
+      renderSchemaToolbar({
+        analysisState: 'initial',
+        error: {
+          errorType: 'HIGH_COMPLEXITY',
+          errorMessage: 'test error msg',
+        },
+      });
+
+      expect(screen.getByTestId('schema-toolbar-complexity-abort-message')).to
+        .be.visible;
+      expect(
+        screen.getByRole('link', { name: 'Learn more' })
+      ).to.have.attribute(
+        'href',
+        'https://www.mongodb.com/docs/manual/data-modeling/design-antipatterns/bloated-documents/'
+      );
+    });
   });
 
   it('renders the sample size count', function () {

packages/compass-schema/src/components/schema-toolbar/schema-toolbar.tsx

@@ -144,10 +144,11 @@ const SchemaToolbar: React.FunctionComponent<SchemaToolbarProps> = ({
           data-testid="schema-toolbar-complexity-abort-message"
           dismissible={true}
         >
-          Analysis was aborted due to: Fields count above 1000. Consider
-          breaking up your data into more collections with smaller documents,
-          and using references to consolidate the data you need.&nbsp;
-          <Link href="https://www.mongodb.com/docs/atlas/schema-suggestions/reduce-document-size/">
+          The analysis was aborted because the number of fields exceeds 1000.
+          Consider breaking up your data into more collections with smaller
+          documents, and using references to consolidate the data you
+          need.&nbsp;
+          <Link href="https://www.mongodb.com/docs/manual/data-modeling/design-antipatterns/bloated-documents/">
             Learn more
           </Link>
         </Banner>

packages/compass-schema/src/stores/store.spec.ts

@@ -100,7 +100,7 @@ describe('Schema Store', function () {
     });
 
     it('defaults the error to empty', function () {
-      expect(store.getState().schemaAnalysis.errorMessage).to.equal('');
+      expect(store.getState().schemaAnalysis.error).to.be.undefined;
     });
 
     it('defaults the schema to null', function () {
@@ -112,15 +112,10 @@ describe('Schema Store', function () {
       sampleStub.resolves([{ name: 'Hans' }, { name: 'Greta' }]);
       await store.dispatch(startAnalysis());
       expect(sampleStub).to.have.been.called;
-      const {
-        analysisState,
-        errorMessage,
-        schema,
-        resultId,
-        analysisStartTime,
-      } = store.getState().schemaAnalysis;
+      const { analysisState, error, schema, resultId, analysisStartTime } =
+        store.getState().schemaAnalysis;
       expect(analysisState).to.equal('complete');
-      expect(!!errorMessage).to.be.false;
+      expect(error).to.be.undefined;
       expect(analysisStartTime).to.not.be.undefined;
       expect(analysisStartTime).to.be.greaterThan(1000);
       expect(schema).not.to.be.null;