Skip to content

Commit 3efa346

Browse files
committed
Simplify Skip comments
1 parent e16afac commit 3efa346

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/Components/test/E2ETest/ServerExecutionTests/NavigationLockPrerenderingTest.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,7 @@ public NavigationLockPrerenderingTest(
2424
public override Task InitializeAsync()
2525
=> InitializeAsync(BrowserFixture.RoutingTestContext);
2626

27-
// See https://github.com/dotnet/aspnetcore/issues/57153
28-
[Fact(Skip = "BeforeUnload alert does not work with Selenium/ChromeDriver after update")]
27+
[Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/57153")]
2928
public void ExternalNavigationIsLockedAfterPrerendering()
3029
{
3130
Navigate("/locked-navigation");

src/Components/test/E2ETest/Tests/RoutingTest.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1155,8 +1155,7 @@ public void NavigationLock_HistoryNavigationWorks_AfterRefresh()
11551155
Browser.Equal("1", () => app.FindElement(By.Id("location-changed-count"))?.Text);
11561156
}
11571157

1158-
// See https://github.com/dotnet/aspnetcore/issues/57153
1159-
[Fact(Skip = "BeforeUnload alert does not work with Selenium/ChromeDriver after update")]
1158+
[Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/57153")]
11601159
public void NavigationLock_CanBlockExternalNavigation()
11611160
{
11621161
SetUrlViaPushState("/");

0 commit comments

Comments
 (0)