From 03d66dd2ef066b29e43c6ebce0cdf733e37cc5d4 Mon Sep 17 00:00:00 2001
From: Sheetal Nandi <shkamat@microsoft.com>
Date: Fri, 7 Dec 2018 12:27:23 -0800
Subject: [PATCH] Emit error tsserver tests into projectErrors

---
 src/testRunner/unittests/projectErrors.ts     | 83 +++++++++++++++++++
 .../unittests/tsserverProjectSystem.ts        | 83 -------------------
 2 files changed, 83 insertions(+), 83 deletions(-)

diff --git a/src/testRunner/unittests/projectErrors.ts b/src/testRunner/unittests/projectErrors.ts
index 6eb364b569..15bbab2a55 100644
--- a/src/testRunner/unittests/projectErrors.ts
+++ b/src/testRunner/unittests/projectErrors.ts
@@ -480,4 +480,87 @@ namespace ts.projectSystem {
             }
         });
     });
+
+    describe("tsserver:: Project Errors dont include overwrite emit error", () => {
+        it("for inferred project", () => {
+            const f1 = {
+                path: "/a/b/f1.js",
+                content: "function test1() { }"
+            };
+            const host = createServerHost([f1, libFile]);
+            const session = createSession(host);
+            openFilesForSession([f1], session);
+
+            const projectService = session.getProjectService();
+            checkNumberOfProjects(projectService, { inferredProjects: 1 });
+            const projectName = projectService.inferredProjects[0].getProjectName();
+
+            const diags = session.executeCommand(<server.protocol.CompilerOptionsDiagnosticsRequest>{
+                type: "request",
+                command: server.CommandNames.CompilerOptionsDiagnosticsFull,
+                seq: 2,
+                arguments: { projectFileName: projectName }
+            }).response as ReadonlyArray<protocol.DiagnosticWithLinePosition>;
+            assert.isTrue(diags.length === 0);
+
+            session.executeCommand(<server.protocol.SetCompilerOptionsForInferredProjectsRequest>{
+                type: "request",
+                command: server.CommandNames.CompilerOptionsForInferredProjects,
+                seq: 3,
+                arguments: { options: { module: ModuleKind.CommonJS } }
+            });
+            const diagsAfterUpdate = session.executeCommand(<server.protocol.CompilerOptionsDiagnosticsRequest>{
+                type: "request",
+                command: server.CommandNames.CompilerOptionsDiagnosticsFull,
+                seq: 4,
+                arguments: { projectFileName: projectName }
+            }).response as ReadonlyArray<protocol.DiagnosticWithLinePosition>;
+            assert.isTrue(diagsAfterUpdate.length === 0);
+        });
+
+        it("for external project", () => {
+            const f1 = {
+                path: "/a/b/f1.js",
+                content: "function test1() { }"
+            };
+            const host = createServerHost([f1, libFile]);
+            const session = createSession(host);
+            const projectService = session.getProjectService();
+            const projectFileName = "/a/b/project.csproj";
+            const externalFiles = toExternalFiles([f1.path]);
+            projectService.openExternalProject(<protocol.ExternalProject>{
+                projectFileName,
+                rootFiles: externalFiles,
+                options: {}
+            });
+
+            checkNumberOfProjects(projectService, { externalProjects: 1 });
+
+            const diags = session.executeCommand(<server.protocol.CompilerOptionsDiagnosticsRequest>{
+                type: "request",
+                command: server.CommandNames.CompilerOptionsDiagnosticsFull,
+                seq: 2,
+                arguments: { projectFileName }
+            }).response as ReadonlyArray<server.protocol.DiagnosticWithLinePosition>;
+            assert.isTrue(diags.length === 0);
+
+            session.executeCommand(<server.protocol.OpenExternalProjectRequest>{
+                type: "request",
+                command: server.CommandNames.OpenExternalProject,
+                seq: 3,
+                arguments: {
+                    projectFileName,
+                    rootFiles: externalFiles,
+                    options: { module: ModuleKind.CommonJS }
+                }
+            });
+            const diagsAfterUpdate = session.executeCommand(<server.protocol.CompilerOptionsDiagnosticsRequest>{
+                type: "request",
+                command: server.CommandNames.CompilerOptionsDiagnosticsFull,
+                seq: 4,
+                arguments: { projectFileName }
+            }).response as ReadonlyArray<server.protocol.DiagnosticWithLinePosition>;
+            assert.isTrue(diagsAfterUpdate.length === 0);
+        });
+    });
 }
diff --git a/src/testRunner/unittests/tsserverProjectSystem.ts b/src/testRunner/unittests/tsserverProjectSystem.ts
index 69de01cf96..214ddabdb8 100644
--- a/src/testRunner/unittests/tsserverProjectSystem.ts
+++ b/src/testRunner/unittests/tsserverProjectSystem.ts
@@ -5214,89 +5214,6 @@ var x = 10;`
         });
     });
 
-    describe("tsserverProjectSystem No overwrite emit error", () => {
-        it("for inferred project", () => {
-            const f1 = {
-                path: "/a/b/f1.js",
-                content: "function test1() { }"
-            };
-            const host = createServerHost([f1, libFile]);
-            const session = createSession(host);
-            openFilesForSession([f1], session);
-
-            const projectService = session.getProjectService();
-            checkNumberOfProjects(projectService, { inferredProjects: 1 });
-            const projectName = projectService.inferredProjects[0].getProjectName();
-
-            const diags = session.executeCommand(<server.protocol.CompilerOptionsDiagnosticsRequest>{
-                type: "request",
-                command: server.CommandNames.CompilerOptionsDiagnosticsFull,
-                seq: 2,
-                arguments: { projectFileName: projectName }
-            }).response as ReadonlyArray<protocol.DiagnosticWithLinePosition>;
-            assert.isTrue(diags.length === 0);
-
-            session.executeCommand(<server.protocol.SetCompilerOptionsForInferredProjectsRequest>{
-                type: "request",
-                command: server.CommandNames.CompilerOptionsForInferredProjects,
-                seq: 3,
-                arguments: { options: { module: ModuleKind.CommonJS } }
-            });
-            const diagsAfterUpdate = session.executeCommand(<server.protocol.CompilerOptionsDiagnosticsRequest>{
-                type: "request",
-                command: server.CommandNames.CompilerOptionsDiagnosticsFull,
-                seq: 4,
-                arguments: { projectFileName: projectName }
-            }).response as ReadonlyArray<protocol.DiagnosticWithLinePosition>;
-            assert.isTrue(diagsAfterUpdate.length === 0);
-        });
-
-        it("for external project", () => {
-            const f1 = {
-                path: "/a/b/f1.js",
-                content: "function test1() { }"
-            };
-            const host = createServerHost([f1, libFile]);
-            const session = createSession(host);
-            const projectService = session.getProjectService();
-            const projectFileName = "/a/b/project.csproj";
-            const externalFiles = toExternalFiles([f1.path]);
-            projectService.openExternalProject(<protocol.ExternalProject>{
-                projectFileName,
-                rootFiles: externalFiles,
-                options: {}
-            });
-
-            checkNumberOfProjects(projectService, { externalProjects: 1 });
-
-            const diags = session.executeCommand(<server.protocol.CompilerOptionsDiagnosticsRequest>{
-                type: "request",
-                command: server.CommandNames.CompilerOptionsDiagnosticsFull,
-                seq: 2,
-                arguments: { projectFileName }
-            }).response as ReadonlyArray<server.protocol.DiagnosticWithLinePosition>;
-            assert.isTrue(diags.length === 0);
-
-            session.executeCommand(<server.protocol.OpenExternalProjectRequest>{
-                type: "request",
-                command: server.CommandNames.OpenExternalProject,
-                seq: 3,
-                arguments: {
-                    projectFileName,
-                    rootFiles: externalFiles,
-                    options: { module: ModuleKind.CommonJS }
-                }
-            });
-            const diagsAfterUpdate = session.executeCommand(<server.protocol.CompilerOptionsDiagnosticsRequest>{
-                type: "request",
-                command: server.CommandNames.CompilerOptionsDiagnosticsFull,
-                seq: 4,
-                arguments: { projectFileName }
-            }).response as ReadonlyArray<server.protocol.DiagnosticWithLinePosition>;
-            assert.isTrue(diagsAfterUpdate.length === 0);
-        });
-    });
-
     describe("tsserverProjectSystem import helpers", () => {
         it("should not crash in tsserver", () => {
             const f1 = {