upe-garanhuns/msw-gama-2022

Análise de Impacto de Melhoria #2: Implementar a função de validação `isPis`

Opened this issue · 0 comments

Análise de Impacto de Melhoria #2: Implementar a função de validação isPis

1) CR de Origem (Issue de origem)

https://user-images.githubusercontent.com/32225687/222295795-cebda5d8-72bd-4461-be73-120841c8908f.png

validatorjs/validator.js#2193

2) Análise de Impacto

  • Link do vídeo: Análise de Impacto

  • Artefatos envolvidos e suas localizações:

    O artefato principal a ser modificado será o arquivo validator.js, localizado no diretório principal do pacote. Será necessário adicionar o novo validador isPis, bem como possíveis funções auxiliares para implementá-lo.

    Também será necessário atualizar a documentação do pacote para incluir a nova funcionalidade e fornecer exemplos de uso.

  • Como será implementada a CR e os efeitos causados pela sua implementação:

    A implementação da nova funcionalidade isPis envolverá as seguintes alterações no código do pacote Validator.js:

    • Adição do novo validador isPis no arquivo validator.js. Isso envolve a definição da função isPis, que receberá um número de PIS como argumento e retornará true se o PIS for válido e false caso contrário. A implementação do validador isPis pode depender da criação de funções auxiliares para a validação do PIS.
    • Atualização do objeto de mensagens de erro utilizado pelo pacote para incluir uma mensagem de erro para o validador isPis. Essa mensagem de erro será retornada caso o PIS não seja válido.
    • Atualização da função validate que será responsável por chamar o novo validador isPis. Isso envolve a inclusão de uma nova condição na função validate para validar o PIS usando a nova função isPis.
    • Atualização da documentação do pacote para incluir a nova funcionalidade e fornecer exemplos de uso.

    A implementação da nova funcionalidade terá como efeito principal a melhoria da qualidade do pacote e a possibilidade de validar PIS de forma mais simples e eficiente. Os impactos no projeto serão positivos, já que a adição de uma nova funcionalidade pode tornar o pacote mais útil e atrair mais desenvolvedores para a comunidade.

    No entanto, a adição de uma nova funcionalidade também pode aumentar a complexidade do pacote e aumentar o risco de bugs ou problemas de desempenho. Portanto, é importante realizar testes abrangentes e buscar revisões por parte de outros desenvolvedores da comunidade para garantir que a nova funcionalidade atenda aos requisitos e padrões do pacote.

  • Complexidade de implementação:

    A complexidade de implementação depende da abordagem escolhida para a validação do PIS. Uma abordagem simples pode ser implementada com poucas linhas de código, enquanto uma abordagem mais complexa pode requerer mais esforço e conhecimento sobre a estrutura do PIS e as regras de validação.

  • Esforço de implementação:

    O esforço de implementação também depende da complexidade da funcionalidade a ser implementada. Se a abordagem escolhida for relativamente simples, o esforço pode ser de algumas horas ou dias. Se a abordagem for mais complexa, pode ser necessário mais tempo e esforço.

  • Macroatividades envolvidas:

    As macroatividades envolvidas na implementação da nova funcionalidade incluem:

    • Análise dos requisitos e documentação da funcionalidade;
    • Implementação da funcionalidade;
    • Testes da funcionalidade;
    • Atualização da documentação do pacote;
    • Criação do pull request para revisão e aceitação da nova funcionalidade.
  • Como os testes serão realizados:

    Os testes podem ser realizados por meio de testes unitários e testes de integração. Os testes unitários devem validar cada parte da implementação do novo validador, enquanto os testes de integração devem validar o comportamento da nova funcionalidade em conjunto com o restante do pacote.

    Para testar o novo validador isPis, podemos criar casos de teste que cobrem diferentes casos de uso, como PISs válidos e inválidos, PISs formatados e não formatados.

  • Riscos previstos e as ações de mitigação:

    Os riscos previstos incluem a possibilidade de introdução de bugs ou problemas de desempenho no pacote, bem como a possibilidade de que a nova funcionalidade não seja aceita pela comunidade ou pelos mantenedores do pacote.

    Para mitigar esses riscos, é importante implementar testes abrangentes e buscar revisões por parte de outros desenvolvedores da comunidade. Também é importante manter a comunicação aberta e transparente com os mantenedores do pacote para garantir que a nova funcionalidade atenda aos requisitos e padrões do pacote.

    Além disso, podemos implementar boas práticas de desenvolvimento, como code reviews, refatoração de código e testes automatizados para minimizar o risco de introduzir bugs ou problemas de desempenho.

3) Estimativa de Implementação

  • Estimativa de Impacto de mudança:
    • Artefatos que serão impactados pela CR:

      1. Código fonte: arquivo index.js;
      2. Documentação: Readme.md do projeto;
    • Tempo em horas para a realização da CR:

      Entre 4 e 8 horas.

  • Estimativa de Esforço:
    • Pouco esforço (Algumas horas de pesquisa/consulta e implementação)
    • Médio esforço (Algumas horas ou dias de pesquisa/consulta e implementação)
    • Muito esforço (Algumas horas, talvez semanas de pesquisa/consulta e implementação)
  • Estimativa de Complexidade:
    • Fácil - Apresenta complexidade relativamente baixa e possui muita assistência da comunidade e/ou da documentação do projeto subjacente.
    • Médio - Apresenta complexidade moderada e/ou possui considerável assistência da comunidade e/ou da documentação do projeto subjacente.
    • Difícil - Apresenta muita complexidade e/ou pouca ou nenhuma assistência da comunidade e/ou da documentação do projeto subjacente.