chore: Upgrade SA

Author: linkdotnet

Directory.Packages.props

@@ -6,7 +6,7 @@
 	<!-- Shared code analyzers used for all projects in the solution -->
 	<ItemGroup Condition="!$(MSBuildProjectName.EndsWith('samples'))">
 		<GlobalPackageReference Include="AsyncFixer" Version="1.6.0" PrivateAssets="All" IncludeAssets="Runtime;Build;Native;contentFiles;Analyzers"/>
-		<GlobalPackageReference Include="SonarAnalyzer.CSharp" Version="9.32.0.97167" PrivateAssets="All" IncludeAssets="Runtime;Build;Native;contentFiles;Analyzers"/>
+		<GlobalPackageReference Include="SonarAnalyzer.CSharp" Version="10.5.0.109200" PrivateAssets="All" IncludeAssets="Runtime;Build;Native;contentFiles;Analyzers"/>
 		<PackageVersion Include="Microsoft.CodeAnalysis.Analyzers" Version="3.11.0" />
 		<PackageVersion Include="Meziantou.Polyfill" Version="1.0.42" />
 	</ItemGroup>
@@ -28,7 +28,7 @@
 
 	<ItemGroup Label="System.Text.Json Vulnerability">
 		<!-- Due to a CVE in System.Text.Json we explicitly reference the latest version of System.Text.Json -->
-		<PackageVersion Include="System.Text.Json" Version="9.0.0"/>
+		<PackageVersion Include="System.Text.Json" Version="9.0.1"/>
 	</ItemGroup>
 
 	<ItemGroup Condition="'$(TargetFramework)' == 'netstandard2.1'">

tests/bunit.testassets/BlazorE2E/AddRemoveChildComponents.razor

@@ -10,7 +10,7 @@ Child components follow.
 
 @code {
     int numAdded = 0;
-    List<string> currentChildrenMessages = new List<string>();
+    private readonly List<string> currentChildrenMessages = new List<string>();
 
     void AddChild()
     {

tests/bunit.testassets/BlazorE2E/DataDashComponent.razor

@@ -1,5 +1,5 @@
 <div id="cool_beans" data-tab="@TabId">@TabId</div>
 
 @code {
-    string TabId = "17";
+    private readonly string TabId = "17";
 }

tests/bunit.testassets/BlazorE2E/DuplicateAttributesComponent.razor

@@ -13,7 +13,7 @@
 </div>
 
 @functions {
-    Dictionary<string, object> elementValues = new Dictionary<string, object>
+	private readonly Dictionary<string, object> elementValues = new Dictionary<string, object>
     {
         { "bool",  true },
         { "string", "middle-value" },

tests/bunit.testassets/BlazorE2E/KeyPressEventComponent.razor

@@ -9,7 +9,7 @@ Type here: <input @onkeypress=OnKeyPressed />
 </ul>
 
 @code {
-    List<string> keysPressed = new List<string>();
+	private readonly List<string> keysPressed = new List<string>();
 
     void OnKeyPressed(KeyboardEventArgs eventArgs)
     {

tests/bunit.testassets/BlazorE2E/MarkupBlockComponent.razor

@@ -28,7 +28,7 @@
     #nullable disable
     bool changeOutput;
 
-    string myMarkup = "<p class='markup-string-value'>This is a <em>markup string</em>.</p>";
+    private readonly string myMarkup = "<p class='markup-string-value'>This is a <em>markup string</em>.</p>";
 
     void EmitMarkupBlock(RenderTreeBuilder builder)
     {

tests/bunit.testassets/BlazorE2E/MovingCheckboxesComponent.razor

@@ -41,15 +41,15 @@
 
 @code {
     #nullable disable
-    List<TodoItem> items = new List<TodoItem>
+    private readonly List<TodoItem> items = new List<TodoItem>
     {
         new TodoItem { Text = "Alpha" },
         new TodoItem { Text = "Beta" },
         new TodoItem { Text = "Gamma", IsDone = true },
         new TodoItem { Text = "Delta", IsDone = true },
     };
 
-    class TodoItem
+    private sealed class TodoItem
     {
         public bool IsDone { get; set; }
         public string Text { get; set; }

tests/bunit.testassets/BlazorE2E/OrderedList.razor

@@ -6,7 +6,7 @@
     }
     @foreach (var item in Items)
     {
-        @Template(new Context { Index = index++, Item = item, });
+        @Template(new Context { Index = index++, Item = item, })
     }
 </ol>
 

tests/bunit.testassets/SampleComponents/AsyncRenderChangesProperty.razor

@@ -1,7 +1,7 @@
 <button id="tick" @onclick="Tick">Tick</button>
 <button id="tock" @onclick="Tock">Tock</button>
 @code {
-	private TaskCompletionSource<object?> _tcs = new TaskCompletionSource<object?>();
+	private readonly TaskCompletionSource<object?> _tcs = new TaskCompletionSource<object?>();
 	public int Counter;
 
 	private Task Tick()

tests/bunit.testassets/SampleComponents/InvokeAsyncInsideContinueWith.razor

@@ -22,13 +22,13 @@
 		{
 			registeredTask = task;
 			delegatedTask = task == null ? null : DelegateTo(task);
-			RenderWhenDone();
+			_ = RenderWhenDone();
 		}
 
 		base.OnParametersSet();
 	}
 
-	private async void RenderWhenDone()
+	private async Task RenderWhenDone()
 	{
 		var task = delegatedTask;
 		if (task != null)
@@ -44,7 +44,7 @@
 
 	private static async Task<object?> DelegateTo(Task task)
 	{
-		await task;//.ConfigureAwait(false);
+		await task;
 		return null;
 	}
 }