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: node 8 #976
test: node 8 #976
Conversation
@fengmk2, thanks for your PR! By analyzing the history of the files in this pull request, we identified @popomore, @dead-horse and @atian25 to be potential reviewers. |
@@ -72,7 +74,9 @@ describe('test/lib/plugins/security.test.js', () => { | |||
}); | |||
|
|||
it('should check csrf on other.json', () => { | |||
return request(app.callback()) | |||
// use prod env to ignore extends properties like frames | |||
mm(app.config, 'env', 'prod'); |
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.
@dead-horse 还是让 egg-onerror 保留那些信息吧,开发阶段有用
好几个库挂了,我修一下 |
Codecov Report
@@ Coverage Diff @@
## master #976 +/- ##
=======================================
Coverage 98.37% 98.37%
=======================================
Files 28 28
Lines 675 675
=======================================
Hits 664 664
Misses 11 11 Continue to review full report at Codecov.
|
onerror 在 unittest 环境是不是不要返回 frame,不然应用也要改一堆 |
+1, @okoala |
eggjs/egg-onerror#14 直接去掉 error detail |
Checklist
npm test
passesAffected core subsystem(s)
Description of change