search

pagarme / superbowleto

:football: A microservice to issue, register and manage boletos
MIT License
5 stars 0 forks source link

Create worker that process background boleto queue #274

Closed evaporei closed 6 years ago

evaporei commented 6 years ago

Description

Creates worker that register boletos which should be registered and for some reason (like the provider being offline) weren't.

Your checklist for this pull request

:rotating_light: Please review this items for a good pull request. :four_leaf_clover:

  1. I've read the project's Contributing Guidelines
  2. My commits are well written and follow pagarme/git-style-guide
  3. My changes are well covered by tests and logs
  4. I've updated the project docs (if needed)
  5. I fell safe about this implementation
  6. I feel comfortable with the code I wrote, and I'm not ashamed to show it to my friends

In a good pull request, everything above is true :relaxed:

this closes https://github.com/pagarme/ghostbusters/issues/9

evaporei commented 6 years ago

Fiz o PR apenas para ficar mais visual quais mudanças já foram feitas. Aqui logging realmente fica mais estranho, mantive desse jeito somente pois o requestId é uma dependência do boletoService. Acho que será necessário uma pequena refatoração da service para que se adeque mais ao worker. Gostaria de falar contigo sobre como usar o SQS de uma forma correta. @grvcoelho

evaporei commented 6 years ago

@grvcoelho Acha que faz sentido realizar algum teste e2e/functional?

grvcoelho commented 6 years ago

Os testes que temos hoje são um pouco funcionais né? Porque eles adicionam coisas na fila, chamam a função e depois verfiicam que os itens foram processados.

A gente só não chama o node worker.js direto, mas de resto fazemos tudo. Acho que podemos deixar do jeito que está :+1:

evaporei commented 6 years ago

Mas nós não testamos nossa funcionalidade de consumir da fila e registar exceto via o handler antigo (resources/processBoletosToRegister). Não testamos o boleto/service/processBoleto também.

evaporei commented 6 years ago

Foi decidido que serão feitos testes para a service de boletos, ou seja, uma nova issue será criada

evaporei commented 6 years ago

https://github.com/pagarme/ghostbusters/issues/73