fix: file name is not unique enough when running tests in parallel

[kobenguyent]

Motivation/Description of the PR

• The fileName is not unique when running tests in parallel and it leads to the issue that attachment couldn't attach to allure report.

Applicable helpers:

• Playwright

Type of change

• 🐛 Bug fix

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)

[EgorBodnar]

a good fix, as a result of our discussion 🙌

The custom attachment fileName should be unique for reporters or it would be rewritten

[EgorBodnar]

Should the fileName be unique for one test or it probably should be unique within one test step too?

• If it should be unique per test, use mocha current test id or nodejs worker.threadId

• If it should be unique within one test use already installed CodeceptJS dependency - uuid to generate really unique id.

So better to delete the new function makeId(length) {}

[kobenguyent]

Should the fileName be unique for one test or it probably should be unique within one test step too?

• If it should be unique per test, use mocha current test id or nodejs worker.threadId
• If it should be unique within one test use already installed CodeceptJS dependency - uuid to generate really unique id.

So better to delete the new function makeId(length) {}

updated per CR. Thanks for pointing it out!