From 8a1952e61ed87a377d7ebad7c25ed1f7dceb4cd3 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 3 Sep 2025 15:29:14 +0000 Subject: [PATCH 1/3] Initial plan From 3b0bd8d7826df0fe370be5e82662092163692476 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 3 Sep 2025 15:41:11 +0000 Subject: [PATCH 2/3] Skip Threading tests due to unreliability - replace QuarantinedTest with Skip attributes Co-authored-by: javiercn <6995051+javiercn@users.noreply.github.com> --- .../E2ETest/Tests/ThreadingAppHostedTest.cs | 12 ++++-------- .../test/E2ETest/Tests/ThreadingAppTest.cs | 18 ++++++------------ 2 files changed, 10 insertions(+), 20 deletions(-) diff --git a/src/Components/test/E2ETest/Tests/ThreadingAppHostedTest.cs b/src/Components/test/E2ETest/Tests/ThreadingAppHostedTest.cs index b9b6bd1db8a5..58e07d320b31 100644 --- a/src/Components/test/E2ETest/Tests/ThreadingAppHostedTest.cs +++ b/src/Components/test/E2ETest/Tests/ThreadingAppHostedTest.cs @@ -32,15 +32,13 @@ protected override void InitializeAsyncCore() WaitUntilLoaded(); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54761")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void HasHeading() { Assert.Equal("Hello, world!", Browser.Exists(By.TagName("h1")).Text); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54761")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void NavMenuHighlightsCurrentLocation() { var activeNavLinksSelector = By.CssSelector(".sidebar a.active"); @@ -66,8 +64,7 @@ public void NavMenuHighlightsCurrentLocation() item => Assert.Equal("Home", item.Text.Trim())); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54761")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void CounterPageCanUseThreads() { // Navigate to "Counter" @@ -84,8 +81,7 @@ public void CounterPageCanUseThreads() Browser.NotEqual("Current count: 0", () => Browser.Exists(By.CssSelector("h1 + p")).Text); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54761")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void HasFetchDataPage() { // Navigate to "Fetch data" diff --git a/src/Components/test/E2ETest/Tests/ThreadingAppTest.cs b/src/Components/test/E2ETest/Tests/ThreadingAppTest.cs index cd430537e177..e238d01bc19c 100644 --- a/src/Components/test/E2ETest/Tests/ThreadingAppTest.cs +++ b/src/Components/test/E2ETest/Tests/ThreadingAppTest.cs @@ -28,22 +28,19 @@ protected override void InitializeAsyncCore() WaitUntilLoaded(); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54761")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void HasTitle() { Assert.Equal("Blazor standalone", Browser.Title); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54761")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void HasHeading() { Assert.Equal("Hello, world!", Browser.Exists(By.TagName("h1")).Text); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54497")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void NavMenuHighlightsCurrentLocation() { var activeNavLinksSelector = By.CssSelector(".sidebar a.active"); @@ -69,8 +66,7 @@ public void NavMenuHighlightsCurrentLocation() item => Assert.Equal("Home", item.Text.Trim())); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/58242")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void CounterPageCanUseThreads() { // Navigate to "Counter" @@ -87,8 +83,7 @@ public void CounterPageCanUseThreads() Browser.NotEqual("Current count: 0", () => Browser.Exists(By.CssSelector("h1 + p")).Text); } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54761")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void HasFetchDataPage() { // Navigate to "Fetch data" @@ -109,8 +104,7 @@ public void HasFetchDataPage() } } - [Fact] - [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/54761")] + [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] public void IsStarted() { // Read from property From dd3bf820ee7d2c05e37e98521e7e3e777d622437 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 3 Sep 2025 16:07:09 +0000 Subject: [PATCH 3/3] Simplify skip messages to use only URL as requested Co-authored-by: javiercn <6995051+javiercn@users.noreply.github.com> --- .../test/E2ETest/Tests/ThreadingAppHostedTest.cs | 8 ++++---- .../test/E2ETest/Tests/ThreadingAppTest.cs | 12 ++++++------ 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/Components/test/E2ETest/Tests/ThreadingAppHostedTest.cs b/src/Components/test/E2ETest/Tests/ThreadingAppHostedTest.cs index 58e07d320b31..ff075400ada3 100644 --- a/src/Components/test/E2ETest/Tests/ThreadingAppHostedTest.cs +++ b/src/Components/test/E2ETest/Tests/ThreadingAppHostedTest.cs @@ -32,13 +32,13 @@ protected override void InitializeAsyncCore() WaitUntilLoaded(); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void HasHeading() { Assert.Equal("Hello, world!", Browser.Exists(By.TagName("h1")).Text); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void NavMenuHighlightsCurrentLocation() { var activeNavLinksSelector = By.CssSelector(".sidebar a.active"); @@ -64,7 +64,7 @@ public void NavMenuHighlightsCurrentLocation() item => Assert.Equal("Home", item.Text.Trim())); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void CounterPageCanUseThreads() { // Navigate to "Counter" @@ -81,7 +81,7 @@ public void CounterPageCanUseThreads() Browser.NotEqual("Current count: 0", () => Browser.Exists(By.CssSelector("h1 + p")).Text); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void HasFetchDataPage() { // Navigate to "Fetch data" diff --git a/src/Components/test/E2ETest/Tests/ThreadingAppTest.cs b/src/Components/test/E2ETest/Tests/ThreadingAppTest.cs index e238d01bc19c..98953991d8b2 100644 --- a/src/Components/test/E2ETest/Tests/ThreadingAppTest.cs +++ b/src/Components/test/E2ETest/Tests/ThreadingAppTest.cs @@ -28,19 +28,19 @@ protected override void InitializeAsyncCore() WaitUntilLoaded(); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void HasTitle() { Assert.Equal("Blazor standalone", Browser.Title); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void HasHeading() { Assert.Equal("Hello, world!", Browser.Exists(By.TagName("h1")).Text); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void NavMenuHighlightsCurrentLocation() { var activeNavLinksSelector = By.CssSelector(".sidebar a.active"); @@ -66,7 +66,7 @@ public void NavMenuHighlightsCurrentLocation() item => Assert.Equal("Home", item.Text.Trim())); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void CounterPageCanUseThreads() { // Navigate to "Counter" @@ -83,7 +83,7 @@ public void CounterPageCanUseThreads() Browser.NotEqual("Current count: 0", () => Browser.Exists(By.CssSelector("h1 + p")).Text); } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void HasFetchDataPage() { // Navigate to "Fetch data" @@ -104,7 +104,7 @@ public void HasFetchDataPage() } } - [Fact(Skip = "Test skipped due to unreliability. See https://github.com/dotnet/aspnetcore/issues/63524")] + [Fact(Skip = "https://github.com/dotnet/aspnetcore/issues/63524")] public void IsStarted() { // Read from property