should test for cases when success and not success