fix: adds grep fallback for win32

detox/package.json

@@ -76,6 +76,7 @@
       "src/utils/environment.js",
       "src/utils/logger.js",
       "src/utils/onTerminate.js",
+      "src/utils/pipeCommands.js",
       "src/utils/sleep.js",
       "AAPT.js",
       "ADB.js",

detox/src/devices/android/AAPT.js

@@ -2,6 +2,7 @@ const _ = require('lodash');
 const Environment = require('../../utils/environment');
 const path = require('path');
 const exec = require('../../utils/exec').execWithRetriesAndLogs;
+const egrep = require('../../utils/pipeCommands').search.fragment;
 const fsext = require('../../utils/fsext');
 
 class AAPT {
@@ -21,7 +22,11 @@ class AAPT {
 
   async getPackageName(apkPath) {
     await this._prepare();
-    const process = await exec(`${this.aaptBin} dump badging "${apkPath}" | grep package:.name=`);
+    const process = await exec(
+      `${this.aaptBin} dump badging "${apkPath}" | ${egrep("package: name=")}`,
+      undefined, undefined, 1
+    );
+
     const packageName = new RegExp(/package: name='([^']+)'/g).exec(process.stdout);
     return packageName[1];
   }

detox/src/devices/android/ADB.js

@@ -1,7 +1,7 @@
 const _ = require('lodash');
 const path = require('path');
 const {execWithRetriesAndLogs, spawnAndLog} = require('../../utils/exec');
-const regexEscape = require('../../utils/regexEscape');
+const pipeCommands = require('../../utils/pipeCommands');
 const EmulatorTelnet = require('./EmulatorTelnet');
 const Environment = require('../../utils/environment');
 
@@ -75,7 +75,10 @@ class ADB {
   }
 
   async pidof(deviceId, bundleId) {
-    const processes = await this.shell(deviceId, `ps | grep "${regexEscape(bundleId)}\\s*$"`).catch(() => '');
+    const bundleIdRegex = pipeCommands.escape.inQuotedRegexp(bundleId) + '\\s*$';
+    const grep = pipeCommands.search.regexp;
+
+    const processes = await this.shell(deviceId, `ps | ${grep(bundleIdRegex)}`).catch(() => '');
     if (!processes) {
       return NaN;
     }

detox/src/utils/pipeCommands.js

@@ -0,0 +1,41 @@
+function win32Implementation() {
+  const escapeInQuotedStringWin32 = (fragment) => fragment.replace(/"/g, '""');
+  const escapeInQuotedRegexpWin32 = escapeInQuotedStringWin32;
+  const searchRegexpWin32 = (pattern) => `findstr /R /C:"${escapeInQuotedStringWin32(pattern)}"`;
+  const searchFragmentWin32 = (fragment) => `findstr /C:"${escapeInQuotedStringWin32(fragment)}"`;
+
+  return {
+    escape: {
+      inQuotedString: escapeInQuotedStringWin32,
+      inQuotedRegexp: escapeInQuotedRegexpWin32,
+    },
+    search: {
+      regexp: searchRegexpWin32,
+      fragment: searchFragmentWin32,
+    },
+  };
+}
+
+function nixImplementation() {
+  const SPECIAL_CHARS = /(["\^\$\[\]\*\.\\])/g;
+
+  const escapeInQuotedStringNix = (fragment) => fragment.replace(/"/g, '\\"');
+  const escapeInQuotedRegexpNix = (fragment) => fragment.replace(SPECIAL_CHARS, "\\$1");
+  const searchRegexpNix = (pattern) => `grep "${escapeInQuotedStringNix(pattern)}"`;
+  const searchFragmentNix = (fragment) => `grep -e "${escapeInQuotedStringNix(fragment)}"`;
+
+  return {
+    escape: {
+      inQuotedString: escapeInQuotedStringNix,
+      inQuotedRegexp: escapeInQuotedRegexpNix,
+    },
+    search: {
+      regexp: searchRegexpNix,
+      fragment: searchFragmentNix,
+    },
+  };
+}
+
+module.exports = process.platform === 'win32'
+  ? win32Implementation()
+  : nixImplementation();

detox/src/utils/regexEscape.js

@@ -1,7 +0,0 @@
-const SPECIAL_CHARS = /([\^\$\[\]\*\.\\])/g;
-
-function regexEscape(exactString) {
-  return exactString.replace(SPECIAL_CHARS, "\\$1");
-}
-
-module.exports = regexEscape;