-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: remove duplicate tests #4136
Conversation
}); | ||
}); | ||
|
||
it("add hot-only option", (done) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e2e tests already present for hot: true
and hot: 'only'
- https://github.com/webpack/webpack-dev-server/blob/master/test/e2e/hot-and-live-reload.test.js
}); | ||
|
||
// TODO: remove this after plugin support is published | ||
it("should create and run server with old parameters order and log deprecation warning", (done) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E2E test already present.
webpack-dev-server/test/e2e/api.test.js
Line 398 in 317e4b9
it(`should work with deprecated API (the order of the arguments in the constructor)`, async () => { |
Codecov Report
@@ Coverage Diff @@
## master #4136 +/- ##
=======================================
Coverage 92.27% 92.27%
=======================================
Files 14 14
Lines 1540 1540
Branches 589 589
=======================================
Hits 1421 1421
Misses 110 110
Partials 9 9 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our CI is now more unstable, we should fix it...
@snitin315 Feel free to merge when CI will green |
For Bugs and Features; did you add new tests?
Motivation / Use-Case
remove duplicate tests.
Breaking Changes
None
Additional Info
No