Merge pull request #239 from Unity-Technologies/UNI-47629-Fixing_cs_console_app_unit_test

lassond · web-flow · commit 10449fb9a7fb · 2018-05-17T16:19:53.000-04:00
UNI-47629 Fixing unit test
diff --git a/tests/UnityTests/Assets/Editor/UseCaseTests/EditorDependancyTest.cs b/tests/UnityTests/Assets/Editor/UseCaseTests/EditorDependancyTest.cs
@@ -64,6 +64,8 @@ public void TestCSharpConsoleApp()
             consoleApp.Start ();
 
             StringBuilder error = new StringBuilder ();
+
+            error.Append (consoleApp.StandardError.ReadToEnd ());
             while (!consoleApp.HasExited) {
                 error.Append (consoleApp.StandardError.ReadToEnd ());
             }
@@ -73,7 +75,6 @@ public void TestCSharpConsoleApp()
             Assert.IsTrue (errorString.Contains (
                 "Unhandled Exception: System.TypeInitializationException: " +
                 "The type initializer for 'Unity.FbxSdk.GlobalsPINVOKE' threw an exception"));
-            Assert.IsTrue (errorString.Contains ("UnityEditor"));
             Assert.IsTrue (errorString.Contains ("InitFbxAllocators"));
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,8 @@ public void TestCSharpConsoleApp()`
`64`	`64`	`consoleApp.Start ();`
`65`	`65`
`66`	`66`	`StringBuilder error = new StringBuilder ();`
	`67`	`+`
	`68`	`+ error.Append (consoleApp.StandardError.ReadToEnd ());`
`67`	`69`	`while (!consoleApp.HasExited) {`
`68`	`70`	`error.Append (consoleApp.StandardError.ReadToEnd ());`
`69`	`71`	`}`
`@@ -73,7 +75,6 @@ public void TestCSharpConsoleApp()`
`73`	`75`	`Assert.IsTrue (errorString.Contains (`
`74`	`76`	`"Unhandled Exception: System.TypeInitializationException: " +`
`75`	`77`	`"The type initializer for 'Unity.FbxSdk.GlobalsPINVOKE' threw an exception"));`
`76`		`- Assert.IsTrue (errorString.Contains ("UnityEditor"));`
`77`	`78`	`Assert.IsTrue (errorString.Contains ("InitFbxAllocators"));`
`78`	`79`	`}`
`79`	`80`	`}`