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.

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

エラーハンドリング対応 #19

Closed
7 tasks done
cucl2-similis opened this issue Nov 2, 2024 · 0 comments · Fixed by #22
Closed
7 tasks done

エラーハンドリング対応 #19

cucl2-similis opened this issue Nov 2, 2024 · 0 comments · Fixed by #22

Comments

@cucl2-similis
Copy link
Owner

cucl2-similis commented Nov 2, 2024

  • Assertions#assertNotNull 修正
    • エラーメッセージ(現状 null is null.)修正
    • カスタム例外追加(テストコード修正)
  • エラーハンドリング追加
    • error.js 追加(現状 console.log
    • テストコード修正(Controller#convertValidator#hasErrors
    • エラーページ遷移テスト作成
      • Jestによる単体テストでは実施が難しいため、Playwright を導入しE2Eテストを実施する。
      • E2Eテスト作成 #20
      • 画面操作によって想定外エラーを発生させることができないため、
        必ず例外送出を行うモックオブジェクトに置き換えてからビルドすることにより、
        想定外エラー発生テスト用サーバを起動してE2Eテストを実行する。
cucl2-similis added a commit that referenced this issue Nov 3, 2024
・カスタム例外追加
・例外メッセージ修正
cucl2-similis added a commit that referenced this issue Nov 3, 2024
・テストケース追加(AssertionError, InvalidCallError)
・テストコード修正(Assertions, Controller, Validator)
cucl2-similis added a commit that referenced this issue Nov 29, 2024
・Errorコンポーネント追加
・テストケース追加(Error)
cucl2-similis added a commit that referenced this issue Dec 2, 2024
・イベントハンドラ内エラー対応
cucl2-similis added a commit that referenced this issue Dec 2, 2024
・テストケース追加(EventHandlerErrorBoundary, InvalidCallError)
cucl2-similis added a commit that referenced this issue Dec 7, 2024
・フォーム送信キャンセル警告を抑制
 ・type変更(submit→button)
 ・submitの既定動作(Enterによるクリック発火)実装
・本番環境のエラー境界でエラーログ出力を抑制
cucl2-similis added a commit that referenced this issue Dec 7, 2024
・テスト実行時のエラー境界ログ出力抑制
・テストケース追加(Form, EventHandlerUtils)
cucl2-similis added a commit that referenced this issue Dec 18, 2024
・ErrorコンポーネントE2Eテスト追加
・想定外エラー発生テスト用サーバ起動処理追加
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant