suppress validation of output paths in transpile scenarios

vladima · vladima · commit b0b0712c73d7 · 2016-02-01T10:36:47.000-08:00
diff --git a/src/compiler/program.ts b/src/compiler/program.ts
@@ -1717,7 +1717,7 @@ namespace ts {
             }
 
             // If the emit is enabled make sure that every output file is unique and not overwriting any of the input files
-            if (!options.noEmit) {
+            if (!options.noEmit && !options.suppressOutputPathCheck) {
                 const emitHost = getEmitHost();
                 const emitFilesSeen = createFileMap<boolean>(!host.useCaseSensitiveFileNames() ? key => key.toLocaleLowerCase() : undefined);
                 forEachExpectedEmitFile(emitHost, (emitFileNames, sourceFiles, isBundledEmit) => {
diff --git a/src/compiler/types.ts b/src/compiler/types.ts
@@ -2426,6 +2426,8 @@ namespace ts {
 
         // Skip checking lib.d.ts to help speed up tests.
         /* @internal */ skipDefaultLibCheck?: boolean;
+        // Do not perform validation of output file name in transpile scenarios
+        /* @internal */ suppressOutputPathCheck?: boolean;
 
         [option: string]: string | number | boolean | TsConfigOnlyOptions;
     }
diff --git a/src/services/services.ts b/src/services/services.ts
@@ -1876,6 +1876,9 @@ namespace ts {
 
         options.isolatedModules = true;
 
+        // transpileModule does not write anything to disk so there is no need to verify that there are no conflicts between input and output paths. 
+        options.suppressOutputPathCheck = true;
+
         // Filename can be non-ts file.
         options.allowNonTsExtensions = true;
 
diff --git a/tests/cases/unittests/transpile.ts b/tests/cases/unittests/transpile.ts
@@ -291,5 +291,14 @@ var x = 0;`,
                 options: { compilerOptions: { jsx: JsxEmit.React, newLine: NewLineKind.LineFeed } }
             })
         });
+        it("transpile .js files", () => {
+            const input = "const a = 10;";
+            const output = `"use strict";\nvar a = 10;\n`;
+            test(input, {
+                expectedOutput: output,
+                options: { compilerOptions: { newLine: NewLineKind.LineFeed, module: ModuleKind.CommonJS }, fileName: "input.js", reportDiagnostics: true },
+                expectedDiagnosticCodes: []
+            });
+        })
     });
 }

Original file line number	Diff line number	Diff line change
`@@ -1717,7 +1717,7 @@ namespace ts {`
`1717`	`1717`	`}`
`1718`	`1718`
`1719`	`1719`	`// If the emit is enabled make sure that every output file is unique and not overwriting any of the input files`
`1720`		`- if (!options.noEmit) {`
	`1720`	`+ if (!options.noEmit && !options.suppressOutputPathCheck) {`
`1721`	`1721`	`const emitHost = getEmitHost();`
`1722`	`1722`	`const emitFilesSeen = createFileMap<boolean>(!host.useCaseSensitiveFileNames() ? key => key.toLocaleLowerCase() : undefined);`
`1723`	`1723`	`forEachExpectedEmitFile(emitHost, (emitFileNames, sourceFiles, isBundledEmit) => {`
Original file line number	Diff line number	Diff line change
`@@ -2426,6 +2426,8 @@ namespace ts {`
`2426`	`2426`
`2427`	`2427`	`// Skip checking lib.d.ts to help speed up tests.`
`2428`	`2428`	`/* @internal */ skipDefaultLibCheck?: boolean;`
	`2429`	`+ // Do not perform validation of output file name in transpile scenarios`
	`2430`	`+ /* @internal */ suppressOutputPathCheck?: boolean;`
`2429`	`2431`
`2430`	`2432`	`[option: string]: string \| number \| boolean \| TsConfigOnlyOptions;`
`2431`	`2433`	`}`