don't set data - never called
cleanup
added more test coverage