Skip to content

Conversation

@cpAdm
Copy link
Contributor

@cpAdm cpAdm commented Dec 30, 2025

Don't poll if the selected test is still selected. Wait for a response from the websocket instead that the test has started (that updates item prop). This polling fetch would fail (sw errors), causing the model to be set to a new TraceModel with an empty traceURI, which in case would make the source call fail (the original issue).

Added assertions to existing test.

Tried instead adding following assertions in runUITest fixture:

expect.soft((page && !page.isClosed()) ? await page.consoleMessages() : []).toHaveLength(0);
expect.soft((page && !page.isClosed()) ? await page.pageErrors() : []).toHaveLength(0);

However many would then fail:
snapshot action must have a pageId (10+ ui-mode tests; e.g. should not duplicate network entries from beforeAll)
Failed to load resource: the server responded with a status of 404 (Not Found) (should run folder / should merge trace events / should merge web assertion events / should show request source context id)

Closes: #38359


const result = item.testCase?.results[0];
if (!result) {
if (!result || item.treeItem?.status === 'scheduled') {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How do we have a test result for something that has not run yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Source temporarily disappears when starting test

2 participants