Fixed issue where beatbars could double up.

Author: neogeek

RhythmGameUtilities.Tests/Mocks.cs

@@ -70,7 +70,7 @@ public void TestCalculateBeatBars()
 
             var beatBars = Utilities.CalculateBeatBars(bpmChanges, resolution, timeSignature, true);
 
-            Assert.That(beatBars.Count, Is.EqualTo(446));
+            Assert.That(beatBars.Count, Is.EqualTo(440));
         }
 
     }

RhythmGameUtilities.Tests/Mocks/song.json

@@ -70,7 +70,7 @@ public void TestCalculateBeatBars()
 
             var beatBars = Utilities.CalculateBeatBars(bpmChanges, resolution, timeSignature, true);
 
-            Assert.That(beatBars.Count, Is.EqualTo(446));
+            Assert.That(beatBars.Count, Is.EqualTo(440));
         }
 
     }

RhythmGameUtilities.Tests/UtilitiesTest.cs

@@ -95,7 +95,7 @@ std::vector<BeatBar> CalculateBeatBars(std::map<int, int> bpmChanges,
         auto startTick = std::get<0>(keyValuePair);
         auto endTick = std::get<1>(keyValuePair);
 
-        for (auto tick = startTick; tick <= endTick; tick += resolution)
+        for (auto tick = startTick; tick < endTick; tick += resolution)
         {
             beatBars.push_back({tick, bpmChanges[startTick]});
 

UnityPackage/Editor/Tests/Mocks.cs

@@ -64,7 +64,7 @@ void testCalculateBeatBars()
 
     auto beatBars = CalculateBeatBars(bpmChanges, 192, 4, true);
 
-    assert(beatBars.size() == 446);
+    assert(beatBars.size() == 440);
 
     std::cout << ".";
 }

UnityPackage/Editor/Tests/UtilitiesTest.cs

@@ -50,7 +50,7 @@ void testCalculateBeatBarsInternal()
         CalculateBeatBarsInternal(&bpmChangesKeys[0], &bpmChangesValues[0],
                                   bpmChanges.size(), 192, 4, true, outSize);
 
-    assert(*outSize == 446);
+    assert(*outSize == 440);
 
     std::cout << ".";
 }