fix(test lenses): Use single instead of double quotes non-Windows environments and escape nested quotes (#175)

* fix: Use single instead of double quotes and escape nested quotes

* Fix formatting

Co-authored-by: Jason Axelson <[email protected]>

Author: Étienne Lévesque

package.json

@@ -411,7 +411,8 @@
     "update-vscode": "node ./node_modules/vscode/bin/install",
     "test": "node ./out/test/runTest.js",
     "lint": "eslint . --ext .js,.ts",
-    "check-formatting": "prettier --check ./src/**/*.ts .*.js "
+    "check-formatting": "prettier --check ./src/**/*.ts .*.js ",
+    "fix-formatting": "prettier --write ./src/**/*.ts .*.js "
   },
   "devDependencies": {
     "@types/glob": "^7.1.3",

src/commands/runTestFromCodeLens.ts

@@ -19,14 +19,29 @@ export default function runFromCodeLens(args: RunArgs): void {
   elixirLsTerminal.sendText(buildTestCommand(args));
 }
 
+function escapeSingleQuotes(s: string): string {
+  return isWindows() ? s : s.replace(/'/g, "'\\''");
+}
+
+function quote(s: string): string {
+  const q = isWindows() ? '"' : `'`;
+  return [q, s, q].join("");
+}
+
+function isWindows(): boolean {
+  return process.platform.includes("win32");
+}
+
 function buildTestCommand(args: RunArgs): string {
   const testFilter = buildTestInclude(
     args.describe,
     args.testName,
     args.module
   );
 
-  return `mix test --exclude test --include "${testFilter}" ${args.filePath}`;
+  return `mix test --exclude test --include ${quote(
+    escapeSingleQuotes(testFilter)
+  )} ${args.filePath}`;
 }
 
 function buildTestInclude(