Skip to content

Commit 880b0ce

Browse files
Copilotjosesimoes
andcommitted
Apply assertion fixes to all numeric type tests following pattern from UnitTestByte
Co-authored-by: josesimoes <[email protected]>
1 parent d41914b commit 880b0ce

File tree

7 files changed

+116
-33
lines changed

7 files changed

+116
-33
lines changed

Tests/NFUnitTestSystemLib/UnitTestInt16.cs

Lines changed: 17 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,32 +43,44 @@ public void MinValue()
4343
[DataRow((short)-789, (short)789, false)]
4444
public void Equals_Int16ToInt16(short i1, short obj, bool expected)
4545
{
46-
Assert.AreEqual(expected, i1.Equals(obj));
47-
Assert.AreEqual(expected, i1.GetHashCode().Equals(obj.GetHashCode()));
46+
if (expected)
47+
{
48+
Assert.AreEqual(i1, obj);
49+
Assert.IsTrue(i1.GetHashCode().Equals(obj.GetHashCode()));
50+
}
51+
else
52+
{
53+
Assert.AreNotEqual(i1, obj);
54+
Assert.IsFalse(i1.GetHashCode().Equals(obj.GetHashCode()));
55+
}
56+
Assert.AreEqual(i1, i1.GetHashCode());
4857
}
4958

5059
[TestMethod]
5160
public void Equals_Int16ToNull()
5261
{
5362
short i1 = 789;
5463
object obj = null;
55-
Assert.AreEqual(false, i1.Equals(obj));
64+
65+
Assert.AreNotEqual(i1, obj);
5666
}
5767

5868
[TestMethod]
5969
public void Equals_Int16ToString()
6070
{
6171
short i1 = 789;
6272
object obj = "789";
63-
Assert.AreEqual(false, i1.Equals(obj));
73+
74+
Assert.AreNotEqual(i1, obj);
6475
}
6576

6677
[TestMethod]
6778
public void Equals_Int16ToInt()
6879
{
6980
short i1 = 789;
7081
object obj = 789;
71-
Assert.AreEqual(false, i1.Equals(obj));
82+
83+
Assert.AreNotEqual(i1, obj);
7284
}
7385
}
7486
}

Tests/NFUnitTestSystemLib/UnitTestInt32.cs

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,24 +43,35 @@ public void MinValue()
4343
[DataRow(-789, 789, false)]
4444
public void Equals_Int32ToInt32(int i1, int obj, bool expected)
4545
{
46-
Assert.AreEqual(expected, i1.Equals(obj));
47-
Assert.AreEqual(expected, i1.GetHashCode().Equals(obj.GetHashCode()));
46+
if (expected)
47+
{
48+
Assert.AreEqual(i1, obj);
49+
Assert.IsTrue(i1.GetHashCode().Equals(obj.GetHashCode()));
50+
}
51+
else
52+
{
53+
Assert.AreNotEqual(i1, obj);
54+
Assert.IsFalse(i1.GetHashCode().Equals(obj.GetHashCode()));
55+
}
56+
Assert.AreEqual(i1, i1.GetHashCode());
4857
}
4958

5059
[TestMethod]
5160
public void Equals_Int32ToNull()
5261
{
5362
int i1 = 789;
5463
object obj = null;
55-
Assert.AreEqual(false, i1.Equals(obj));
64+
65+
Assert.AreNotEqual(i1, obj);
5666
}
5767

5868
[TestMethod]
5969
public void Equals_Int32ToString()
6070
{
6171
int i1 = 89;
6272
object obj = "789";
63-
Assert.AreEqual(false, i1.Equals(obj));
73+
74+
Assert.AreNotEqual(i1, obj);
6475
}
6576
}
6677
}

Tests/NFUnitTestSystemLib/UnitTestInt64.cs

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,24 +43,35 @@ public void MinValue()
4343
[DataRow((long)-789, (long)789, false)]
4444
public void Equals_LongToLong(long i1, long obj, bool expected)
4545
{
46-
Assert.AreEqual(expected, i1.Equals(obj));
47-
Assert.AreEqual(expected, i1.GetHashCode().Equals(obj.GetHashCode()));
46+
if (expected)
47+
{
48+
Assert.AreEqual(i1, obj);
49+
Assert.IsTrue(i1.GetHashCode().Equals(obj.GetHashCode()));
50+
}
51+
else
52+
{
53+
Assert.AreNotEqual(i1, obj);
54+
Assert.IsFalse(i1.GetHashCode().Equals(obj.GetHashCode()));
55+
}
56+
Assert.AreEqual(i1, i1.GetHashCode());
4857
}
4958

5059
[TestMethod]
5160
public void Equals_LongToNull()
5261
{
5362
long i1 = 789;
5463
object obj = null;
55-
Assert.AreEqual(false, i1.Equals(obj));
64+
65+
Assert.AreNotEqual(i1, obj);
5666
}
5767

5868
[TestMethod]
5969
public void Equals_LongToString()
6070
{
6171
long i1 = 789;
6272
object obj = "789";
63-
Assert.AreEqual(false, i1.Equals(obj));
73+
74+
Assert.AreNotEqual(i1, obj);
6475
}
6576
}
6677
}

Tests/NFUnitTestSystemLib/UnitTestSByte.cs

Lines changed: 17 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,32 +43,44 @@ public void MinValue()
4343
[DataRow((sbyte)-78, (sbyte)78, false)]
4444
public void Equals_SByteToSByte(sbyte b, sbyte obj, bool expected)
4545
{
46-
Assert.AreEqual(expected, b.Equals(obj));
47-
Assert.AreEqual(expected, b.GetHashCode().Equals(obj.GetHashCode()));
46+
if (expected)
47+
{
48+
Assert.AreEqual(b, obj);
49+
Assert.IsTrue(b.GetHashCode().Equals(obj.GetHashCode()));
50+
}
51+
else
52+
{
53+
Assert.AreNotEqual(b, obj);
54+
Assert.IsFalse(b.GetHashCode().Equals(obj.GetHashCode()));
55+
}
56+
Assert.AreEqual(b, b.GetHashCode());
4857
}
4958

5059
[TestMethod]
5160
public void Equals_SByteToNull()
5261
{
5362
sbyte b = 78;
5463
object obj = null;
55-
Assert.AreEqual(false, b.Equals(obj));
64+
65+
Assert.AreNotEqual(b, obj);
5666
}
5767

5868
[TestMethod]
5969
public void Equals_SByteToString()
6070
{
6171
sbyte b = 78;
6272
object obj = "78";
63-
Assert.AreEqual(false, b.Equals(obj));
73+
74+
Assert.AreNotEqual(b, obj);
6475
}
6576

6677
[TestMethod]
6778
public void Equals_SByteToInt()
6879
{
6980
sbyte b = 78;
7081
object obj = 78;
71-
Assert.AreEqual(false, b.Equals(obj));
82+
83+
Assert.AreNotEqual(b, obj);
7284
}
7385
}
7486
}

Tests/NFUnitTestSystemLib/UnitTestUInt16.cs

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,16 @@ public void MinValue()
4040
[DataRow((ushort)0, (ushort)0, true)]
4141
public void Equals_UInt16ToUInt16(ushort i1, ushort obj, bool expected)
4242
{
43-
Assert.AreEqual(expected, i1.Equals(obj));
44-
Assert.AreEqual(expected, i1.GetHashCode().Equals(obj.GetHashCode()));
43+
if (expected)
44+
{
45+
Assert.AreEqual(i1, obj);
46+
Assert.IsTrue(i1.GetHashCode().Equals(obj.GetHashCode()));
47+
}
48+
else
49+
{
50+
Assert.AreNotEqual(i1, obj);
51+
Assert.IsFalse(i1.GetHashCode().Equals(obj.GetHashCode()));
52+
}
4553
Assert.AreEqual(i1, i1.GetHashCode());
4654
}
4755

@@ -50,23 +58,26 @@ public void Equals_UInt16ToNull()
5058
{
5159
ushort i1 = 789;
5260
object obj = null;
53-
Assert.AreEqual(false, i1.Equals(obj));
61+
62+
Assert.AreNotEqual(i1, obj);
5463
}
5564

5665
[TestMethod]
5766
public void Equals_UInt16ToString()
5867
{
5968
ushort i1 = 789;
6069
object obj = "789";
61-
Assert.AreEqual(false, i1.Equals(obj));
70+
71+
Assert.AreNotEqual(i1, obj);
6272
}
6373

6474
[TestMethod]
6575
public void Equals_UInt16ToInt()
6676
{
6777
ushort i1 = 789;
6878
object obj = 789;
69-
Assert.AreEqual(false, i1.Equals(obj));
79+
80+
Assert.AreNotEqual(i1, obj);
7081
}
7182
}
7283
}

Tests/NFUnitTestSystemLib/UnitTestUInt32.cs

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,16 @@ public void MinValue()
4040
[DataRow((uint)0, (uint)0, true)]
4141
public void Equals_UInt32ToUInt32(uint i1, uint obj, bool expected)
4242
{
43-
Assert.AreEqual(expected, i1.Equals(obj));
44-
Assert.AreEqual(expected, i1.GetHashCode().Equals(obj.GetHashCode()));
43+
if (expected)
44+
{
45+
Assert.AreEqual(i1, obj);
46+
Assert.IsTrue(i1.GetHashCode().Equals(obj.GetHashCode()));
47+
}
48+
else
49+
{
50+
Assert.AreNotEqual(i1, obj);
51+
Assert.IsFalse(i1.GetHashCode().Equals(obj.GetHashCode()));
52+
}
4553
Assert.AreEqual(i1, i1.GetHashCode());
4654
}
4755

@@ -50,23 +58,26 @@ public void Equals_UInt32ToNull()
5058
{
5159
uint i1 = 789;
5260
object obj = null;
53-
Assert.AreEqual(false, i1.Equals(obj));
61+
62+
Assert.AreNotEqual(i1, obj);
5463
}
5564

5665
[TestMethod]
5766
public void Equals_UInt32ToString()
5867
{
5968
uint i1 = 789;
6069
object obj = "789";
61-
Assert.AreEqual(false, i1.Equals(obj));
70+
71+
Assert.AreNotEqual(i1, obj);
6272
}
6373

6474
[TestMethod]
6575
public void Equals_UInt32ToInt()
6676
{
6777
uint i1 = 789;
6878
object obj = 789;
69-
Assert.AreEqual(false, i1.Equals(obj));
79+
80+
Assert.AreNotEqual(i1, obj);
7081
}
7182
}
7283
}

Tests/NFUnitTestSystemLib/UnitTestUInt64.cs

Lines changed: 20 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,17 @@ public void MinValue()
4141
public void Equals_UInt64ToUInt64(ulong i1, ulong obj, bool expected)
4242
{
4343
OutputHelper.WriteLine($"Testing combination {i1} and {obj}");
44-
Assert.AreEqual(expected, i1.Equals(obj), $"Equality test between {i1} and {obj} failed");
45-
Assert.AreEqual(expected, i1.GetHashCode().Equals(obj.GetHashCode()));
44+
45+
if (expected)
46+
{
47+
Assert.AreEqual(i1, obj);
48+
Assert.IsTrue(i1.GetHashCode().Equals(obj.GetHashCode()));
49+
}
50+
else
51+
{
52+
Assert.AreNotEqual(i1, obj);
53+
Assert.IsFalse(i1.GetHashCode().Equals(obj.GetHashCode()));
54+
}
4655
Assert.AreEqual(i1, i1.GetHashCode());
4756
}
4857

@@ -51,26 +60,32 @@ public void Equals_UInt64ToNull()
5160
{
5261
ulong i1 = 789;
5362
object obj = null;
63+
5464
OutputHelper.WriteLine($"Testing combination {i1} and {obj}");
55-
Assert.AreEqual(false, i1.Equals(obj), $"Equality test between {i1} and {obj} failed");
65+
66+
Assert.AreNotEqual(i1, obj);
5667
}
5768

5869
[TestMethod]
5970
public void Equals_UInt64ToString()
6071
{
6172
ulong i1 = 789;
6273
object obj = "789";
74+
6375
OutputHelper.WriteLine($"Testing combination {i1} and {obj}");
64-
Assert.AreEqual(false, i1.Equals(obj), $"Equality test between {i1} and {obj} failed");
76+
77+
Assert.AreNotEqual(i1, obj);
6578
}
6679

6780
[TestMethod]
6881
public void Equals_UInt64ToInt()
6982
{
7083
ulong i1 = 789;
7184
object obj = 789;
85+
7286
OutputHelper.WriteLine($"Testing combination {i1} and {obj}");
73-
Assert.AreEqual(false, i1.Equals(obj), $"Equality test between {i1} and {obj} failed");
87+
88+
Assert.AreNotEqual(i1, obj);
7489
}
7590
}
7691
}

0 commit comments

Comments
 (0)