COMPASS-2110: Investigate timeouts on Travis

.travis.yml

@@ -1,4 +1,4 @@
-sudo: required
+sudo: false
 dist: trusty
 language: node_js
 node_js:
@@ -24,7 +24,7 @@ before_script:
   - export DISPLAY=:99.0
   - sh -e /etc/init.d/xvfb start
   - npm run check
-script: npm run test
+script: DEBUG=* npm run test -- --functional
 cache:
   directories:
     - $HOME/.electron

test/functional/collections.test.js

@@ -1,6 +1,6 @@
 const Connection = require('mongodb-connection-model');
 const DataService = require('mongodb-data-service');
-const { launchCompass, quitCompass} = require('./support/spectron-support');
+const { launchCompass, quitCompass } = require('./support/spectron-support');
 
 /**
  * Global connection model for this test.

test/functional/databases.test.js

@@ -39,15 +39,6 @@ describe('#databases', function() {
       });
     });
 
-    after(function(done) {
-      dataService.connect(function() {
-        dataService.dropDatabase('music', function() {
-          dataService.disconnect();
-          done();
-        });
-      });
-    });
-
     context('when the escape key is pressed', function() {
       it('closes the create databases modal', function() {
         return client

test/functional/schema.test.js

@@ -62,8 +62,6 @@ describe('#schema', function() {
 
     it('shows a schema on refresh', function() {
       return client
-        .clickDatabaseInSidebar('music')
-        .waitForSidebar('collection')
         .goToCollection('music', 'artists')
         .clickApplyFilterButtonFromSchemaTab()
         .waitForStatusBar()

test/functional/support/packages/spectron-connect.js

@@ -6,6 +6,7 @@ function addWaitConnectCommands(client) {
    * Wait for the connect screen to finish loading.
    */
   client.addCommand('waitForConnectView', function() {
+    console.log('-> waitForConnectView');
     return this.waitForVisibleInCompass(selector('connect-form'));
   });
 }
@@ -26,6 +27,7 @@ function addClickConnectCommands(client) {
    * click the Connect button on the connect screen.
    */
   client.addCommand('clickConnectButton', function() {
+    console.log('-> clickConnectButton');
     return this.click(selector('connect-button'));
   });
 }
@@ -38,6 +40,7 @@ function addInputConnectCommands(client) {
    * @param {Object} model - The connection model.
    */
   client.addCommand('inputConnectionDetails', function(model) {
+    console.log('-> inputConnectionDetails');
     const that = this;
     let sequence = Promise.resolve();
 

test/functional/support/packages/spectron-database-ddl.js

@@ -53,6 +53,8 @@ function addGetDatabaseDDLCommands(client) {
    * Gets the human-readable text of the Databases Tab.
    */
   client.addCommand('getDatabasesTabText', function() {
+    console.log('-> getDatabasesTabText');
+    console.timeEnd('connectToCompass -> getDatabasesTabText');
     return this.getText(selector('databases-tab'));
   });
 }

test/functional/support/packages/spectron-home.js

@@ -6,6 +6,7 @@ function addWaitHomeCommands(client) {
    * Wait for the home screen to finish loading.
    */
   client.addCommand('waitForHomeView', function() {
+    console.log('-> waitForHomeView');
     return this.waitForVisibleInCompass(selector('home-view'));
   });
 }

test/functional/support/packages/spectron-schema.js

@@ -13,6 +13,7 @@ function addClickSchemaCommands(client) {
    * Click the apply filter button from the schema tab.
    */
   client.addCommand('clickApplyFilterButtonFromSchemaTab', function() {
+    console.log('clickApplyFilterButtonFromSchemaTab');
     const base = selector('schema-content');
     const button = `${base} ${selector('apply-filter-button')}`;
     return this.waitForVisibleInCompass(button).click(button);
@@ -34,6 +35,7 @@ function addGetSchemaCommands(client) {
    * Get the sampling message on the schema tab.
    */
   client.addCommand('getSamplingMessageFromSchemaTab', function() {
+    console.log('getSamplingMessageFromSchemaTab');
     const base = selector('schema-content');
     const div = `${base} .sampling-message`;
     return this.waitForVisibleInCompass(div).getText(div);

test/functional/support/packages/spectron-sidebar.js

@@ -8,13 +8,15 @@ function addWaitSidebarCommands(client) {
    * @param {String} type - One of database or collection.
    */
   client.addCommand('waitForSidebar', function(type) {
+    console.log('waitForSidebar');
     return this.waitForVisibleInCompass(selector('sidebar-' + type));
   });
 
   /**
    * Wait for the instance refresh to finish.
    */
   client.addCommand('waitForInstanceRefresh', function() {
+    console.log('waitForInstanceRefresh');
     const button = selector('instance-refresh-button');
     const icon = `${button} i.fa-spin`;
     return this.waitForVisibleInCompass(icon, true);
@@ -35,6 +37,7 @@ function addClickSidebarCommands(client) {
    * Click the instance refresh button in the top right corner of the sidebar.
    */
   client.addCommand('clickInstanceRefreshIcon', function() {
+    console.log('clickInstanceRefreshIcon');
     const button = selector('instance-refresh-button');
     return this
       .waitForVisibleInCompass(button).click(button);
@@ -46,6 +49,7 @@ function addClickSidebarCommands(client) {
    * @param {String} name - The full collection name.
    */
   client.addCommand('clickCollectionInSidebar', function(name) {
+    console.log('clickCollectionInSidebar');
     const base = `${selector('sidebar-collection')}[title='${name}']`;
     return this.waitForVisibleInCompass(base).click(base);
   });
@@ -56,6 +60,7 @@ function addClickSidebarCommands(client) {
    * @param {String} name - The database name.
    */
   client.addCommand('clickDatabaseInSidebar', function(name) {
+    console.log('clickDatabaseInSidebar');
     const base = `${selector('sidebar-database')}[title='${name}']`;
     return this.waitForVisibleInCompass(base).click(base);
   });

test/functional/support/packages/spectron-status-bar.js

@@ -3,6 +3,7 @@ function addWaitStatusBarCommands(client) {
    * Waits for the status bar to finish its progress and unlock the page.
    */
   client.addCommand('waitForStatusBar', function() {
+    console.log('-> waitForStatusBar');
     return this.waitForVisibleInCompass('#statusbar', true);
   });
 }

test/functional/support/packages/spectron-workflow.js

@@ -23,6 +23,8 @@ function addWorkflowCommands(client) {
    * Connect to Compass
    */
   client.addCommand('connectToCompass', function(connection) {
+    console.timeEnd('launchCompass -> connectToCompass');
+    console.time('connectToCompass -> getDatabasesTabText');
     return this
       .waitForConnectView()
       .inputConnectionDetails(connection)
@@ -36,6 +38,7 @@ function addWorkflowCommands(client) {
    * Go to collection, assumes refresh is required
    */
   client.addCommand('goToCollection', function(database, collection) {
+    console.log('goToCollection');
     return this
       .clickInstanceRefreshIcon()
       .waitForInstanceRefresh()

test/functional/support/spectron-support.js

@@ -52,25 +52,69 @@ function addCommands(client) {
   addWorkflowCommands(client);
 }
 
+const cleanupElectronChromeDriver = () => {
+  const { spawn } = require('child_process');
+  const killall = spawn('killall', ['chromedriver']);
+
+  killall.stdout.on('data', (data) => {
+    console.log(`ps stdout: ${data}`);
+  });
+
+  killall.stderr.on('data', (data) => {
+    console.log(`ps stderr: ${data}`);
+  });
+
+  killall.on('close', (code) => {
+    if (code !== 0) {
+      console.log(`ps process exited with code ${code}`);
+    }
+  });
+};
+
+const printProcessInfo = () => {
+  // From https://nodejs.org/docs/latest/api/child_process.html#child_process_class_childprocess
+  const { spawn } = require('child_process');
+  const ps = spawn('ps', ['-ef']);
+
+  ps.stdout.on('data', (data) => {
+    console.log(`ps stdout: ${data}`);
+  });
+
+  ps.stderr.on('data', (data) => {
+    console.log(`ps stderr: ${data}`);
+  });
+
+  ps.on('close', (code) => {
+    if (code !== 0) {
+      console.log(`ps process exited with code ${code}`);
+    }
+  });
+};
+
 /**
  * Call launchCompass in beforeEach for all UI tests:
  *
  * @returns {Promise} Promise that resolves when app starts.
  */
 function launchCompass() {
+  printProcessInfo();
+  cleanupElectronChromeDriver();
+  console.time('launchCompass -> connectToCompass');
   return new App(ROOT).launch(addCommands);
 }
 
 /**
  * Call quitCompass in afterEach for all UI tests:
 
  * @param {Object} app - The running application
- * @param {Function} done - The callback to execute when finished.
  *
- * @returns {Promise}    Promise that resolves when app stops.
+ * @returns {Promise}    Promise that resolves when app stops or is undefined.
  */
 function quitCompass(app) {
-  return app.quit();
+  if (app === undefined || app === null) {
+    return Promise.resolve().then(printProcessInfo).then(cleanupElectronChromeDriver);
+  }
+  return app.quit().then(printProcessInfo).then(cleanupElectronChromeDriver);
 }
 
 /**