reporter icon indicating copy to clipboard operation
reporter copied to clipboard

Reporter to ESM

Open olexandr13 opened this issue 1 year ago • 2 comments

Rewrite reporter to use ESM syntax. Description: previously we used commonjs syntax inside reporter (require), not we changed it to esm (import). It affects just project internally and should not break anything. But the changes are huge and everything may be broken.

Release: 2.0.0-beta-esm

Affected areas:

  • everything
  • every adapter (except cucumber <=6)
  • every testomat function (artifact, step, log)
  • each functionality should be tested with both esm and commonjs syntax

olexandr13 avatar Sep 15 '24 07:09 olexandr13

https://github.com/testomatio/reporter/pull/360

olexandr13 avatar Sep 15 '24 07:09 olexandr13

Testing checklist:

  • [x] frameworks
    • [x] playwright @olexandr13
    • [x] jest
    • [x] mocha @olexandr13
    • [x] wdio @olexandr13
    • [x] cucumber @olexandr13
    • [x] cypress @olexandr13
    • [x] codecept @olexandr13
    • [ ] jasmine (low priority, not used widely)
    • [x] vitest @olexandr13
  • [x] xml reporter @olexandr13
  • [x] html report @olexandr13
  • [x] testomat functions/services (artifact, step, log) @olexandr13
  • [ ] pipes @olexandr13
    • [x] debug
    • [x] html
    • [x] other pipes (github, gitlab) just invoke and check no errors; (not tested: csv, bitbucket)
  • [x] artifacts upload
  • [x] cli @olexandr13

  • [x] everything above should be tested:
    • [x] using both syntaxes
      • [x] commonjs (require) @olexandr13
      • [x] esm (import) @olexandr13 also recommend to set type to commonjs or module in package.json when using appropriate module system
    • [x] specifying both paths to reporters:
      • [x] old (full) (node_nodules/@testomatio/reporter/lib/adapter/playwright.js) @olexandr13
      • [x] new (short)@testomatio/reporter/playwright) @@olexandr13
  • [x] Types @olexandr13

olexandr13 avatar Dec 14 '24 23:12 olexandr13