Handle Pester Describe block strings with single quotes inside it (#1729)

bergmeister · TylerLeonhardt · commit fc0d52bd15bb · 2019-02-08T14:52:00.000-08:00
* Handle describe block strings with single quotes inside it.

* fix replace call to handle expression with more than one single quote as well
diff --git a/src/features/PesterTests.ts b/src/features/PesterTests.ts
@@ -72,6 +72,10 @@ export class PesterTestsFeature implements IFeature {
 
         if (describeBlockName) {
             launchConfig.args.push("-TestName");
+            // Escape single quotes inside double quotes by doubling them up
+            if (describeBlockName.includes("'")) {
+                describeBlockName = describeBlockName.replace(/'/g, "''");
+            }
             launchConfig.args.push(`'${describeBlockName}'`);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -72,6 +72,10 @@ export class PesterTestsFeature implements IFeature {`
`72`	`72`
`73`	`73`	`if (describeBlockName) {`
`74`	`74`	`launchConfig.args.push("-TestName");`
	`75`	`+ // Escape single quotes inside double quotes by doubling them up`
	`76`	`+ if (describeBlockName.includes("'")) {`
	`77`	`+ describeBlockName = describeBlockName.replace(/'/g, "''");`
	`78`	`+ }`
`75`	`79`	launchConfig.args.push(`'${describeBlockName}'`);
`76`	`80`	`}`
`77`	`81`