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

CompetitionDAO: ClassificationTC #36

Open
jordiniubo929 opened this issue Jun 6, 2013 · 4 comments
Open

CompetitionDAO: ClassificationTC #36

jordiniubo929 opened this issue Jun 6, 2013 · 4 comments
Assignees

Comments

@jordiniubo929
Copy link
Collaborator

per obtenir les ClassificationTC de les Competition, a CompetitionDAO s'ha fet l'operacio findClassificationTC(String name) amb el nom de la competicio.

Falta fer els tests

També es té que inserir a la DB cada ClassificationTC per a cada Team respecte a Competition. On es realitzaria aquesta operació?

Faltarien els testos d'aquesta també.

@ghost ghost assigned jordiniubo929 Jun 13, 2013
@josepma
Copy link
Owner

josepma commented Jun 13, 2013

addClassificationTC() ja està feta i testejada. Però cal testejar findClassificationTC que crec que no va bé.
Ho podreu fer???

Heu d'afegir testos a CompetitionDAOImplTest i CompetitionDAOImplIntegTest.

@josepma
Copy link
Owner

josepma commented Jun 13, 2013

Intentaré afegir la classificació de 0 punts per cada equip que s'afegeix a una competició. Per això retocaré:

CompetitionDAO.addTeam(comp, team)

@jordiniubo929
Copy link
Collaborator Author

testos a CompetitionDAOImplTest i CompetitionDAOImplIntegTest fets

@jordiniubo929
Copy link
Collaborator Author

Ja s'afegeix la classificació de 0 punts en CompetitionDAO.addTeam(comp, team)

S'han afegit algun test i s'han modificat alguns altres.

Ara l'operació CompetitionDAO.addClassificationTC crec que ha perdut gran part del sentit i potser es tindria que refactoritzar. Això afectaria bastants tests crec

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

No branches or pull requests

2 participants