Merge pull request #3103 from peppy/fix-testing-regression

Fix fatal test result on temporary file deletion failures

osu.Game.Tests/Visual/TestCasePlaySongSelect.cs

@@ -54,6 +54,12 @@ private class TestSongSelect : PlaySongSelect
             public new BeatmapCarousel Carousel => base.Carousel;
         }
 
+        protected override void Dispose(bool isDisposing)
+        {
+            factory.ResetDatabase();
+            base.Dispose(isDisposing);
+        }
+
         [BackgroundDependencyLoader]
         private void load()
         {

osu.Game/Tests/Visual/OsuTestCase.cs

@@ -62,7 +62,16 @@ protected override void Dispose(bool isDisposing)
             }
 
             if (localStorage.IsValueCreated)
-                localStorage.Value.DeleteDirectory(".");
+            {
+                try
+                {
+                    localStorage.Value.DeleteDirectory(".");
+                }
+                catch
+                {
+                    // we don't really care if this fails; it will just leave folders lying around from test runs.
+                }
+            }
         }
 
         protected override ITestCaseTestRunner CreateRunner() => new OsuTestCaseTestRunner();