test case works..with some todo