From 63aaab804b50691b13a13862043f759cf971deb3 Mon Sep 17 00:00:00 2001 From: Yuya Ochiai Date: Mon, 14 Mar 2016 21:28:43 +0900 Subject: [PATCH 1/2] Try to stabilize tests pause() was missing before getUrl() in some tests. --- test/browser_test.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/browser_test.js b/test/browser_test.js index a30ae701..837b89bb 100644 --- a/test/browser_test.js +++ b/test/browser_test.js @@ -56,6 +56,7 @@ describe('electron-mattermost', function() { it('should show settings.html when there is no config file', function() { return client .init() + .pause(1000) .getUrl().then(function(url) { var p = path.parse(url); p.base.should.equal('settings.html'); @@ -69,6 +70,7 @@ describe('electron-mattermost', function() { })); return client .init() + .pause(1000) .getUrl().then(function(url) { var p = path.parse(url); p.base.should.equal('index.html'); @@ -83,6 +85,7 @@ describe('electron-mattermost', function() { })); return client .init() + .pause(1000) .getUrl().then(function(url) { var p = path.parse(url); p.base.should.equal('index.html'); From 361b71a04af9c13b4acdb3ba65655432a6b8acb5 Mon Sep 17 00:00:00 2001 From: Yuya Ochiai Date: Mon, 14 Mar 2016 21:38:50 +0900 Subject: [PATCH 2/2] Add waiting when switching tabs --- test/browser_test.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/test/browser_test.js b/test/browser_test.js index 837b89bb..e254d019 100644 --- a/test/browser_test.js +++ b/test/browser_test.js @@ -158,12 +158,14 @@ describe('electron-mattermost', function() { it('should show only the selected team', function() { return client .init() - .waitForVisible('#mattermostView0') + .pause(1000) + .waitForVisible('#mattermostView0', 1000) .isVisible('#mattermostView1').then(function(visility) { visility.should.be.false(); }) .click('#teamTabItem1') - .waitForVisible('#mattermostView1') + .pause(1000) + .waitForVisible('#mattermostView1', 1000) .isVisible('#mattermostView0').then(function(visility) { visility.should.be.false(); })