Added initial support for filtering platform specific tests in cli (wix#435)

detox test -c ios.sim.release --platform ios will skip tests marked with :android: in their name
detox test -c android.emu.release --platform android will skip tests marked with :ios: in their name

Author: rotemmiz

detox/local-cli/detox-test.js

@@ -4,17 +4,24 @@ const program = require('commander');
 const path = require('path');
 const cp = require('child_process');
 program
-  .option('-o, --runner-config [config]', `Test runner config file, defaults to e2e/mocha.opts for mocha and e2e/config.json' for jest`)
-  .option('-l, --loglevel [value]', 'info, debug, verbose, silly, wss')
-  .option('-c, --configuration [device configuration]', 'Select a device configuration from your defined configurations,'
-                                                        + 'if not supplied, and there\'s only one configuration, detox will default to it')
-  .option('-r, --reuse', 'Reuse existing installed app (do not delete and re-install) for a faster run.')
-  .option('-u, --cleanup', 'Shutdown simulator when test is over, useful for CI scripts, to make sure detox exists cleanly with no residue')
+  .option('-o, --runner-config [config]',
+    `Test runner config file, defaults to e2e/mocha.opts for mocha and e2e/config.json' for jest`)
+  .option('-l, --loglevel [value]',
+    'info, debug, verbose, silly, wss')
+  .option('-c, --configuration [device configuration]',
+    'Select a device configuration from your defined configurations, if not supplied, and there\'s only one configuration, detox will default to it')
+  .option('-r, --reuse',
+    'Reuse existing installed app (do not delete and re-install) for a faster run.')
+  .option('-u, --cleanup',
+    'Shutdown simulator when test is over, useful for CI scripts, to make sure detox exists cleanly with no residue')
   .option('-d, --debug-synchronization [value]',
-    'When an action/expectation takes a significant amount of time use this option to print device synchronization status. '
+    'When an action/expectation takes a significant amount of time use this option to print device synchronization status.'
     + 'The status will be printed if the action takes more than [value]ms to complete')
-  .option('-a, --artifacts-location [path]', 'Artifacts destination path (currently will contain only logs). '
-                                             + 'If the destination already exists, it will be removed first')
+  .option('-a, --artifacts-location [path]',
+    'Artifacts destination path (currently will contain only logs). If the destination already exists, it will be removed first')
+  .option('-p, --platform [ios/android]',
+    'Run platform specific tests. Runs tests with invert grep on \':platform:\', '
+    + 'e.g test with substring \':ios:\' in its name will not run when passing \'--platform android\'')
   .parse(process.argv);
 
 const config = require(path.join(process.cwd(), 'package.json')).detox;
@@ -45,17 +52,19 @@ function runMocha() {
   const reuse = program.reuse ? `--reuse` : '';
   const artifactsLocation = program.artifactsLocation ? `--artifacts-location ${program.artifactsLocation}` : '';
   const configFile = runnerConfig ? `--opts ${runnerConfig}` : '';
+  const platform = program.platform ? `--grep ${getPlatformSpecificString(program.platform)} --invert` : '';
 
   const debugSynchronization = program.debugSynchronization ? `--debug-synchronization ${program.debugSynchronization}` : '';
-  const command = `node_modules/.bin/mocha ${testFolder} ${configFile} ${configuration} ${loglevel} ${cleanup} ${reuse} ${debugSynchronization} ${artifactsLocation}`;
+  const command = `node_modules/.bin/mocha ${testFolder} ${configFile} ${configuration} ${loglevel} ${cleanup} ${reuse} ${debugSynchronization} ${platform} ${artifactsLocation}`;
 
   console.log(command);
   cp.execSync(command, {stdio: 'inherit'});
 }
 
 function runJest() {
   const configFile = runnerConfig ? `--config=${runnerConfig}` : '';
-  const command = `node_modules/.bin/jest ${testFolder} ${configFile} --runInBand`;
+  const platform = program.platform ? `--testNamePattern='^((?!${getPlatformSpecificString(program.platform)}).)*$'` : '';
+  const command = `node_modules/.bin/jest ${testFolder} ${configFile} --runInBand ${platform}`;
   console.log(command);
   cp.execSync(command, {
     stdio: 'inherit',
@@ -70,16 +79,29 @@ function runJest() {
   });
 }
 
-
 function getDefaultRunnerConfig() {
   let defaultConfig;
   switch (runner) {
     case 'mocha':
       defaultConfig = 'e2e/mocha.opts';
       break;
     case 'jest':
-      defaultConfig = 'e2e/config.json'
+      defaultConfig = 'e2e/config.json';
+      break;
+    default:
+      console.log(`Missing 'runner-config' value in detox config in package.json, using '${defaultConfig}' as default for ${runner}`);
   }
-  console.log(`Missing 'runner-config' value in detox config in package.json, using '${defaultConfig}' as default for ${runner}`);
+
   return defaultConfig;
+}
+
+function getPlatformSpecificString(platform) {
+  let platformRevertString;
+  if (platform === 'ios') {
+    platformRevertString = ':android:';
+  } else if (platform === 'android') {
+    platformRevertString = ':ios:';
+  }
+
+  return platformRevertString;
 }

examples/demo-react-native/e2e/init.js

@@ -1,4 +1,3 @@
-require('babel-polyfill');
 const detox = require('detox');
 const config = require('../package.json').detox;