Add graph ordering tests

RyanCavanaugh · RyanCavanaugh · commit 70fa29b627af · 2018-05-29T10:00:54.000-07:00
diff --git a/src/compiler/tsbuild.ts b/src/compiler/tsbuild.ts
@@ -650,7 +650,6 @@ namespace ts {
             }
         }
 
-        // TODO Accept parsedCommandLine instead?
         function buildSingleProject(proj: ResolvedConfigFileName): BuildResultFlags {
             if (context.options.dry) {
                 reportDiagnostic(createCompilerDiagnostic(Diagnostics.Would_build_project_0, proj));
diff --git a/src/harness/unittests/tsbuild.ts b/src/harness/unittests/tsbuild.ts
@@ -228,7 +228,15 @@ namespace ts {
         });
 
         it("orders the graph correctly - multiple parts of the same graph in various orders", () => {
-            // TODO add cases here
+            checkGraphOrdering(["A"], ["A", "B", "C", "D", "E"]);
+            checkGraphOrdering(["A", "C", "D"], ["A", "B", "C", "D", "E"]);
+            checkGraphOrdering(["D", "C", "A"], ["A", "B", "C", "D", "E"]);
+        });
+
+        it("orders the graph correctly - other orderings", () => {
+            checkGraphOrdering(["F"], ["F", "E"]);
+            checkGraphOrdering(["E"], ["E"]);
+            checkGraphOrdering(["F", "C", "A"], ["A", "B", "C", "D", "E", "F"]);
         });
 
         function checkGraphOrdering(rootNames: string[], expectedBuildSet: string[]) {

Original file line number	Diff line number	Diff line change
`@@ -650,7 +650,6 @@ namespace ts {`
`650`	`650`	`}`
`651`	`651`	`}`
`652`	`652`
`653`		`- // TODO Accept parsedCommandLine instead?`
`654`	`653`	`function buildSingleProject(proj: ResolvedConfigFileName): BuildResultFlags {`
`655`	`654`	`if (context.options.dry) {`
`656`	`655`	`reportDiagnostic(createCompilerDiagnostic(Diagnostics.Would_build_project_0, proj));`