Ensure host is in a running state before running tests further

osu.Framework/Testing/TestCase.cs

@@ -16,6 +16,7 @@
 using OpenTK;
 using OpenTK.Graphics;
 using System.Threading.Tasks;
+using System.Threading;
 
 namespace osu.Framework.Testing
 {
@@ -46,6 +47,11 @@ public void SetupGameHost()
                 throw new InvalidCastException($"The test runner must be a {nameof(Game)}.");
 
             runTask = Task.Factory.StartNew(() => host.Run(game), TaskCreationOptions.LongRunning);
+            while (!game.IsLoaded)
+            {
+                checkForErrors();
+                Thread.Sleep(10);
+            }
         }
 
         [OneTimeTearDown]
@@ -77,7 +83,18 @@ public void SetupTest()
         }
 
         [TearDown]
-        public void RunTests() => runner.RunTestBlocking(this);
+        public void RunTests()
+        {
+            checkForErrors();
+            runner.RunTestBlocking(this);
+            checkForErrors();
+        }
+
+        private void checkForErrors()
+        {
+            if (runTask.Exception != null)
+                throw runTask.Exception;
+        }
 
         /// <summary>
         /// Most derived usages of this start with TestCase. This will be removed for display purposes.

osu.Framework/Testing/TestCaseTestRunner.cs

@@ -2,6 +2,7 @@
 // Licensed under the MIT Licence - https://raw.githubusercontent.com/ppy/osu-framework/master/LICENCE
 
 using System;
+using System.Diagnostics;
 using System.Threading;
 using osu.Framework.Allocation;
 using osu.Framework.Configuration;
@@ -46,7 +47,7 @@ private void load(GameHost host, FrameworkConfigManager config)
 
             protected override void Dispose(bool isDisposing)
             {
-                volume.Value = volumeAtStartup;
+                if (volume != null) volume.Value = volumeAtStartup;
                 base.Dispose(isDisposing);
             }
 
@@ -65,7 +66,10 @@ protected override void LoadComplete()
             /// <param name="test">The <see cref="TestCase"/> to run.</param>
             public void RunTestBlocking(TestCase test)
             {
+                Trace.Assert(host != null, $"Ensure this runner has been loaded before calling {nameof(RunTestBlocking)}");
+
                 bool completed = false;
+                Exception exception = null;
 
                 void complete()
                 {
@@ -90,13 +94,16 @@ void complete()
                         Scheduler.AddDelayed(complete, time_between_tests);
                     }, e =>
                     {
+                        exception = e;
                         complete();
-                        throw new Exception("The test case threw an exception while running", e);
                     });
                 });
 
-                while (!completed && host?.ExecutionState != ExecutionState.Stopped)
+                while (!completed && host.ExecutionState == ExecutionState.Running)
                     Thread.Sleep(10);
+
+                if (exception != null)
+                    throw exception;
             }
         }
     }