build(deps): Bump SonarAnalyzer.CSharp from 9.7.0.75501 to 9.9.0.77355

bUnit-dev · Sep 5, 2023 · 12f1cfd · 12f1cfd
1 parent 025edb0
commit 12f1cfd
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 17 deletions.
diff --git a/Directory.Build.props b/Directory.Build.props
@@ -51,7 +51,7 @@
 	<!-- Shared code analyzers used for all projects in the solution -->
 	<ItemGroup Label="Code Analyzers">
 		<PackageReference Include="AsyncFixer" Version="1.6.0" PrivateAssets="All" />
-		<PackageReference Include="SonarAnalyzer.CSharp" Version="9.7.0.75501" PrivateAssets="All" />
+		<PackageReference Include="SonarAnalyzer.CSharp" Version="9.9.0.77355" PrivateAssets="All" />
 	</ItemGroup>
 
 	<ItemGroup Label="Implicit usings"

diff --git a/...t.web/JSInterop/InvocationHandlers/Implementation/VirtualizeJSRuntimeInvocationHandler.cs b/...t.web/JSInterop/InvocationHandlers/Implementation/VirtualizeJSRuntimeInvocationHandler.cs
@@ -24,11 +24,10 @@ internal sealed class VirtualizeJSRuntimeInvocationHandler : JSRuntimeInvocation
 
 			var dotNetObjectReferenceVirtualizeJsInteropType = typeof(DotNetObjectReference<>).MakeGenericType(virtualizeJsInteropType);
 
-			var dotNetObjectReferenceValuePropertyInfo = dotNetObjectReferenceVirtualizeJsInteropType
-				.GetProperty("Value", BindingFlags.Public | BindingFlags.Instance)
+			var dotNetObjectReferenceValuePropertyInfo = dotNetObjectReferenceVirtualizeJsInteropType.GetProperty("Value", BindingFlags.Public | BindingFlags.Instance)
 				?? throw new InvalidOperationException("Did not find the Value property on the DotNetObjectReference<VirtualizeJsInterop> type.");
 
-			var onSpacerBeforeVisibleMethodInfo = virtualizeJsInteropType?.GetMethod("OnSpacerBeforeVisible")
+			var onSpacerBeforeVisibleMethodInfo = virtualizeJsInteropType.GetMethod("OnSpacerBeforeVisible")
 				?? throw new InvalidOperationException("Did not find the OnSpacerBeforeVisible method on the VirtualizeJsInterop type.");
 
 			return (dotNetObjectReferenceValuePropertyInfo, onSpacerBeforeVisibleMethodInfo);

diff --git a/tests/bunit.web.testcomponents.tests/RazorTesting/RazorTestSourceInformationProviderTest.cs b/tests/bunit.web.testcomponents.tests/RazorTesting/RazorTestSourceInformationProviderTest.cs
@@ -32,8 +32,8 @@ public void Test001(Type target, int testNumber, int expectedLineNumber)
 		var sourceInfo = sut.GetSourceInformation(target, GetTest(target, testNumber), testNumber);
 
 		sourceInfo.ShouldNotBeNull();
-		sourceInfo?.FileName.ShouldEndWith($"SampleComponents{Path.DirectorySeparatorChar}{target.Name}.razor", Case.Insensitive);
-		sourceInfo?.LineNumber.ShouldBe(expectedLineNumber);
+		sourceInfo.FileName.ShouldEndWith($"SampleComponents{Path.DirectorySeparatorChar}{target.Name}.razor", Case.Insensitive);
+		sourceInfo.LineNumber.ShouldBe(expectedLineNumber);
 	}
 
 	public void Dispose() => renderer.Dispose();

diff --git a/tests/bunit.web.testcomponents.tests/bunit.web.testcomponents.tests.csproj b/tests/bunit.web.testcomponents.tests/bunit.web.testcomponents.tests.csproj
@@ -15,7 +15,6 @@
 			<!-- some test fails with > 4.1.0 -->
 			<NoWarn>NU1605</NoWarn>
 		</PackageReference>
-		<PackageReference Update="SonarAnalyzer.CSharp" Version="9.7.0.75501" />	
 	</ItemGroup>
 
 	<ItemGroup>

diff --git a/tests/bunit.web.tests/TestDoubles/Authorization/FakeAuthenticationStateProviderTest.cs b/tests/bunit.web.tests/TestDoubles/Authorization/FakeAuthenticationStateProviderTest.cs
@@ -15,8 +15,8 @@ public async Task Test001()
 		// assert
 		Assert.NotNull(authState.User);
 		Assert.NotNull(authState.User.Identity);
-		Assert.Equal("TestUser", authState?.User?.Identity?.Name);
-		Assert.True(authState?.User?.Identity?.IsAuthenticated);
+		Assert.Equal("TestUser", authState.User?.Identity?.Name);
+		Assert.True(authState.User?.Identity?.IsAuthenticated);
 	}
 
 	[Fact(DisplayName = "Create unauthenticated AuthenticationState")]

diff --git a/tests/bunit.web.tests/TestDoubles/Authorization/FakeAuthorizationPolicyProviderTest.cs b/tests/bunit.web.tests/TestDoubles/Authorization/FakeAuthorizationPolicyProviderTest.cs
@@ -45,10 +45,10 @@ public async Task Test003()
 
 		// assert
 		Assert.NotNull(policy);
-		Assert.Equal(1, policy?.AuthenticationSchemes?.Count);
-		Assert.Equal("TestScheme:FooBar", policy?.AuthenticationSchemes?[0]);
-		Assert.Equal(1, policy?.Requirements?.Count);
-		Assert.IsType<TestPolicyRequirement>(policy?.Requirements?[0]);
+		Assert.Equal(1, policy.AuthenticationSchemes?.Count);
+		Assert.Equal("TestScheme:FooBar", policy.AuthenticationSchemes?[0]);
+		Assert.Equal(1, policy.Requirements?.Count);
+		Assert.IsType<TestPolicyRequirement>(policy.Requirements?[0]);
 	}
 
 	[Fact(DisplayName = "Get policy based on name not in the PolicyProvider.")]
@@ -63,10 +63,10 @@ public async Task Test004()
 
 		// assert
 		Assert.NotNull(policy);
-		Assert.Equal(1, policy?.AuthenticationSchemes?.Count);
-		Assert.Equal("TestScheme:OtherPolicy", policy?.AuthenticationSchemes?[0]);
-		Assert.Equal(1, policy?.Requirements?.Count);
-		Assert.IsType<TestPolicyRequirement>(policy?.Requirements?[0]);
+		Assert.Equal(1, policy.AuthenticationSchemes?.Count);
+		Assert.Equal("TestScheme:OtherPolicy", policy.AuthenticationSchemes?[0]);
+		Assert.Equal(1, policy.Requirements?.Count);
+		Assert.IsType<TestPolicyRequirement>(policy.Requirements?[0]);
 	}
 
 	[Fact(DisplayName = "Set Policies with empty scheme name.")]