code review Index.controller
most tests run through