code style
pending tests for controllers