Use FluentAssertions.Analyzers (#3314)

Author: Porges

src/ApiService/FunctionalTests/FunctionalTests.csproj

@@ -14,6 +14,10 @@
 
   <ItemGroup>
     <PackageReference Include="FluentAssertions" Version="6.11.0" />
+    <PackageReference Include="FluentAssertions.Analyzers" Version="0.21.0">
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+      <PrivateAssets>all</PrivateAssets>
+    </PackageReference>
     <PackageReference Include="Microsoft.Identity.Client" Version="4.52.0" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.6.2" />
     <PackageReference Include="System.Net.Http" Version="4.3.4" />

src/ApiService/FunctionalTests/packages.lock.json

@@ -17,6 +17,12 @@
           "System.Configuration.ConfigurationManager": "4.4.0"
         }
       },
+      "FluentAssertions.Analyzers": {
+        "type": "Direct",
+        "requested": "[0.21.0, )",
+        "resolved": "0.21.0",
+        "contentHash": "5CprzHStF627DqPytBnjRMXT2dvmXrP4XwVK4+jRllU5JH8SC6tytyyuOOd06tzjDPHU8YbDZ7/JtgJSiMb8RA=="
+      },
       "Microsoft.Identity.Client": {
         "type": "Direct",
         "requested": "[4.52.0, )",

src/ApiService/IntegrationTests/AgentCommandsTests.cs

@@ -49,7 +49,7 @@ await Context.InsertAll(new[] {
 
         var pendingNodeCommand = BodyAs<PendingNodeCommand>(result);
         pendingNodeCommand.Envelope.Should().NotBeNull();
-        pendingNodeCommand.Envelope?.Command.Should().BeEquivalentTo(command);
-        pendingNodeCommand.Envelope?.MessageId.Should().Be(messageId);
+        pendingNodeCommand.Envelope!.Command.Should().BeEquivalentTo(command);
+        pendingNodeCommand.Envelope.MessageId.Should().Be(messageId);
     }
 }

src/ApiService/IntegrationTests/IntegrationTests.csproj

@@ -6,6 +6,10 @@
 
   <ItemGroup>
     <PackageReference Include="FluentAssertions" Version="6.11.0" />
+    <PackageReference Include="FluentAssertions.Analyzers" Version="0.21.0">
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+      <PrivateAssets>all</PrivateAssets>
+    </PackageReference>
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.6.2" />
     <PackageReference Include="Moq" Version="4.18.4" />
     <PackageReference Include="System.Security.Cryptography.Pkcs" Version="7.0.2" />

src/ApiService/IntegrationTests/packages.lock.json

@@ -17,6 +17,12 @@
           "System.Configuration.ConfigurationManager": "4.4.0"
         }
       },
+      "FluentAssertions.Analyzers": {
+        "type": "Direct",
+        "requested": "[0.21.0, )",
+        "resolved": "0.21.0",
+        "contentHash": "5CprzHStF627DqPytBnjRMXT2dvmXrP4XwVK4+jRllU5JH8SC6tytyyuOOd06tzjDPHU8YbDZ7/JtgJSiMb8RA=="
+      },
       "Microsoft.NET.Test.Sdk": {
         "type": "Direct",
         "requested": "[17.6.2, )",

src/ApiService/Tests/Tests.csproj

@@ -5,6 +5,10 @@
   </PropertyGroup>
 
   <ItemGroup>
+    <PackageReference Include="FluentAssertions.Analyzers" Version="0.21.0">
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+      <PrivateAssets>all</PrivateAssets>
+    </PackageReference>
     <PackageReference Include="FsCheck" Version="2.16.5" />
     <PackageReference Include="FsCheck.Xunit" Version="2.16.5" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.6.2" />

src/ApiService/Tests/TruncationTests.cs

@@ -39,8 +39,8 @@ public static void TestNestedTruncation() {
 
         var truncatedEvent = eventCrashReported.Truncate(3) as EventCrashReported;
         truncatedEvent.Should().NotBeNull();
-        truncatedEvent?.Report.Executable.Should().Be("SOM");
-        truncatedEvent?.Report.CallStack.Count.Should().Be(0);
+        truncatedEvent!.Report.Executable.Should().Be("SOM");
+        truncatedEvent.Report.CallStack.Count.Should().Be(0);
     }
 
     [Fact]

src/ApiService/Tests/packages.lock.json

@@ -17,6 +17,12 @@
           "System.Configuration.ConfigurationManager": "4.4.0"
         }
       },
+      "FluentAssertions.Analyzers": {
+        "type": "Direct",
+        "requested": "[0.21.0, )",
+        "resolved": "0.21.0",
+        "contentHash": "5CprzHStF627DqPytBnjRMXT2dvmXrP4XwVK4+jRllU5JH8SC6tytyyuOOd06tzjDPHU8YbDZ7/JtgJSiMb8RA=="
+      },
       "FsCheck": {
         "type": "Direct",
         "requested": "[2.16.5, )",