Changed all constructors to private.

Added additional unit test for operators.
TensionDev · Dec 30, 2024 · 320b754 · 320b754
1 parent 2fe6426
commit 320b754
Show file tree

Hide file tree

Showing 8 changed files with 210 additions and 8 deletions.
diff --git a/.github/workflows/package-release.yml b/.github/workflows/package-release.yml
@@ -21,10 +21,10 @@ jobs:
     # Steps represent a sequence of tasks that will be executed as part of the job
     steps:
       # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
 
       - name: Setup .NET 8
-        uses: actions/setup-dotnet@v1
+        uses: actions/setup-dotnet@v3
         with:
           dotnet-version: 8.0.x
           source-url: https://nuget.pkg.github.com/${{ github.repository_owner }}/index.json

diff --git a/UUIDUtil/UUIDv1.cs b/UUIDUtil/UUIDv1.cs
@@ -30,7 +30,7 @@ public class UUIDv1
         protected internal static readonly Object s_initLock = new Object();
         protected internal static readonly Object s_clockLock = new Object();
 
-        protected UUIDv1()
+        private UUIDv1()
         {
         }
 

diff --git a/UUIDUtil/UUIDv3.cs b/UUIDUtil/UUIDv3.cs
@@ -25,7 +25,7 @@ namespace TensionDev.UUID
     /// </summary>
     public class UUIDv3
     {
-        protected UUIDv3()
+        private UUIDv3()
         {
         }
 

diff --git a/UUIDUtil/UUIDv4.cs b/UUIDUtil/UUIDv4.cs
@@ -23,7 +23,7 @@ namespace TensionDev.UUID
     /// </summary>
     public class UUIDv4
     {
-        protected UUIDv4()
+        private UUIDv4()
         {
         }
 

diff --git a/UUIDUtil/UUIDv5.cs b/UUIDUtil/UUIDv5.cs
@@ -25,7 +25,7 @@ namespace TensionDev.UUID
     /// </summary>
     public class UUIDv5
     {
-        protected UUIDv5()
+        private UUIDv5()
         {
         }
 

diff --git a/UUIDUtil/UUIDv6.cs b/UUIDUtil/UUIDv6.cs
@@ -29,7 +29,7 @@ public class UUIDv6
         protected internal static readonly Object s_initLock = new Object();
         protected internal static readonly Object s_clockLock = new Object();
 
-        protected UUIDv6()
+        private UUIDv6()
         {
         }
 

diff --git a/UUIDUtil/UUIDv7.cs b/UUIDUtil/UUIDv7.cs
@@ -29,7 +29,7 @@ public class UUIDv7
         protected internal static UInt16 s_counter = 0;
         protected internal static readonly Object s_counterLock = new Object();
 
-        protected UUIDv7()
+        private UUIDv7()
         {
         }
 

diff --git a/XUnitTestProjectUUID/UnitTestUuid.cs b/XUnitTestProjectUUID/UnitTestUuid.cs
@@ -321,5 +321,207 @@ public void TestToString7()
 
             Assert.Throws<FormatException>(() => { uuid.ToString("C"); });
         }
+
+        [Fact]
+        public void TestOperatorEquals1()
+        {
+            object other = new object();
+            string vs = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid == other;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorEquals2()
+        {
+            string vs = "{7d444840-9dc0-11d1-b245-5ffdce74fad2}";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid1 == uuid2;
+            Assert.True(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorEquals3()
+        {
+            TensionDev.UUID.Uuid other = null;
+            string vs = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid == other;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorNotEquals1()
+        {
+            object other = new object();
+            string vs = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid != other;
+            Assert.True(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorNotEquals2()
+        {
+            string vs = "{7d444840-9dc0-11d1-b245-5ffdce74fad2}";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid1 != uuid2;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorNotEquals3()
+        {
+            TensionDev.UUID.Uuid other = null;
+            string vs = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid != other;
+            Assert.True(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorLessThan1()
+        {
+            string vs1 = "7d444830-9dc0-11d1-b245-5ffdce74fad2";
+            string vs2 = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs1);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs2);
+
+            bool actualResult = uuid1 < uuid2;
+            Assert.True(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorLessThan2()
+        {
+            string vs = "{7d444840-9dc0-11d1-b245-5ffdce74fad2}";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid1 < uuid2;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorLessThan3()
+        {
+            TensionDev.UUID.Uuid other = null;
+            string vs = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid < other;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorGreaterThan1()
+        {
+            string vs1 = "7d444830-9dc0-11d1-b245-5ffdce74fad2";
+            string vs2 = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs1);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs2);
+
+            bool actualResult = uuid1 > uuid2;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorGreaterThan2()
+        {
+            string vs = "{7d444840-9dc0-11d1-b245-5ffdce74fad2}";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid1 > uuid2;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorGreaterThan3()
+        {
+            TensionDev.UUID.Uuid other = null;
+            string vs = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid > other;
+            Assert.True(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorLessThanOrEqual1()
+        {
+            string vs1 = "7d444830-9dc0-11d1-b245-5ffdce74fad2";
+            string vs2 = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs1);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs2);
+
+            bool actualResult = uuid1 <= uuid2;
+            Assert.True(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorLessThanOrEqual2()
+        {
+            string vs = "{7d444840-9dc0-11d1-b245-5ffdce74fad2}";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid1 <= uuid2;
+            Assert.True(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorLessThanOrEqual3()
+        {
+            TensionDev.UUID.Uuid other = null;
+            string vs = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid <= other;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorGreaterThanOrEqual1()
+        {
+            string vs1 = "7d444830-9dc0-11d1-b245-5ffdce74fad2";
+            string vs2 = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs1);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs2);
+
+            bool actualResult = uuid1 >= uuid2;
+            Assert.False(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorGreaterThanOrEqual2()
+        {
+            string vs = "{7d444840-9dc0-11d1-b245-5ffdce74fad2}";
+            TensionDev.UUID.Uuid uuid1 = TensionDev.UUID.Uuid.Parse(vs);
+            TensionDev.UUID.Uuid uuid2 = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid1 >= uuid2;
+            Assert.True(actualResult);
+        }
+
+        [Fact]
+        public void TestOperatorGreaterThanOrEqual3()
+        {
+            TensionDev.UUID.Uuid other = null;
+            string vs = "7d444840-9dc0-11d1-b245-5ffdce74fad2";
+            TensionDev.UUID.Uuid uuid = TensionDev.UUID.Uuid.Parse(vs);
+
+            bool actualResult = uuid >= other;
+            Assert.True(actualResult);
+        }
     }
 }