Visitar URL original
[test-only] setup playwright test by v-scharf · Pull Request #102 · owncloud/web-extensions · GitHub
Skip to content

[test-only] setup playwright test - #102

Merged
v-scharf merged 1 commit into
mainfrom
setupPlaywrightTest1
Nov 16, 2024
Merged

v-scharf merged 1 commit into
mainfrom
setupPlaywrightTest1

Conversation

@v-scharf

@v-scharf v-scharf commented Nov 15, 2024 •

Copy link
Copy Markdown
Contributor
  • deleted cucumber from test setup.
  • instaled playwright as the only framework for ui testing, what gives us:
    • using vscode extension we can run the test by pressing the button
    • It is easy to select a browser for the test. or run the test in all browsers at once
    • It is easy to disable/enable headless mode or run test and see result in trace viewer
    • debuging testing has become much more convenient
  • created example test createDrawio file
  • here test structure. I wan to keep in the web-apps. and use pagesObjects and helpers in a separate folder (see structure here Readmi )
  • tests uses POM like here https://playwright.dev/docs/pom

here I tried to demonstrate how to run test using platwright extension

Screen.Recording.2024-11-15.at.14.00.50.mov

@CLAassistant

CLAassistant commented Nov 15, 2024 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@JammingBen JammingBen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Awesome stuff, just a few small remarks from my side.

The unit test suite seems to pick up the e2e .spec.ts files. You can avoid this by adding the following to the vite.config.ts files of the apps:

test: {
    exclude: ['**/e2e/**']
}

Comment thread package.json Resolved
Comment thread support/READMI.md Resolved
@v-scharf
v-scharf force-pushed the setupPlaywrightTest1 branch from 222559a to bb0ca42 Compare November 15, 2024 14:36
@v-scharf
v-scharf requested a review from JammingBen November 15, 2024 14:36
Comment thread package-lock.json Resolved
@v-scharf
v-scharf force-pushed the setupPlaywrightTest1 branch from bb0ca42 to 714a4de Compare November 15, 2024 15:22

@JammingBen JammingBen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@v-scharf
v-scharf merged commit 2a7e16e into main Nov 16, 2024
@v-scharf
v-scharf deleted the setupPlaywrightTest1 branch November 16, 2024 07:08
@v-scharf v-scharf mentioned this pull request Nov 16, 2024
@saw-jan saw-jan mentioned this pull request Dec 16, 2024
6 of 7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants