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 status student #896

Merged
merged 2 commits into from
Dec 23, 2024
Merged

Conversation

GabrielChagas14
Copy link
Collaborator

Motivação

TCDA-806: No município de Poço Dantas a média de aprovação para a recuperação final (5) é diferente da média de aprovação comum (7). O sistema estava considerando apenas a média 7 por isso o status não estava sendo atualizado corretamente

Alterações Realizadas

Foi alterado o arquivo CalculateFinalMediaUsecase.php de modo que agora a recuperação final é salva no campo "rec_final" da tabela de grades resultse não mais no "final_media" assim o usecase ChageStudentStatusByGradeUsecase.php pode verificar corretamente se a média pós recuperação final é superior a média de aprovação

Fluxo de Teste

Teste 1

Com um dump de Poço Dantas

Testar se um aluno que não tenha atingido a média 7, ou seja uma aluno que vai para recuperação final, ao atingir a média 5 tem o status aprovado.

Teste 2

testar se o cálculo do status está correto para diferentes tipos de cálculo de aprovação em estrutura de unidades e avaliações distintas ("média", "soma", "média semestral", "maior", "menor")

Migrations Utilizadas

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

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

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

@GabrielChagas14 GabrielChagas14 changed the title first commit Fix status student Dec 18, 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

@igorgoncalves igorgoncalves merged commit f0b3b5c into main Dec 23, 2024
5 of 6 checks passed
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.

4 participants