-
Notifications
You must be signed in to change notification settings - Fork 69
Description
Description
Right now, full-page screenshots are saved in the images database column, the same way user assertView calls are. This full-page screenshot gets a stateName calculated as error_${N}, where N is the number of user assertView commands.
This creates a conflict: if a user calls an assertView with stateName: 'error_1', their screenshot will be missing from the report, overwritten by the automatic full-page screenshot.
This should be fixed. The easiest solution is to change the state's name to something unique that a user would never choose, for example, __testplane_full_page__.
Verify latest release
- I verified that the issue exists in the latest html-reporter release
Html-reporter version
10.19.0
Last html-reporter version that worked
No response
Link to the code that reproduces this issue or a replay of the bug
No response
Reproduction steps
it('some test, async ({browser}) => {
await browser.url('...') // any url
await browser.assertView('error_1', 'body');
throw new Error('any-error');
});Actual Behavior
AssertView state screenshot is not shown:
Expected Behavior
AssertView state screenshot is shown (like it does with other state names):
Which Node.js version are you using?
20.18.1
Internal issue: https://nda.ya.ru/t/dpHfw1XF7JEZzk
Metadata
Metadata
Assignees
Labels
Type
Projects
Status