-
Notifications
You must be signed in to change notification settings - Fork 661
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
[OWASP ZAP]管理画面>商品管理>商品一覧のテストを追加 #5332
Conversation
Codecov Report
@@ Coverage Diff @@
## 4.1 #5332 +/- ##
=========================================
Coverage 68.55% 68.55%
Complexity 6159 6159
=========================================
Files 463 463
Lines 25279 25279
=========================================
Hits 17331 17331
Misses 7948 7948
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
let page: Page; | ||
test.beforeAll(async () => { | ||
await zapClient.setMode(Mode.Protect); | ||
await zapClient.newSession('/zap/wrk/sessions/admin_index', true); |
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.
@kurozumi
セッション名は以下でお願いします。
admin_index -> admin_product
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.
修正しました。
@kurozumi ありがとうございます、マージしました |
概要(Overview・Refs Issue)
方針(Policy)
実装に関する補足(Appendix)
テスト(Test)
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目