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

test: creating mcdc tests to get_current_year_and_period method #226

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

LuisMiranda10
Copy link

Descrição

Tipo de alteração

  • Bug fix (alteração que corrige um problema)
  • New feature (alteração que adiciona funcionalidade)
  • Breaking change (correção ou funcionalidade que altera o comportamento de outras partes do sistema de maneira significativa)
  • Documentation update (alteração na documentação)

Como isso foi testado?

  • Teste A
  • Teste B

Checklist

  • Meu código segue as diretrizes de contribuição deste projeto
  • Realizei uma revisão pessoal do meu código
  • Comentei meu código, especialmente em áreas de difícil compreensão
  • Fiz alterações correspondentes na documentação
  • Minhas alterações não geram novos warnings ou erros
  • Adicionei testes que comprovam que minha correção é eficaz ou que minha funcionalidade está funcionando corretamente
  • Todos os testes unitários novos e existentes passam localmente com minhas alterações

Copy link

vercel bot commented Jul 23, 2024

@LuisMiranda10 is attempting to deploy a commit to the Sua Grade UnB Team on Vercel.

A member of the Team first needs to authorize it.

@@ -239,3 +239,12 @@ def sort_by_priority(self):
lambda _class: self.schedule_info[_class.schedule]["priority"], priority)), reverse=True)

return self.schedules

def test_discipline_hour(self, discipline: str, hours: dict[str], departaments: dict[str], choice_disciplines: dict[str]):
Copy link

Choose a reason for hiding this comment

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

Function test_discipline_hour has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Copy link

codeclimate bot commented Aug 13, 2024

Code Climate has analyzed commit f526e4b and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

View more on Code Climate.

@caio-felipee
Copy link
Collaborator

Boa tarde! Vi que o template do Pull Request ainda não foi preenchido. Esse PR ainda terá atualizações?

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.

2 participants