fixed success and fail