Migrate NUnit assertions to the constraint model in preparation of the NUnit 4 upgrade

src/AcceptanceTests/AcceptanceTests.csproj

@@ -18,6 +18,8 @@
   <ItemGroup>
     <PackageReference Include="NServiceBus.AcceptanceTests.Sources" Version="8.1.1" GeneratePathProperty="true" />
     <PackageReference Include="GitHubActionsTestLogger" Version="2.4.1" />
+    <PackageReference Include="NUnit" Version="3.14.0" />
+    <PackageReference Include="NUnit.Analyzers" Version="4.3.0" />
   </ItemGroup>
 
   <ItemGroup>

src/AcceptanceTests/Outbox/When_trying_to_share_outbox_transaction.cs

@@ -20,8 +20,11 @@ public async Task Should_share_it()
                 .Done(c => c.OrderAckReceived == 1)
                 .Run(TimeSpan.FromSeconds(20));
 
-            Assert.AreEqual(1, context.OrderAckReceived, "Order ack should have been received");
-            Assert.AreEqual(context.TestRunId.ToString(), context.Value, "TestRunId should have been stored in dictionary");
+            Assert.Multiple(() =>
+            {
+                Assert.That(context.OrderAckReceived, Is.EqualTo(1), "Order ack should have been received");
+                Assert.That(context.Value, Is.EqualTo(context.TestRunId.ToString()), "TestRunId should have been stored in dictionary");
+            });
         }
 
         class Context : ScenarioContext

src/AcceptanceTests/SagaDataStorage/When_storing_saga_with_custom_saga_attribute.cs

@@ -32,7 +32,7 @@ public async Task Should_use_custom_collection_name()
             {
                 var conditionalValue = await collection.TryGetValueAsync(tx, context.SagaId.Value);
 
-                Assert.True(conditionalValue.HasValue);
+                Assert.That(conditionalValue.HasValue, Is.True);
             }
         }
 

src/AcceptanceTests/SagaDataStorage/When_storing_saga_with_default_collection_mapping.cs

@@ -29,7 +29,7 @@ public async Task Should_use_saga_data_type_name()
             {
                 var conditionalValue = await collection.TryGetValueAsync(tx, context.SagaId.Value);
 
-                Assert.True(conditionalValue.HasValue);
+                Assert.That(conditionalValue.HasValue, Is.True);
             }
         }
 

src/AcceptanceTests/SagaDataStorage/When_storing_saga_with_high_contention.cs

@@ -21,9 +21,12 @@ public async Task Should_succeed_without_retries()
                 .Done(c => c.Done)
                 .Run();
 
-            Assert.AreEqual(0, context.NumberOfRetries);
-            Assert.IsTrue(context.MessagesSent);
-            Assert.IsTrue(context.SagaStarted);
+            Assert.Multiple(() =>
+            {
+                Assert.That(context.NumberOfRetries, Is.EqualTo(0));
+                Assert.That(context.MessagesSent, Is.True);
+                Assert.That(context.SagaStarted, Is.True);
+            });
         }
 
         public class Context : ScenarioContext

src/ApiTests/ApiStandardTests.cs

@@ -19,14 +19,17 @@ public void VerifyFeatureNaming()
         {
             foreach (var featureType in GetFeatures())
             {
-                Assert.AreEqual("NServiceBus.Features", featureType.Namespace, "Features should be in the NServiceBus.Features namespace. " + featureType.FullName);
-                Assert.IsFalse(featureType.Name.EndsWith("Feature"), "Features should not be suffixed with 'Feature'. " + featureType.FullName);
+                Assert.Multiple(() =>
+                {
+                    Assert.That(featureType.Namespace, Is.EqualTo("NServiceBus.Features"), "Features should be in the NServiceBus.Features namespace. " + featureType.FullName);
+                    Assert.That(featureType.Name, Does.Not.EndWith("Feature"), "Features should not be suffixed with 'Feature'. " + featureType.FullName);
+                });
                 if (featureType.IsPublic)
                 {
                     var constructorInfo = featureType.GetConstructor(BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public, null, new Type[]
                     {
                     }, null);
-                    Assert.IsFalse(constructorInfo.IsPublic, "Features should have an internal constructor. " + featureType.FullName);
+                    Assert.That(constructorInfo.IsPublic, Is.False, "Features should have an internal constructor. " + featureType.FullName);
                 }
             }
         }
@@ -51,7 +54,7 @@ public void NonPublicShouldHaveSimpleNamespace()
                     x.Namespace != "NServiceBus.Persistence.ServiceFabric").ToList();
             if (types.Count > 0)
             {
-                Assert.IsEmpty(types, "Non public types should have 'NServiceBus.Persistence.ServiceFabric' namespace\r\n" + string.Join(Environment.NewLine, types.Select(x => x.FullName)));
+                Assert.That(types, Is.Empty, "Non public types should have 'NServiceBus.Persistence.ServiceFabric' namespace\r\n" + string.Join(Environment.NewLine, types.Select(x => x.FullName)));
             }
         }
 
@@ -71,7 +74,7 @@ public void LoggersShouldBeStaticField()
                 {
                     if (field.FieldType == typeof(ILog))
                     {
-                        Assert.IsTrue(field.IsStatic, "Logger fields should be static " + type.FullName);
+                        Assert.That(field.IsStatic, Is.True, "Logger fields should be static " + type.FullName);
                     }
                 }
             }
@@ -82,9 +85,12 @@ public void VerifyBehaviorNaming()
         {
             foreach (var featureType in GetBehaviors())
             {
-                Assert.IsFalse(featureType.IsPublic, "Behaviors should internal " + featureType.FullName);
-                Assert.AreEqual("NServiceBus", featureType.Namespace, "Behaviors should be in the NServiceBus namespace since it reduces the 'wall of text' problem when looking at pipeline stack traces. " + featureType.FullName);
-                Assert.IsTrue(featureType.Name.EndsWith("Terminator") || featureType.Name.EndsWith("Behavior") || featureType.Name.EndsWith("Connector"), "Behaviors should be suffixed with 'Behavior' or 'Connector'. " + featureType.FullName);
+                Assert.Multiple(() =>
+                {
+                    Assert.That(featureType.IsPublic, Is.False, "Behaviors should internal " + featureType.FullName);
+                    Assert.That(featureType.Namespace, Is.EqualTo("NServiceBus"), "Behaviors should be in the NServiceBus namespace since it reduces the 'wall of text' problem when looking at pipeline stack traces. " + featureType.FullName);
+                    Assert.That(featureType.Name.EndsWith("Terminator") || featureType.Name.EndsWith("Behavior") || featureType.Name.EndsWith("Connector"), Is.True, "Behaviors should be suffixed with 'Behavior' or 'Connector'. " + featureType.FullName);
+                });
             }
         }
 
@@ -93,7 +99,7 @@ public void VerifyAttributesAreSealed()
         {
             foreach (var attributeType in GetAttributeTypes())
             {
-                Assert.IsTrue(attributeType.IsSealed, attributeType.FullName + " should be sealed.");
+                Assert.That(attributeType.IsSealed, Is.True, attributeType.FullName + " should be sealed.");
             }
         }
 

src/ApiTests/ApiTests.csproj

@@ -13,6 +13,7 @@
     <PackageReference Include="GitHubActionsTestLogger" Version="2.4.1" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.0" />
     <PackageReference Include="NUnit" Version="3.14.0" />
+    <PackageReference Include="NUnit.Analyzers" Version="4.3.0" />
     <PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />
     <PackageReference Include="Particular.Approvals" Version="0.6.0" />
     <PackageReference Include="PublicApiGenerator" Version="11.1.0" />

src/PersistenceTests/PersistenceTests.csproj

@@ -16,13 +16,16 @@
     <ProjectReference Include="..\TestRunner\TestRunner.csproj" />
   </ItemGroup>
 
-  <ItemGroup>
-    <!-- DO NOT REMOVE Microsoft.Azure.ServiceFabric.Data, it is added so that dependabot knows about version changes-->
+  <ItemGroup Label="Pinned packages to prevent transitive dependencies issues">
     <PackageReference Include="Microsoft.ServiceFabric.Data" Version="6.1.1390" />
-    <!-- DO NOT REMOVE Newtonsoft.Json, it is added so that dependabot knows about version changes-->
     <PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
+  </ItemGroup>
+
+  <ItemGroup>
     <PackageReference Include="NServiceBus.PersistenceTests.Sources" Version="8.1.6" />
     <PackageReference Include="GitHubActionsTestLogger" Version="2.4.1" />
+    <PackageReference Include="NUnit" Version="3.14.0" />
+    <PackageReference Include="NUnit.Analyzers" Version="4.3.0" />
   </ItemGroup>
 
 </Project>

src/TestHarness/R.cs

@@ -1,6 +1,7 @@
 namespace TestHarness
 {
     using System;
+    using System.Collections;
     using System.Collections.Generic;
     using System.Collections.Specialized;
     using System.Fabric;
@@ -155,7 +156,7 @@ static async Task TearDown(CancellationToken cancellationToken = default)
             }
         }
 
-        public static IEnumerable<ITestCaseData> GetTestCases()
+        public static IEnumerable GetTestCases()
         {
             string[] enumerable;
             try

src/TestHarness/TestHarness.csproj

@@ -15,7 +15,9 @@
   <ItemGroup>
     <PackageReference Include="GitHubActionsTestLogger" Version="2.4.1" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.0" />
-    <PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />
+    <PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />    
+    <PackageReference Include="NUnit" Version="3.14.0" />
+    <PackageReference Include="NUnit.Analyzers" Version="4.3.0" />
   </ItemGroup>
 
 </Project>

src/TestRunner/TestRunner.csproj

@@ -10,6 +10,7 @@
   <ItemGroup>
     <PackageReference Include="Microsoft.ServiceFabric.Services.Remoting" Version="6.1.1390" />
     <PackageReference Include="NUnit" Version="3.14.0" />
+    <PackageReference Include="NUnit.Analyzers" Version="4.3.0" />
   </ItemGroup>
 
 </Project>

src/Tests/Outbox/ServiceFabricOutboxTransactionTests.cs

@@ -27,8 +27,11 @@ public async Task Commit_completes_transaction()
             {
                 var value = await dictionary.TryGetValueAsync(tx, "Key");
 
-                Assert.True(value.HasValue);
-                Assert.AreEqual("Value", value.Value);
+                Assert.Multiple(() =>
+                {
+                    Assert.That(value.HasValue, Is.True);
+                    Assert.That(value.Value, Is.EqualTo("Value"));
+                });
             }
         }
 
@@ -43,7 +46,7 @@ public async Task Dispose_transaction_owned_without_complete_rolls_back()
             {
                 var value = await dictionary.TryGetValueAsync(tx, "Key");
 
-                Assert.False(value.HasValue);
+                Assert.That(value.HasValue, Is.False);
             }
         }
 

src/Tests/SynchronizedStorage/ServiceFabricStorageSessionTests.cs

@@ -34,7 +34,7 @@ public async Task CompleteAsync_transaction_owned_completes_transaction()
                 var value = await dictionary.TryGetValueAsync(tx, "Key");
 
                 Assert.True(value.HasValue);
-                Assert.AreEqual("Value", value.Value);
+                Assert.That(value.Value, Is.EqualTo("Value"));
             }
         }
 
@@ -112,7 +112,7 @@ public async Task Dispose_transaction_not_owned_does_not_rollback_transaction()
                 var value = await dictionary.TryGetValueAsync(tx, "Key");
 
                 Assert.True(value.HasValue);
-                Assert.AreEqual("Value", value.Value);
+                Assert.That(value.Value, Is.EqualTo("Value"));
             }
         }
 

src/Tests/Testing/TestableStorageSessionTests.cs

@@ -41,8 +41,11 @@ public async Task CanBeUsed()
             {
                 var value = await dictionary.TryGetValueAsync(tx, "Key");
 
-                Assert.True(value.HasValue);
-                Assert.AreEqual("Value", value.Value);
+                Assert.Multiple(() =>
+                {
+                    Assert.That(value.HasValue, Is.True);
+                    Assert.That(value.Value, Is.EqualTo("Value"));
+                });
             }
         }
 

src/Tests/Tests.csproj

@@ -16,7 +16,9 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="NServiceBus.Testing" Version="8.1.0" />
+    <PackageReference Include="NServiceBus.Testing" Version="8.1.0" />    
+    <PackageReference Include="NUnit" Version="3.14.0" />
+    <PackageReference Include="NUnit.Analyzers" Version="4.3.0" />
   </ItemGroup>
 
 </Project>