Skip to content
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.

Sign up for GitHub

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

Jump to bottom

異常なデータ #12

Closed
4 tasks done
jiro4989 opened this issue Jul 15, 2019 · 2 comments
Closed
4 tasks done

異常なデータ #12

jiro4989 opened this issue Jul 15, 2019 · 2 comments

Comments

@jiro4989
Copy link
Owner

jiro4989 commented Jul 15, 2019 •
edited
Loading

目星空のケース。
同様のことは他にも起きる可能性がある

jiro4989/coc-radar#32 で発覚

  • そもそも探索者のページを削除されているケース
  • ページが存在するが、探索者の能力値に不正な値が入っているケース
  • そもそもパラメータの項目が削除されてしまっているケース
  • ネットワークの不調でデータが取得できないケース（リトライする）
@jiro4989
Copy link
Owner Author

異常終了するケースについて、色々考慮が必要。

  • そもそも探索者のページを削除されているケース
  • ページが存在するが、探索者の能力値に不正な値が入っているケース
  • そもそもパラメータの項目が削除されてしまっているケース
  • ネットワークの不調でデータが取得できないケース（リトライする）

@jiro4989 jiro4989 mentioned this issue Jul 15, 2019
Closed
@jiro4989
Copy link
Owner Author

あとメインロジックのテストケース不十分とも考えられる。
異常系の全網羅テストを設ける必要がある。

jiro4989 added a commit that referenced this issue Jul 15, 2019
@jiro4989
🐛 404ページのときはスキップするように修正 (#12)
80aaba2
jiro4989 added a commit that referenced this issue Jul 16, 2019
@jiro4989
🐛 探索者の能力値が存在しないときに、諸々スキップして異常終了する可能性が合ったのを修正 (#12)
ed6546c
jiro4989 added a commit that referenced this issue Jul 16, 2019
@jiro4989
✅ processJsonのテスト追加 (#12)
ad562bc
jiro4989 added a commit that referenced this issue Jul 16, 2019
@jiro4989
Merge pull request #13 from jiro4989/hotfix/#12_illegal
cae240c 
(#12) 404のスキップ、warnログの追加、異常データの無視
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jiro4989