fix: mochawesome helper with unique screenshots

[Lando1n]

Motivation/Description of the PR

Screenshots started to fail to link when using the uniqueScreenshotName: true value on the Mochawesome helper.

This is due to the uuid suffix being added with a different method than before. Instead of re-implementing the same _getUUID function, it was moved into the testToFileName function with option unique.

Applicable helpers:

• Playwright`
• Puppeteer
• WebDriver
• REST
• FileHelper
• Appium
• TestCafe
• Mochawesome

Applicable plugins:

• allure
• autoDelay
• autoLogin
• customLocator
• pauseOnFail
• coverage
• retryFailedStep
• screenshotOnFail
• selenoid
• stepByStepReport
• stepTimeout
• wdio
• subtitles

Type of change

• 🔥 Breaking changes
• 🚀 New functionality
• 🐛 Bug fix
• 🧹 Chore
• 📋 Documentation changes/updates
• ♨️ Hot fix
• 🔨 Markdown files fix - not related to source code
• 💅 Polish code

Checklist:

• Tests have been added
• Documentation has been added (Run npm run docs)
• Lint checking (Run npm run lint)
• Local tests are passed (Run npm test)

[kobenguyent]

Could you please verify if the screenshot works correctly in reports? There was a previous issue with a broken screenshot link in reports, so it would be helpful to have a check. Thank you.

The fix was:
#4889

[Lando1n]

@kobenguyent - correct, it's now working for unique and non-unique screenshots.

Report:

With output directory like:

[kobenguyent]

@Lando1n may you think we could add some UTs for those changes in https://github.com/codeceptjs/CodeceptJS/blob/3.x/test/unit/plugin/screenshotOnFail_test.js

[Lando1n]

@Lando1n may you think we could add some UTs for those changes in https://github.com/codeceptjs/CodeceptJS/blob/3.x/test/unit/plugin/screenshotOnFail_test.js

For sure, will add.

[Lando1n]

@kobenguyent - added some tests, but they were reliant on a small Mochawesome helper modification in order to properly mock.

[Lando1n]

@kobenguyent - any idea why mochawesome isn't a dependency already? Seeing that test failure.

[kobenguyent]

@kobenguyent - any idea why mochawesome isn't a dependency already? Seeing that test failure.

I guess we shall keep this mochawesome as dev-deps right @DavertMik ?

[Copilot]

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Files not reviewed (1)

• package.json: Language not supported

Comments suppressed due to low confidence (1)

test/unit/plugin/screenshotOnFail_test.js:137

• Consider adding additional test cases that cover a broader range of configuration options and edge cases for unique screenshot naming to ensure consistent behavior across different scenarios.

// TODO: write more tests for different options