Due to frequent instances of timeouts during test execution, we've decided to remove the WaitUntil = WaitUntilState.NetworkIdle property from the GotoAsync invocation. From the Playwright documentation, it has been suggested NOT to use the NetworkIdle property due to its nature of timing out explicitly at 500ms which is undesirable in a real-world scenario (since we can never expect real world network reliability within such tight tolerances).
By removing the explicit setting, the WaitUntil property defaults to the JavaScript Window.Load event. This makes this function call more robust as it waits until the entire page loads and will timeout only at the predefined default timeout.
Checklist
[x] The code change is covered by unit tests. I have added tests that prove my fix is effective or that my feature works
[x] I have performed end-to-end test locally.
[x] New and existing unit tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas
[N/A] I have made corresponding changes to the documentation
Description
Due to frequent instances of timeouts during test execution, we've decided to remove the
WaitUntil = WaitUntilState.NetworkIdle
property from theGotoAsync
invocation. From the Playwright documentation, it has been suggested NOT to use theNetworkIdle
property due to its nature of timing out explicitly at 500ms which is undesirable in a real-world scenario (since we can never expect real world network reliability within such tight tolerances).By removing the explicit setting, the
WaitUntil
property defaults to the JavaScriptWindow.Load
event. This makes this function call more robust as it waits until the entire page loads and will timeout only at the predefined default timeout.Checklist