Print more information on log line interop test failures

[nathaniel-may]

resolves #5658

Description

Adds more information to std out when these log tests fail. Also refactors the code to be higher quality.

Reviewers

This has a few whitespace changes. My non-whitespace changes start at 167 of the original. You can start at test_deserialize_serialize_is_unchanged which is the test. This calls the helper functions that have been modified.

The biggest risk of this change is that I could have weakened the test so it would pass in more situations. I, of course, don't suspect this is the case but it should be verified by the reviewer.

Checklist

• I have read the contributing guide and understand what's expected of me
• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
• I have run changie new to create a changelog entry

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

[McKnight-42]

The Rust looks fine from what I know of rust, logic seems fair, curious about the possible weakening though.

[nathaniel-may]

@McKnight-42 the risk with changing tests like this is that they could be accidentally modified to fail less often. I've frequently found tests in old code bases that could never fail which are obviously not good tests. I don't think this modification falls into that category, but I wanted to call it out as something for reviewers to look for since we're modifying a test.