Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Split up slow Playwright tests #12741

Merged
merged 4 commits into from
Jul 10, 2024
Merged

Split up slow Playwright tests #12741

merged 4 commits into from
Jul 10, 2024

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Jul 9, 2024

To optimise parallelism which naively distributes the spec files between the 6 runners

Deals with:

Slow test file: read-receipts/redactions.spec.ts (5.4m)
Slow test file: read-receipts/new-messages.spec.ts (3.9m)
Slow test file: read-receipts/high-level.spec.ts (3.6m)
Slow test file: read-receipts/editing-messages.spec.ts (3.1m)
Slow test file: read-receipts/reactions.spec.ts (2.2m)
Slow test file: crypto/crypto.spec.ts (2.4m)
Slow test file: settings/appearance-user-settings-tab/appearance-user-settings-tab.spec.ts (1.2m)
Slow test file: composer/composer.spec.ts (1.1m)
Slow test file: crypto/verification.spec.ts (1.1m)

To optimise parallelism

Deals with:

```
Slow test file: read-receipts/redactions.spec.ts (5.4m)
Slow test file: read-receipts/new-messages.spec.ts (3.9m)
Slow test file: read-receipts/high-level.spec.ts (3.6m)
Slow test file: read-receipts/editing-messages.spec.ts (3.1m)
Slow test file: read-receipts/reactions.spec.ts (2.2m)
Slow test file: crypto/crypto.spec.ts (2.4m)
Slow test file: settings/appearance-user-settings-tab/appearance-user-settings-tab.spec.ts (1.2m)
Slow test file: composer/composer.spec.ts (1.1m)
Slow test file: crypto/verification.spec.ts (1.1m)
```

Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy t3chguy added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Jul 9, 2024
Signed-off-by: Michael Telatynski <[email protected]>
… t3chguy/split-slow-tests

Signed-off-by: Michael Telatynski <[email protected]>

# Conflicts:
#	playwright/e2e/crypto/crypto.spec.ts
Signed-off-by: Michael Telatynski <[email protected]>
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Somewhat impossible to review meaningfully, but the idea is good and I'll trust that you can C&P code.

@t3chguy t3chguy added this pull request to the merge queue Jul 10, 2024
Merged via the queue into develop with commit c894beb Jul 10, 2024
29 checks passed
@t3chguy t3chguy deleted the t3chguy/split-slow-tests branch July 10, 2024 09:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants