testing console tests in the ci (#83)

* testing console tests in the ci

* console: making cypress wait for the server to start

* console: fixing failing tests

* console: update failing test

* console: cleaned up modify tests

* console: fixed a failing test for api-explorer

.circleci/config.yml

@@ -37,7 +37,7 @@ refs:
     filters:
       branches:
         ignore: /^release-v.*/
-  
+
 
 
 version: 2
@@ -142,14 +142,14 @@ jobs:
     - attach_workspace:
         at: /build
     - restore_cache:
-        key: 
+        key:
           console-npm-cache-{{ checksum "console/package.json" }}-{{ checksum "console/package-lock.json" }}
     - run:
         name: install dependencies
         working_directory: console
         command: make ci-deps
     - save_cache:
-        key: 
+        key:
           console-npm-cache-{{ checksum "console/package.json" }}-{{ checksum "console/package-lock.json" }}
         paths:
         - console/node_modules
@@ -157,9 +157,9 @@ jobs:
         - ~/.cache
     - *wait_for_postgres
     # ignore console test for now
-    # - run:
-    #     name: test console
-    #     command: .circleci/test-console.sh
+    - run:
+        name: test console
+        command: .circleci/test-console.sh
     - run:
         name: build console
         working_directory: console
@@ -194,7 +194,7 @@ workflows:
   build_and_test:
     jobs:
     - test_and_build_server: *filter_only_vtags
-    - test_and_build_cli: 
+    - test_and_build_cli:
         <<: *filter_only_vtags
         requires:
         - test_and_build_server
@@ -203,7 +203,7 @@ workflows:
         requires:
         - test_and_build_server
         - test_and_build_cli
-    - deploy: 
+    - deploy:
         <<: *filter_only_vtags_dev_release_branches
         requires:
         - test_and_build_console

console/cypress/integration/api-explorer/graphql/spec.js

@@ -1,13 +1,10 @@
 /* eslint import/prefer-default-export: 0 */
 
-import { baseUrl } from '../../../helpers/dataHelpers';
+import { getElementFromAlias } from '../../../helpers/dataHelpers';
 
 export const openAPIExplorer = () => {
   //eslint-disable-line
   // Open API Explorer
-  cy.get('a')
-    .contains('API Explorer')
-    .click();
+  cy.get(getElementFromAlias('api-explorer')).click();
   cy.wait(3000);
-  cy.url().should('eq', `${baseUrl}/api-explorer`);
 };

console/cypress/integration/api-explorer/graphql/test.js

@@ -4,6 +4,7 @@
 import { openAPIExplorer } from './spec';
 
 import { setMetaData } from '../../validators/validators';
+import { testMode } from '../../../helpers/common';
 
 const setup = () => {
   describe('Setup route', () => {
@@ -23,5 +24,7 @@ export const runApiExplorerTests = () => {
   });
 };
 
-// setup();
-// runApiExplorerTests();
+if (testMode !== 'cli') {
+  setup();
+  runApiExplorerTests();
+}

console/cypress/integration/data/modify/spec.js

@@ -1,6 +1,7 @@
 import {
   baseUrl,
   getTableName,
+  getColName,
   getElementFromAlias,
 } from '../../../helpers/dataHelpers';
 
@@ -29,17 +30,6 @@ export const passMTCreateTable = () => {
   validateCT(getTableName(0, testName), 'success');
 };
 
-export const makeid = () => {
-  let text = '';
-  const possible = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz';
-
-  for (let i = 0; i < 5; i += 1) {
-    text += possible.charAt(Math.floor(Math.random() * possible.length));
-  }
-
-  return text;
-};
-
 export const passMTCheckRoute = () => {
   // Click on the create table button
   cy.get(getElementFromAlias('table-modify')).click();
@@ -66,58 +56,25 @@ export const failMTWithoutColName = () => {
     `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
   );
 
-  validateColumn(getTableName(0, testName), ['something'], 'failure');
+  validateColumn(getTableName(0, testName), [getColName(2)], 'failure');
 };
 
 export const failMTWithoutColType = () => {
-  cy.get(getElementFromAlias('column-name')).type('something');
+  cy.get(getElementFromAlias('column-name')).type(getColName(2));
   cy.get(getElementFromAlias('add-column-button')).click();
   // cy.get('.notification-error').click();
   cy.url().should(
     'eq',
     `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
   );
-  validateColumn(getTableName(0, testName), ['something'], 'failure');
-};
-
-export const failMTDuplicateColumns = () => {
-  cy.get(getElementFromAlias('column-name')).type('{selectall}{del}');
-  cy.get(getElementFromAlias('column-name')).type('id');
-  cy.get(getElementFromAlias('data-type')).select('integer');
-  cy.get(getElementFromAlias('add-column-button')).click();
-  // Check for an alert
-  cy.wait(2500);
-  // cy.get('.notification-error').click();
-  // Check if the route didn't change
-  cy.url().should(
-    'eq',
-    `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
-  );
-};
-
-export const Addcolumn = () => {
-  cy.get(getElementFromAlias('column-name')).type('{selectall}{del}');
-  const name = makeid();
-  cy.get(getElementFromAlias('column-name')).type(name);
-  cy.get(getElementFromAlias('data-type')).select('integer');
-  cy.get(getElementFromAlias('default-value')).type('{selectall}{del}');
-  cy.get(getElementFromAlias('nullable-checkbox')).check();
-  cy.get(getElementFromAlias('add-column-button')).click();
-  cy.wait(2500);
-  // cy.get('.notification-success').click();
-  cy.wait(7000);
-  cy.url().should(
-    'eq',
-    `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
-  );
-  validateColumn(getTableName(0, testName), [name], 'success');
+  validateColumn(getTableName(0, testName), [getColName(2)], 'failure');
 };
 
 export const Addcolumnnullable = () => {
   cy.get(getElementFromAlias('column-name')).type('{selectall}{del}');
-  cy.get(getElementFromAlias('column-name')).type('some');
+  cy.get(getElementFromAlias('column-name')).type(getColName(3));
   cy.get(getElementFromAlias('data-type')).select('Text');
-  cy.get(getElementFromAlias('nullable-checkbox')).uncheck();
+  cy.get(getElementFromAlias('nullable-checkbox')).uncheck({ force: true });
   cy.get(getElementFromAlias('add-column-button')).click();
   cy.wait(2500);
   // cy.get('.notification-error').click();
@@ -125,22 +82,7 @@ export const Addcolumnnullable = () => {
     'eq',
     `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
   );
-  validateColumn(getTableName(0, testName), ['some'], 'failure');
-};
-
-export const failMTWrongDefault = () => {
-  cy.get(getElementFromAlias('column-name')).type('{selectall}{del}');
-  cy.get(getElementFromAlias('column-name')).type('some');
-  cy.get(getElementFromAlias('default-value')).type('some');
-  cy.get(getElementFromAlias('data-type')).select('Integer');
-  cy.get(getElementFromAlias('add-column-button')).click();
-  cy.wait(2500);
-  // cy.get('.notification-error').click();
-  cy.url().should(
-    'eq',
-    `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
-  );
-  validateColumn(getTableName(0, testName), ['some'], 'failure');
+  validateColumn(getTableName(0, testName), [getColName(3)], 'failure');
 };
 
 export const Addcolumnname = name => {
@@ -150,49 +92,27 @@ export const Addcolumnname = name => {
 
   cy.get(getElementFromAlias('add-column-button')).click();
   cy.wait(5000);
-  // cy.get('.notification-success').click();
-  cy.url().should(
-    'eq',
-    `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
-  );
   validateColumn(getTableName(0, testName), [name], 'success');
 };
 
-export const passMTAddTwoCols = () => {
-  Addcolumn();
-  Addcolumn();
+export const passMTAddColumn = () => {
+  cy.get(getElementFromAlias('column-name')).type('{selectall}{del}');
+  cy.get(getElementFromAlias('column-name')).type(getColName(0));
+  cy.get(getElementFromAlias('data-type')).select('integer');
+  cy.get(getElementFromAlias('add-column-button')).click();
+  cy.wait(2500);
+  // cy.get('.notification-success').click();
+  validateColumn(getTableName(0, testName), [getColName(0)], 'success');
 };
 
 export const Movetocolumn = () => {
-  Addcolumnname('somes');
-  cy.get(getElementFromAlias('edit-somes')).click();
-};
-
-export const passMTChangeColType = () => {
-  Movetocolumn();
-  cy.get('.form-horizontal')
-    .children()
-    .get('select')
-    .first()
-    .select('Text');
-  cy.get(getElementFromAlias('save-button')).click();
-  cy.wait(5000);
-  // cy.get('.notification-success').click();
-  cy.get('button')
-    .contains('Close')
-    .click();
-  cy.url().should(
-    'eq',
-    `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
-  );
-  validateColumn(getTableName(0, testName), ['somes'], 'success');
+  Addcolumnname(getColName(1));
+  cy.get(getElementFromAlias(`edit-${getColName(1)}`)).click();
 };
 
 export const passMTDeleteCol = () => {
-  cy.get(getElementFromAlias('edit-somes')).click();
-  cy.get('button')
-    .contains('Remove')
-    .click();
+  cy.get(getElementFromAlias(`edit-${getColName(0)}`)).click();
+  cy.get(getElementFromAlias('remove-button')).click();
   cy.on('window:alert', str => {
     expect(str === 'Are you sure you want to delete?').to.be.true;
   });
@@ -202,7 +122,7 @@ export const passMTDeleteCol = () => {
     'eq',
     `${baseUrl}/data/schema/public/tables/${getTableName(0, testName)}/modify`
   );
-  validateColumn(getTableName(0, testName), ['somes'], 'failure');
+  validateColumn(getTableName(0, testName), [getColName(0)], 'failure');
 };
 
 export const passMTDeleteTableCancel = () => {
@@ -224,18 +144,6 @@ export const passMTDeleteTable = () => {
   validateCT(getTableName(0, testName), 'failure');
 };
 
-export const failMTRemoveNullable = () => {
-  cy.get(getElementFromAlias('table-insert-rows')).click();
-  cy.get('input[placeholder = "integer"]').type('{selectall}{del}');
-  cy.get('input[placeholder="integer"]').type(123);
-  cy.get(getElementFromAlias('insert-save-button')).click();
-  cy.get('input[placeholder = "integer"]').type('{selectall}{del}');
-  cy.get('input[placeholder="integer"]').type(1234);
-  cy.get(getElementFromAlias('insert-save-button')).click();
-  cy.get(getElementFromAlias('table-modify')).click();
-  Addcolumnnullable();
-};
-
 export const setValidationMetaData = () => {
   setMetaData();
 };

console/cypress/integration/data/modify/test.js

@@ -1,4 +1,3 @@
-/* eslint no-unused-vars: 0 */
 /* eslint import/prefer-default-export: 0 */
 
 import {
@@ -7,14 +6,10 @@ import {
   passMTCreateTable,
   failMTWithoutColName,
   failMTWithoutColType,
-  failMTDuplicateColumns,
-  passMTAddTwoCols,
+  passMTAddColumn,
   passMTDeleteTableCancel,
   passMTDeleteCol,
-  passMTChangeColType,
   passMTDeleteTable,
-  failMTRemoveNullable,
-  failMTWrongDefault,
 } from './spec';
 
 import { testMode } from '../../../helpers/common';
@@ -39,11 +34,7 @@ export const runModifyTableTests = () => {
     it('Modify table button opens the correct route', passMTCheckRoute);
     it('Fails to add column without column name', failMTWithoutColName);
     it('Fails without type selected', failMTWithoutColType);
-    it('Fail duplicate column', failMTDuplicateColumns);
-    it('Fail for removing nullable after addition', failMTRemoveNullable);
-    it('Fail with wrong default', failMTWrongDefault);
-    it('Add 2 Columns', passMTAddTwoCols);
-    it('Moving to column and change type', passMTChangeColType);
+    it('Add a column', passMTAddColumn);
     it('Delete the column', passMTDeleteCol);
     it('Delete Table Cancel', passMTDeleteTableCancel);
     it('Delete table', passMTDeleteTable);

console/cypress/integration/test_complete.js

@@ -14,7 +14,7 @@ import { runApiExplorerTests } from './api-explorer/graphql/test';
 const setup = () => {
   it('Visit the index route', () => {
     // Wait for the server to start
-    cy.wait(20000);
+    cy.wait(60000);
     // Visit the index route
     cy.visit('/data/schema/public');
     cy.wait(7000);

console/package-lock.json

@@ -5757,12 +5757,14 @@
         "balanced-match": {
           "version": "1.0.0",
           "resolved": "https://registry.npmjs.org/balanced-match/-/balanced-match-1.0.0.tgz",
-          "integrity": "sha1-ibTRmasr7kneFk6gK4nORi1xt2c="
+          "integrity": "sha1-ibTRmasr7kneFk6gK4nORi1xt2c=",
+          "optional": true
         },
         "brace-expansion": {
           "version": "1.1.11",
           "resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-1.1.11.tgz",
           "integrity": "sha512-iCuPHDFgrHX7H2vEI/5xpz07zSHB00TpugqhmYtVmMO6518mCuRMoOYFldEBl0g187ufozdaHgWKcYFb61qGiA==",
+          "optional": true,
           "requires": {
             "balanced-match": "1.0.0",
             "concat-map": "0.0.1"
@@ -5777,17 +5779,20 @@
         "code-point-at": {
           "version": "1.1.0",
           "resolved": "https://registry.npmjs.org/code-point-at/-/code-point-at-1.1.0.tgz",
-          "integrity": "sha1-DQcLTQQ6W+ozovGkDi7bPZpMz3c="
+          "integrity": "sha1-DQcLTQQ6W+ozovGkDi7bPZpMz3c=",
+          "optional": true
         },
         "concat-map": {
           "version": "0.0.1",
           "resolved": "https://registry.npmjs.org/concat-map/-/concat-map-0.0.1.tgz",
-          "integrity": "sha1-2Klr13/Wjfd5OnMDajug1UBdR3s="
+          "integrity": "sha1-2Klr13/Wjfd5OnMDajug1UBdR3s=",
+          "optional": true
         },
         "console-control-strings": {
           "version": "1.1.0",
           "resolved": "https://registry.npmjs.org/console-control-strings/-/console-control-strings-1.1.0.tgz",
-          "integrity": "sha1-PXz0Rk22RG6mRL9LOVB/mFEAjo4="
+          "integrity": "sha1-PXz0Rk22RG6mRL9LOVB/mFEAjo4=",
+          "optional": true
         },
         "core-util-is": {
           "version": "1.0.2",
@@ -5904,7 +5909,8 @@
         "inherits": {
           "version": "2.0.3",
           "resolved": "https://registry.npmjs.org/inherits/-/inherits-2.0.3.tgz",
-          "integrity": "sha1-Yzwsg+PaQqUC9SRmAiSA9CCCYd4="
+          "integrity": "sha1-Yzwsg+PaQqUC9SRmAiSA9CCCYd4=",
+          "optional": true
         },
         "ini": {
           "version": "1.3.5",
@@ -5916,6 +5922,7 @@
           "version": "1.0.0",
           "resolved": "https://registry.npmjs.org/is-fullwidth-code-point/-/is-fullwidth-code-point-1.0.0.tgz",
           "integrity": "sha1-754xOG8DGn8NZDr4L95QxFfvAMs=",
+          "optional": true,
           "requires": {
             "number-is-nan": "1.0.1"
           }
@@ -5930,6 +5937,7 @@
           "version": "3.0.4",
           "resolved": "https://registry.npmjs.org/minimatch/-/minimatch-3.0.4.tgz",
           "integrity": "sha512-yJHVQEhyqPLUTgt9B83PXu6W3rx4MvvHvSUvToogpwoGDOUQ+yDrR0HRot+yOCdCO7u4hX3pWft6kWBBcqh0UA==",
+          "optional": true,
           "requires": {
             "brace-expansion": "1.1.11"
           }
@@ -5937,12 +5945,14 @@
         "minimist": {
           "version": "0.0.8",
           "resolved": "https://registry.npmjs.org/minimist/-/minimist-0.0.8.tgz",
-          "integrity": "sha1-hX/Kv8M5fSYluCKCYuhqp6ARsF0="
+          "integrity": "sha1-hX/Kv8M5fSYluCKCYuhqp6ARsF0=",
+          "optional": true
         },
         "minipass": {
           "version": "2.2.4",
           "resolved": "https://registry.npmjs.org/minipass/-/minipass-2.2.4.tgz",
           "integrity": "sha512-hzXIWWet/BzWhYs2b+u7dRHlruXhwdgvlTMDKC6Cb1U7ps6Ac6yQlR39xsbjWJE377YTCtKwIXIpJ5oP+j5y8g==",
+          "optional": true,
           "requires": {
             "safe-buffer": "5.1.1",
             "yallist": "3.0.2"
@@ -5961,6 +5971,7 @@
           "version": "0.5.1",
           "resolved": "https://registry.npmjs.org/mkdirp/-/mkdirp-0.5.1.tgz",
           "integrity": "sha1-MAV0OOrGz3+MR2fzhkjWaX11yQM=",
+          "optional": true,
           "requires": {
             "minimist": "0.0.8"
           }
@@ -6041,7 +6052,8 @@
         "number-is-nan": {
           "version": "1.0.1",
           "resolved": "https://registry.npmjs.org/number-is-nan/-/number-is-nan-1.0.1.tgz",
-          "integrity": "sha1-CXtgK1NCKlIsGvuHkDGDNpQaAR0="
+          "integrity": "sha1-CXtgK1NCKlIsGvuHkDGDNpQaAR0=",
+          "optional": true
         },
         "object-assign": {
           "version": "4.1.1",
@@ -6053,6 +6065,7 @@
           "version": "1.4.0",
           "resolved": "https://registry.npmjs.org/once/-/once-1.4.0.tgz",
           "integrity": "sha1-WDsap3WWHUsROsF9nFC6753Xa9E=",
+          "optional": true,
           "requires": {
             "wrappy": "1.0.2"
           }
@@ -6174,6 +6187,7 @@
           "version": "1.0.2",
           "resolved": "https://registry.npmjs.org/string-width/-/string-width-1.0.2.tgz",
           "integrity": "sha1-EYvfW4zcUaKn5w0hHgfisLmxB9M=",
+          "optional": true,
           "requires": {
             "code-point-at": "1.1.0",
             "is-fullwidth-code-point": "1.0.0",

console/src/components/Main/Main.js

@@ -75,7 +75,7 @@ const Main = ({ children, location, migrationModeProgress, currentSchema }) => {
                     }
                     to={appPrefix + '/api-explorer'}
                   >
-                    <div className={styles.iconCenter}>
+                    <div className={styles.iconCenter} data-test="api-explorer">
                       <i
                         title="API Explorer"
                         className="fa fa-flask"