phlux/mattermostest
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Improve the way to skip tests on specific condition

Browse Source
This commit is contained in:
Yuya Ochiai
2016-06-25 17:05:51 +09:00
parent 580bebdc66
commit 08aca0f235
2 changed files with 33 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
test/modules/environment.js
View File

@@ -35,12 +35,11 @@ module.exports = {
chaiAsPromised.transferPromiseness = app.transferPromiseness chaiAsPromised.transferPromiseness = app.transferPromiseness
return app; return app;
}, },
shouldTestForPlatforms: function(testCase, platforms) { // execute the test only when `condition` is true
if (platforms.indexOf(process.platform) !== -1) { shouldTest: function(it, condition) {
return; return condition ? it : it.skip;
} },
else { isOneOf(platforms) {
testCase.skip(); return (platforms.indexOf(process.platform) !== -1);
}
} }
} }

4
test/specs/browser/settings_test.js
View File

@@ -68,8 +68,8 @@ describe('browser/settings.html', function() {
}); });
[true, false].forEach(function(v) { [true, false].forEach(function(v) {
it(`should be saved and loaded: ${v}`, function() { env.shouldTest(it, env.isOneOf(['win32', 'linux']))
env.shouldTestForPlatforms(this, ['win32', 'linux']); (`should be saved and loaded: ${v}`, function() {
addClientCommands(this.app.client); addClientCommands(this.app.client);
return this.app.client return this.app.client
.loadSettingsPage() .loadSettingsPage()