#89 [US02] - Apagar conta

[Joao-Moura]

Descrição

• Adicionando funcionalidade de deletar as contas de usuários (tanto produtor quanto consumidor)

Está consertando alguma issue aberta?

• 89

Tarefas gerais realizadas

• Adição da viewset UserDeleteAPIView para deleção de usuários
• Novo custom router CustomDeleteRouter para métodos DELETE
• Nova rota users/delete
• Testes unitários e de integração

Testando as alterações

1. Registre um productor

---

2. Registre um customer

---

3. Registre um ou mais anúncios para o productor

---

4. Favorite um desses anúncios com o customer

---

5. Acesse a rota users/delete autenticada com o token de quem será apagado do banco, com o seguinte json no código no corpo da requisição

{
    'password': {senha do user}
}

Caso tudo de certo, o código 204 será retornado; OBS: Caso o user que foi apagado for um productor, perceba que os anúncios dele também foram apagados.

---

Para rodar os testes unitários

1. Ligue o docker-compose

2. Rode o código de testes (-v 2 para o verbose) apenas do User

   docker exec 20202-hortum_web_1 bash -c "python manage.py test hortum.users.tests.DeleteUserAPIViewTestCase -v 2"

3. Caso queira rodar todos os testes com o coverage

   docker exec 20202-hortum_web_1 bash -c "coverage run manage.py test && coverage report -m && coverage erase"

4. Cheque se todos os testes passaram

   
   System check identified no issues (0 silenced).
   test_delete_valid_customer (hortum.users.tests.DeleteUserAPIViewTestCase) ... ok
   test_delete_valid_productor (hortum.users.tests.DeleteUserAPIViewTestCase) ... ok
   test_delete_with_invalid_password_user (hortum.users.tests.DeleteUserAPIViewTestCase) ... ok

---

Ran 3 tests in 2.299s

OK Destroying test database for alias 'default' ('test_hortum')...

[codecov-commenter]

Codecov Report

Merging #201 (4a328ee) into main (d63c0d2) will increase coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #201      +/-   ##
==========================================
+ Coverage   99.81%   99.82%   +0.01%     
==========================================
  Files          40       40              
  Lines        1053     1143      +90     
==========================================
+ Hits         1051     1141      +90     
  Misses          2        2              

Flag	Coverage Δ
unittests	99.82% <100.00%> (+0.01%)	:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/hortum/routers.py	100.00% <100.00%> (ø)
src/hortum/users/serializer.py	100.00% <100.00%> (ø)
src/hortum/users/tests.py	100.00% <100.00%> (ø)
src/hortum/users/urls.py	100.00% <100.00%> (ø)
src/hortum/users/viewsets.py	100.00% <100.00%> (ø)
src/hortum/urls.py	100.00% <0.00%> (ø)
src/hortum/settings.py	100.00% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d63c0d2...4a328ee. Read the comment docs.

[Joao-Moura]

@VitorLamego entendo, também esperava uma mensagem, só que como é uma Response padrão do django fica até mais dificil de alterar. Além disso a announcements/update também retorna só o 204 quando é para apagar.

[VitorLamego]

Tranquilo !! Parabéns pelo trabalho.