Delete cancelled billing agreement

You should never do side-effects in `map`. Instead, you should use `each` for side-effects. But, then there's the return value...

What about this? 

```ruby
billing_agreement = Maybe(BillingAgreement.eager_load ...... /* load agreement here */ )

billing_agreement.each { |ba| ba.destroy }

return billing_agreement.is_some?
```