AC-758 Fixed VisitRoomDAO tests

[rishabh-997]

Description of what I changed

Issue I worked on

JIRA Issue: https://issues.openmrs.org/browse/AC-758

Checklist: I completed these to help reviewers :)

• My pull request only contains ONE single commit
  (the number above, next to the 'Commits' tab is 1).

• I have added tests to cover my changes. (If you refactored
  existing code that was well tested you do not have to add tests)

• All new and existing tests passed.

• My pull request is based on the latest changes of the master branch.

[codecov-io]

Codecov Report

Merging #738 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #738   +/-   ##
=======================================
  Coverage   11.76%   11.76%           
=======================================
  Files         238      238           
  Lines       10646    10646           
  Branches     1027     1027           
=======================================
  Hits         1252     1252           
  Misses       9312     9312           
  Partials       82       82           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3778fdf...f25e8af. Read the comment docs.

[VibhorChinda]

LGTM :)

[f4ww4z]

@rishabh-997 Can you create a helper function to compare the equality of 2 VisitEntity objects? Like you did at #739 . It will make the code a lot easier and remove repetition.

[rishabh-997]

@rishabh-997 Can you create a helper function to compare the equality of 2 VisitEntity objects? Like you did at #739 . It will make the code a lot easier and remove repetition.

yeah sure sir. will make it ready.

[f4ww4z]

@rishabh-997 looks good.