-
Notifications
You must be signed in to change notification settings - Fork 4.2k
chore: two-line trace view (2) #36055
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
Conversation
} | ||
{ | ||
const error = await expect(locator).not.toHaveAttribute('checked', /.*/, { timeout: 1000 }).catch(e => e); | ||
expect(error.message).toContain('expect.not.toHaveAttribute with timeout 1000ms'); | ||
expect(stripAnsi(error.message)).toContain(`Timed out 1000ms waiting for expect(locator).not.toHaveAttribute(expected)`); | ||
expect(stripAnsi(error.message)).toContain(`- Expect not toHaveAttribute with timeout 1000ms`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we actually need stripAnsi here?
@@ -991,7 +991,7 @@ test('should record nested steps, even after timeout', async ({ runInlineTest }, | |||
' fixture: barPage', | |||
' barPage teardown', | |||
' step in barPage teardown', | |||
' page.close', | |||
' Close context', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be "Close page", shouldn't it? An in other places too.
@microsoft-github-policy-service rerun |
Test results for "tests 1"4 failed 7 flaky39208 passed, 804 skipped Merge workflow run. |
No description provided.