[release/6.0] Backport 'Fix Blazor RemoteAuthenticatorView processing the same action multiple times' (#44718)

* Fix Blazor RemoteAuthenticatorView processing the same action multiple times
* Fix components-e2e pipeline (#44402)

Author: MackinnonBuck

src/Components/WebAssembly/WebAssembly.Authentication/src/RemoteAuthenticatorViewCore.cs

@@ -18,6 +18,7 @@ public class RemoteAuthenticatorViewCore<TAuthenticationState> : ComponentBase w
         private string _message;
         private RemoteAuthenticationApplicationPathsOptions _applicationPaths;
         private string _action;
+        private string _lastHandledAction;
 
         /// <summary>
         /// Gets or sets the <see cref="RemoteAuthenticationActions"/> action the component needs to handle.
@@ -165,6 +166,13 @@ protected override void BuildRenderTree(RenderTreeBuilder builder)
         /// <inheritdoc />
         protected override async Task OnParametersSetAsync()
         {
+            if (_lastHandledAction == Action)
+            {
+                // Avoid processing the same action more than once.
+                return;
+            }
+
+            _lastHandledAction = Action;
             switch (Action)
             {
                 case RemoteAuthenticationActions.LogIn:

src/Components/WebAssembly/WebAssembly.Authentication/test/RemoteAuthenticatorCoreTests.cs

@@ -116,6 +116,55 @@ public async Task AuthenticationManager_Login_NavigatesToLoginFailureOnError()
             Assert.Equal("https://www.example.com/base/authentication/login-failed", remoteAuthenticator.Navigation.Uri.ToString());
         }
 
+        [Fact]
+        public async Task AuthenticationManager_Callbacks_OnlyExecuteOncePerAction()
+        {
+            // Arrange
+            var (remoteAuthenticator, renderer, authServiceMock, jsRuntime) = CreateAuthenticationManager(
+                "https://www.example.com/base/authentication/login-callback?code=1234");
+
+            authServiceMock.CompleteSignInCallback = s => Task.FromResult(new RemoteAuthenticationResult<RemoteAuthenticationState>()
+            {
+                Status = RemoteAuthenticationStatus.Success,
+            });
+
+            authServiceMock.CompleteSignOutCallback = s => Task.FromResult(new RemoteAuthenticationResult<RemoteAuthenticationState>()
+            {
+                Status = RemoteAuthenticationStatus.Success,
+            });
+
+            var logInCallbackInvocationCount = 0;
+            var logOutCallbackInvocationCount = 0;
+
+            var parameterDictionary = new Dictionary<string, object>
+            {
+                [_action] = RemoteAuthenticationActions.LogInCallback,
+                [_onLogInSucceded] = new EventCallbackFactory().Create<RemoteAuthenticationState>(
+                    remoteAuthenticator,
+                    (state) => logInCallbackInvocationCount++),
+                [_onLogOutSucceeded] = new EventCallbackFactory().Create<RemoteAuthenticationState>(
+                    remoteAuthenticator,
+                    (state) => logOutCallbackInvocationCount++)
+            };
+
+            var initialParameters = ParameterView.FromDictionary(parameterDictionary);
+
+            parameterDictionary[_action] = RemoteAuthenticationActions.LogOutCallback;
+
+            var finalParameters = ParameterView.FromDictionary(parameterDictionary);
+
+            // Act
+            await renderer.Dispatcher.InvokeAsync<object>(() => remoteAuthenticator.SetParametersAsync(initialParameters));
+            await renderer.Dispatcher.InvokeAsync<object>(() => remoteAuthenticator.SetParametersAsync(initialParameters));
+
+            await renderer.Dispatcher.InvokeAsync<object>(() => remoteAuthenticator.SetParametersAsync(finalParameters));
+            await renderer.Dispatcher.InvokeAsync<object>(() => remoteAuthenticator.SetParametersAsync(finalParameters));
+
+            // Assert
+            Assert.Equal(1, logInCallbackInvocationCount);
+            Assert.Equal(1, logOutCallbackInvocationCount);
+        }
+
         [Fact]
         public async Task AuthenticationManager_LoginCallback_ThrowsOnRedirectResult()
         {

src/Components/test/E2ETest/Infrastructure/WebDriverExtensions/BasicTestAppAuthenticationWebDriverExtensions.cs

@@ -22,13 +22,11 @@ public static void SignInAs(this IWebDriver browser, Uri baseUri, string usernam
             {
                 // Some tests need to change the authentication state without discarding the
                 // original page, but this adds several seconds of delay
-                var javascript = (IJavaScriptExecutor)browser;
                 var originalWindow = browser.CurrentWindowHandle;
-                javascript.ExecuteScript("window.open()");
-                browser.SwitchTo().Window(browser.WindowHandles.Last());
+                browser.SwitchTo().NewWindow(WindowType.Tab);
                 browser.Navigate(baseUri, baseRelativeUri, noReload: false);
                 browser.Exists(By.CssSelector("h1#authentication"));
-                javascript.ExecuteScript("window.close()");
+                browser.Close();
                 browser.SwitchTo().Window(originalWindow);
             }
             else