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

Problemes a InscriptionTeamTest #18

Open
josepma opened this issue May 20, 2013 · 5 comments
Open

Problemes a InscriptionTeamTest #18

josepma opened this issue May 20, 2013 · 5 comments
Assignees
Labels

Comments

@josepma
Copy link
Owner

josepma commented May 20, 2013

He fet modificacions a l'entitat Team perquè estava mal fet el mapeig amb Competition.

Concretament, hi havia l'atribut:

class Team{

Competition competition;

}

I havia de ser

class Team{

List competitions;

}

Però aquesta modificació ha afectat InscriptionTeamTest, que ara no s'executa correctament. Podeu resoldre-ho?

@ghost ghost assigned dhernandez88 May 20, 2013
@dhernandez88
Copy link
Collaborator

Ho hem mirat bastant però no trobem el error, al fer un add() a la llista de competitions salta una NullPointerException, però la llista sembla ben inicialitzada al constructor.

Hem provat a fer-ho pas per pas:

List competition = new ArrayList();
competitions = team1.getCompetitions();
competitions222.add(comp);
team1.setCompetitions(competitions222);

Pero salta la exception al fer el add. Hi ha algun problema per fer un add a una llista que està buida?

@dhernandez88
Copy link
Collaborator

El InscriptionTeamIntegTest no s'ha pogut provar si funciona per que fins ahir faltaba implementar la getCompetitionms del CompetitionDAO.

@josepma
Copy link
Owner Author

josepma commented Jun 3, 2013

Sisplau, a InscriptionTeamIntegTest queda per resoldre el test competitionInscriptionPeriodClosed. En particular, cal refer l'operació periodOpen.

També us proposo que simplifiqueu checkForException(comp,team)

A InscriptionTeamTest, en alguns testos només cal comprovar que es fa la interacció correcta. No cal verificar que la BD s'ha modificat adequadament perquè, com és un mock , en realitat, no es modifica.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants
@josepma @dhernandez88 and others