Merge pull request microsoft#7999 from Microsoft/fixHostOptionality

vladima · vladima · commit d280b1b7bc9f · 2016-04-11T09:34:43.000-07:00
fix issue with optionality of CompilerHost
diff --git a/src/compiler/program.ts b/src/compiler/program.ts
@@ -959,15 +959,16 @@ namespace ts {
 
         let resolvedTypeReferenceDirectives: Map<ResolvedTypeReferenceDirective> = {};
         let fileProcessingDiagnostics = createDiagnosticCollection();
-        let skipDefaultLib = options.noLib;
-        const programDiagnostics = createDiagnosticCollection();
-        const currentDirectory = host.getCurrentDirectory();
-        const supportedExtensions = getSupportedExtensions(options);
 
         const start = new Date().getTime();
 
         host = host || createCompilerHost(options);
 
+        let skipDefaultLib = options.noLib;
+        const programDiagnostics = createDiagnosticCollection();
+        const currentDirectory = host.getCurrentDirectory();
+        const supportedExtensions = getSupportedExtensions(options);
+
         // Map storing if there is emit blocking diagnostics for given input
         const hasEmitBlockingDiagnostics = createFileMap<boolean>(getCanonicalFileName);
 
diff --git a/tests/cases/unittests/reuseProgramStructure.ts b/tests/cases/unittests/reuseProgramStructure.ts
@@ -368,5 +368,11 @@ module ts {
             assert.isTrue(!program_3.structureIsReused);
             checkResolvedTypeDirectivesCache(program_1, "/a.ts", { "typedefs": { resolvedFileName: "/types/typedefs/index.d.ts", primary: true } });
         });
-    })
+    });
+
+    describe("host is optional", () => {
+        it("should work if host is not provided", () => {
+            createProgram([], {});
+        })
+    });
 }
