Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quarantine Microsoft.AspNetCore.Components.E2ETest.Tests.RoutingTest.NavigationLock_CanBlockExternalNavigation and friends #57153

Open
MackinnonBuck opened this issue Aug 2, 2024 · 5 comments
Assignees
Labels
area-blazor Includes: Blazor, Razor Components test-failure

Comments

@MackinnonBuck
Copy link
Member

Failing Test(s)

  • Microsoft.AspNetCore.Components.E2ETest.ServerExecutionTests.NavigationLockPrerenderingTest.NavigationIsLockedAfterPrerendering
  • Microsoft.AspNetCore.Components.E2ETest.Tests.RoutingTest.NavigationLock_CanBlockExternalNavigation
  • Microsoft.AspNetCore.Components.E2ETest.ServerExecutionTests.ServerRoutingTest.NavigationLock_CanBlockExternalNavigation

Error Message

OpenQA.Selenium.NoAlertPresentException : no such alert

Stacktrace

   at OpenQA.Selenium.WebDriver.UnpackAndThrowOnError(Response errorResponse, String commandToExecute)
   at OpenQA.Selenium.WebDriver.ExecuteAsync(String driverCommandToExecute, Dictionary`2 parameters)
   at OpenQA.Selenium.WebDriver.InternalExecute(String driverCommandToExecute, Dictionary`2 parameters)
   at OpenQA.Selenium.TargetLocator.Alert()
   at Microsoft.AspNetCore.Components.E2ETest.Tests.RoutingTest.NavigationLock_CanBlockExternalNavigation() in /home/vsts/work/1/s/src/Components/test/E2ETest/Tests/RoutingTest.cs:line 1100

Build

https://dev.azure.com/dnceng-public/public/_build/results?buildId=764364&view=ms.vss-test-web.build-test-results-tab

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-blazor Includes: Blazor, Razor Components label Aug 2, 2024
@javiercn javiercn added this to the 9.0-rc2 milestone Aug 5, 2024
@javiercn
Copy link
Member

javiercn commented Aug 5, 2024

@MackinnonBuck this issue has to remain open, isn't it?

@MackinnonBuck
Copy link
Member Author

@javiercn yes, I believe so.

@SteveSandersonMS
Copy link
Member

SteveSandersonMS commented Aug 6, 2024

Is this some breaking change inside a newer version of Selenium?

@amcasey
Copy link
Member

amcasey commented Aug 12, 2024

Is this affecting release/8.0 as well?

@halter73
Copy link
Member

I noticed this is still happening when I tried unquarantining the tests in #57978. Two of the failures were OpenQA.Selenium.NoAlertPresentException : no such alert as seen above, but one of them showed a different error:

OpenQA.Selenium.JavaScriptException : javascript error: Blazor is not defined
(Session info: chrome=128.0.6613.137)

   at OpenQA.Selenium.WebDriver.UnpackAndThrowOnError(Response errorResponse, String commandToExecute)
   at OpenQA.Selenium.WebDriver.ExecuteAsync(String driverCommandToExecute, Dictionary`2 parameters)
   at Microsoft.AspNetCore.Components.E2ETest.Tests.RoutingTest.SetAbsluteUrlViaPushState(String absoluteUri, Boolean forceLoad) in /home/vsts/work/1/s/src/Components/test/E2ETest/Tests/RoutingTest.cs:line 1792
   at Microsoft.AspNetCore.Components.E2ETest.Tests.RoutingTest.NavigationLock_CanBlockExternalNavigation() in /home/vsts/work/1/s/src/Components/test/E2ETest/Tests/RoutingTest.cs:line 1099
   at InvokeStub_RoutingTest.NavigationLock_CanBlockExternalNavigation(Object, Object, IntPtr*)
   at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

https://dev.azure.com/dnceng-public/public/_build/results?buildId=814364&view=ms.vss-test-web.build-test-results-tab&runId=21102442&resultId=101176&paneView=debug

@halter73 halter73 removed this from the 9.0-rc2 milestone Sep 22, 2024
halter73 added a commit that referenced this issue Sep 22, 2024
This reverts commit 763b6c6.
@mkArtakMSFT mkArtakMSFT added this to the .NET 10 Planning milestone Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components test-failure
Projects
None yet
Development

No branches or pull requests

6 participants