Another batch of mapping tests refactored #2734
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ref: #2708
Same logic as #2732 for this batch. Most of the changes in existing fixtures are meant to try and get them to have similar mapping definitions across mapping providers, which hopefully makes maintaining the tests simpler in the long run as one case is only needed for all 4 mapping configurations. Unfortunately, some quirks and differences in the mapping drivers themselves make this tricky in some places (and consolidating these mapping tests is starting to show that, like the loggable embeddable configs); those should be revisited at some point down the road.
The deleted files are all mapping test classes that are now merged into a single test class.