firebase / firebase/firebaseui-web-react
Usability in testing scenarios - unhandled promise rejections
- 主要言語
- JavaScript
- スター
- 1.3k
- フォーク
- 246
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Hey @nicolasgarnier !
I've been using your component for a hot minute, implementing OAuth in an app I'm contributing to.
There are 2 unhandled promise rejections in this plugin that will automatically fail in a test environment where .env variables are missing.
Is there a special way this plugin should be tested?
If not would it be feasible to implement a catch for each of the 2 used promises in order for it not to have to be mocked out completely?
I tested it in a fork locally and it works like a charm for me, if I should open a PR for this, please let me know!
コントリビューションガイド
調査の方向性
issue にはファイルもテストも指定されていません。まず React wrapper 内の 2 つの promise を見つけ、.env 変数がない状態で失敗を再現してください。テスト環境で未処理の promise rejection が発生せず、OAuth の動作が維持されていれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- firebase, javascript, react
- 領域
- frontend, testing
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100