Salvar nos eventos o histórico de alterações de username

[aprendendofelipe]

Estudando as possibilidades para revalidar as páginas estáticas sob demanda para diminuir a utilização do cache da Vercel (#1724 (comment)), optei por usar os eventos para analisar o que precisa ser revalidado. Usar os eventos facilita se um dia quisermos criar uma fila ou levar o processo de revalidação para outro servidor que monitora e reage de acordo com os eventos.

Para revalidar as páginas antigas e retornar 404 quando um usuário alterar seu username, precisamos guardar o antigo username no evento. Precisamos atualizar os antigos para 404, pois se deixar os dados antigos em cache, eles não vão ser consistentes com as novas mudanças nos conteúdos, como votos, comentários e qualquer outra modificação que só vai afetar o endereço novo.

Mudanças realizadas

Com isso em mente, adicionei os metadados nos eventos de alteração do username:

metadata.username = {
  old: originalUser.username,
  new: updatedUser.username,
};

Ao adicionar testes para a modificação, notei que nem todos os tipos de eventos possuíam testes, então já criei para todos.

Movi os poucos que existiam em tests/integration/api/v1/events/get.test.js para junto dos novos que criei em tests/integration/models/event.test.js.

E os testes mostraram que o evento do tipo update:content:text_child nunca era criado, mas apenas o update:content:text_root no seu lugar. Então já corrigi isso.

Adicionei o método delete ao requestBuilder para os testes de evento de banimento.

Outra mudança foi alterar events.findAll() para orchestrator.getLastEvent(), pois findAll era utilizado apenas nos testes, e com as modificação atuais, apenas o último evento precisa ser buscado.

Por fim, adicionei .soft nos expect(responde.status) para o teste poder continuar mesmo se receber um status incorreto. Assim aumenta as chances de termos uma mensagem de erro com mais detalhes do porque o teste falhou, já que olhar para o responseBody normalmente ajuda mais.

Tipo de mudança

• Correção de bug
• Nova funcionalidade

Checklist:

• As modificações não geram novos logs de erro ou aviso (warning).
• Eu adicionei testes que provam que a correção ou novo recurso funciona conforme esperado.
• Tanto os novos testes quanto os antigos estão passando localmente.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
tabnews	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 20, 2024 10:02pm

[Rafatcb]

Boa ideia em criar um arquivo separado para os testes de evento 👍