chore: test utils cleanup #4180
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #4180 +/- ##
=====================================
Coverage 81.8% 81.8%
=====================================
Files 29 29
Lines 1561 1561
Branches 409 409
=====================================
Hits 1277 1277
Misses 224 224
Partials 60 60Continue to review full report at Codecov.
|
packages/common/src/index.js
Outdated
| export { default as Modes } from './modes' | ||
| export { default as NuxtConfig } from './nuxt.config' | ||
| export { default as Options } from './options' | ||
| export * from './utils' | ||
| export const Utils = _Utils |
There was a problem hiding this comment.
Now we export an object Utils includes all functions in ./utils and also all functions on index, it looks kind of confusing, we'd better keep only one behaviour.
There was a problem hiding this comment.
@clarkdo gotcha, makes sense. I'll modify.
|
@clarkdo updated, tests passing :) |
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Refactor
test/utilsto better separate Nuxt-specific imports/exports and generally clean up these files' headers. Also removed a comment mentioning acreateSportsSelectionViewwhich doesn't appear to be related to Nuxt at all.