josepma / fofo

Programming project of the subject Models de procés
0 stars 2 forks source link

RefereeDAOImpl, RefereeDAOImplTest, RefereeDAOImplIntegTest #27

Closed josepma closed 11 years ago

josepma commented 11 years ago

Almenys caldria testejar + implementar les ops:

addReferee(ref) findRefereeByNif(nif) findAllReferees()

anat0lius commented 11 years ago

Als tests d'integració...

Al fer un persist d'un Referee, en addReferee, surt aquest error:

Internal Exception: java.sql.SQLSyntaxErrorException: 'EMAIL' is not a column in table or VTI 'FOFOUSER.REFEREE'.Error Code: 20000 Call:INSERT INTO REFEREE (NIF, NAME, EMAIL) VALUES (?, ?, ?) bind => [refereeNif, refereeName, referee@mail.com]

-En el setUp ja m'asseguro de donar-li un email -A Referee també he modificat el equals per a que també compari email. -Si trec l'atribut email de Referee, puc fer persist bé, sino no. -He provat de posar l'anotació @Column per email, pero no.

anat0lius commented 11 years ago

Estrany, pero es veu que només pase a windows aquest error.

josepma commented 11 years ago

He vist alguns defectes en el test public void testFindRefereeByMatch() .

Considereu aquestes millores:

@Test
public void testFindRefereeByMatch() throws Exception{
    Match m = new Match(new Team("home"),new Team("visitant"));        
    //referee.getMatches().add(m); ****LA INSERCIO D'UN PARTIT A UN REFEREE NO ES FA AIXI
    //                                                SINO AMB L'OP. MatchDAO.addRefereeToMatch, que ja està feta
    //m.setReferee(referee);           ****IDEM.

    mDao.insertMatch(m);   //PRIMER INSERIM EL MATCH
    refDao.addReferee(referee);  //DESPRES INSERIM EL REFEREE

    mDao.addRefereeToMatch(m.getIdMatch(),referee.getNif()); //ARA VINCULEM LES DUES COSES

    assertEquals(referee,refDao.findRefereeByMatch(m.getIdMatch()));
}

Però anteció!! Per poder inserir un match, hauran d'estar inserits els teams corresponents. Sisplau, feu-ho al setup, tot això.