BFD-4267: Use negative values for synthetic data + some logging fixes

[aschey-forpeople]

JIRA Ticket:
BFD-4267

What Does This PR Do?

• Updates synthetic data generator to always use negative values for bene_sk and clm_uniq_id
  • This makes it easier to keep track of which data is synthetic when we mix synthetic and real data together
• Adds new parameters to the data generator to randomize the number of claims per patient
• A few fixes for the data generator when using the benes-sandbox.csv file
  • Adds additional CI tests to ensure that the data generator works with this file
• Logging improvements for MDC values and when an error is redirected to the /error controller
• Fixes status code for an unauthorized request to return 401 instead of 400 - I think I accidentally changed this a bit ago

What Should Reviewers Watch For?

If you're reviewing this PR, please check for these things in particular:

What Security Implications Does This PR Have?

Please indicate if this PR does any of the following:

• Adds any new software dependencies

• Modifies any security controls

• Adds new transmission or storage of data

• Any other changes that could possibly affect security?

• I have considered the above security implications as it relates to this PR. (If one or more of the above apply, it cannot be merged without the ISSO or team security engineer's (@sb-benohe) approval.)

• I have created tests to sufficiently ensure the reliability of my code, if applicable. If this is a modification to an existing piece of code, I have audited the associated tests to ensure everything works as expected.

Validation

Have you fully verified and tested these changes? Is the acceptance criteria met? Please provide reproducible testing instructions, code snippets, or screenshots as applicable.

tests

[brick-green]

approved with one comment