Skip to content

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

Open
nathanipti wants to merge 9 commits into
devfrom
fix/frequency_teacher
Open

fix: couldn't register the class on the absent day#723
nathanipti wants to merge 9 commits into
devfrom
fix/frequency_teacher

Conversation

@nathanipti
Copy link
Copy Markdown
Contributor

@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?

@sonarqubecloud
Copy link
Copy Markdown

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

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
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
Copy Markdown
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
@igorgoncalves igorgoncalves force-pushed the dev branch 2 times, most recently from 1e59e3c to f12660b Compare August 7, 2025 14:04
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