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

Test Failure: BlazorServerTemplateWorks_NoAuth #30761

Closed
sebastienros opened this issue Mar 9, 2021 · 4 comments · Fixed by #45682
Closed

Test Failure: BlazorServerTemplateWorks_NoAuth #30761

sebastienros opened this issue Mar 9, 2021 · 4 comments · Fixed by #45682
Assignees
Labels
area-blazor Includes: Blazor, Razor Components feature-playwright 🎭 This issue is related to Playwright E2E Project Template Tests feature-templates test-failure
Milestone

Comments

@sebastienros
Copy link
Member

Build: https://dev.azure.com/dnceng/public/_build/results?buildId=1027022&view=logs&j=8c9dd6ae-5017-5a70-997c-c2386fed4e85&t=67839355-6a03-55d6-fbd7-5d56c2fd9ea1

  Failed Templates.Test.BlazorServerTemplateTest.BlazorServerTemplateWorks_NoAuth(browserKind: Chromium) [2 m 50 s]
  Error Message:
   System.TimeoutException : Timeout 30000ms exceeded.
=========================== logs ===========================
waiting for selector "h1+p >> text=Current count: 1" to be visible
============================================================
Note: use DEBUG=pw:api environment variable and rerun to capture Playwright logs.
  Stack Trace:
     at PlaywrightSharp.Transport.Connection.SendMessageToServerAsync[T](String guid, String method, Object args, Boolean ignoreNullValues, JsonSerializerOptions serializerOptions, Boolean treatErrorPropertyAsError) in /Users/neo/Documents/Coding/Personal/playwright-sharp/src/PlaywrightSharp/Transport/Connection.cs:line 163
   at PlaywrightSharp.Frame.WaitForSelectorAsync(Boolean isPageCall, String selector, Nullable`1 state, Nullable`1 timeout) in /Users/neo/Documents/Coding/Personal/playwright-sharp/src/PlaywrightSharp/Frame.cs:line 645
   at Templates.Test.BlazorServerTemplateTest.TestBasicNavigation(IPage page) in /_/src/ProjectTemplates/BlazorTemplates.Tests/BlazorServerTemplateTest.cs:line 200
   at Templates.Test.BlazorServerTemplateTest.BlazorServerTemplateWorks_NoAuth(BrowserKind browserKind) in /_/src/ProjectTemplates/BlazorTemplates.Tests/BlazorServerTemplateTest.cs:line 74
   at Templates.Test.BlazorServerTemplateTest.BlazorServerTemplateWorks_NoAuth(BrowserKind browserKind) in /_/src/ProjectTemplates/BlazorTemplates.Tests/BlazorServerTemplateTest.cs:line 101
@sebastienros sebastienros changed the title Test Failure: Test Failure: BlazorServerTemplateWorks_NoAuth Mar 9, 2021
sebastienros added a commit that referenced this issue Mar 9, 2021
@javiercn javiercn added area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates feature-templates area-blazor Includes: Blazor, Razor Components and removed area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates labels Mar 9, 2021
@javiercn
Copy link
Member

javiercn commented Mar 9, 2021

This is failing to wait for the navigation to be intercepted.

@javiercn javiercn added this to the Next sprint planning milestone Mar 9, 2021
@ghost
Copy link

ghost commented Mar 9, 2021

Thanks for contacting us.
We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. We will evaluate the request when we are planning the work for the next milestone. To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@TanayParikh
Copy link
Contributor

Tests currently skipped via #40129, will be unskipped via #38818 along with being re-enabled in helix.

@TanayParikh TanayParikh added the feature-playwright 🎭 This issue is related to Playwright E2E Project Template Tests label Feb 10, 2022
@ghost
Copy link

ghost commented May 24, 2022

Thanks for contacting us.

We're moving this issue to the .NET 7 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@mkArtakMSFT mkArtakMSFT modified the milestones: .NET 7 Planning, 7.0-rc2 Aug 10, 2022
@mkArtakMSFT mkArtakMSFT modified the milestones: 7.0-rc2, .NET 8 Planning Sep 7, 2022
@mkArtakMSFT mkArtakMSFT modified the milestones: .NET 8 Planning, 8.0-MQ Sep 15, 2022
@TanayParikh TanayParikh linked a pull request Dec 22, 2022 that will close this issue
TanayParikh added a commit that referenced this issue Dec 23, 2022
@mkArtakMSFT mkArtakMSFT modified the milestones: 8.0-MQ, 8.0-preview1 Jan 4, 2023
wtgodbe added a commit that referenced this issue Feb 14, 2023
* Version Bumps

* Run with `Test: Windows Server x64`

* Install Playwright in CI

* Add running playwright locally instructions

* Test Cleanup

* Remove Selenium

* Prebuild for Playwright install

* Update default-build.yml

* Update default-build.yml

* Update default-build.yml

* Update default-build.yml

* Update default-build.yml

* Playwright install in test

* New template test pipeline

* Update components-e2e-tests.yml

* Update components-template-tests.yml

* Update components-e2e-tests.yml

* Update components-e2e-tests.yml

* Update components-e2e-tests.yml

* Update components-template-tests.yml

* Update components-e2e-tests.yml

* Update components-e2e-tests.yml

* Update components-e2e-tests.yml

* Revert new pipeline changes

* RestoreAdditionalProjectSources

* Update Templates.Blazor.Tests.csproj

* PR Feedback

* RunBlazorPlaywrightTemplateTests

* Socket logic

* Fixed socket wait logic

Fixes #30761

* Remove redundant process wait for exit

We're already in the `Exited` Event:

There are two ways of being notified when the associated process exits: synchronously and asynchronously. Synchronous notification means calling the WaitForExit method to block the current thread until the process exits. Asynchronous notification uses the Exited event, which allows the calling thread to continue execution in the meantime. In the latter case, EnableRaisingEvents must be set to true for the calling application to receive the Exited event.

https://learn.microsoft.com/dotnet/api/system.diagnostics.process.exited?view=net-7.0

* Update BlazorWasmTemplateTest.cs

* Update Templates.Blazor.Tests.csproj

* Revert App.Ref, add src in props

* Config updates

* Update RequiresDelayedBuildProjects.props

* Update Templates.Blazor.Tests.csproj

* Skip failing tests

* Add #46430

* Update Templates.Blazor.Tests.csproj

* Update BlazorWasmTemplateTest.cs

* Update BlazorWasmTemplateTest.cs

---------

Co-authored-by: William Godbe <wigodbe@microsoft.com>
@ghost ghost locked as resolved and limited conversation to collaborators Mar 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-blazor Includes: Blazor, Razor Components feature-playwright 🎭 This issue is related to Playwright E2E Project Template Tests feature-templates test-failure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants