search

gems-uff / sapos

SAPOS main goal is to ease the management of information related to graduate programs such as enrollments, courses, advisement, scholarships, requirements, among others.
http://gems-uff.github.io/sapos/
MIT License
28 stars 14 forks source link

Valor NULL em prazo máximo de Tipo de Etapa #314

Closed braganholo closed 3 years ago

braganholo commented 4 years ago

Recentemente tivemos um bug em produção que foi causado porque o SAPOS permitiu cadastrar um Tipo de Etapa com valor NULL (vazio) para prazo máximo em Meses. Como o SAPOS soma esses valores de prazo máximo para calcular quando um determinado aluno tem que cumprir uma determinada etapa, e ele não sabe somar NULL, dá um erro. É preciso adicionar um teste antes de salvar o tipo de etapa, não deixando nenhum dos valores de prazo máximo (dias, meses e períodos) ser informado como vazio. Note que o valor pode ser zero, mas não pode ser vazio.

Carlos-Eduardo-Cabral-da-Cunha commented 3 years ago

Foram adicionados testes rspec e validações para o modelo PhaseDuration que verificam se os campos de prazo máximo de períodos, meses e dias são números inteiros.