better naming. writing testcases