Add more terminal link test cases

Tyriar · Tyriar · commit d5d8d98cf3ae · 2018-09-22T13:50:04.000-07:00
Related microsoft#59052
diff --git a/src/vs/workbench/parts/terminal/test/electron-browser/terminalLinkHandler.test.ts b/src/vs/workbench/parts/terminal/test/electron-browser/terminalLinkHandler.test.ts
@@ -62,7 +62,9 @@ suite('Workbench - TerminalLinkHandler', () => {
 					'c:/a/long/path',
 					'c:\\a\\long\\path',
 					'c:\\mixed/slash\\path',
-					'a/relative/path'
+					'a/relative/path',
+					'plain/path',
+					'plain\\path'
 				];
 
 				const supportedLinkFormats: LinkFormatInfo[] = [
@@ -179,6 +181,19 @@ suite('Workbench - TerminalLinkHandler', () => {
 
 			stub.restore();
 		});
+		test('Windows - spaces', () => {
+			const linkHandler = new TestTerminalLinkHandler(new TestXterm(), Platform.Windows, null, null, null, null);
+			linkHandler.initialCwd = 'C:\\base dir';
+
+			let stub = sinon.stub(path, 'join', function (arg1: string, arg2: string) {
+				return arg1 + '\\' + arg2;
+			});
+			assert.equal(linkHandler.preprocessPath('./src/file1'), 'C:\\base dir\\./src/file1');
+			assert.equal(linkHandler.preprocessPath('src\\file2'), 'C:\\base dir\\src\\file2');
+			assert.equal(linkHandler.preprocessPath('C:\\absolute\\path\\file3'), 'C:\\absolute\\path\\file3');
+
+			stub.restore();
+		});
 
 		test('Linux', () => {
 			const linkHandler = new TestTerminalLinkHandler(new TestXterm(), Platform.Linux, null, null, null, null);
