code review 7/10 - do not merge

looks pretty good! keep writing tests and take a look at my comments in your code