From ce129400f01d3ca34b9bd0ee237fc5da13dd1f70 Mon Sep 17 00:00:00 2001
From: andy840119 <andy840119@gmail.com>
Date: Thu, 7 Dec 2023 00:30:31 +0800
Subject: [PATCH] use  that text index helper instead.

---
 LrcParser.Tests/Model/TextIndexTest.cs | 84 ++++++++++++--------------
 1 file changed, 38 insertions(+), 46 deletions(-)

diff --git a/LrcParser.Tests/Model/TextIndexTest.cs b/LrcParser.Tests/Model/TextIndexTest.cs
index 5273ec4..80ec53a 100644
--- a/LrcParser.Tests/Model/TextIndexTest.cs
+++ b/LrcParser.Tests/Model/TextIndexTest.cs
@@ -1,8 +1,7 @@
 // Copyright (c) karaoke.dev <contact@karaoke.dev>. Licensed under the MIT Licence.
 // See the LICENCE file in the repository root for full licence text.
 
-using System;
-using LrcParser.Model;
+using LrcParser.Tests.Helper;
 using NUnit.Framework;
 
 namespace LrcParser.Tests.Model;
@@ -10,65 +9,58 @@ namespace LrcParser.Tests.Model;
 [TestFixture]
 public class TextIndexTest
 {
-    [TestCase(1, 1)]
-    [TestCase(1.5, 1.5)]
-    [TestCase(-1.5, -1.5)]
-    public void TestOperatorEqual(double tone1, double tone2)
+    [TestCase("1,start")]
+    [TestCase("1,end")]
+    [TestCase("-1,start")]
+    public void TestOperatorEqual(string textIndex1)
     {
-        Assert.AreEqual(numberToTextIndex(tone1), numberToTextIndex(tone2));
+        Assert.AreEqual(TestCaseTextIndexHelper.ParseTextIndex(textIndex1), TestCaseTextIndexHelper.ParseTextIndex(textIndex1));
     }
 
-    [TestCase(-1, 1)]
-    [TestCase(1, -1)]
-    [TestCase(1.5, -1.5)]
-    [TestCase(1.5, 1)]
-    [TestCase(-1.5, -1)]
-    [TestCase(-1.5, -2)]
-    public void TestOperatorNotEqual(double tone1, double tone2)
+    [TestCase("-1,start", "1,start")]
+    [TestCase("1,start", "-1,start")]
+    [TestCase("1,end", "-1,end")]
+    [TestCase("1,end", "1,start")]
+    [TestCase("-2,end", "-1,start")]
+    [TestCase("-2,end", "-2,start")]
+    public void TestOperatorNotEqual(string textIndex1, string textIndex2)
     {
-        Assert.AreNotEqual(numberToTextIndex(tone1), numberToTextIndex(tone2));
+        Assert.AreNotEqual(TestCaseTextIndexHelper.ParseTextIndex(textIndex1), TestCaseTextIndexHelper.ParseTextIndex(textIndex2));
     }
 
-    [TestCase(1, 0, true)]
-    [TestCase(1, 0.5, true)]
-    [TestCase(1, 1, false)]
-    [TestCase(1, 1.5, false)]
-    public void TestOperatorGreater(double tone1, double tone2, bool match)
+    [TestCase("1,start", "0,start", true)]
+    [TestCase("1,start", "0,end", true)]
+    [TestCase("1,start", "1,start", false)]
+    [TestCase("1,start", "1,end", false)]
+    public void TestOperatorGreater(string textIndex1, string textIndex2, bool match)
     {
-        Assert.AreEqual(numberToTextIndex(tone1) > numberToTextIndex(tone2), match);
+        Assert.AreEqual(TestCaseTextIndexHelper.ParseTextIndex(textIndex1) > TestCaseTextIndexHelper.ParseTextIndex(textIndex2), match);
     }
 
-    [TestCase(1, 0, true)]
-    [TestCase(1, 0.5, true)]
-    [TestCase(1, 1, true)]
-    [TestCase(1, 1.5, false)]
-    public void TestOperatorGreaterOrEqual(double tone1, double tone2, bool match)
+    [TestCase("1,start", "0,start", true)]
+    [TestCase("1,start", "0,end", true)]
+    [TestCase("1,start", "1,start", true)]
+    [TestCase("1,start", "1,end", false)]
+    public void TestOperatorGreaterOrEqual(string textIndex1, string textIndex2, bool match)
     {
-        Assert.AreEqual(numberToTextIndex(tone1) >= numberToTextIndex(tone2), match);
+        Assert.AreEqual(TestCaseTextIndexHelper.ParseTextIndex(textIndex1) >= TestCaseTextIndexHelper.ParseTextIndex(textIndex2), match);
     }
 
-    [TestCase(-1, 0, true)]
-    [TestCase(-1, -0.5, true)]
-    [TestCase(-1, -1, false)]
-    [TestCase(-1, -1.5, false)]
-    public void TestOperatorLess(double tone1, double tone2, bool match)
+    [TestCase("-1,start", "0,start", true)]
+    [TestCase("-1,start", "-1,end", true)]
+    [TestCase("-1,start", "-1,start", false)]
+    [TestCase("-1,start", "-2,end", false)]
+    public void TestOperatorLess(string textIndex1, string textIndex2, bool match)
     {
-        Assert.AreEqual(numberToTextIndex(tone1) < numberToTextIndex(tone2), match);
+        Assert.AreEqual(TestCaseTextIndexHelper.ParseTextIndex(textIndex1) < TestCaseTextIndexHelper.ParseTextIndex(textIndex2), match);
     }
 
-    [TestCase(-1, 0, true)]
-    [TestCase(-1, -0.5, true)]
-    [TestCase(-1, -1, true)]
-    [TestCase(-1, -1.5, false)]
-    public void TestOperatorLessOrEqual(double tone1, double tone2, bool match)
+    [TestCase("-1,start", "0,start", true)]
+    [TestCase("-1,start", "-1,end", true)]
+    [TestCase("-1,start", "-1,start", true)]
+    [TestCase("-1,start", "-2,end", false)]
+    public void TestOperatorLessOrEqual(string textIndex1, string textIndex2, bool match)
     {
-        Assert.AreEqual(numberToTextIndex(tone1) <= numberToTextIndex(tone2), match);
-    }
-
-    private static TextIndex numberToTextIndex(double tone)
-    {
-        var half = Math.Abs(tone) % 1 == 0.5;
-        var scale = tone < 0 ? (int)tone - (half ? 1 : 0) : (int)tone;
-        return new TextIndex(scale, half ? IndexState.End : IndexState.Start);
+        Assert.AreEqual(TestCaseTextIndexHelper.ParseTextIndex(textIndex1) <= TestCaseTextIndexHelper.ParseTextIndex(textIndex2), match);
     }
 }