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

fix: couldn't register the class on the absent day #723

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

nathanipti
Copy link
Collaborator

@nathanipti nathanipti commented May 27, 2024

Motivação

Quando o professor for sinalizado com falta nos dias XX, ele estaria inabilitado para registrar a aula que não estava presente no dia que foi registrado sua falta.

Foi Realizado alguns testes na plataforma https://demo.tag.ong.br, e foi verificado que, mesmo o professor sendo sinalizado como infrequente ele conseguia registrar a aula normalmente. A equipe da SEMED informou que é de extrema importância a necessidade que nesse caso especifico, o professor que tenha sido sinalizado como infrequente, seja impossibilitado de realizar o registro de aula no dia que não estava presente.

Alterações Realizadas

Validações foram realizadas para impedir que professores registrem suas aulas nos dias em que estiveram ausentes.

Fluxo de Teste

🧪 Test 01

Tipo de usuário: Professor
Testar apenas: Diário eletrônico -> aulas ministradas
 
Neste caso, testar se, para os dias de falta, o professor está sendo impedido de registrar o conteúdo da aula. Ou seja, verificar se, para os dias de falta, é mostrado: ‘Não é possível registrar a aula, pois o professor não estava presente neste dia.’

🧪 Teste 02

Tipo de usuário: Professor
Recurso a ser testado: Diário eletrônico -> Frequência
 
Objetivo do teste: Verificar se o sistema impede que o professor registre a frequência nos dias em que esteve ausente.
 
Passos do teste:
1. Acesse o sistema como um professor.
2. Navegue até a seção 'Diário eletrônico' -> 'Frequência'.
3. Tente registrar a frequência para os dias em que o professor esteve ausente.
 
Resultado esperado: O sistema deve impedir que o professor registre a frequência nos dias em que esteve ausente. Se o sistema bloquear o registro de frequência para esses dias, o teste será considerado bem-sucedido. Caso contrário, o teste terá falhado.

🧪 Teste 03

Tipo de Usuário: Professor 
Recurso a Ser Testado: Diário de Classe
 
Objetivo do Teste: Verificar se o sistema não exibe a tabela de alunos para as datas em que o professor esteve ausente.
 
Passos do Teste:
 
1. Acesse o sistema como professor.
2. Navegue até a seção ‘Diário de Classe’.
3. Selecione uma data na qual o professor esteve ausente.
 
Resultado Esperado: O teste é considerado bem-sucedido se a tabela de alunos não for exibida para uma data em que o professor esteve ausente. Caso contrário, o teste falha.

Migrations Utilizadas

2024-07-11_frequency_teacher

Checklist de revisão

  • O número da versão foi alterado no arquivo config.php?
  • Foi adicionada uma descrição das alterações no arquivo de CHANGELOG?
  • O pull request passou na avaliação do SonarLint?
  • O pull request está nomeado corretamente seguindo o padrão de nomes de branchs?

Copy link

sonarcloud bot commented May 27, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

Copy link

sonarcloud bot commented Jul 11, 2024

Copy link
Collaborator

@Gusta-Santos Gusta-Santos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@AnotherOne07 AnotherOne07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍🏾

@nathanipti nathanipti added Pronto para merge reviews feitas e aprovadas and removed Aguardando review 2 Pronto para merge reviews feitas e aprovadas labels Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants