-
Notifications
You must be signed in to change notification settings - Fork 8
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportBase: 88.32% // Head: 88.32% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## release-next #498 +/- ##
================================================
- Coverage 88.32% 88.32% -0.01%
================================================
Files 109 109
Lines 9445 9444 -1
Branches 1299 1299
================================================
- Hits 8342 8341 -1
Misses 1095 1095
Partials 8 8
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
需要有 test case 保证 render html 的链路不会错误地引入不支持的 node 依赖 |
理论上测试 Node.js 环境是合理的, 天马场景是因为对执行环境做了定制, 我们也没有办法去模拟这个定制环境吧 当然冒烟测试还是得有... |
有一个 polyfill 文件修改了 process,执行 renderHTML 方法前引入,应该就能保证 renderHTML 链路上对于 node 环境不过多依赖 |
或者把 SSR 产物放到 vm 里去执行,也可以这样测试 |
consola 里面依赖 process 变量,ssr 产物里尽量可能少依赖(因为天马场景下需要模拟这部分变量)