firebase / firebase/firebase-admin-java

[FR] Open up FirebaseMessagingException

Aberta
#722 3 comentários 2 reações 0 responsáveis Ver no GitHub
needs-triage type: feature request
Linguagem predominante
Java
Estrelas
620
Forks
305
Merge médio
3h 23min
PRs com merge (30d)
1

Descrição

**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.

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Start by locating FirebaseMessagingException and reviewing its constructors and existing tests. Confirm that tests can instantiate exceptions for the UNREGISTERED and INVALID_ARGUMENT error codes while preserving the SDK's intended exception behavior; done when those scenarios are testable.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
firebase, java
Domínio
api, backend
Tipo de issue
Funcionalidade
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.