Make tests check entire state

elastic · Mar 28, 2022 · 466c4ab · 466c4ab
1 parent de38f33
commit 466c4ab
Showing 1 changed file with 53 additions and 17 deletions.
diff --git a/...ent_sources/components/add_source/add_external_connector/external_connector_logic.test.ts b/...ent_sources/components/add_source/add_external_connector/external_connector_logic.test.ts
@@ -45,6 +45,16 @@ describe('ExternalConnectorLogic', () => {
     insecureUrl: true,
   };
 
+  const DEFAULT_VALUES_SUCCESS: ExternalConnectorValues = {
+    ...DEFAULT_VALUES,
+    externalConnectorApiKey: 'asdf1234',
+    externalConnectorUrl: 'https://www.elastic.co',
+    formDisabled: false,
+    insecureUrl: false,
+    dataLoading: false,
+    sourceConfigData,
+  };
+
   beforeEach(() => {
     jest.clearAllMocks();
     mount();
@@ -61,38 +71,56 @@ describe('ExternalConnectorLogic', () => {
       });
 
       it('turns off the data loading flag', () => {
-        expect(ExternalConnectorLogic.values.dataLoading).toEqual(false);
+        expect(ExternalConnectorLogic.values).toEqual({
+          ...DEFAULT_VALUES_SUCCESS,
+          dataLoading: false,
+        });
       });
 
       it('saves the external url', () => {
-        expect(ExternalConnectorLogic.values.externalConnectorUrl).toEqual(
-          sourceConfigData.configuredFields.externalConnectorUrl
-        );
+        expect(ExternalConnectorLogic.values).toEqual({
+          ...DEFAULT_VALUES_SUCCESS,
+          externalConnectorUrl: sourceConfigData.configuredFields.externalConnectorUrl,
+        });
       });
 
       it('saves the source config', () => {
-        expect(ExternalConnectorLogic.values.sourceConfigData).toEqual(sourceConfigData);
+        expect(ExternalConnectorLogic.values).toEqual({
+          ...DEFAULT_VALUES_SUCCESS,
+          sourceConfigData,
+        });
       });
 
       it('sets undefined url to empty string', () => {
-        ExternalConnectorLogic.actions.fetchExternalSourceSuccess({
+        const newSourceConfigData = {
           ...sourceConfigData,
           configuredFields: {
             ...sourceConfigData.configuredFields,
             externalConnectorUrl: undefined,
           },
+        };
+        ExternalConnectorLogic.actions.fetchExternalSourceSuccess(newSourceConfigData);
+        expect(ExternalConnectorLogic.values).toEqual({
+          ...DEFAULT_VALUES_SUCCESS,
+          externalConnectorUrl: '',
+          insecureUrl: true,
+          sourceConfigData: newSourceConfigData,
         });
-        expect(ExternalConnectorLogic.values.externalConnectorUrl).toEqual('');
       });
       it('sets undefined api key to empty string', () => {
-        ExternalConnectorLogic.actions.fetchExternalSourceSuccess({
+        const newSourceConfigData = {
           ...sourceConfigData,
           configuredFields: {
             ...sourceConfigData.configuredFields,
             externalConnectorApiKey: undefined,
           },
+        };
+        ExternalConnectorLogic.actions.fetchExternalSourceSuccess(newSourceConfigData);
+        expect(ExternalConnectorLogic.values).toEqual({
+          ...DEFAULT_VALUES_SUCCESS,
+          externalConnectorApiKey: '',
+          sourceConfigData: newSourceConfigData,
         });
-        expect(ExternalConnectorLogic.values.externalConnectorApiKey).toEqual('');
       });
     });
 
@@ -104,7 +132,7 @@ describe('ExternalConnectorLogic', () => {
 
         ExternalConnectorLogic.actions.saveExternalConnectorConfigSuccess('external');
 
-        expect(ExternalConnectorLogic.values.buttonLoading).toEqual(false);
+        expect(ExternalConnectorLogic.values).toEqual({ ...DEFAULT_VALUES, buttonLoading: false });
       });
     });
 
@@ -116,39 +144,47 @@ describe('ExternalConnectorLogic', () => {
 
         ExternalConnectorLogic.actions.saveExternalConnectorConfigError();
 
-        expect(ExternalConnectorLogic.values.buttonLoading).toEqual(false);
+        expect(ExternalConnectorLogic.values).toEqual({ ...DEFAULT_VALUES, buttonLoading: false });
       });
     });
 
     describe('setExternalConnectorApiKey', () => {
       it('updates the api key', () => {
         ExternalConnectorLogic.actions.setExternalConnectorApiKey('abcd1234');
 
-        expect(ExternalConnectorLogic.values.externalConnectorApiKey).toEqual('abcd1234');
+        expect(ExternalConnectorLogic.values).toEqual({
+          ...DEFAULT_VALUES,
+          externalConnectorApiKey: 'abcd1234',
+        });
       });
     });
 
     describe('setExternalConnectorUrl', () => {
       it('updates the url', () => {
         ExternalConnectorLogic.actions.setExternalConnectorUrl('https://www.elastic.co');
 
-        expect(ExternalConnectorLogic.values.externalConnectorUrl).toEqual(
-          'https://www.elastic.co'
-        );
+        expect(ExternalConnectorLogic.values).toEqual({
+          ...DEFAULT_VALUES,
+          externalConnectorUrl: 'https://www.elastic.co',
+          insecureUrl: false,
+        });
       });
     });
     describe('setUrlValidation', () => {
       it('updates the url validation', () => {
         ExternalConnectorLogic.actions.setUrlValidation(false);
 
-        expect(ExternalConnectorLogic.values.urlValid).toEqual(false);
+        expect(ExternalConnectorLogic.values).toEqual({ ...DEFAULT_VALUES, urlValid: false });
       });
     });
     describe('setShowInsecureUrlCallout', () => {
       it('updates the url validation', () => {
         ExternalConnectorLogic.actions.setShowInsecureUrlCallout(true);
 
-        expect(ExternalConnectorLogic.values.showInsecureUrlCallout).toEqual(true);
+        expect(ExternalConnectorLogic.values).toEqual({
+          ...DEFAULT_VALUES,
+          showInsecureUrlCallout: true,
+        });
       });
     });
   });