Unless I'm getting confused somehow, it seems that the tests introduced in 2/2 already pass in 1/2, so they don't really justify the changes in 2/2. More in general, I think it would be useful to first add the failing tests with todo and then, in a separated commit, add the code changes that fix the tests. That makes it easier for the reviewer to see why the commit is useful.