[Question] Skipped tests mark as failed

[judithhartmann]

Hey,
we have some tests that we skip conditionally, therefore the beforeEach and the beforeAll are run before these tests.
If playwright detects an error in these steps, it will fail this test. So far so good.

We have retries enabled, so when the retry of the test makes it to the test.skip it skips the test
In the end the test is marked as FAILED in the final results, even though it was 1 failure run, 1 skip run (success kinda).
Is this the desired behaviour? I feel like it should be marked as skipped.

[aslushnikov]

This is a dupe of #17652