Open eburzlaff opened 1 year ago
Hi @eburzlaff! You have a good start with unit testing the User and Property entities and daos. You implemented the GenericDao appropriately to avoid a bunch of duplication between entity-specific daos.
[ ] The relationships between the User and Property should be tested more fully. For example
add a user with properties. Confirm you can access the properties through the user.
If you delete a user that has properties, what should happen to their properties?
What if a property is deleted? What happens to the user. Write tests to confirm what should happen does happen.
[ ] Can you add another assertion here to confirm the id? https://github.com/eburzlaff/RealEstateApp/blob/80021321bc18cc86acd22911a4106991b766d1a1/src/test/java/com/rsproperties/persistence/PropertyTest.java#L55
Once those are added, please run all tests and include a screenshot showing all tests are passing. You can include the screenshot right in your issue response here, or put it in a screenshots directory within your project. Whichever you prefer is fine, just let me know where to look for it. Thanks!
I have completed week's 5 content in my project.
Checkpoint 2 is also ready.
Plan to submit remaining work as soon as possible.