Compare Qua.TimingGroups properly

Quaver.API.Tests/Quaver/TestCaseQua.cs

@@ -267,6 +267,26 @@ public void StableSorting()
             Assert.Equal(unsorted.Bookmarks, sorted.Bookmarks, BookmarkInfo.ByValueComparer);
         }
 
+        [Fact]
+        public void TimingGroupsComparison()
+        {
+            var q1 = new Qua();
+            var q2 = new Qua();
+            Assert.True(q1.EqualByValue(q2));
+
+            q2.TimingGroups["a"] = new ScrollGroup();
+            Assert.False(q1.EqualByValue(q2));
+
+            q1.TimingGroups["a"] = new ScrollGroup();
+            Assert.True(q1.EqualByValue(q2));
+
+            ((ScrollGroup)q2.TimingGroups["a"]).ScrollVelocities.Add(new SliderVelocityInfo {Multiplier = 2});
+            Assert.False(q1.EqualByValue(q2));
+
+            ((ScrollGroup)q1.TimingGroups["a"]).ScrollVelocities.Add(new SliderVelocityInfo {Multiplier = 2});
+            Assert.True(q1.EqualByValue(q2));
+        }
+
         [Fact]
         public void SVNormalization()
         {

Quaver.API/Maps/Qua.cs

@@ -259,7 +259,7 @@ public bool EqualByValue(Qua other) =>
             TimingPoints.SequenceEqual(other.TimingPoints, TimingPointInfo.ByValueComparer) &&
             SliderVelocities.SequenceEqual(other.SliderVelocities, SliderVelocityInfo.ByValueComparer) &&
             InitialScrollVelocity == other.InitialScrollVelocity &&
-            TimingGroups.SequenceEqual(other.TimingGroups) &&
+            CompareTimingGroups(other.TimingGroups) &&
             BPMDoesNotAffectScrollVelocity == other.BPMDoesNotAffectScrollVelocity &&
             LegacyLNRendering == other.LegacyLNRendering &&
             HasScratchKey == other.HasScratchKey &&
@@ -270,6 +270,19 @@ public bool EqualByValue(Qua other) =>
             Bookmarks.SequenceEqual(other.Bookmarks, BookmarkInfo.ByValueComparer) &&
             RandomizeModifierSeed == other.RandomizeModifierSeed;
 
+        private bool CompareTimingGroups(Dictionary<string, TimingGroup> other)
+        {
+            if (TimingGroups == null || other == null || TimingGroups.Count != other.Count)
+                return false;
+            foreach (var (key, value) in TimingGroups)
+            {
+                if (!other.TryGetValue(key, out var otherGroup) || !otherGroup.Equals(value))
+                    return false;
+            }
+
+            return true;
+        }
+
         private static IDeserializer Deserializer =>
             new DeserializerBuilder()
                 .IgnoreUnmatchedProperties()

Quaver.API/Maps/Structures/ScrollGroup.cs

@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Linq;
 using MoonSharp.Interpreter;
 using Quaver.API.Helpers;
 using YamlDotNet.Serialization;
@@ -11,23 +12,38 @@
     ///     (<see cref="Qua.SliderVelocities"/>)
     /// </summary>
     [MoonSharpUserData]
     public class ScrollGroup : TimingGroup
     {
-        public float InitialScrollVelocity
-        {
-            get;
-            [MoonSharpHidden] set;
-        } = 1;
+        public float InitialScrollVelocity { get; [MoonSharpHidden] set; } = 1;
 
-        public List<SliderVelocityInfo> ScrollVelocities
-        {
-            get;
-            [MoonSharpHidden] set;
-        } = new List<SliderVelocityInfo>();
+        public List<SliderVelocityInfo> ScrollVelocities { get; [MoonSharpHidden] set; } =
+            new List<SliderVelocityInfo>();
 
         public SliderVelocityInfo GetScrollVelocityAt(double time)
         {
             return ScrollVelocities.AtTime((float)time);
         }
+
+        /// <summary>
+        /// </summary>
+        /// <param name="other"></param>
+        /// <returns></returns>
+        protected bool Equals(ScrollGroup other)
+        {
+            return base.Equals(other) && InitialScrollVelocity.Equals(other.InitialScrollVelocity) &&
+                   ScrollVelocities.SequenceEqual(other.ScrollVelocities, SliderVelocityInfo.ByValueComparer);
+        }
+
+        /// <summary>
+        /// </summary>
+        /// <param name="obj"></param>
+        /// <returns></returns>
+        public override bool Equals(object obj)
+        {
+            if (obj is null) return false;
+            if (ReferenceEquals(this, obj)) return true;
+            if (obj.GetType() != GetType()) return false;
+            return Equals((ScrollGroup)obj);
+        }
     }
 }

Quaver.API/Maps/Structures/TimingGroup.cs

@@ -16,7 +16,7 @@
     ///     to the <see cref="Id"/> of the desired <see cref="TimingGroup"/>.
     /// </summary>
     [MoonSharpUserData]
     public abstract class TimingGroup
     {
         /// <summary>
         ///     The color of the layer (default is white)
@@ -35,5 +35,26 @@
             byte.TryParse(tb, out var b)
                 ? Color.FromArgb(r, g, b)
                 : Color.White;
+
+        /// <summary>
+        /// </summary>
+        /// <param name="other"></param>
+        /// <returns></returns>
+        protected bool Equals(TimingGroup other)
+        {
+            return ColorRgb == other.ColorRgb;
+        }
+
+        /// <summary>
+        /// </summary>
+        /// <param name="obj"></param>
+        /// <returns></returns>
+        public override bool Equals(object obj)
+        {
+            if (obj is null) return false;
+            if (ReferenceEquals(this, obj)) return true;
+            if (obj.GetType() != GetType()) return false;
+            return Equals((TimingGroup)obj);
+        }
     }
 }