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

feat: adding svelte component testing support #23553

Merged
merged 14 commits into from
Aug 26, 2022
Merged

Conversation

ZachJW34
Copy link
Contributor

User facing changelog

Svelte Component Testing Support

Additional details

Merging of the Svelte Feature branch into develop, no additional changes from the two previously merged and approved PRS: #23519 and #23441

Documentation is WIP: cypress-io/cypress-documentation#4677

Steps to test

You can scaffold a new svelte project with npm init vite and choose Svelte (the scaffold has outdated peer deps so update @sveltejs/vite-plugin-svelte and vite to latest). Then you can point yarn cypress:open at the svelte project and get to testing!

There are also some system tests in system-tests/test/component_testing_spec.ts that test both webpack and vite.

How has the user experience changed?

Screen.Recording.2022-08-25.at.2.06.27.PM.mov

PR Tasks

  • Have tests been added/updated?
  • Has the original issue (or this PR, if no issue exists) been tagged with a release in ZenHub? (user-facing changes only)
  • Has a PR for user-facing changes been opened in cypress-documentation?
  • Have API changes been updated in the type definitions?

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Aug 25, 2022

Thanks for taking the time to open a PR!

@cypress
Copy link

cypress bot commented Aug 25, 2022



Test summary

39609 0 3371 0Flakiness 1


Run details

Project cypress
Status Passed
Commit b8cd5fe
Started Aug 26, 2022 5:14 PM
Ended Aug 26, 2022 5:28 PM
Duration 14:13 💡
OS Linux Debian - 11.3
Browser Multiple

View run in Cypress Dashboard ➡️


Flakiness

cypress/e2e/next.cy.ts Flakiness
1 Working with next-12 > should live-reload on src changes

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@ZachJW34 ZachJW34 changed the title feat: adding svelte support for Cypress (#23519) feat: adding svelte component testing support Aug 26, 2022
@ZachJW34
Copy link
Contributor Author

I fixed up the types in e53f3a1

Before:
Screen Shot 2022-08-25 at 8 57 03 PM

After:
Screen Shot 2022-08-25 at 8 57 42 PM

Copy link
Contributor

@lmiller1990 lmiller1990 left a comment

Choose a reason for hiding this comment

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

I approved all previous PRs, this looks to be 💯

Couple of failing CI steps, but looks like Percy just failing to finalize.

@mike-plummer mike-plummer requested a review from a team August 26, 2022 14:05
Copy link
Contributor

@mike-plummer mike-plummer left a comment

Choose a reason for hiding this comment

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

Amazing work! 💯 Validated Vite JS/TS and Webpack, all bootstrap and run tests as expected. Just a couple minor issues on the docs

npm/svelte/README.md Outdated Show resolved Hide resolved
## Install

- Requires Svelte >= 3
- Requires Cypress v10.6.0 or later
Copy link
Contributor

Choose a reason for hiding this comment

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

version number incorrect

npm/svelte/README.md Outdated Show resolved Hide resolved
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.

4 participants