Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[test-only] setup playwright test #102

Merged
merged 1 commit into from
Nov 16, 2024
Merged

Conversation

ScharfViktor
Copy link
Contributor

@ScharfViktor ScharfViktor commented Nov 15, 2024

  • 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
Copy link

CLAassistant commented Nov 15, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@JammingBen JammingBen left a comment

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/**']
}

package.json Outdated Show resolved Hide resolved
packages/web-app-cast/l10n/translations.json Outdated Show resolved Hide resolved
support/READMI.md Outdated Show resolved Hide resolved
packages/web-app-draw-io/tests/e2e/createDrawIo.spec.ts Outdated Show resolved Hide resolved
package-lock.json Outdated Show resolved Hide resolved
Copy link
Contributor

@JammingBen JammingBen left a comment

Choose a reason for hiding this comment

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

👍

@ScharfViktor ScharfViktor merged commit 2a7e16e into main Nov 16, 2024
2 checks passed
@ScharfViktor ScharfViktor deleted the setupPlaywrightTest1 branch November 16, 2024 07:08
@ScharfViktor ScharfViktor mentioned this pull request Nov 16, 2024
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