issues
search
malikra
/
TTS-RM
0
stars
0
forks
source link
Instructor Feedback - Week 5
#3
Open
pawaitemadisoncollege
opened
6 years ago
pawaitemadisoncollege
commented
6 years ago
Good work starting to implement the generic dao.
[ ] Consider using .equals to more thoroughly compare entities.
https://github.com/malikra/TTS-RM/blob/8154566d68f5e951dab31c233af63ef569422788/src/test/java/com/malik/persistance/TaskDaoTest.java#L48
[ ] Can you get rid of the UserDao and rely only on the generic dao?
[ ] What is your current code coverage in the persistence package?
[ ] Remember javadoc!
[ ] What should the cascade delete behavior be for associated entities?
Good work starting to implement the generic dao.