Adding E2E Testing Automation #755
Conversation
There was a problem hiding this comment.
Is abstracting actions and assertions in a class best practice? Just curious, there's a lot of LoC regardless for e2e right.
Is it overkill in that case to adopt some kind of builder pattern (return *self on each object function)?
eg.
await somePage
.doStuff()
.expectStuff();| @@ -0,0 +1,86 @@ | |||
| # Playwright Tests with TypeScript | |||
|
|
|||
| This repository contains end-to-end tests for a web application using Playwright and TypeScript. The tests are organized using the Page Object Model (POM) to enhance maintainability and readability. | |||
There was a problem hiding this comment.
nit: Should probably describe this in other words, as repository is quite a trademarked word in the git context. Also, confusing as it is, apps/web is a legacy naming we never got around to update, but (correct me if I'm wrong) it's essentially the life events web app.
There was a problem hiding this comment.
On another note, a tl;dr description on POM and best practice would be cool 😎
| - Node.js (>=14.x) | ||
| - npm (>=6.x) |
| ### Installation | ||
|
|
||
| 1. Clone the repository: | ||
| ```sh | ||
| git clone https://github.com/your-repo/playwright-tests.git | ||
| cd playwright-tests | ||
| ``` | ||
|
|
||
| 2. Install dependencies: | ||
| ```sh | ||
| npm install | ||
| ``` |
| ### Running Tests | ||
|
|
||
| To run all tests, use the following command: | ||
| ```sh | ||
| npx playwright test | ||
|
|
||
| npx playwright test --grep @critical |
There was a problem hiding this comment.
Do we need two Running Tests sections?
|
|
||
| For more information on configuring Playwright, visit the [official documentation](https://playwright.dev/docs/test-configuration). | ||
|
|
||
| ### Running Tests |
There was a problem hiding this comment.
General question, is there any merit in describing (tl;dr fashion) what headed eg. means, or are we supposed to know this already (me problem)?
Ticket:
Description
Type
Checklist:
Screenshots:
N/A