test: remove duplicate tests#4136
Conversation
snitin315
requested review from
alexander-akait,
anshumanv,
hiroppy and
knagaitsev
as code owners
| }); | ||
| }); | ||
|
|
||
| it("add hot-only option", (done) => { |
There was a problem hiding this comment.
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.
E2E test already present.
webpack-dev-server/test/e2e/api.test.js
Line 398 in 317e4b9
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.
|
alexander-akait
left a comment
alexander-akait
left a comment
There was a problem hiding this comment.
Our CI is now more unstable, we should fix it...
|
@snitin315 Feel free to merge when CI will green |
2 participants
For Bugs and Features; did you add new tests?
Motivation / Use-Case
remove duplicate tests.
Breaking Changes
None
Additional Info
No