Feat/database auth refactor

[clintonbrito]

• [x] renomear tabela de user_with_email para auth
• [x] remover coluna email da tabela user
• [x] criar coluna auth_id na tabela user e relacionar essa coluna auth_id com a tabela auth
• [x] criar coluna onboarding (boolean) na tabela auth
• [x] criar coluna refresh_token na tabela auth

[sonarcloud[bot]]

Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarCloud

[clintonbrito]

@felipesouza91 estava conversando com o @emerson-oliveira e o Juliano sobre o impedimento que estou tendo no momento que é ter que editar os testes que foram escritos antes para fazerem passar porque só com essas mudanças que fiz até o momento já fez quebrar uma porrada de testes.

Para não perdermos tempo com os testes tendo em vista que iremos refatorar muita coisa mesmo, porque acaba que estou quebrando a cabeça para fazer testes passarem que talvez nem existam no futuro, sobretudo por conta de testes envolvendo a antiga arquitetura do projeto que será refatorada, pensamos em duas possibilidades:

1) Colocar skip em todos os testes que quebrarem enquanto estiver trabalhando tanto na refatoração quanto no desenvolvimento de novas features já na nova arquitetura para passar no CI/CD do GitHub e depois editar os testes com base nos arquivos que realmente serão usados no futuro; 2) Deletar os arquivos de testes para destravar aqui no GH o CI/CD e lá na frente investir um tempo só no desenvolvimento de testes do que realmente será utilizado.

O que você acha? Alguma outra possibilidade?

[felipesouza91]

@clintonbrito não acho uma boa remover os testes. Infelizmente vai ser algo que tem que ser feito. O que pode fazer e eu ver isso quando fizer o Merge do meu refactor com a branch de Dev. Pois o que eu estou fazendo vai quebrar e gerar conflitos com o que você está fazendo.

No caso, os testes estão impedindo você de prosseguir no desenvolvimento ou e só na hora de fazer o push ?

Pode fazer o seguinte da Skip em tudo e só deixa o teste da task que você tiver fazendo funcionado.

Tem que informar o @emerson-oliveira que o covereage vai cair de forma drástica pra ele ajustar as métricas no Github actions.

[clintonbrito]

@felipesouza91

No caso, os testes estão impedindo você de prosseguir no desenvolvimento ou e só na hora de fazer o push ?

Por enquanto, mais na hora de fazer o push mesmo. O GitHub Actions impede de proceder com o merge.

Pode fazer o seguinte da Skip em tudo e só deixa o teste da task que você tiver fazendo funcionado.

Certo. Vou dar skip e só focar em testes da task.

Tem que informar o @emerson-oliveira que o covereage vai cair de forma drástica pra ele ajustar as métricas no Github actions.

Ok, depois ele responde aqui sobre isso.

[felipesouza91]

@clintonbrito show vamos seguir nesse ritmo quando eu tiver finalizando do refactor, aí a gente vê com o @emerson-oliveira e @htamagnus de dar uma parada nas task par poder mergear isso tudo e resolver os conflitos

[emerson-oliveira]

@clintonbrito , @felipesouza91, vi que conseguiram avançar com a discussão em relação as testes. Muito bom!! Acredito que no momento de refactor essa abordagem do skip vai ser a melhor. Porém, vamos ter que atualizar nossa lista de débitos técnicos.

Acredito que podemos criar esses itens como issues aqui dentro do github para em breve retomar sem falta.