Apply suggestions from code review

Co-authored-by: Pranav K <[email protected]>

Author: captainsafia

src/Components/Components/src/Routing/Router.cs

@@ -192,7 +192,7 @@ internal virtual void Refresh(bool isNavigationIntercepted)
             }
         }
 
-        private async ValueTask<bool> RunOnNavigateAsync(string path, Task prevOnNavigate)
+        private async ValueTask<bool> RunOnNavigateAsync(string path, Task previousOnNavigate)
         {
             // If this router instance does not provide an OnNavigateAsync parameter
             // then we render the component associated with the route as per usual.
@@ -206,7 +206,7 @@ private async ValueTask<bool> RunOnNavigateAsync(string path, Task prevOnNavigat
             _onNavigateCts?.Cancel();
             // Then make sure that the task has been completed cancelled or
             // completed before continuing with the execution of this current task.
-            await prevOnNavigate;
+            await previousOnNavigate;
 
             // Create a new cancellation token source for this instance
             _onNavigateCts = new CancellationTokenSource();

src/Components/Components/test/Routing/RouterTest.cs

@@ -79,7 +79,7 @@ async Task OnNavigateAsync(NavigationContext args)
             router.Object.RunOnNavigateWithRefreshAsync("feb", false);
 
             // Assert refresh should've only been called once for the second route
-            router.Verify(x => x.Refresh(It.IsAny<bool>()), Times.Once());
+            router.Verify(x => x.Refresh(false), Times.Once());
         }
 
         private Mock<Router> CreateMockRouter()