OBPIH-7610 restructure date formatter and parser classes away from DateUtil
:sparkles: Description of Change
Link to GitHub issue or Jira ticket: https://pihemr.atlassian.net/browse/OBPIH-7610
Description: Sorry once again for a long PR. 95% of it is just moving classes into new places so I'm going to go over it in tech huddle because a full review wouldn't be worth the time IMO.
I got fed up with the structure of all these asX methods being in DateUtil so I moved them into individual XParser classes. That does a better job of encapsulating the logic and allows us to do the Date -> Instant/LocalDate refactor easier (see my comments in the individual files for details)
Codecov Report
:x: Patch coverage is 38.16794% with 162 lines in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 9.81%. Comparing base (1bb7314) to head (8f0caf6).
:warning: Report is 122 commits behind head on develop.
Additional details and impacted files
@@ Coverage Diff @@
## develop #5661 +/- ##
============================================
+ Coverage 9.12% 9.81% +0.69%
- Complexity 1170 1352 +182
============================================
Files 701 725 +24
Lines 45281 45942 +661
Branches 10851 10973 +122
============================================
+ Hits 4131 4510 +379
- Misses 40497 40725 +228
- Partials 653 707 +54
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@ewaterman Does this PR have a ticket?
I'm merging this PR to unblock my other work that I'm trying to wrap up before the break but as usual please feel free to leave review comments and I can address in a new PR.