Update failing test coverage and fix onFail being called too often

osu.Game.Tests/Visual/Gameplay/TestScenePlayerScoreSubmission.cs

@@ -179,7 +179,6 @@ namespace osu.Game.Tests.Visual.Gameplay
             addFakeHit();
 
             AddUntilStep("wait for fail", () => Player.GameplayState.HasFailed);
-            AddStep("exit", () => Player.Exit());
 
             AddUntilStep("wait for submission", () => Player.SubmittedScore != null);
             AddAssert("ensure failing submission", () => Player.SubmittedScore.ScoreInfo.Passed == false);

osu.Game/Screens/Play/Player.cs

@@ -933,6 +933,7 @@ namespace osu.Game.Screens.Play
             if (GameplayState.Mods.OfType<IApplicableFailOverride>().Any(m => m.RestartOnFail))
                 Restart(true);
 
+            OnFail();
             return true;
         }
 

osu.Game/Screens/Play/SubmittingPlayer.cs

@@ -54,8 +54,6 @@ namespace osu.Game.Screens.Play
             }
 
             AddInternal(new PlayerTouchInputDetector());
-
-            HealthProcessor.Failed += onFail;
         }
 
         protected override void LoadAsyncComplete()
@@ -167,10 +165,11 @@ namespace osu.Game.Screens.Play
             spectatorClient.BeginPlaying(token, GameplayState, Score);
         }
 
-        private bool onFail()
+        protected override void OnFail()
         {
+            base.OnFail();
+
             submitFromFailOrQuit();
-            return true;
         }
 
         public override bool OnExiting(ScreenExitEvent e)