we probably want to assert here something right a non exception or that no exception has been thrown? or is this good just wondering if we need to be more explicit.