We had a couple of helpers to remove some of the bookkeeping of setting
up the tests. This seemed like a good idea at the time, but it meant
that we'd end up with awkward names for the helpers we made.
As we add more tests with different requirements, it's actually harder
to know what to use in each test with the helpers. So, we ditch them in
exchange for calling each thing explicitly.
This is a bit more work for each call-site, but it means that we don't
have to keep inventing new func names that don't accurately describe
what they're doing, or when to use them.
We had a couple of helpers to remove some of the bookkeeping of setting up the tests. This seemed like a good idea at the time, but it meant that we'd end up with awkward names for the helpers we made.
As we add more tests with different requirements, it's actually harder to know what to use in each test with the helpers. So, we ditch them in exchange for calling each thing explicitly.
This is a bit more work for each call-site, but it means that we don't have to keep inventing new func names that don't accurately describe what they're doing, or when to use them.