Skip to content

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

Merged
JessicaSachs merged 45 commits into
chore/cutover-to-bundled-react-mountfrom
feat/swap-vite-fresh-and-webpack-fresh
Apr 19, 2022
Merged

feat: removing all references for "fresh" dev servers (webpack-dev-server-fresh and vite-dev-server-fresh)#21094
JessicaSachs merged 45 commits into
chore/cutover-to-bundled-react-mountfrom
feat/swap-vite-fresh-and-webpack-fresh

Conversation

@JessicaSachs
Copy link
Copy Markdown
Contributor

No description provided.

@cypress
Copy link
Copy Markdown

cypress Bot commented Apr 14, 2022



Test summary

17871 1 217 0Flakiness 3


Run details

Project cypress
Status Failed
Commit 61d5c31
Started Apr 19, 2022 8:39 PM
Ended Apr 19, 2022 8:54 PM
Duration 14:01 💡
OS Linux Debian - 10.10
Browser Multiple

View run in Cypress Dashboard ➡️


Failures

cypress/e2e/commands/net_stubbing.cy.ts Failed
1 network stubbing > waiting and aliasing > can timeout waiting on a single response using "alias.response"

Flakiness

net_stubbing.cy.ts Flakiness
1 network stubbing > intercepting request > can delay and throttle a StaticResponse
2 network stubbing > waiting and aliasing > can timeout waiting on a single response using "alias"
navigation.cy.js Flakiness
1 src/cy/commands/navigation > #visit > window immediately resolves and doesn't reload when visiting the same URL with hashes

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

Base automatically changed from chore/cutover-to-bundled-vue-mount to chore/cutover-to-bundled-react-mount April 15, 2022 06:26
}

if (devServer.bundler === 'vite') {
return { devServer: require('@cypress/vite-dev-server-fresh').devServer, objApi: true }
return { devServer: require('../../../../../npm/vite-dev-server').devServer, objApi: true }
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.

Should this be relative?

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.

Addressed in 2b8faab

@JessicaSachs JessicaSachs marked this pull request as ready for review April 19, 2022 21:00
@JessicaSachs JessicaSachs requested review from a team as code owners April 19, 2022 21:00
@JessicaSachs JessicaSachs requested review from jennifer-shehane and removed request for a team April 19, 2022 21:00
@JessicaSachs JessicaSachs merged commit 19cb287 into chore/cutover-to-bundled-react-mount Apr 19, 2022
@JessicaSachs JessicaSachs deleted the feat/swap-vite-fresh-and-webpack-fresh branch April 19, 2022 21:00
@JessicaSachs
Copy link
Copy Markdown
Contributor Author

merging into feature branch

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.

3 participants