firebase / firebase/firebaseui-web-react

Usability in testing scenarios - unhandled promise rejections

オープン
#54 コメント 0 件 リアクション 4 件 担当者 0 名 GitHub で見る
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。