Code looks good, on the face of it (I'm not familiar with this codebase).
I dislike one- or two-letter names, even in tests, but that's just personal preference.
https://codereview.appspot.com/6935068/
« Back to merge proposal
Code looks good, on the face of it (I'm not familiar with this
codebase).
I dislike one- or two-letter names, even in tests, but that's just
personal preference.
https:/ /codereview. appspot. com/6935068/