#902: Editing the getPersonById() to stop Sting Id to int and adding more test cases to personService test

[josephbate]

Pull Requests Requirements

• The PR title includes a brief description of the work done, including the
  Issue number if applicable.
• The PR includes a video showing the changes for the work done.
• The PR title follows conventional commit label standards.
• The changes confirm to the OpenElis Global x3 Styleguide and design
  documentation.
• The changes include tests or are validated by existing tests.
• I have read and agree to the Contributing Guidelines of this project.

Summary

1. added more test in the persion and patient service classes
2. made a change on getPersonById(String personId) to stoped coverting the id from string to interger which caused an illegal data type exception showing there is a mismatch in the parameter request

Screenshots

[Add relevant screenshots here if applicable]

Related Issue

see: https://github.com/orgs/I-TECH-UW/projects/6/views/1?pane=issue&itemId=57668194

[Add a link to the related issue or mention it here if applicable]

Other

[Add any additional information or notes here]

[josephbate]

hello @mozzy11 please go through this as

[mozzy11]

@josephbate can you see the build failure ??
Always make sure to run mvn spotless:apply to properly format your code otherwise the CI build will fail on un-formated code

[github-actions]

👋 Hi, @josephbate,
Conflicts have been detected against the base branch. Please rebase your branch against the base branch.

---

This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

[mozzy11]

@josephbate did you see the CI build failure ??