Skip to content
This repository was archived by the owner on Jul 7, 2026. It is now read-only.

feat: init playwright#44

Open
parovozik787 wants to merge 2 commits into
masterfrom
feature/playwright
Open

feat: init playwright#44
parovozik787 wants to merge 2 commits into
masterfrom
feature/playwright

Conversation

@parovozik787

Copy link
Copy Markdown

No description provided.

@commit-lint

commit-lint Bot commented Jun 23, 2022

Copy link
Copy Markdown

Features

Continuous Integration

Contributors

parovozik787, anttibull

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@netlify

netlify Bot commented Jun 23, 2022

Copy link
Copy Markdown

Deploy Preview for builder69 ready!

Name Link
🔨 Latest commit 2a13895
🔍 Latest deploy log https://app.netlify.com/sites/builder69/deploys/62b9f7c0c97a5a00087cbd41
😎 Deploy Preview https://deploy-preview-44--builder69.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@anttibull anttibull left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

думаю краще перейментувати комміт і pr в test/playwright

Comment thread .gitignore
npm-debug.log*
yarn-debug.log*
yarn-error.log*
/test-results/

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
/test-results/
/test-results/

Comment thread package.json
"version": "0.1.0",
"private": false,
"dependencies": {
"@playwright/test": "^1.22.2",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

думаю це в дев депенденсі краще буде

Comment thread tests/example.spec.ts
@@ -0,0 +1,12 @@
import { test, expect } from "@playwright/test";

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

я би це обернув в describe, також треба додати тайтл до теста, test нічого не описує що тест робить

Comment thread tests/example.spec.ts
await expect(page.locator("text=Learn React")).toHaveAttribute(
"href",
"https://reactjs.org"
);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

я би ще додав тест щоб перевірити що лінка правильна і кнопка робоча, при переході ми попадаємо в доку

@parovozik787 parovozik787 Jun 27, 2022

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

трошки не розумію, що означає лінка правильна
Типу зробити перевірку, що:

  • кнопка не disabled
  • при кліку ми переходимо на нову сторінку з url ="https://reactjs.org"?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

я трохи незрозуміло написав: при кліку ми переходимо на нову сторінку з url ="https://reactjs.org/"? цього вистачить, так як якшо ми попадемо на https://reactjs.org/, то кнопка вже enabled і лінка значить правильна

@@ -0,0 +1,22 @@
name: Playwright Tests

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

в принципі цей файл можеш видалить, так як цей конфіг треба буде рефакторить і зараз він не працює

@anttibull
anttibull requested a review from quatrolavita June 24, 2022 07:33
@anttibull anttibull linked an issue Jun 24, 2022 that may be closed by this pull request
@anttibull
anttibull force-pushed the feature/playwright branch from db559a9 to c662380 Compare June 27, 2022 16:33
parovozik787 and others added 2 commits June 27, 2022 21:32
* feat: init playwright

* ci(playwright): add running e2e tests

* ci: add wait netlify gh action

Co-authored-by: parovozik <[email protected]>
@anttibull
anttibull force-pushed the feature/playwright branch from 69eeb9f to 2a13895 Compare June 27, 2022 18:32
@parovozik787
parovozik787 requested a review from anttibull June 29, 2022 19:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add basic tests

2 participants