Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • garbus-frontend garbus-frontend
  • Project information
    • Project information
    • Activity
    • Labels
    • Planning hierarchy
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GarbUs
  • garbus-frontendgarbus-frontend
  • Merge requests
  • !49

Merged
Created Jun 24, 2021 by Angelo Calebe Araujo da Rocha@angelo.rochaOwner

FIX firebase.messaging estava causando falhas nos testes

  • Overview 0
  • Commits 1
  • Changes 3

FIX Notificações causando falhas nos testes

PushNotifications.ts foi refatorado, sendo transformado em uma classe utilizando o padrão Singleton, para garantir que o Firebase seja inicializado uma única vez.

A inicialização do Firebase agora é lazy, ou seja, ocorrerá apenas quando um dos métodos de notificações for utilizado.

Desta forma, os testes não serão impactados, a não ser que seja implementado algum teste em um componente que inicialize as notificações. Neste caso, outra abordagem será necessária.

Também foi aplicado um tratamento para exibir um toast de erro caso não seja possível verificar o status das notificações na rota /notifications/is-disabled.

Autores: @angelo.rocha

Checklist

Deixe um dos ícones de acordo com a sua resposta para cada pergunta. (legenda abaixo)

  • ✅ interface funciona nos tamanhos de tela suportados
  • ✅ interface segue especificação no Figma
  • ✅ passa nos testes funcionais definidos para a tarefa/story
  • 🤷‍♀️ documentação atualizada
  • ✅ código dentro dos padrões
  • ❌ adiciona dependências externas
  • 🤷‍♀️ dependências externas adicionadas foram aprovadas pelos AGES III

Legenda:

  • ✅: sim (funciona/builda/documentação atualizada/...)
  • ⚠️: parcialmente (partes não funcionam/apenas documentação pendente/...)
  • ❌: não (não builda/não funciona/não segue padrões/sem documentação/...)
  • 🤷‍♀️: não se aplica (não tenho como testar no iOS/não envolve interface/...)
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix/notifications_breaking_tests