Looks OK to me, this is pretty foundational and I'd like to see how this is used, but as a starting point it's fine. Also looking at upcoming tests would make it easier to see how it all fits together.
« Back to merge proposal
Looks OK to me, this is pretty foundational and I'd like to see how this is used, but as a starting point it's fine. Also looking at upcoming tests would make it easier to see how it all fits together.