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

Migrate stencil e2e tests to playwright #1258

Closed
37 of 38 tasks
BenPag opened this issue Feb 15, 2024 · 0 comments · Fixed by #1359
Closed
37 of 38 tasks

Migrate stencil e2e tests to playwright #1258

BenPag opened this issue Feb 15, 2024 · 0 comments · Fixed by #1359
Assignees

Comments

@BenPag
Copy link
Collaborator

BenPag commented Feb 15, 2024

The current E2E testing setup in Stencil relies on its own testing framework, which lacks transparency for debugging purposes and utilizes Puppeteer, known for its slower performance. With #1226 we introduced Playwright, a state-of-the-art E2E testing tool. This task keeps an overview about which tests needs to migrate to playwright.

@BenPag BenPag added the new This issue has been recently added and has no estimation label Feb 15, 2024
@BenPag BenPag self-assigned this Feb 15, 2024
@janivo janivo removed the new This issue has been recently added and has no estimation label Feb 22, 2024
janivo pushed a commit that referenced this issue Apr 3, 2024
Part of #1258

## Proposed Changes
- remove stencil e2e test, because of css class testsing
janivo pushed a commit that referenced this issue Apr 4, 2024
…wright tests (#1352)

Part of #1258

## Proposed Changes

- migrate existing stencil e2e tests to jest spec tests
janivo pushed a commit that referenced this issue Apr 5, 2024
…st (#1355)

Part of #1258

## Proposed Changes

- migrate left stencil e2e test to spec test
janivo pushed a commit that referenced this issue Apr 5, 2024
…ests (#1335)

Part of #1258

## Proposed Changes

- migrate existing stencil e2e tests to jest spec tests
BenPag added a commit that referenced this issue Apr 5, 2024
#1356)

Part of #1258 

## Proposed Changes

- migrate existing stencil e2e tests to jest spec tests
- add some playwright tests
janivo added a commit that referenced this issue Apr 5, 2024
… tests (#1349)

Part of #1258

## Proposed Changes
- migrate existing stencil e2e tests to jest spec tests

---------

Co-authored-by: Jan-Niklas Voß <[email protected]>
janivo pushed a commit that referenced this issue Apr 8, 2024
…playwright (#1256)

Part of #1258

## Proposed Changes

- migrate existing stencil e2e tests to playwright
janivo pushed a commit that referenced this issue Apr 10, 2024
…right (#1375)

Part of #1258

## Proposed Changes

- migrate tests to playwright

<!--
## Things to check

- [ ] Does the change need to be documented?
- [ ] Does any existing example code needs to be updated?
- [ ] Is the change properly tested?
- [ ] Is it helpful to provide another example to demonstrate the new
feature?
- [ ] Are there other code lines that need to be modified?
-->
janivo pushed a commit that referenced this issue Apr 15, 2024
…aywright (#1374)

Part of #1258 

## Proposed Changes

- migrate tests to playwright
- add spec test to test event api
janivo pushed a commit that referenced this issue Apr 15, 2024
…o playwright (#1364)

Part of #1258 

## Proposed Changes

- remove `ino-table` stencil e2e tests
- migrate and add playwright tests for `ino-table-header-cell`
janivo added a commit that referenced this issue Apr 16, 2024
…#1331)

Part of #1258

## Proposed Changes

- migrate existing stencil e2e tests to playwright

<!--
## Things to check

- [ ] Does the change need to be documented?
- [ ] Does any existing example code needs to be updated?
- [ ] Is the change properly tested?
- [ ] Is it helpful to provide another example to demonstrate the new
feature?
- [ ] Are there other code lines that need to be modified?
-->

---------

Co-authored-by: Jan-Niklas Voß <[email protected]>
janivo added a commit that referenced this issue Apr 17, 2024
@janivo janivo reopened this Apr 18, 2024
@janivo janivo closed this as completed May 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 a pull request may close this issue.

2 participants