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

chore: moving over many vue + vite system tests to use object API instead of function API #21080

Conversation

JessicaSachs
Copy link
Contributor

@JessicaSachs JessicaSachs commented Apr 14, 2022

See the parent Pull Request #21079 for the latest details.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Apr 14, 2022

Thanks for taking the time to open a PR!

@cypress
Copy link

cypress bot commented Apr 14, 2022



Test summary

17872 0 217 0Flakiness 5


Run details

Project cypress
Status Passed
Commit 1b9ad1e
Started Apr 15, 2022 2:55 AM
Ended Apr 15, 2022 3:10 AM
Duration 14:13 💡
OS Linux Debian - 10.10
Browser Multiple

View run in Cypress Dashboard ➡️


Flakiness

commands/net_stubbing.cy.ts Flakiness
1 network stubbing > intercepting request > can delay and throttle a StaticResponse
2 ... > when body contains unicode
commands/xhr.cy.js Flakiness
1 ... > no status when request isnt forced 404
cypress/proxy-logging.cy.ts Flakiness
1 Proxy Logging > request logging > xhr log has response body/status code when xhr response is logged second
2 Proxy Logging > request logging > xhr log has response body/status code when xhr response is logged second

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

@lmiller1990 lmiller1990 marked this pull request as ready for review April 15, 2022 06:25
@lmiller1990 lmiller1990 requested review from a team as code owners April 15, 2022 06:25
@lmiller1990 lmiller1990 requested review from jennifer-shehane and removed request for a team April 15, 2022 06:25
@lmiller1990
Copy link
Contributor

image

Fixed these guys, gonna merge into the feat branch @JessicaSachs.

@lmiller1990 lmiller1990 merged commit 298e995 into chore/cutover-to-bundled-react-mount Apr 15, 2022
@lmiller1990 lmiller1990 deleted the chore/cutover-to-bundled-vue-mount branch April 15, 2022 06:26
lmiller1990 added a commit that referenced this pull request Apr 20, 2022
… API (#21079)

* removing vite-dev-server local dependency from react-vite-ts-configured system test

* moving some CRA examples over to use the object api for setup

* fixing issue where function API was broken by object API for cy config + devservers

* adding deeply nested react import to project-fixtures for cra

* finishes cutting over cypress/react for sys tests

* chore: adding circle for this feature branch

* chore: moving over many vue + vite system tests to use object API instead of function API (#21080)

* doing webpack-dev-server cutovers

* removing more webpack-dev-server refrences

* fixing snapshots

* bumping yarn.lock

* wip

* fix test

* fix assertion

Co-authored-by: Lachlan Miller <lachlan.miller.1990@outlook.com>

* feat: removing all references for "fresh" dev servers (webpack-dev-server-fresh and vite-dev-server-fresh) (#21094)

Co-authored-by: Lachlan Miller <lachlan.miller.1990@outlook.com>
Co-authored-by: Zachary Williams <ZachJW34@gmail.com>

* chore: add dev-servers as deps to server to be included in the binary (#21091)

* fix bad merge

* fix next types and webpack-dev-server- resolve

Co-authored-by: Lachlan Miller <lachlan.miller.1990@outlook.com>
Co-authored-by: Zachary Williams <ZachJW34@gmail.com>
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.

2 participants