firebase / firebase/firebase-admin-java

[FR] Open up FirebaseMessagingException

オープン
#722 コメント 3 件 リアクション 2 件 担当者 0 名 GitHub で見る
needs-triage type: feature request
主要言語
Java
スター
620
フォーク
305
平均マージ
3時間 23分
マージ済み PR(30日)
1

説明

**Is your feature request related to a problem? Please describe.**
FirebaseMessagingException is final and has package private/private constructors. This prevents using the exception in tests. The firebase recommendation to deal with invalid tokens is to react on error code UNREGISTERED and INVALID_ARGUMENT. In order to test my code that deals with this scenario I would need to create a FirebaseMessagingException with the specified error codes which I can't.

**Describe the solution you'd like**
Open up FirebaseMessagingException to allow instantiation in tests.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず FirebaseMessagingException を見つけ、そのコンストラクタと既存のテストを確認します。SDK の意図した例外動作を維持したまま、テストでエラーコード UNREGISTERED と INVALID_ARGUMENT の例外をインスタンス化できることを確認します。これらのシナリオをテストできれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
firebase, java
領域
api, backend
issue の種類
機能追加
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

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

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