Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support webmock 1.20.3+ #83

Merged
merged 1 commit into from Dec 8, 2014
Merged

Conversation

slaunay
Copy link
Contributor

@slaunay slaunay commented Dec 6, 2014

When running bundle install it pulled webmock 1.20.4 that produced the following errors when running the tests:

test_adds_id_key_when_configured(ElasticsearchOutput):
ArgumentError: #with method invoked with no arguments. Either options hash or block must be specified.
    /path/to/.rvm/gems/ruby-1.9.3-p551/gems/webmock-1.20.4/lib/webmock/request_pattern.rb:23:in `with'
    /path/to/.rvm/gems/ruby-1.9.3-p551/gems/webmock-1.20.4/lib/webmock/request_stub.rb:13:in `with'
    test/plugin/test_out_elasticsearch.rb:33:in `stub_elastic_ping'
    test/plugin/test_out_elasticsearch.rb:356:in `test_adds_id_key_when_configured'

- was making test failed with recent webmock (1.20.4):
  1) Error:
test_adds_id_key_when_configured(ElasticsearchOutput):
ArgumentError: #with method invoked with no arguments. Either options hash or block must be specified.
    /path/to/.rvm/gems/ruby-1.9.3-p551/gems/webmock-1.20.4/lib/webmock/request_pattern.rb:23:in `with'
    /path/to/.rvm/gems/ruby-1.9.3-p551/gems/webmock-1.20.4/lib/webmock/request_stub.rb:13:in `with'
    test/plugin/test_out_elasticsearch.rb:33:in `stub_elastic_ping'
    test/plugin/test_out_elasticsearch.rb:356:in `test_adds_id_key_when_configured'

- more information:
bblimke/webmock@5554c0d
pitr added a commit that referenced this pull request Dec 8, 2014
@pitr pitr merged commit 81dcfbb into uken:master Dec 8, 2014
@pitr
Copy link
Contributor

pitr commented Dec 8, 2014

thanks @slaunay. Btw, I fixed formatting of your comment with triple backticks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants