MSTEST0017 Assertion arguments should be passed in the correct order.

'actual' and 'expected'/'notExpected' arguments have been swapped.

Author: satano

src/MigrationTools.Clients.TfsObjectModel.Tests/Processors/TfsTeamSettingsProcessorTests.cs

@@ -1,5 +1,4 @@
-using Microsoft.Extensions.DependencyInjection;
-using Microsoft.VisualStudio.TestTools.UnitTesting;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
 using MigrationTools.Processors.Infrastructure;
 
 namespace MigrationTools.Processors.Tests
@@ -28,7 +27,7 @@ public void TfsTeamSettingsProcessorConfigureTest()
             };
             var x = GetTfsTeamSettingsProcessor(y);
             Assert.IsNotNull(x);
-            Assert.AreEqual(x.Options.SourceName, "TfsTeamSettingsSource");
+            Assert.AreEqual("TfsTeamSettingsSource", x.Options.SourceName);
             Assert.IsNotNull(x.Source);
         }
 
@@ -41,4 +40,4 @@ public void TfsTeamSettingsProcessorNoEnrichersTest()
             Assert.AreEqual(ProcessingStatus.Complete, processor.Status);
         }
     }
-}
+}

src/MigrationTools.Clients.TfsObjectModel.Tests/Tools/TfsRevisionManagerTests.cs

@@ -4,7 +4,6 @@
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.VisualStudio.TestTools.UnitTesting;
 using MigrationTools.DataContracts;
-using MigrationTools.Tests;
 using MigrationTools.Tools;
 
 
@@ -38,8 +37,7 @@ public void TfsRevisionManagerToolInSync1()
 
             var revs = processorEnricher.GetRevisionsToMigrate(source, target);
 
-            Assert.AreEqual(revs.Count, 0);
-
+            Assert.AreEqual(0, revs.Count);
         }
 
         [TestMethod(), TestCategory("L0")]