From 23ed77f2c6f10135e3b849e5adf96008372b9cd4 Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Sat, 17 Jul 2021 14:33:02 +0900
Subject: [PATCH] Fix test failure under visual tests due to double firing

---
 .../UserInterface/TestSceneBeatSyncedContainer.cs  | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/osu.Game.Tests/Visual/UserInterface/TestSceneBeatSyncedContainer.cs b/osu.Game.Tests/Visual/UserInterface/TestSceneBeatSyncedContainer.cs
index 6b56f339d8..e5bcc08924 100644
--- a/osu.Game.Tests/Visual/UserInterface/TestSceneBeatSyncedContainer.cs
+++ b/osu.Game.Tests/Visual/UserInterface/TestSceneBeatSyncedContainer.cs
@@ -65,19 +65,16 @@ namespace osu.Game.Tests.Visual.UserInterface
 
             AddStep($"set mistimed to {(allowMistimed ? "allowed" : "disallowed")}", () => beatContainer.AllowMistimedEventFiring = allowMistimed);
 
-            AddStep("bind event", () =>
+            AddStep("Set time before zero", () =>
             {
                 beatContainer.NewBeat = (i, timingControlPoint, effectControlPoint, channelAmplitudes) =>
                 {
                     lastActuationTime = gameplayClockContainer.CurrentTime;
                     lastTimingPoint = timingControlPoint;
                     lastBeatIndex = i;
+                    beatContainer.NewBeat = null;
                 };
-            });
 
-            AddStep("Set time before zero", () =>
-            {
-                lastBeatIndex = null;
                 gameplayClockContainer.Seek(-1000);
             });
 
@@ -99,19 +96,14 @@ namespace osu.Game.Tests.Visual.UserInterface
             int? lastBeatIndex = null;
             double? lastBpm = null;
 
-            AddStep("bind event", () =>
+            AddStep("Set time before zero", () =>
             {
                 beatContainer.NewBeat = (i, timingControlPoint, effectControlPoint, channelAmplitudes) =>
                 {
                     lastBeatIndex = i;
                     lastBpm = timingControlPoint.BPM;
                 };
-            });
 
-            AddStep("Set time before zero", () =>
-            {
-                lastBeatIndex = null;
-                lastBpm = null;
                 gameplayClockContainer.Seek(-1000);
             });