replay icon indicating copy to clipboard operation
replay copied to clipboard

Provide a better status report for multiple recordings

Open ergunsh opened this issue 2 years ago • 4 comments

Recorder panel and @puppeteer/replay library makes it a lot easier to start writing E2E tests. For a small project of mine, I was able to have a basic E2E setup using Recorder panel in around 5 minutes :)

Though currently running multiple files doesn't provide a good status report at the end: i.e. these replays succeeded and this failed with reason X.

So, if we provide a better status report; users will be able to use this as a basic recorder runner maybe to have their initial tests in their codebase.

ergunsh avatar May 13 '22 12:05 ergunsh

I believe you can specify a directory using shell expansion. E.g., replay test/resources/*.json. Alternatively, one can use xargs and similar utilities together with find or ls. I am not sure if we really should support glob parsing inside the lib. Adding some improved reporting sounds good to me.

OrKoN avatar May 13 '22 12:05 OrKoN

Oh, yes; you're totally right and we have the ability to specify multiple files with shell expansion. I'm updating this as a feature request for better reporting :)

ergunsh avatar May 13 '22 12:05 ergunsh

Hello!

I've implemented status report with #224. I've used cli-table3 to create the table structure. You can see the output below.

Light mode: Screen Shot 2022-07-13 at 23 36 28

Dark mode: Screen Shot 2022-07-13 at 23 37 30

This is a draft implementation. I'm waiting for your feedback. 🙂

puskuruk avatar Jul 13 '22 20:07 puskuruk

Recorder panel and @puppeteer/replay library makes it a lot easier to start writing E2E tests. For a small project of mine, I was able to have a basic E2E setup using Recorder panel in around 5 minutes :)

Though currently running multiple files doesn't provide a good status report at the end: i.e. these replays succeeded and this failed with reason X.

So, if we provide a better status report; users will be able to use this as a basic recorder runner maybe to have their initial tests in their codebase.

https://github.com/puppeteer/replay/issues/133#issue-1235181301

Lowlife84 avatar Jul 31 '22 15:07 Lowlife84

This seems to have been fixed by #224.

nickserv avatar Aug 18 '22 01:08 nickserv