Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tdd #13169

Closed
wants to merge 3 commits into from
Closed

Tdd #13169

wants to merge 3 commits into from

Conversation

acamposs
Copy link

@acamposs acamposs commented Aug 14, 2024

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.

Copy link

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.

@nusoss-bot
Copy link

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

@nusoss-bot
Copy link

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

@weiquu
Copy link
Contributor

weiquu commented Sep 2, 2024

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 weiquu closed this Sep 2, 2024
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.

3 participants