From 45c0b74151c7f547838a98786c5859194ad3f442 Mon Sep 17 00:00:00 2001
From: Salman Ahmed <frenzibyte@gmail.com>
Date: Tue, 4 May 2021 23:41:46 +0300
Subject: [PATCH] Use LINQ select for data assigning for simplicity

To avoid having to read through all of frames and ensure nothing is failing there
---
 .../NonVisual/FramedReplayInputHandlerTest.cs | 46 ++++++++++---------
 1 file changed, 24 insertions(+), 22 deletions(-)

diff --git a/osu.Game.Tests/NonVisual/FramedReplayInputHandlerTest.cs b/osu.Game.Tests/NonVisual/FramedReplayInputHandlerTest.cs
index fe1186bf95..a9f9dfdc83 100644
--- a/osu.Game.Tests/NonVisual/FramedReplayInputHandlerTest.cs
+++ b/osu.Game.Tests/NonVisual/FramedReplayInputHandlerTest.cs
@@ -284,31 +284,33 @@ public void TestReplayFramesSortStability()
         {
             const double repeating_time = 5000;
 
+            int incrementingData = 0;
+
             // add a range of frames randomized in time but have a "data" assigned to them in ascending order.
             replay.Frames.AddRange(new[]
             {
-                new TestReplayFrame(repeating_time, true, 0),
-                new TestReplayFrame(0, true, 1),
-                new TestReplayFrame(3000, true, 2),
-                new TestReplayFrame(repeating_time, true, 3),
-                new TestReplayFrame(repeating_time, true, 4),
-                new TestReplayFrame(6000, true, 5),
-                new TestReplayFrame(9000, true, 6),
-                new TestReplayFrame(repeating_time, true, 7),
-                new TestReplayFrame(repeating_time, true, 8),
-                new TestReplayFrame(1000, true, 9),
-                new TestReplayFrame(11000, true, 10),
-                new TestReplayFrame(21000, true, 11),
-                new TestReplayFrame(4000, true, 12),
-                new TestReplayFrame(repeating_time, true, 13),
-                new TestReplayFrame(repeating_time, true, 14),
-                new TestReplayFrame(8000, true, 15),
-                new TestReplayFrame(2000, true, 16),
-                new TestReplayFrame(7000, true, 17),
-                new TestReplayFrame(repeating_time, true, 18),
-                new TestReplayFrame(repeating_time, true, 19),
-                new TestReplayFrame(10000, true, 20),
-            });
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(0, true),
+                new TestReplayFrame(3000, true),
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(6000, true),
+                new TestReplayFrame(9000, true),
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(1000, true),
+                new TestReplayFrame(11000, true),
+                new TestReplayFrame(21000, true),
+                new TestReplayFrame(4000, true),
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(8000, true),
+                new TestReplayFrame(2000, true),
+                new TestReplayFrame(7000, true),
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(repeating_time, true),
+                new TestReplayFrame(10000, true),
+            }.Select(f => new TestReplayFrame(f.Time, true, incrementingData++)));
 
             replay.HasReceivedAllFrames = true;