Aula.bateCom()

[williamhjcho]

if(this.diaDaSemana != outraAula.getDiaDaSemana()){
    return false;
}else if (this.bateHorarioCom(outraAula.getPeriodo())){
    return true;
}

return false;

reduzível para

return this.diaDaSemana == outraAula.getDiaDaSemana()
        && this.bateHorarioCom(outraAula.getPeriodo());

[calebepb]

Foi confirmado que os testes existentes validam essa refatoração? Se não, construir testes para garantir o antes/depois.

[ErikAoki]

Existe um teste dentro dos pacotes de teste que testa esse método com um objeto teste, e ao reduzir o código do jeito que foi sugerido, o teste continua aprovando.

[calebepb]

Perfeito. Falta apenas fazer o Pull-Request #185 ser aprovado.

[ErikAoki]

Fiquei meio confuso, então eu faria uma branch issue-185 e faria o pull-request? E seria #185 ao invés de #115 mesmo?