Merge pull request #1019 from telerik/vladimirov/fix-tests

rosen-vladimirov · web-flow · commit 6da50075c033 · 2017-11-01T10:02:18.000+02:00
Fix unit tests on Windows
diff --git a/test/unit-tests/mobile/android/logcat-helper.ts b/test/unit-tests/mobile/android/logcat-helper.ts
@@ -11,14 +11,17 @@ class ChildProcessStub {
 	public spawn(command: string, args?: string[], options?: any): childProcess.ChildProcess {
 		ChildProcessStub.methodCallCount++;
 		let pathToExecutable = "";
+		let shell = "";
 		if (this.isWin) {
 			pathToExecutable = "type";
+			shell = "cmd";
 		} else {
 			pathToExecutable = "cat";
 		}
 		pathToExecutable = path.join(pathToExecutable);
 		const pathToSample = path.join(__dirname, "valid-sample.txt");
-		return childProcess.spawn(pathToExecutable, [pathToSample]);
+
+		return childProcess.spawn(pathToExecutable, [pathToSample], { shell });
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -11,14 +11,17 @@ class ChildProcessStub {`
`11`	`11`	`public spawn(command: string, args?: string[], options?: any): childProcess.ChildProcess {`
`12`	`12`	`ChildProcessStub.methodCallCount++;`
`13`	`13`	`let pathToExecutable = "";`
	`14`	`+ let shell = "";`
`14`	`15`	`if (this.isWin) {`
`15`	`16`	`pathToExecutable = "type";`
	`17`	`+ shell = "cmd";`
`16`	`18`	`} else {`
`17`	`19`	`pathToExecutable = "cat";`
`18`	`20`	`}`
`19`	`21`	`pathToExecutable = path.join(pathToExecutable);`
`20`	`22`	`const pathToSample = path.join(__dirname, "valid-sample.txt");`
`21`		`- return childProcess.spawn(pathToExecutable, [pathToSample]);`
	`23`	`+`
	`24`	`+ return childProcess.spawn(pathToExecutable, [pathToSample], { shell });`
`22`	`25`	`}`
`23`	`26`	`}`
`24`	`27`