reminder to add tests re input validation