Feat/database auth refactor

[clintonbrito]

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

[sonarcloud]

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.

[sonarcloud]

Quality Gate failed

Failed conditions
8.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

[felipesouza91]

Realizado merge da branch dev para essa. Apos realizado os ajustes. Ficou alinhado com o @emerson-oliveira que a melhor abordagem seria remover o relacionamento entre as tabelas Auth e User, deixando esse relacionamento de forma fraca via codigo. Pois o Auth seria considerado um provedor de acesso externo, similar ao clreack. Com isso ao Realizar o cadastro via a rota de Auth da aplicação, e criado um registro na tabela Auth, e uma pre cadastro do usuario na tabela User, aonde o relacionamento entre essas duas tabelas é via o external_auth_id.

Dessa forma a modelagem do negocio fica desacoplada nos modelos e serviços de authenticação.

Criado teste para a Webhook Middleware que estava incompleto.

[felipesouza91]

Ajustado conforme alinhado com o Emerson.