Do not emit files if noEmit is specified

sheetalkamat · sheetalkamat · commit acedf3c2472e · 2015-11-30T12:46:53.000-08:00
Handles microsoft#5799
diff --git a/src/compiler/declarationEmitter.ts b/src/compiler/declarationEmitter.ts
@@ -1675,7 +1675,7 @@ namespace ts {
     /* @internal */
     export function writeDeclarationFile(declarationFilePath: string, sourceFiles: SourceFile[], isBundledEmit: boolean, host: EmitHost, resolver: EmitResolver, emitterDiagnostics: DiagnosticCollection) {
         const emitDeclarationResult = emitDeclarations(host, resolver, emitterDiagnostics, declarationFilePath, sourceFiles, isBundledEmit);
-        const emitSkipped = emitDeclarationResult.reportedDeclarationError || host.isEmitBlocked(declarationFilePath);
+        const emitSkipped = emitDeclarationResult.reportedDeclarationError || host.isEmitBlocked(declarationFilePath) || host.getCompilerOptions().noEmit;
         if (!emitSkipped) {
             const declarationOutput = emitDeclarationResult.referencePathsOutput
                 + getDeclarationOutput(emitDeclarationResult.synchronousDeclarationOutput, emitDeclarationResult.moduleElementDeclarationEmitInfo);
diff --git a/src/compiler/emitter.ts b/src/compiler/emitter.ts
@@ -8221,7 +8221,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, Promi
         function emitFile({ jsFilePath, sourceMapFilePath, declarationFilePath}: { jsFilePath: string, sourceMapFilePath: string, declarationFilePath: string },
             sourceFiles: SourceFile[], isBundledEmit: boolean) {
             // Make sure not to write js File and source map file if any of them cannot be written
-            if (!host.isEmitBlocked(jsFilePath)) {
+            if (!host.isEmitBlocked(jsFilePath) && !compilerOptions.noEmit) {
                 emitJavaScript(jsFilePath, sourceMapFilePath, sourceFiles, isBundledEmit);
             }
             else {
diff --git a/src/harness/compilerRunner.ts b/src/harness/compilerRunner.ts
@@ -142,7 +142,7 @@ class CompilerBaselineRunner extends RunnerBase {
 
             it("Correct JS output for " + fileName, () => {
                 if (hasNonDtsFiles && this.emit) {
-                    if (result.files.length === 0 && result.errors.length === 0) {
+                    if (!options.noEmit && result.files.length === 0 && result.errors.length === 0) {
                         throw new Error("Expected at least one js file to be emitted or at least one error to be created.");
                     }
 
diff --git a/tests/baselines/reference/jsFileCompilationBindErrors.js b/tests/baselines/reference/jsFileCompilationBindErrors.js

Original file line number	Diff line number	Diff line change
`@@ -8221,7 +8221,7 @@ var __awaiter = (this && this.__awaiter) \|\| function (thisArg, _arguments, Promi`
`8221`	`8221`	`function emitFile({ jsFilePath, sourceMapFilePath, declarationFilePath}: { jsFilePath: string, sourceMapFilePath: string, declarationFilePath: string },`
`8222`	`8222`	`sourceFiles: SourceFile[], isBundledEmit: boolean) {`
`8223`	`8223`	`// Make sure not to write js File and source map file if any of them cannot be written`
`8224`		`- if (!host.isEmitBlocked(jsFilePath)) {`
	`8224`	`+ if (!host.isEmitBlocked(jsFilePath) && !compilerOptions.noEmit) {`
`8225`	`8225`	`emitJavaScript(jsFilePath, sourceMapFilePath, sourceFiles, isBundledEmit);`
`8226`	`8226`	`}`
`8227`	`8227`	`else {`
Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ class CompilerBaselineRunner extends RunnerBase {`
`142`	`142`
`143`	`143`	`it("Correct JS output for " + fileName, () => {`
`144`	`144`	`if (hasNonDtsFiles && this.emit) {`
`145`		`- if (result.files.length === 0 && result.errors.length === 0) {`
	`145`	`+ if (!options.noEmit && result.files.length === 0 && result.errors.length === 0) {`
`146`	`146`	`throw new Error("Expected at least one js file to be emitted or at least one error to be created.");`
`147`	`147`	`}`
`148`	`148`