front: add speed space chart e2e test

[Maymanaf]

Closes #8651
The visual assertion using the snapshot is only performed on Chrome due to issue #9909

[anisometropie]

Review & tested, looking good

[anisometropie]

btw, the fixup fa8fd20 should not go in the readme commit

[Maymanaf]

btw, the fixup fa8fd20 should not go in the readme commit

Yes my bad , I will fix that

[Yohh]

one thing that may be important:

when the tests are launched, snapshots are created with the name you give, but three times each with -chromium-linux.png or -firefox-linux.png added, so we have yet 16 png files, when I launch them on macos, playwright generates 16 new files with -chromium-darwin.png or -firefox-darwin.png

[Maymanaf]

one thing that may be important:

when the tests are launched, snapshots are created with the name you give, but three times each with -chromium-linux.png or -firefox-linux.png added, so we have yet 16 png files, when I launch them on macos, playwright generates 16 new files with -chromium-darwin.png or -firefox-darwin.png

Good catch! I think we can add a control to run the snapshot assertion only on Linux. The method toHaveScreenshot() generates images based on the operating system and browser, and we can't maintain multiple screenshots for every OS and browser combination
What do you think ?

[Yohh]

that's the wisest solution I think

[Yohh]

lgtm, thanks

[emersion]

Looks a lot better, thanks for fixing the comments! Here are a few more.

[emersion]

There is still one remaining comment for a small cleanup: #9829 (comment)

If you're not willing to do it, you can resolve it.

Thanks!