From d7f17457e4d25f03bf274f2ac935932e525418e6 Mon Sep 17 00:00:00 2001 From: Eriksen Costa Date: Wed, 21 Jul 2010 15:34:08 -0300 Subject: [PATCH] Code refactoring --- ieducar/lib/App/Model/IedFinder.php | 472 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- ieducar/tests/unit/App/Model/IedFinderTest.php | 102 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------- 2 files changed, 305 insertions(+), 269 deletions(-) diff --git a/ieducar/lib/App/Model/IedFinder.php b/ieducar/lib/App/Model/IedFinder.php index 273fe94..37166ba 100644 --- a/ieducar/lib/App/Model/IedFinder.php +++ b/ieducar/lib/App/Model/IedFinder.php @@ -63,85 +63,222 @@ class App_Model_IedFinder extends CoreExt_Entity } /** - * Retorna uma instância de RegraAvaliacao_Model_Regra a partir dos dados - * da matrícula. + * Retorna um nome de curso, procurando pelo seu código. + * @param int $id + * @return string|FALSE + */ + public static function getCurso($id) + { + $curso = self::addClassToStorage('clsPmieducarCurso', NULL, + 'include/pmieducar/clsPmieducarCurso.inc.php'); + $curso->cod_curso = $id; + $curso = $curso->detalhe(); + return $curso['nm_curso']; + } + + /** + * Retorna um array com as informações da série a partir de seu código. * - * @param int $codMatricula - * @param RegraAvaliacao_Model_RegraDataMapper $mapper - * @return RegraAvaliacao_Model_Regra + * @param int $codSerie + * @return array * @throws App_Model_Exception */ - public static function getRegraAvaliacaoPorMatricula($codMatricula, - RegraAvaliacao_Model_RegraDataMapper $mapper = NULL) + public static function getSerie($codSerie) { - $matricula = self::getMatricula($codMatricula); - $serie = self::getSerie($matricula['ref_ref_cod_serie']); + // Recupera clsPmieducarSerie do storage de classe estático + $serie = self::addClassToStorage('clsPmieducarSerie', NULL, + 'include/pmieducar/clsPmieducarSerie.inc.php'); - if (is_null($mapper)) { - require_once 'RegraAvaliacao/Model/RegraDataMapper.php'; - $mapper = new RegraAvaliacao_Model_RegraDataMapper(); + // Usa o atributo público para depois chamar o método detalhe() + $serie->cod_serie = $codSerie; + $serie = $serie->detalhe(); + + if (FALSE === $serie) { + throw new App_Model_Exception( + sprintf('Série com o código "%d" não existe.', $codSerie) + ); } - return $mapper->find($serie['regra_avaliacao_id']); + return $serie; } /** - * Retorna um array de instâncias ComponenteCurricular_Model_Componente ao - * qual um aluno cursa através de sua matrícula. + * Retorna todas as séries cadastradas na tabela pmieducar.serie, selecionando + * opcionalmente pelo código da instituição. + * @param int $instituicaoId + * @return array + */ + public static function getSeries($instituicaoId = NULL) + { + $serie = self::addClassToStorage('clsPmieducarSerie', NULL, + 'include/pmieducar/clsPmieducarSerie.inc.php'); + + // Carrega as séries + $serie->setOrderby('ref_cod_curso ASC, cod_serie ASC, etapa_curso ASC'); + $serie = $serie->lista(NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, + NULL, NULL, NULL, NULL, $instituicaoId); + + $series = array(); + foreach ($serie as $key => $val) { + $series[$val['cod_serie']] = $val; + } + + return $series; + } + + /** + * Retorna as turmas de uma escola. + * @param int $escola + * @return array (cod_turma => nm_turma) + */ + public static function getTurmas($escola) + { + $turma = self::addClassToStorage('clsPmieducarTurma', NULL, + 'include/pmieducar/clsPmieducarTurma.inc.php'); + + // Carrega as turmas da escola + $turma->setOrderBy('nm_turma ASC'); + $turmas = $turma->lista(NULL, NULL, NULL, NULL, $escola); + + $ret = array(); + foreach ($turmas as $turma) { + $ret[$turma['cod_turma']] = $turma['nm_turma']; + } + + return $ret; + } + + /** + * Retorna o total de módulos do ano letivo da escola ou turma (caso o ano + * escolar do curso não seja "padrão"). Retorna um array com o total de + * módulos atribuídos ao ano letivo e o nome do módulo. Ex: * - * Exclui todas os componentes curriculares ao qual o aluno está dispensado - * de cursar. + * + * 4, + * 'nome' => 'Bimestre' + * ); + * * - * @param int $codMatricula - * @param ComponenteCurricular_Model_ComponenteDataMapper $mapper + * @param int $codEscola + * @param int $codCurso + * @param int $codTurma + * @param int|NULL $ano Ano letivo da escola ou turma. Opcional. * @return array - * @throws App_Model_Exception */ - public static function getComponentesPorMatricula($codMatricula, - ComponenteCurricular_Model_ComponenteDataMapper $mapper = NULL) + public static function getModulo($codEscola, $codCurso, $codTurma, + $ano = NULL) { - $matricula = self::getMatricula($codMatricula); + $modulos = array(); - $codEscola = $matricula['ref_ref_cod_escola']; - $codSerie = $matricula['ref_ref_cod_serie']; + $curso = self::addClassToStorage('clsPmieducarCurso', NULL, + 'include/pmieducar/clsPmieducarCurso.inc.php'); - $serie = self::getSerie($codSerie); + $curso->cod_curso = $codCurso; + $curso = $curso->detalhe(); - // Disciplinas da escola na série em que o aluno está matriculado - $disciplinas = self::getEscolaSerieDisciplina($codSerie, $codEscola); + $padraoAnoEscolar = $curso['padrao_ano_escolar'] == 1 ? TRUE : FALSE; - // Dispensas do aluno - $disciplinasDispensa = self::getDisciplinasDispensadasPorMatricula( - $codMatricula, $codSerie, $codEscola + // Segue o padrão + if (TRUE == $padraoAnoEscolar) { + $escolaAnoLetivo = self::addClassToStorage('clsPmieducarEscolaAnoLetivo', + NULL, 'include/pmieducar/clsPmieducarEscolaAnoLetivo.inc.php'); + + $anosEmAndamento = $escolaAnoLetivo->lista($codEscola, $ano, NULL, NULL, + 1, NULL, NULL, NULL, NULL, 1); + + // Pela restrição na criação de anos letivos, eu posso confiar no primeiro + // e único resultado que deve ter retornado + if (FALSE !== $anosEmAndamento && 1 == count($anosEmAndamento)) { + $ano = array_shift($anosEmAndamento); + $ano = $ano['ano']; + } + else { + throw new App_Model_Exception('Existem vários anos escolares em andamento.'); + } + + $anoLetivoModulo = self::addClassToStorage('clsPmieducarAnoLetivoModulo', + NULL, 'include/pmieducar/clsPmieducarAnoLetivoModulo.inc.php'); + + $modulos = $anoLetivoModulo->lista($ano, $codEscola); + } + else { + $turmaModulo = self::addClassToStorage('clsPmieducarTurmaModulo', + NULL, 'include/pmieducar/clsPmieducarTurmaModulo.inc.php'); + + $modulos = $turmaModulo->lista($codTurma); + } + + if (FALSE === $modulos) { + return 0; + } + + // Total de módulos + $total = count($modulos); + + // Código do tipo de módulo + $modulo = array_shift($modulos); + $codModulo = $modulo['ref_cod_modulo']; + + // Recupera do regstry o objeto legado + $modulo = self::addClassToStorage('clsPmieducarModulo', NULL, + 'include/pmieducar/clsPmieducarModulo.inc.php'); + + $modulo->cod_modulo = $codModulo; + $modulo = $modulo->detalhe(); + $modulo = $modulo['nm_tipo']; + + return array( + 'total' => $total, + 'nome' => $modulo ); + } - // Instancia um data mapper caso nenhum seja provido - if (is_null($mapper)) { + /** + * Retorna array com as referências de pmieducar.escola_serie_disciplina + * a modules.componente_curricular ('ref_ref_cod_disciplina'). + * + * @param int $codSerie + * @param int $codEscola + * @param bool $hydrate + * @return array + * @throws App_Model_Exception + */ + public static function getEscolaSerieDisciplina($codSerie, $codEscola, $hydrate = FALSE) + { + // Disciplinas na série na escola + $escolaSerieDisciplina = self::addClassToStorage('clsPmieducarEscolaSerieDisciplina', + NULL, 'include/pmieducar/clsPmieducarEscolaSerieDisciplina.inc.php'); + + $disciplinasEscolaSerie = $escolaSerieDisciplina->lista($codSerie, $codEscola, NULL, 1); + + if (FALSE === $disciplinasEscolaSerie) { + throw new App_Model_Exception( + sprintf('Nenhuma disciplina para a série (%d) e a escola (%d) informados', + $codSerie, $codEscola) + ); + } + + if ($hydrate) { require_once 'ComponenteCurricular/Model/ComponenteDataMapper.php'; $mapper = new ComponenteCurricular_Model_ComponenteDataMapper(); } - // Seleciona os componentes curriculares em que o aluno está cursando - $componentes = array(); - - foreach ($disciplinas as $disciplina) { - if (in_array($disciplina['ref_cod_disciplina'], $disciplinasDispensa)) { + $disciplinas = array(); + foreach ($disciplinasEscolaSerie as $disciplinaEscolaSerie) { + if ($hydrate) { + $disciplinas[] = $mapper->find($disciplinaEscolaSerie['ref_cod_disciplina']); continue; } - $componenteCurricular = $mapper->findComponenteCurricularAnoEscolar( - $disciplina['ref_cod_disciplina'], - $codSerie + $disciplinas[] = array( + 'ref_cod_disciplina' => $disciplinaEscolaSerie['ref_cod_disciplina'], + 'carga_horaria' => $disciplinaEscolaSerie['carga_horaria'] ); - - if (!is_null($disciplina['carga_horaria'])) { - $componenteCurricular->cargaHoraria = $disciplina['carga_horaria']; - } - - $componentes[$componenteCurricular->id] = $componenteCurricular; } - return $componentes; + return $disciplinas; } /** @@ -206,75 +343,85 @@ class App_Model_IedFinder extends CoreExt_Entity } /** - * Retorna um array com as informações da série a partir de seu código. + * Retorna uma instância de RegraAvaliacao_Model_Regra a partir dos dados + * da matrícula. * - * @param int $codSerie - * @return array + * @param int $codMatricula + * @param RegraAvaliacao_Model_RegraDataMapper $mapper + * @return RegraAvaliacao_Model_Regra * @throws App_Model_Exception */ - public static function getSerie($codSerie) + public static function getRegraAvaliacaoPorMatricula($codMatricula, + RegraAvaliacao_Model_RegraDataMapper $mapper = NULL) { - // Recupera clsPmieducarSerie do storage de classe estático - $serie = self::addClassToStorage('clsPmieducarSerie', NULL, - 'include/pmieducar/clsPmieducarSerie.inc.php'); - - // Usa o atributo público para depois chamar o método detalhe() - $serie->cod_serie = $codSerie; - $serie = $serie->detalhe(); + $matricula = self::getMatricula($codMatricula); + $serie = self::getSerie($matricula['ref_ref_cod_serie']); - if (FALSE === $serie) { - throw new App_Model_Exception( - sprintf('Série com o código "%d" não existe.', $codSerie) - ); + if (is_null($mapper)) { + require_once 'RegraAvaliacao/Model/RegraDataMapper.php'; + $mapper = new RegraAvaliacao_Model_RegraDataMapper(); } - return $serie; + return $mapper->find($serie['regra_avaliacao_id']); } /** - * Retorna array com as referências de pmieducar.escola_serie_disciplina - * a modules.componente_curricular ('ref_ref_cod_disciplina'). + * Retorna um array de instâncias ComponenteCurricular_Model_Componente ao + * qual um aluno cursa através de sua matrícula. * - * @param int $codSerie - * @param int $codEscola - * @param bool $hydrate + * Exclui todas os componentes curriculares ao qual o aluno está dispensado + * de cursar. + * + * @param int $codMatricula + * @param ComponenteCurricular_Model_ComponenteDataMapper $mapper * @return array * @throws App_Model_Exception */ - public static function getEscolaSerieDisciplina($codSerie, $codEscola, $hydrate = FALSE) + public static function getComponentesPorMatricula($codMatricula, + ComponenteCurricular_Model_ComponenteDataMapper $mapper = NULL) { - // Disciplinas na série na escola - $escolaSerieDisciplina = self::addClassToStorage('clsPmieducarEscolaSerieDisciplina', - NULL, 'include/pmieducar/clsPmieducarEscolaSerieDisciplina.inc.php'); + $matricula = self::getMatricula($codMatricula); - $disciplinasEscolaSerie = $escolaSerieDisciplina->lista($codSerie, $codEscola, NULL, 1); + $codEscola = $matricula['ref_ref_cod_escola']; + $codSerie = $matricula['ref_ref_cod_serie']; - if (FALSE === $disciplinasEscolaSerie) { - throw new App_Model_Exception( - sprintf('Nenhuma disciplina para a série (%d) e a escola (%d) informados', - $codSerie, $codEscola) - ); - } + $serie = self::getSerie($codSerie); - if ($hydrate) { + // Disciplinas da escola na série em que o aluno está matriculado + $disciplinas = self::getEscolaSerieDisciplina($codSerie, $codEscola); + + // Dispensas do aluno + $disciplinasDispensa = self::getDisciplinasDispensadasPorMatricula( + $codMatricula, $codSerie, $codEscola + ); + + // Instancia um data mapper caso nenhum seja provido + if (is_null($mapper)) { require_once 'ComponenteCurricular/Model/ComponenteDataMapper.php'; $mapper = new ComponenteCurricular_Model_ComponenteDataMapper(); } - $disciplinas = array(); - foreach ($disciplinasEscolaSerie as $disciplinaEscolaSerie) { - if ($hydrate) { - $disciplinas[] = $mapper->find($disciplinaEscolaSerie['ref_cod_disciplina']); + // Seleciona os componentes curriculares em que o aluno está cursando + $componentes = array(); + + foreach ($disciplinas as $disciplina) { + if (in_array($disciplina['ref_cod_disciplina'], $disciplinasDispensa)) { continue; } - $disciplinas[] = array( - 'ref_cod_disciplina' => $disciplinaEscolaSerie['ref_cod_disciplina'], - 'carga_horaria' => $disciplinaEscolaSerie['carga_horaria'] + $componenteCurricular = $mapper->findComponenteCurricularAnoEscolar( + $disciplina['ref_cod_disciplina'], + $codSerie ); + + if (!is_null($disciplina['carga_horaria'])) { + $componenteCurricular->cargaHoraria = $disciplina['carga_horaria']; + } + + $componentes[$componenteCurricular->id] = $componenteCurricular; } - return $disciplinas; + return $componentes; } /** @@ -328,153 +475,6 @@ class App_Model_IedFinder extends CoreExt_Entity } /** - * Retorna o total de módulos do ano letivo da escola ou turma (caso o ano - * escolar do curso não seja "padrão"). Retorna um array com o total de - * módulos atribuídos ao ano letivo e o nome do módulo. Ex: - * - * - * 4, - * 'nome' => 'Bimestre' - * ); - * - * - * @param int $codEscola - * @param int $codCurso - * @param int $codTurma - * @param int|NULL $ano Ano letivo da escola ou turma. Opcional. - * @return array - */ - public static function getModulo($codEscola, $codCurso, $codTurma, - $ano = NULL) - { - $modulos = array(); - - $curso = self::addClassToStorage('clsPmieducarCurso', NULL, - 'include/pmieducar/clsPmieducarCurso.inc.php'); - - $curso->cod_curso = $codCurso; - $curso = $curso->detalhe(); - - $padraoAnoEscolar = $curso['padrao_ano_escolar'] == 1 ? TRUE : FALSE; - - // Segue o padrão - if (TRUE == $padraoAnoEscolar) { - $escolaAnoLetivo = self::addClassToStorage('clsPmieducarEscolaAnoLetivo', - NULL, 'include/pmieducar/clsPmieducarEscolaAnoLetivo.inc.php'); - - $anosEmAndamento = $escolaAnoLetivo->lista($codEscola, $ano, NULL, NULL, - 1, NULL, NULL, NULL, NULL, 1); - - // Pela restrição na criação de anos letivos, eu posso confiar no primeiro - // e único resultado que deve ter retornado - if (FALSE !== $anosEmAndamento && 1 == count($anosEmAndamento)) { - $ano = array_shift($anosEmAndamento); - $ano = $ano['ano']; - } - else { - throw new App_Model_Exception('Existem vários anos escolares em andamento.'); - } - - $anoLetivoModulo = self::addClassToStorage('clsPmieducarAnoLetivoModulo', - NULL, 'include/pmieducar/clsPmieducarAnoLetivoModulo.inc.php'); - - $modulos = $anoLetivoModulo->lista($ano, $codEscola); - } - else { - $turmaModulo = self::addClassToStorage('clsPmieducarTurmaModulo', - NULL, 'include/pmieducar/clsPmieducarTurmaModulo.inc.php'); - - $modulos = $turmaModulo->lista($codTurma); - } - - if (FALSE === $modulos) { - return 0; - } - - // Total de módulos - $total = count($modulos); - - // Código do tipo de módulo - $modulo = array_shift($modulos); - $codModulo = $modulo['ref_cod_modulo']; - - // Recupera do regstry o objeto legado - $modulo = self::addClassToStorage('clsPmieducarModulo', NULL, - 'include/pmieducar/clsPmieducarModulo.inc.php'); - - $modulo->cod_modulo = $codModulo; - $modulo = $modulo->detalhe(); - $modulo = $modulo['nm_tipo']; - - return array( - 'total' => $total, - 'nome' => $modulo - ); - } - - /** - * Retorna todas as séries cadastradas na tabela pmieducar.serie, selecionando - * opcionalmente pelo código da instituição. - * @param int $instituicaoId - * @return array - */ - public static function getSeries($instituicaoId = NULL) - { - $serie = self::addClassToStorage('clsPmieducarSerie', NULL, - 'include/pmieducar/clsPmieducarSerie.inc.php'); - - // Carrega as séries - $serie->setOrderby('ref_cod_curso ASC, cod_serie ASC, etapa_curso ASC'); - $serie = $serie->lista(NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, - NULL, NULL, NULL, NULL, $instituicaoId); - - $series = array(); - foreach ($serie as $key => $val) { - $series[$val['cod_serie']] = $val; - } - - return $series; - } - - /** - * Retorna um nome de curso, procurando pelo seu código. - * @param int $id - * @return string|FALSE - */ - public static function getCurso($id) - { - $curso = self::addClassToStorage('clsPmieducarCurso', NULL, - 'include/pmieducar/clsPmieducarCurso.inc.php'); - $curso->cod_curso = $id; - $curso = $curso->detalhe(); - return $curso['nm_curso']; - } - - /** - * Retorna as turmas de uma escola. - * @param int $escola - * @return array (cod_turma => nm_turma) - */ - public static function getTurmas($escola) - { - $turma = self::addClassToStorage('clsPmieducarTurma', NULL, - 'include/pmieducar/clsPmieducarTurma.inc.php'); - - // Carrega as turmas da escola - $turma->setOrderBy('nm_turma ASC'); - $turmas = $turma->lista(NULL, NULL, NULL, NULL, $escola); - - $ret = array(); - foreach ($turmas as $turma) { - $ret[$turma['cod_turma']] = $turma['nm_turma']; - } - - return $ret; - } - - /** * @see CoreExt_Entity_Validatable#getDefaultValidatorCollection() */ public function getDefaultValidatorCollection() diff --git a/ieducar/tests/unit/App/Model/IedFinderTest.php b/ieducar/tests/unit/App/Model/IedFinderTest.php index 23bcfd4..d4e13ce 100644 --- a/ieducar/tests/unit/App/Model/IedFinderTest.php +++ b/ieducar/tests/unit/App/Model/IedFinderTest.php @@ -101,7 +101,7 @@ class App_Model_IedFinderTest extends UnitBaseTest return $tabela; } - public function testCarregaNomeDeCursoPorCodigo() + public function testGetCurso() { $returnValue = array( 'nm_curso' => 'Ensino Fundamental' @@ -117,45 +117,63 @@ class App_Model_IedFinderTest extends UnitBaseTest 'clsPmieducarCurso', $mock, NULL, TRUE); $curso = App_Model_IedFinder::getCurso(1); - $this->assertEquals($returnValue['nm_curso'], $curso); + $this->assertEquals( + $returnValue['nm_curso'], $curso, + '::getCurso() retorna o nome do curso através de uma busca pelo código.' + ); } - public function testCarregaSeries() + public function testGetInstituicoes() { - $returnValue = array(1 => array('cod_serie' => 1, 'nm_serie' => 'Pré')); + $returnValue = array(array('cod_instituicao' => 1, 'nm_instituicao' => 'Instituição')); + $expected = array(1 => 'Instituição'); - $mock = $this->getCleanMock('clsPmieducarSerie'); + $mock = $this->getCleanMock('clsPmieducarInstituicao'); $mock->expects($this->once()) ->method('lista') ->will($this->returnValue($returnValue)); // Registra a instância no repositório de classes de CoreExt_Entity - $instance = CoreExt_Entity::addClassToStorage( - 'clsPmieducarSerie', $mock, NULL, TRUE); + $instance = App_Model_IedFinder::addClassToStorage( + 'clsPmieducarInstituicao', $mock); - $series = App_Model_IedFinder::getSeries(1); - $this->assertEquals($returnValue, $series); + $instituicoes = App_Model_IedFinder::getInstituicoes(); + $this->assertEquals( + $expected, $instituicoes, + '::getInstituicoes() retorna todas as instituições cadastradas.' + ); } - public function testCarregaInstituicoes() + public function testGetSeries() { - $returnValue = array(array('cod_instituicao' => 1, 'nm_instituicao' => 'Instituição')); - $expected = array(1 => 'Instituição'); + $returnValue = array( + 1 => array('cod_serie' => 1, 'ref_ref_cod_instituicao' => 1, 'nm_serie' => 'Pré'), + 2 => array('cod_serie' => 2, 'ref_ref_cod_instituicao' => 2, 'nm_serie' => 'Pré') + ); - $mock = $this->getCleanMock('clsPmieducarInstituicao'); - $mock->expects($this->once()) + $mock = $this->getCleanMock('clsPmieducarSerie'); + $mock->expects($this->exactly(2)) ->method('lista') - ->will($this->returnValue($returnValue)); + ->will($this->onConsecutiveCalls($returnValue, array($returnValue[1]))); // Registra a instância no repositório de classes de CoreExt_Entity - $instance = App_Model_IedFinder::addClassToStorage( - 'clsPmieducarInstituicao', $mock); + $instance = CoreExt_Entity::addClassToStorage( + 'clsPmieducarSerie', $mock, NULL, TRUE); - $instituicoes = App_Model_IedFinder::getInstituicoes(); - $this->assertEquals($expected, $instituicoes); + $series = App_Model_IedFinder::getSeries(); + $this->assertEquals( + $returnValue, $series, + '::getSeries() retorna todas as séries cadastradas.' + ); + + $series = App_Model_IedFinder::getSeries(1); + $this->assertEquals( + array(1 => $returnValue[1]), $series, + '::getSeries() retorna todas as séries de uma instituição.' + ); } - public function testCarregaTurmas() + public function testGetTurmas() { $returnValue = array(1 => array('cod_turma' => 1, 'nm_turma' => 'Primeiro ano')); $expected = array(1 => 'Primeiro ano'); @@ -170,7 +188,10 @@ class App_Model_IedFinderTest extends UnitBaseTest 'clsPmieducarTurma', $mock, NULL, TRUE); $turmas = App_Model_IedFinder::getTurmas(1); - $this->assertEquals($expected, $turmas); + $this->assertEquals( + $expected, $turmas, + '::getTurmas() retorna todas as turmas de uma escola.' + ); } public function testGetMatricula() @@ -222,10 +243,13 @@ class App_Model_IedFinderTest extends UnitBaseTest CoreExt_Entity::addClassToStorage('clsPmieducarCurso', $cursoMock, NULL, TRUE); $matricula = App_Model_IedFinder::getMatricula(1); - $this->assertEquals($expected, $matricula); + $this->assertEquals( + $expected, $matricula, + '::getMatricula() retorna os dados (escola, série, curso, turma e carga horária) de uma matrícula.' + ); } - public function testInstanciaRegraDeAvaliacaoPorMatricula() + public function testGetRegraAvaliacaoPorMatricula() { $expected = new RegraAvaliacao_Model_Regra(array( 'id' => 1, @@ -268,13 +292,16 @@ class App_Model_IedFinderTest extends UnitBaseTest ->will($this->returnValue($expected)); $regraAvaliacao = App_Model_IedFinder::getRegraAvaliacaoPorMatricula(1, $mapperMock); - $this->assertEquals($expected, $regraAvaliacao); + $this->assertEquals( + $expected, $regraAvaliacao, + '::getRegraAvaliacaoPorMatricula() retorna a regra de avaliação de uma matrícula.' + ); } /** - * @depends App_Model_IedFinderTest::testInstanciaRegraDeAvaliacaoPorMatricula + * @depends App_Model_IedFinderTest::testGetRegraAvaliacaoPorMatricula */ - public function testDisciplinasPorMatricula() + public function testGetComponentesPorMatricula() { $componentes = array( new ComponenteCurricular_Model_Componente( @@ -346,13 +373,16 @@ class App_Model_IedFinderTest extends UnitBaseTest $expected[3] = clone($expected[3]); $expected[3]->cargaHoraria = 60; - $this->assertEquals($expected, $disciplinas); + $this->assertEquals( + $expected, $disciplinas, + '::getComponentesPorMatricula() retorna um array de ComponenteCurricular_Model_Componente para uma matrícula.' + ); } /** - * @depends App_Model_IedFinderTest::testInstanciaRegraDeAvaliacaoPorMatricula + * @depends App_Model_IedFinderTest::testGetRegraAvaliacaoPorMatricula */ - public function testModulosDeUmCursoPadraoAnoEscolar() + public function testGetQuantidadeDeModulosMatricula() { $returnEscolaAno = array( array('ref_cod_escola' => 1, 'ano' => 2009, 'andamento' => 1, 'ativo' => 1) @@ -408,13 +438,16 @@ class App_Model_IedFinderTest extends UnitBaseTest $modulos = App_Model_IedFinder::getQuantidadeDeModulosMatricula(1); - $this->assertEquals(4, $modulos); + $this->assertEquals( + 4, $modulos, + '::getQuantidadeDeModulosMatricula() retorna a quantidade de módulos para uma matrícula de ano escolar padrão (curso padrão ano escolar).' + ); } /** - * @depends App_Model_IedFinderTest::testInstanciaRegraDeAvaliacaoPorMatricula + * @depends App_Model_IedFinderTest::testGetRegraAvaliacaoPorMatricula */ - public function testEtapasDeUmCursoAnoNaoPadrao() + public function testGetQuantidadeDeModulosMatriculaCursoAnoNaoPadrao() { // Curso não padrão $returnCurso = array('cod_curso' => 1, 'carga_horaria' => 800, 'hora_falta' => (50 / 60), 'padrao_ano_escolar' => 0); @@ -444,6 +477,9 @@ class App_Model_IedFinderTest extends UnitBaseTest $etapas = App_Model_IedFinder::getQuantidadeDeModulosMatricula(1); - $this->assertEquals(4, $etapas); + $this->assertEquals( + 4, $etapas, + '::getQuantidadeDeModulosMatricula() retorna a quantidade de módulos para uma matrícula de um ano escolar não padrão (curso não padrão).' + ); } } \ No newline at end of file -- libgit2 0.21.2