feat(allure-playwright): initial implementation for allure-playwright

[pavelfeldman]

Fixes #296, microsoft/playwright#7034

This is initial implementation for Playwright reporter. I already booked allure-playwright npm package for it, please let me know who I should transfer ownership over it to. I did not push that package.

Please let me know what you think about it and I can add more tests for it.

[CLAassistant]

All committers have signed the CLA.

[just-boris]

Hello!

Thanks for the PR! There are known build failures, will be fixed soon #298
I will also try to run it locally and play with the new reporter in my demo project

[pavelfeldman]

I added a couple more tests.

[baev]

I already booked allure-playwright npm package for it, please let me know who I should transfer ownership over it to. I did not push that package.

https://www.npmjs.com/~baev

Can we also add language and framework labels for each test?

[pavelfeldman]

https://www.npmjs.com/~baev

Done.

Can we also add language and framework labels for each test?

Done.

[pavelfeldman]

Tell me if I should kick myself out of the owners for allure-playwright or you want someone from Playwright team there.

[pavelfeldman]

Playwright v1.13 is out and I published this package as experimental-allure-playwright to let users try it out.

[mpuertao]

@baev good day. When will the changes be approved?

[baev]

@pavelfeldman could you please fix the build?

[pavelfeldman]

You probably want to merge #298, otherwise current master is broken:

allure-jasmine: test/label.test.ts(10,14): error TS2339: Property 'describe' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/label.test.ts(11,16): error TS2339: Property 'it' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/status.test.ts(10,15): error TS2339: Property 'describe' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/status.test.ts(11,17): error TS2339: Property 'it' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/status.test.ts(25,15): error TS2339: Property 'describe' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/status.test.ts(26,17): error TS2339: Property 'it' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/status.test.ts(40,15): error TS2339: Property 'describe' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/status.test.ts(41,17): error TS2339: Property 'xit' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/status.test.ts(55,15): error TS2339: Property 'xdescribe' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/status.test.ts(56,17): error TS2339: Property 'it' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/step.test.ts(10,15): error TS2339: Property 'describe' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/step.test.ts(11,17): error TS2339: Property 'it' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/step.test.ts(35,15): error TS2339: Property 'describe' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/step.test.ts(36,17): error TS2339: Property 'it' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/step.test.ts(74,15): error TS2339: Property 'describe' does not exist on type 'JasmineTestEnv'.
allure-jasmine: test/step.test.ts(75,17): error TS2339: Property 'it' does not exist on type 'JasmineTestEnv'.

[pavelfeldman]

@baev ^^

[pavelfeldman]

It should build/test now, but mocha coverage fails for me locally...
^^ @baev

[baev]

It should build/test now, but mocha coverage fails for me locally...

Fixed coverage in master, please rebase (also prettier/eslint config was added, please consider aligning code style as well)

[pavelfeldman]

Your eslint is quite unusual, but should be aligned now.

[baev]

@pavelfeldman

> allure-playwright@2.0.0-beta.10 test /home/runner/work/allure-js/allure-js/packages/allure-playwright
> playwright test -c test

·undefined
F···

  1) screenshot-on-failure.spec.ts:19:1 › should capture screenshot on failure =====================

    Error: expect(received).toEqual(expected) // deep equality

    - Expected  - 7
    + Received  + 1

    - Array [
    -   Object {
    -     "file": "/home/runner/work/allure-js/allure-js/packages/allure-playwright/test-results/screenshot-on-failure-should-capture-screenshot-on-failure/test-results/a-test-name/test-failed-1.png",
    -     "name": "screenshot",
    -     "type": "image/png",
    -   },
    - ]
    + Array []

      42 |     },
      43 |   );
    > 44 |   expect(result).toEqual([
         |                  ^
      45 |     {
      46 |       name: "screenshot",
      47 |       type: "image/png",

        at /home/runner/work/allure-js/allure-js/packages/allure-playwright/test/screenshot-on-failure.spec.ts:44:18
        at processTicksAndRejections (internal/process/task_queues.js:95:5)
        at WorkerRunner._runTestWithBeforeHooks (/home/runner/work/allure-js/allure-js/node_modules/@playwright/test/lib/test/workerRunner.js:390:7)


  1 failed
    screenshot-on-failure.spec.ts:19:1 › should capture screenshot on failure ======================
  4 passed (8s)

[pavelfeldman]

@baev that test was running full Playwright and we need to install browser deps and/or include GH action for that. That's one line in yaml, but I removed the test for now to unblock it as is.

[baev]

@pavelfeldman @just-boris thanks!