Webhook create method returning false negative about mailgun-ruby HOT 4 CLOSED

Checked out the code and thought I saw the reason:

def create(domain, action, url = '')
  res = @client.post("domains/#{domain}/webhooks", id: action, url: url)
  res.to_h['webhook'] == url && res.to_h[message] == 'Webhook has been created'
end

According to the API docs at https://documentation.mailgun.com/en/latest/api-webhooks.html#examples the 2nd check should be

  res.to_h['webhook']['url']== url && res.to_h[message] == 'Webhook has been created'

The VCR cassette is correct, but the 'webhook_spec' success is a bit misleading because it's not testing the webhook wrapper. It would be best to test the webhook spec directly, but I'm guessing this is more of a broader issue :)

I'll submit a PR with the fix

from mailgun-ruby.

Hey, thanks for the PR - I've merged it! I'll try and cut another release in a few days when I get some time to tackle some other bugs as well. Thanks a bunch!

from mailgun-ruby.

Thanks, @pirogoeth! If I get time I'd love to help build out more of the library as we're going to be using it more heavily at GymLeads going forward 👍

from mailgun-ruby.

@abury That's great to hear! It would be awesome to have you on-board.

from mailgun-ruby.