teammates icon indicating copy to clipboard operation
teammates copied to clipboard

Tdd

Open acamposs opened this issue 1 year ago • 2 comments

Neste PR, foi implementado um novo recurso de pesquisa de estudantes utilizando o padrão TDD (Test-Driven Development). As etapas principais do TDD foram seguidas como descrito abaixo:

  1. Criação dos Testes:

    • Objetivo: Implementar a funcionalidade de busca de estudantes em todo o sistema.
    • Testes Criados: Foram elaborados testes unitários para verificar a precisão da busca por substring no nome dos estudantes. Os testes garantem que a busca:
      • Encontre corretamente estudantes cujo nome corresponde ao termo de pesquisa.
      • Retorne uma lista vazia quando nenhum estudante corresponder ao termo.
  2. Implementação do Código:

    • Código Adicionado: A funcionalidade de busca foi implementada na classe Logic com o método searchStudentsInWholeSystem.
    • Lógica: O método divide o termo de pesquisa em partes e verifica se o nome completo do estudante contém essas partes. Se houver correspondência, o estudante é adicionado aos resultados.
  3. Refinamento e Correção:

    • Ajustes: Após a implementação, o código foi ajustado para lidar com exceções e garantir que a lista retornada esteja no formato correto.
    • Correções: Foram corrigidos problemas de nomenclatura e tipo de dados para assegurar a integração correta com o restante do sistema.
  4. Documentação e Pull Request:

    • Documentação: O processo de desenvolvimento foi documentado, incluindo a descrição dos testes, o ciclo de TDD seguido e as capturas de tela dos testes em execução (se aplicável).
    • Pull Request: Este PR inclui a implementação completa da funcionalidade de busca, bem como os testes associados para garantir a qualidade e a precisão da nova funcionalidade.

Esta abordagem garante que o novo recurso está bem testado e funciona conforme o esperado, atendendo aos requisitos definidos.

acamposs avatar Aug 14 '24 03:08 acamposs

Hi @acamposs, thank you for your interest in contributing to TEAMMATES! However, your PR does not appear to follow our contribution guidelines:

  • Title must start with the issue number the PR is fixing in square brackets, e.g. [#<issue-number>]
  • Description must reference the issue number the PR is fixing, e.g. Fixes #<issue-number> (or Part of #<issue-number> if the PR does not address the issue fully)

Please address the above before we proceed to review your PR.

github-actions[bot] avatar Aug 14 '24 03:08 github-actions[bot]

Folks, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...

nusoss-bot avatar Aug 21 '24 15:08 nusoss-bot

Folks, This PR seems to be stalling (no activities for the past 11 days). :snail: :cry: Hope someone can get it to move forward again soon...

nusoss-bot avatar Sep 02 '24 03:09 nusoss-bot

Closing this PR because I'm not certain this addresses an open issue. Feel free to reopen if I'm mistaken. However, do also note the following (which is also stated in our documentation):

  • A PR must address an open issue. If there isn't an open issue, do open one first so that we can determine if it's something that should be addressed or not
  • Formatting changes should not be included in the PR. I noticed many changed lines were due to formatting changes, which I assume was caused by some format on save option. Do turn that option off - we do not enforce any particular style, and adding unnecessary formatting changes to the PR makes reviewing a lot harder
  • Do write comments and PR descriptions in English, since we use that for our communications. You can pass your comments through a translation engine before posting them

weiquu avatar Sep 02 '24 09:09 weiquu