Should this be return_value= instead of side_effect=?