Incorrect video file attached on failure

[MaryGeraseva]

Currently, when test scenarios reuse the browser session the attached video file on the failure may not accurately reflect the failed test case.
Instead of attaching the video associated with the failed scenario, the report attaches the first test video.

config sample:

browser: {
reuseSession: true,
logLevel: 'warn',
timeout: {
page: configs.browserCapabilities.PAGE_TIMEOUT,
visible: configs.browserCapabilities.VISIBLE_TIMEOUT,
},
capabilities: {
browserName: configs.browserCapabilities.BROWSER,
headless: configs.browserCapabilities.HEADLESS
},
video: {
event: ['onFail'],
dir: 'video',
size: { width: 640, height: 480 },
attach: true
}
},

tests sample:

Feature: Demo

Scenario: Login
When I log in DAP

Scenario: Demo#1
When I click 'HomePage > Header > Domains'
Then I expect current url to contain 'email-domains'

Scenario: Demo#2
When I click 'HomePage > Header > Users'
Then I expect current url to contain 'users'

Scenario: Demo#3
When I click 'HomePage > Header > Apps'
Then I expect current url to contain 'apps'

Scenario: Demo#4
When I click 'HomePage > Header > Users'
Then I expect current url to contain 'failed'

[AlexGalichenko]

Currently, there is no flexible control of video recording
opened feature requests
microsoft/playwright#4821
microsoft/playwright#14258

I'll take a look if it somehow can be overcome