Skip to content

feat: delete image route#28

Merged
GabrielViellCastilho merged 1 commit intodevelopfrom
feat/delete-image
Apr 28, 2026
Merged

feat: delete image route#28
GabrielViellCastilho merged 1 commit intodevelopfrom
feat/delete-image

Conversation

@GabrielViellCastilho
Copy link
Copy Markdown
Contributor

Remoção de imagens de usuário e empresa


Tipo do PR

  • feature (nova funcionalidade)
  • fix (correção de bug)
  • refactor (refatoração sem mudança de comportamento)
  • chore (tarefas técnicas, melhorias internas, configs, etc.)

O que foi feito nesta branch?

  • Implementado endpoint para remoção da foto de perfil do usuário
  • Implementado endpoint para remoção da imagem da empresa
  • Integração com storage local para exclusão física dos arquivos
  • Atualização do banco de dados removendo referência da imagem (setando como null)
  • Tratamento de erros:
    • Usuário/empresa não encontrados (404)
    • Usuário/empresa sem imagem (400)
  • Garantia de consistência entre sistema de arquivos e banco de dados

Instruções de como testar

  1. Rodar a aplicação

    npm run start:dev
  2. Acessar o Swagger
    http://localhost:3000/api

  3. Fazer login com usuário admin
    email: admin@pro4tech.com
    password: Pro4Tech

  4. Testar remoção de imagem de usuário

    • Fazer upload de imagem primeiro (POST /files/profile)
    • Executar DELETE /files/profile
    • Verificar se a imagem foi removida
    • Fazer GET /files/profile/:id e confirmar retorno null
  5. Testar remoção de imagem de empresa

    • Fazer upload da imagem da empresa
    • Executar DELETE /files/company/:companyId
    • Verificar se a imagem foi removida do banco e do storage
  6. Testar cenários de erro

    • Deletar imagem inexistente → deve retornar 400
    • Deletar usuário/empresa inexistente → deve retornar 404

Revisores

@GabrielViellCastilho GabrielViellCastilho merged commit 407ce3a into develop Apr 28, 2026
3 checks passed
@GabrielViellCastilho GabrielViellCastilho deleted the feat/delete-image branch April 28, 2026 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants