Skip to content

Refatorações e ajustes finais da Milestone 3 #61

Description

@GuiMRDS

Contexto

Toda Milestone avança o projeto, mas também deixa arestas a serem aparadas. Desta vez não vai ser diferente.

Execução

Já vou trazer aqui o que faltou da Milestone passada, mas que ao final esta lista irá com certeza aumentar:

  • Exportar do model user ou activation quais são as features padrão pós ativação para importar isso nos testes e não precisar atualizar manualmente baterias que esperam por esses itens. E para cobrir/travar essa lista e evitar erros por desatenção, fazer um teste unitário que espera manualmente as features. Esta excelente recomendação foi de um aluno, mas que não estou mais encontrando quem foi. Caso a pessoa leia isso, por favor se identificar para eu marcar aqui 🙏

  • O aluno kazluBR apontou que ficou faltando aplicar a tolerância de 5s na comparação do timestamp no teste With valid token que está no PATCH contra /api/v1/activations/[token_id]. Como referência ele compartilhou o commit que fez em seu projeto.

  • Se certificar do uso certo do NOW() com Fuso Horário nas queries, pois no Postgres forçar o Fuso pode causar comportamentos inesperados, como a remoção da informação do fuso. Origem da pesquisa começou através deste comentário do aluno Heinemann

  • Conseguir atualizar o próprio username com um case diferente. Pull request de referência.

  • Adicionar um limite mínimo de dias para o npm aceitar baixar dependências e evitar ataques à cadeia de suprimentos. Para bloquear versões publicadas há menos de 7 dias, use min-release-age=7 dentro do arquivo .npmrc.

  • Aparentemente o endpoint /migrations não está mais funcionando nos ambientes da Vercel depois das últimas atualizações, apesar de localmente e CI estarem funcionando. Isto foi reportado neste comentário do aluno leandl onde já sugere um ajuste. Um ajuste com outro ângulo foi sugerido por Andrei que é no arquivo next.config.js colocar o seguinte:

    module.exports = {
      outputFileTracingIncludes: {
        "/api/v1/migrations": ["./infra/migrations/*.js"],
      },
    };

    A explicação do funcionamento disto está nesta documentação do framework.

  • …outros ajustes que inclusive podem ser sugeridos nessa issue, basta usar os comentários abaixo 🤝

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions