Refactor and fix functionality dealing with ReadPartitionIdRequestDetails

[codeforgreen]

closes #5694

Summary of changes:

• populate ReadPartitionIdRequestDetails for all invocations of IRequestPartitionHelperSvc which involve computing partitionId for read requests (including search, history, mdm related operations and system operations like bulk import/export, reindex). Do not allow this parameter to be passed as null.
• Optimize calling pattern by adding new methods to IRequestPartitionHelperSvc such that caller no longer needs to pass null arguments.
• Add test for bulk import with patientId partitioning and move existing bulk export tests.

[github-actions]

Formatting check succeeded!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2f5ffe7) 81.32% compared to head (1c12eef) 83.38%.
Report is 448 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #5691      +/-   ##
============================================
+ Coverage     81.32%   83.38%   +2.05%     
- Complexity    23650    26886    +3236     
============================================
  Files          1425     1679     +254     
  Lines         86399   103799   +17400     
  Branches      11677    13149    +1472     
============================================
+ Hits          70265    86554   +16289     
- Misses        10947    11612     +665     
- Partials       5187     5633     +446     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[michaelabuckley]

This seems careful and safe.

Made a minor suggestion about test name.

I'm not sure about replacing the parameter object pattern with overloaded methods, but at least they are all non-null. I think this would have been cleaner with overloads or builders on the underlying ReadPartitionIdRequestDetails instead of the service layer. It shows up in all the mock changes for the new methods names.