Skip to content

Commit 14f91dc

Browse files
committed
Minor issues fixes in tests.
1 parent c62bdcc commit 14f91dc

21 files changed

Lines changed: 36 additions & 34 deletions

tests/QuikGraph.Data.Tests/DataRelationEdgeTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ public void Equals()
7979
Assert.IsFalse(relation1.Equals(relation3));
8080
Assert.IsFalse(relation3.Equals(relation1));
8181

82-
Assert.AreNotEqual(relation1, null);
82+
Assert.AreNotEqual(null, relation1);
8383
Assert.IsFalse(relation1.Equals(null));
8484

8585
#region Local function

tests/QuikGraph.Graphviz.Tests/Dot/GraphvizColorTests.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,9 @@ public void Equals()
8585
Assert.IsTrue(color1 != color6);
8686
Assert.AreNotEqual(color1, color6);
8787

88-
Assert.AreNotEqual(color1, null);
88+
Assert.AreNotEqual(null, color1);
89+
Assert.IsFalse(color1.Equals(null));
90+
Assert.AreNotEqual(new TestClass(), color1);
8991
Assert.AreNotEqual(color1, new TestClass());
9092
}
9193

tests/QuikGraph.Tests/Algorithms/Condensation/CondensededEdgeTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using System;
1+
using System;
22
using NUnit.Framework;
33
using QuikGraph.Algorithms.Condensation;
44
using QuikGraph.Tests.Structures;
@@ -99,7 +99,7 @@ public void Equals()
9999
Assert.AreNotEqual(edge1, edge3);
100100
Assert.AreNotEqual(edge1, edge4);
101101

102-
Assert.AreNotEqual(edge1, null);
102+
Assert.AreNotEqual(null, edge1);
103103
}
104104
}
105105
}

tests/QuikGraph.Tests/Algorithms/Condensation/MergedEdgeTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using System;
1+
using System;
22
using NUnit.Framework;
33
using QuikGraph.Algorithms.Condensation;
44
using QuikGraph.Tests.Structures;
@@ -119,7 +119,7 @@ public void Equals()
119119
Assert.AreNotEqual(edge1, edge3);
120120
Assert.AreNotEqual(edge1, edge4);
121121

122-
Assert.AreNotEqual(edge1, null);
122+
Assert.AreNotEqual(null, edge1);
123123
}
124124

125125
[Test]

tests/QuikGraph.Tests/Algorithms/TSP/TaskPriorityTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using NUnit.Framework;
1+
using NUnit.Framework;
22
using QuikGraph.Algorithms.TSP;
33

44
namespace QuikGraph.Tests.Algorithms.TSP
@@ -50,7 +50,7 @@ public void Equals()
5050
Assert.IsTrue(priority1 != priority5);
5151
Assert.IsTrue(priority5 != priority1);
5252

53-
Assert.AreNotEqual(priority1, null);
53+
Assert.AreNotEqual(null, priority1);
5454
Assert.IsFalse(priority1.Equals(null));
5555
Assert.IsFalse(priority1 == null);
5656
Assert.IsFalse(null == priority1);

tests/QuikGraph.Tests/Structures/Edges/EdgeTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using System;
1+
using System;
22
using NUnit.Framework;
33

44
namespace QuikGraph.Tests.Structures
@@ -56,7 +56,7 @@ public void Equals()
5656
Assert.IsFalse(edge1.Equals(edge2));
5757
Assert.IsFalse(edge2.Equals(edge1));
5858

59-
Assert.AreNotEqual(edge1, null);
59+
Assert.AreNotEqual(null, edge1);
6060
Assert.IsFalse(edge1.Equals(null));
6161
}
6262

tests/QuikGraph.Tests/Structures/Edges/EquatableEdgeTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using System;
1+
using System;
22
using NUnit.Framework;
33

44
namespace QuikGraph.Tests.Structures
@@ -58,7 +58,7 @@ public void Equals()
5858
Assert.IsFalse(edge1.Equals(edge3));
5959
Assert.IsFalse(edge3.Equals(edge1));
6060

61-
Assert.AreNotEqual(edge1, null);
61+
Assert.AreNotEqual(null, edge1);
6262
Assert.IsFalse(edge1.Equals(null));
6363
}
6464

tests/QuikGraph.Tests/Structures/Edges/EquatableTaggedEdgeTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public void Equals()
7373
Assert.IsFalse(edge4.Equals(edge6));
7474
Assert.IsFalse(edge6.Equals(edge4));
7575

76-
Assert.AreNotEqual(edge1, null);
76+
Assert.AreNotEqual(null, edge1);
7777
Assert.IsFalse(edge1.Equals(null));
7878
}
7979

tests/QuikGraph.Tests/Structures/Edges/EquatableTermEdgeTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using System;
1+
using System;
22
using NUnit.Framework;
33

44
namespace QuikGraph.Tests.Structures
@@ -99,7 +99,7 @@ public void Equals()
9999
Assert.IsTrue(edge5.Equals(edge6));
100100
Assert.IsTrue(edge6.Equals(edge5));
101101

102-
Assert.AreNotEqual(edge1, null);
102+
Assert.AreNotEqual(null, edge1);
103103
Assert.IsFalse(edge1.Equals(null));
104104
}
105105

tests/QuikGraph.Tests/Structures/Edges/EquatableUndirectedEdgeTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using System;
1+
using System;
22
using NUnit.Framework;
33

44
namespace QuikGraph.Tests.Structures
@@ -60,7 +60,7 @@ public void Equals()
6060
Assert.IsTrue(edge1.Equals(edge2));
6161
Assert.IsTrue(edge2.Equals(edge1));
6262

63-
Assert.AreNotEqual(edge1, null);
63+
Assert.AreNotEqual(null, edge1);
6464
Assert.IsFalse(edge1.Equals(null));
6565
}
6666

0 commit comments

Comments
 (0)