From 3197f599bb2a9d5a1f516416293d5ac0ab0e636b Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Thu, 15 Jul 2021 16:01:46 +0900
Subject: [PATCH] Add failing test showing `OnNewBeat` can execute far away
 from an actual beat

---
 .../TestSceneBeatSyncedContainer.cs           | 29 ++++++++++++++++++-
 1 file changed, 28 insertions(+), 1 deletion(-)

diff --git a/osu.Game.Tests/Visual/UserInterface/TestSceneBeatSyncedContainer.cs b/osu.Game.Tests/Visual/UserInterface/TestSceneBeatSyncedContainer.cs
index 3ca6bf782a..5bfaf35bf1 100644
--- a/osu.Game.Tests/Visual/UserInterface/TestSceneBeatSyncedContainer.cs
+++ b/osu.Game.Tests/Visual/UserInterface/TestSceneBeatSyncedContainer.cs
@@ -56,7 +56,34 @@ namespace osu.Game.Tests.Visual.UserInterface
         }
 
         [Test]
-        public void TestFirstBeatAtFirstTimingPoint()
+        public void TestSeekBackDoesntPlayMidBeat()
+        {
+            int? lastBeatIndex = null;
+            double? lastActuationTime = null;
+            TimingControlPoint lastTimingPoint = null;
+
+            AddStep("bind event", () =>
+            {
+                beatContainer.NewBeat = (i, timingControlPoint, effectControlPoint, channelAmplitudes) =>
+                {
+                    lastActuationTime = gameplayClockContainer.CurrentTime;
+                    lastTimingPoint = timingControlPoint;
+                    lastBeatIndex = i;
+                };
+            });
+
+            AddStep("Set time before zero", () =>
+            {
+                lastBeatIndex = null;
+                gameplayClockContainer.Seek(-1000);
+            });
+
+            AddUntilStep("wait for trigger", () => lastBeatIndex != null);
+            AddAssert("trigger is near beat length", () => lastActuationTime != null && lastBeatIndex != null && Precision.AlmostEquals(lastTimingPoint.Time + lastBeatIndex.Value * lastTimingPoint.BeatLength, lastActuationTime.Value, 32));
+        }
+
+        [Test]
+        public void TestNegativeBeatsStillUsingBeatmapTiming()
         {
             int? lastBeatIndex = null;
             double? lastBpm = null;