the match and exclude functions make each test case slightly more compact, but they don't seem to provide significant value beyond that, and they obfuscate the underlying data structure that is being tested. I would prefer to have the test cases be slightly longer but easier to understand at a glance, if that sounds good?
This may be slightly easier to review if you make the diff ignore whitespace changes.
the
matchandexcludefunctions make each test case slightly more compact, but they don't seem to provide significant value beyond that, and they obfuscate the underlying data structure that is being tested. I would prefer to have the test cases be slightly longer but easier to understand at a glance, if that sounds good?This may be slightly easier to review if you make the diff ignore whitespace changes.