elastomer-client 0.7.0 → 0.8.1

data/lib/elastomer/client/template.rb

@@ -36,7 +36,7 @@ module Elastomer
       #
       # Returns the response body as a Hash
       def get( params = {} )
-        response = client.get '/_template/{template}', update_params(params, :action => 'template.get')
+        response = client.get "/_template/{template}", update_params(params, :action => "template.get")
         response.body
       end
 
@@ -48,7 +48,7 @@ module Elastomer
       #
       # Returns the response body as a Hash
       def create( template, params = {} )
-        response = client.put '/_template/{template}', update_params(params, :body => template, :action => 'template.create')
+        response = client.put "/_template/{template}", update_params(params, :body => template, :action => "template.create")
         response.body
       end
 
@@ -59,7 +59,7 @@ module Elastomer
       #
       # Returns the response body as a Hash
       def delete( params = {} )
-        response = client.delete '/_template/{template}', update_params(params, :action => 'template.delete')
+        response = client.delete "/_template/{template}", update_params(params, :action => "template.delete")
         response.body
       end
 

data/lib/elastomer/client/warmer.rb

@@ -1,17 +1,6 @@
 module Elastomer
   class Client
 
-    # Provides access to warmer API commands.
-    # See http://www.elasticsearch.org/guide/reference/api/admin-indices-warmers/
-    #
-    # index_name  - The name of the index as a String
-    # warmer_name - The name of the warmer as a String
-    #
-    # Returns a Warmer instance.
-    def warmer(index_name, warmer_name)
-      Warmer.new(self, index_name, warmer_name)
-    end
-
     class Warmer
 
       # Create a new Warmer helper for making warmer API requests.
@@ -21,8 +10,8 @@ module Elastomer
       # name       - The name of the warmer as a String
       def initialize(client, index_name, name)
         @client     = client
-        @index_name = @client.assert_param_presence(index_name, 'index name')
-        @name       = @client.assert_param_presence(name, 'warmer name')
+        @index_name = @client.assert_param_presence(index_name, "index name")
+        @name       = @client.assert_param_presence(name, "warmer name")
       end
 
       attr_reader :client, :index_name, :name
@@ -39,7 +28,7 @@ module Elastomer
       #
       # Returns the response body as a Hash
       def create(query, params = {})
-        response = client.put '/{index}{/type}/_warmer/{warmer}', defaults.update(params.update(:body => query))
+        response = client.put "/{index}{/type}/_warmer/{warmer}", defaults.update(params.update(:body => query))
         response.body
       end
 
@@ -50,7 +39,7 @@ module Elastomer
       #
       # Returns the response body as a Hash
       def delete(params = {})
-        response = client.delete '/{index}{/type}/_warmer/{warmer}', defaults.update(params)
+        response = client.delete "/{index}{/type}/_warmer/{warmer}", defaults.update(params)
         response.body
       end
 
@@ -61,7 +50,7 @@ module Elastomer
       #
       # Returns the response body as a Hash
       def get(params = {})
-        response = client.get '/{index}{/type}/_warmer/{warmer}', defaults.update(params)
+        response = client.get "/{index}{/type}/_warmer/{warmer}", defaults.update(params)
         response.body
       end
 

data/lib/elastomer/core_ext/time.rb

@@ -1,4 +1,4 @@
-require 'time'
+require "time"
 
 class Time
   def to_json(ignore = nil)

data/lib/elastomer/middleware/encode_json.rb

@@ -8,8 +8,8 @@ module Elastomer
     #
     # Doesn't try to encode bodies that already are in string form.
     class EncodeJson < Faraday::Middleware
-      CONTENT_TYPE = 'Content-Type'.freeze
-      MIME_TYPE    = 'application/json'.freeze
+      CONTENT_TYPE = "Content-Type".freeze
+      MIME_TYPE    = "application/json".freeze
 
       def call(env)
         match_content_type(env) do |data|
@@ -31,16 +31,16 @@ module Elastomer
 
       def process_request?(env)
         type = request_type(env)
-        has_body?(env) and (type.empty? or type == MIME_TYPE)
+        has_body?(env) && (type.empty? || type == MIME_TYPE)
       end
 
       def has_body?(env)
-        body = env[:body] and !(body.respond_to?(:to_str) and body.empty?)
+        (body = env[:body]) && !(body.respond_to?(:to_str) && body.empty?)
       end
 
       def request_type(env)
         type = env[:request_headers][CONTENT_TYPE].to_s
-        type = type.split(';', 2).first if type.index(';')
+        type = type.split(";", 2).first if type.index(";")
         type
       end
     end

data/lib/elastomer/middleware/opaque_id.rb

@@ -1,4 +1,4 @@
-require 'securerandom'
+require "securerandom"
 
 module Elastomer
   module Middleware
@@ -19,7 +19,7 @@ module Elastomer
     # header](https://github.com/elasticsearch/elasticsearch/issues/1202)
     # for more details.
     class OpaqueId < ::Faraday::Middleware
-      X_OPAQUE_ID = 'X-Opaque-Id'.freeze
+      X_OPAQUE_ID = "X-Opaque-Id".freeze
       COUNTER_MAX = 2**32 - 1
 
       # Faraday middleware implementation.
@@ -47,7 +47,7 @@ module Elastomer
         t = Thread.current
 
         unless t.key? :opaque_id_base
-          t[:opaque_id_base]    = (SecureRandom.urlsafe_base64(12) + '%08x').freeze
+          t[:opaque_id_base]    = (SecureRandom.urlsafe_base64(12) + "%08x").freeze
           t[:opaque_id_counter] = -1
         end
 

data/lib/elastomer/middleware/parse_json.rb

@@ -3,8 +3,8 @@ module Elastomer
 
     # Parse response bodies as JSON.
     class ParseJson < Faraday::Middleware
-      CONTENT_TYPE = 'Content-Type'.freeze
-      MIME_TYPE    = 'application/json'.freeze
+      CONTENT_TYPE = "Content-Type".freeze
+      MIME_TYPE    = "application/json".freeze
 
       def call(environment)
         @app.call(environment).on_complete do |env|
@@ -16,19 +16,19 @@ module Elastomer
 
       # Parse the response body.
       def parse(body)
-        MultiJson.load(body) if body.respond_to?(:to_str) and !body.strip.empty?
+        MultiJson.load(body) if body.respond_to?(:to_str) && !body.strip.empty?
       rescue StandardError, SyntaxError => e
         raise Faraday::Error::ParsingError, e
       end
 
       def process_response?(env)
         type = response_type(env)
-        type.empty? or type == MIME_TYPE
+        type.empty? || type == MIME_TYPE
       end
 
       def response_type(env)
         type = env[:response_headers][CONTENT_TYPE].to_s
-        type = type.split(';', 2).first if type.index(';')
+        type = type.split(";", 2).first if type.index(";")
         type
       end
     end

data/lib/elastomer/notifications.rb

@@ -1,6 +1,6 @@
-require 'active_support/notifications'
-require 'securerandom'
-require 'elastomer/client'
+require "active_support/notifications"
+require "securerandom"
+require "elastomer/client"
 
 module Elastomer
 
@@ -40,7 +40,7 @@ module Elastomer
     end
 
     # The name to subscribe to for notifications
-    NAME = 'request.client.elastomer'.freeze
+    NAME = "request.client.elastomer".freeze
 
     # Internal: Execute the given block and provide instrumentation info to
     # subscribers. The name we use for subscriptions is

data/lib/elastomer/version.rb

@@ -1,5 +1,5 @@
 module Elastomer
-  VERSION = '0.7.0'
+  VERSION = "0.8.1"
 
   def self.version
     VERSION

data/script/bootstrap

@@ -7,3 +7,5 @@ if bundle check 1>/dev/null 2>&1; then
 else
     exec bundle install --binstubs --path vendor/gems "$@"
 fi
+
+overcommit --install

data/script/console

@@ -1,9 +1,9 @@
 #!/usr/bin/env ruby
-require 'irb'
-require 'rubygems'
-require 'bundler/setup'
+require "irb"
+require "rubygems"
+require "bundler/setup"
 
-$LOAD_PATH.unshift 'lib'
-require 'elastomer/client'
+$LOAD_PATH.unshift "lib"
+require "elastomer/client"
 
 IRB.start

data/test/assertions.rb

@@ -4,7 +4,7 @@ module Minitest::Assertions
   # in index responses. Check for either one so we are compatible
   # with 0.90 and 1.0.
   def assert_created(response)
-    assert response['created'] || response['ok'], 'document was not created'
+    assert response["created"] || response["ok"], "document was not created"
   end
 
   #COMPATIBILITY
@@ -12,7 +12,7 @@ module Minitest::Assertions
   # in many responses. Check for either one so we are compatible
   # with 0.90 and 1.0.
   def assert_acknowledged(response)
-    assert response['acknowledged'] || response['ok'], 'document was not acknowledged'
+    assert response["acknowledged"] || response["ok"], "document was not acknowledged"
   end
 
   #COMPATIBILITY
@@ -20,32 +20,32 @@ module Minitest::Assertions
   # get document response. Check for either one so we are compatible
   # with 0.90 and 1.0.
   def assert_found(response)
-    assert response['found'] || response['exists'], 'document was not found'
+    assert response["found"] || response["exists"], "document was not found"
   end
 
   def refute_found(response)
-    refute response['found'] || response['exists'], 'document was unexpectedly found'
+    refute response["found"] || response["exists"], "document was unexpectedly found"
   end
 
   #COMPATIBILITY
   # ES 1.0 replaced the 'ok' attribute in the bulk response item with a
   # 'status' attribute. Here we check for either one for compatibility
   # with 0.90 and 1.0.
-  def assert_bulk_index(item, message='bulk index did not succeed')
-    ok     = item['index']['ok']
-    status = item['index']['status']
+  def assert_bulk_index(item, message="bulk index did not succeed")
+    ok     = item["index"]["ok"]
+    status = item["index"]["status"]
     assert ok == true || status == 201, message
   end
 
-  def assert_bulk_create(item, message='bulk create did not succeed')
-    ok     = item['create']['ok']
-    status = item['create']['status']
+  def assert_bulk_create(item, message="bulk create did not succeed")
+    ok     = item["create"]["ok"]
+    status = item["create"]["status"]
     assert ok == true || status == 201, message
   end
 
-  def assert_bulk_delete(item, message='bulk delete did not succeed')
-    ok     = item['delete']['ok']
-    status = item['delete']['status']
+  def assert_bulk_delete(item, message="bulk delete did not succeed")
+    ok     = item["delete"]["ok"]
+    status = item["delete"]["status"]
     assert ok == true || status == 200, message
   end
 
@@ -55,20 +55,22 @@ module Minitest::Assertions
   # ES 0.90 doesn't have the "mappings" element:
   # mapping["test-index"]["doco"]
   def assert_mapping_exists(response, type, message="mapping expected to exist, but doesn't")
-    mapping = if response.has_key?('mappings')
-      response['mappings'][type]
-    else
-      response[type]
-    end
+    mapping =
+      if response.has_key?("mappings")
+        response["mappings"][type]
+      else
+        response[type]
+      end
     refute_nil mapping, message
   end
 
   def assert_property_exists(response, type, property, message="property expected to exist, but doesn't")
-    mapping = if response.has_key?('mappings')
-      response['mappings'][type]
-    else
-      response[type]
-    end
-    assert mapping['properties'].has_key?(property), message
+    mapping =
+      if response.has_key?("mappings")
+        response["mappings"][type]
+      else
+        response[type]
+      end
+    assert mapping["properties"].has_key?(property), message
   end
 end

data/test/client/bulk_test.rb

@@ -1,27 +1,27 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 describe Elastomer::Client::Bulk do
 
   before do
-    @name  = 'elastomer-bulk-test'
+    @name  = "elastomer-bulk-test"
     @index = $client.index(@name)
 
     unless @index.exists?
       @index.create \
-        :settings => { 'index.number_of_shards' => 1, 'index.number_of_replicas' => 0 },
+        :settings => { "index.number_of_shards" => 1, "index.number_of_replicas" => 0 },
         :mappings => {
           :tweet => {
             :_source => { :enabled => true }, :_all => { :enabled => false },
             :properties => {
-              :message => { :type => 'string', :analyzer => 'standard' },
-              :author  => { :type => 'string', :index => 'not_analyzed' }
+              :message => { :type => "string", :analyzer => "standard" },
+              :author  => { :type => "string", :index => "not_analyzed" }
             }
           },
           :book => {
             :_source => { :enabled => true }, :_all => { :enabled => false },
             :properties => {
-              :title  => { :type => 'string', :analyzer => 'standard' },
-              :author => { :type => 'string', :index => 'not_analyzed' }
+              :title  => { :type => "string", :analyzer => "standard" },
+              :author => { :type => "string", :index => "not_analyzed" }
             }
           }
         }
@@ -34,7 +34,7 @@ describe Elastomer::Client::Bulk do
     @index.delete if @index.exists?
   end
 
-  it 'performs bulk index actions' do
+  it "performs bulk index actions" do
     body = [
       '{"index" : {"_id":"1", "_type":"tweet", "_index":"elastomer-bulk-test"}}',
       '{"author":"pea53", "message":"just a test tweet"}',
@@ -45,16 +45,16 @@ describe Elastomer::Client::Bulk do
     body = body.join "\n"
     h = $client.bulk body
 
-    assert_bulk_index(h['items'][0])
-    assert_bulk_index(h['items'][1])
+    assert_bulk_index(h["items"][0])
+    assert_bulk_index(h["items"][1])
 
     @index.refresh
 
-    h = @index.docs('tweet').get :id => 1
-    assert_equal 'pea53', h['_source']['author']
+    h = @index.docs("tweet").get :id => 1
+    assert_equal "pea53", h["_source"]["author"]
 
-    h = @index.docs('book').get :id => 1
-    assert_equal 'John Scalzi', h['_source']['author']
+    h = @index.docs("book").get :id => 1
+    assert_equal "John Scalzi", h["_source"]["author"]
 
 
     body = [
@@ -66,239 +66,239 @@ describe Elastomer::Client::Bulk do
     body = body.join "\n"
     h = $client.bulk body, :index => @name
 
-    assert_bulk_index h['items'].first, 'expected to index a book'
-    assert_bulk_delete h['items'].last, 'expected to delete a book'
+    assert_bulk_index h["items"].first, "expected to index a book"
+    assert_bulk_delete h["items"].last, "expected to delete a book"
 
     @index.refresh
 
-    h = @index.docs('book').get :id => 1
-    assert !h['exists'], 'was not successfully deleted'
+    h = @index.docs("book").get :id => 1
+    assert !h["exists"], "was not successfully deleted"
 
-    h = @index.docs('book').get :id => 2
-    assert_equal 'Tolkien', h['_source']['author']
+    h = @index.docs("book").get :id => 2
+    assert_equal "Tolkien", h["_source"]["author"]
   end
 
-  it 'supports a nice block syntax' do
+  it "supports a nice block syntax" do
     h = @index.bulk do |b|
-      b.index :_id => 1,   :_type => 'tweet', :author => 'pea53', :message => 'just a test tweet'
-      b.index :_id => nil, :_type => 'book', :author => 'John Scalzi', :title => 'Old Mans War'
+      b.index :_id => 1,   :_type => "tweet", :author => "pea53", :message => "just a test tweet"
+      b.index :_id => nil, :_type => "book", :author => "John Scalzi", :title => "Old Mans War"
     end
-    items = h['items']
+    items = h["items"]
 
-    assert_instance_of Fixnum, h['took']
+    assert_instance_of Fixnum, h["took"]
 
-    assert_bulk_index h['items'].first
-    assert_bulk_create h['items'].last
+    assert_bulk_index h["items"].first
+    assert_bulk_create h["items"].last
 
-    book_id = items.last['create']['_id']
+    book_id = items.last["create"]["_id"]
     assert_match %r/^\S{20,22}$/, book_id
 
     @index.refresh
 
-    h = @index.docs('tweet').get :id => 1
-    assert_equal 'pea53', h['_source']['author']
+    h = @index.docs("tweet").get :id => 1
+    assert_equal "pea53", h["_source"]["author"]
 
-    h = @index.docs('book').get :id => book_id
-    assert_equal 'John Scalzi', h['_source']['author']
+    h = @index.docs("book").get :id => book_id
+    assert_equal "John Scalzi", h["_source"]["author"]
 
 
     h = @index.bulk do |b|
-      b.index  :_id => '', :_type => 'book', :author => 'Tolkien', :title => 'The Silmarillion'
-      b.delete :_id => book_id, :_type => 'book'
+      b.index  :_id => "", :_type => "book", :author => "Tolkien", :title => "The Silmarillion"
+      b.delete :_id => book_id, :_type => "book"
     end
-    items = h['items']
+    items = h["items"]
 
-    assert_bulk_create h['items'].first, 'expected to create a book'
-    assert_bulk_delete h['items'].last, 'expected to delete a book'
+    assert_bulk_create h["items"].first, "expected to create a book"
+    assert_bulk_delete h["items"].last, "expected to delete a book"
 
-    book_id2 = items.first['create']['_id']
+    book_id2 = items.first["create"]["_id"]
     assert_match %r/^\S{20,22}$/, book_id2
 
     @index.refresh
 
-    h = @index.docs('book').get :id => book_id
-    assert !h['exists'], 'was not successfully deleted'
+    h = @index.docs("book").get :id => book_id
+    assert !h["exists"], "was not successfully deleted"
 
-    h = @index.docs('book').get :id => book_id2
-    assert_equal 'Tolkien', h['_source']['author']
+    h = @index.docs("book").get :id => book_id2
+    assert_equal "Tolkien", h["_source"]["author"]
   end
 
-  it 'allows documents to be JSON strings' do
+  it "allows documents to be JSON strings" do
     h = @index.bulk do |b|
-      b.index  '{"author":"pea53", "message":"just a test tweet"}', :_id => 1, :_type => 'tweet'
-      b.create '{"author":"John Scalzi", "title":"Old Mans War"}',  :_id => 1, :_type => 'book'
+      b.index  '{"author":"pea53", "message":"just a test tweet"}', :_id => 1, :_type => "tweet"
+      b.create '{"author":"John Scalzi", "title":"Old Mans War"}',  :_id => 1, :_type => "book"
     end
-    items = h['items']
+    items = h["items"]
 
-    assert_instance_of Fixnum, h['took']
+    assert_instance_of Fixnum, h["took"]
 
-    assert_bulk_index h['items'].first
-    assert_bulk_create h['items'].last
+    assert_bulk_index h["items"].first
+    assert_bulk_create h["items"].last
 
     @index.refresh
 
-    h = @index.docs('tweet').get :id => 1
-    assert_equal 'pea53', h['_source']['author']
+    h = @index.docs("tweet").get :id => 1
+    assert_equal "pea53", h["_source"]["author"]
 
-    h = @index.docs('book').get :id => 1
-    assert_equal 'John Scalzi', h['_source']['author']
+    h = @index.docs("book").get :id => 1
+    assert_equal "John Scalzi", h["_source"]["author"]
 
 
     h = @index.bulk do |b|
-      b.index '{"author":"Tolkien", "title":"The Silmarillion"}', :_id => 2, :_type => 'book'
-      b.delete :_id => 1, :_type => 'book'
+      b.index '{"author":"Tolkien", "title":"The Silmarillion"}', :_id => 2, :_type => "book"
+      b.delete :_id => 1, :_type => "book"
     end
-    items = h['items']
+    items = h["items"]
 
-    assert_bulk_index h['items'].first, 'expected to index a book'
-    assert_bulk_delete h['items'].last, 'expected to delete a book'
+    assert_bulk_index h["items"].first, "expected to index a book"
+    assert_bulk_delete h["items"].last, "expected to delete a book"
 
     @index.refresh
 
-    h = @index.docs('book').get :id => 1
-    assert !h['exists'], 'was not successfully deleted'
+    h = @index.docs("book").get :id => 1
+    assert !h["exists"], "was not successfully deleted"
 
-    h = @index.docs('book').get :id => 2
-    assert_equal 'Tolkien', h['_source']['author']
+    h = @index.docs("book").get :id => 2
+    assert_equal "Tolkien", h["_source"]["author"]
   end
 
-  it 'executes a bulk API call when a request size is reached' do
+  it "executes a bulk API call when a request size is reached" do
     ary = []
     ary << @index.bulk(:request_size => 300) do |b|
       2.times { |num|
-        document = {:_id => num, :_type => 'tweet', :author => 'pea53', :message => "tweet #{num} is a 100 character request"}
+        document = {:_id => num, :_type => "tweet", :author => "pea53", :message => "tweet #{num} is a 100 character request"}
         ary << b.index(document)
       }
       ary.compact!
       assert_equal 0, ary.length
 
       7.times { |num|
-        document = {:_id => num+2, :_type => 'tweet', :author => 'pea53', :message => "tweet #{num+2} is a 100 character request"}
+        document = {:_id => num+2, :_type => "tweet", :author => "pea53", :message => "tweet #{num+2} is a 100 character request"}
         ary << b.index(document)
       }
       ary.compact!
       assert_equal 3, ary.length
 
-      document = {:_id => 10, :_type => 'tweet', :author => 'pea53', :message => "tweet 10 is a 102 character request"}
+      document = {:_id => 10, :_type => "tweet", :author => "pea53", :message => "tweet 10 is a 102 character request"}
       ary << b.index(document)
     end
     ary.compact!
 
     assert_equal 4, ary.length
-    ary.each { |a| a['items'].each { |b| assert_bulk_index(b) } }
+    ary.each { |a| a["items"].each { |b| assert_bulk_index(b) } }
 
     @index.refresh
-    h = @index.docs.search :q => '*:*', :search_type => 'count'
+    h = @index.docs.search :q => "*:*", :search_type => "count"
 
-    assert_equal 10, h['hits']['total']
+    assert_equal 10, h["hits"]["total"]
   end
 
-  it 'executes a bulk API call when an action count is reached' do
+  it "executes a bulk API call when an action count is reached" do
     ary = []
     ary << @index.bulk(:action_count => 3) do |b|
       2.times { |num|
-        document = {:_id => num, :_type => 'tweet', :author => 'pea53', :message => "this is tweet number #{num}"}
+        document = {:_id => num, :_type => "tweet", :author => "pea53", :message => "this is tweet number #{num}"}
         ary << b.index(document)
       }
       ary.compact!
       assert_equal 0, ary.length
 
       7.times { |num|
-        document = {:_id => num+2, :_type => 'tweet', :author => 'pea53', :message => "this is tweet number #{num+2}"}
+        document = {:_id => num+2, :_type => "tweet", :author => "pea53", :message => "this is tweet number #{num+2}"}
         ary << b.index(document)
       }
       ary.compact!
       assert_equal 3, ary.length
 
-      document = {:_id => 10, :_type => 'tweet', :author => 'pea53', :message => "this is tweet number 10"}
+      document = {:_id => 10, :_type => "tweet", :author => "pea53", :message => "this is tweet number 10"}
       ary << b.index(document)
     end
     ary.compact!
 
     assert_equal 4, ary.length
-    ary.each { |a| a['items'].each { |b| assert_bulk_index(b) } }
+    ary.each { |a| a["items"].each { |b| assert_bulk_index(b) } }
 
     @index.refresh
-    h = @index.docs.search :q => '*:*', :search_type => 'count'
+    h = @index.docs.search :q => "*:*", :search_type => "count"
 
-    assert_equal 10, h['hits']['total']
+    assert_equal 10, h["hits"]["total"]
   end
 
-  it 'uses :id from parameters' do
+  it "uses :id from parameters" do
     response = @index.bulk do |b|
-      document = { :_type => 'tweet', :author => 'pea53', :message => 'just a test tweet' }
-      params = { :id => 'foo' }
+      document = { :_type => "tweet", :author => "pea53", :message => "just a test tweet" }
+      params = { :id => "foo" }
 
       b.index document, params
     end
 
-    assert_instance_of Fixnum, response['took']
+    assert_instance_of Fixnum, response["took"]
 
-    items = response['items']
+    items = response["items"]
     assert_bulk_index(items[0])
 
-    assert_equal 'foo', items[0]['index']['_id']
+    assert_equal "foo", items[0]["index"]["_id"]
   end
 
-  it 'supports symbol and string parameters' do
+  it "supports symbol and string parameters" do
     response = @index.bulk do |b|
-      doc1 = { :author => 'pea53', :message => 'a tweet about foo' }
-      b.index doc1, { :id => 'foo', :type => 'tweet' }
+      doc1 = { :author => "pea53", :message => "a tweet about foo" }
+      b.index doc1, { :id => "foo", :type => "tweet" }
 
-      doc2 = { :author => 'pea53', :message => 'a tweet about bar' }
-      b.index doc2, { 'id' => 'bar', 'type' => 'tweet' }
+      doc2 = { :author => "pea53", :message => "a tweet about bar" }
+      b.index doc2, { "id" => "bar", "type" => "tweet" }
     end
 
-    assert_instance_of Fixnum, response['took']
+    assert_instance_of Fixnum, response["took"]
 
-    items = response['items']
+    items = response["items"]
     assert_bulk_index(items[0])
     assert_bulk_index(items[1])
 
-    assert_equal 'a tweet about foo', @index.docs('tweet').get(:id => 'foo')['_source']['message']
-    assert_equal 'a tweet about bar', @index.docs('tweet').get(:id => 'bar')['_source']['message']
+    assert_equal "a tweet about foo", @index.docs("tweet").get(:id => "foo")["_source"]["message"]
+    assert_equal "a tweet about bar", @index.docs("tweet").get(:id => "bar")["_source"]["message"]
   end
 
   it 'doesn\'t override parameters from the document' do
-    document = { :_id => 1, :_type => 'tweet', :author => 'pea53', :message => 'just a test tweet' }
+    document = { :_id => 1, :_type => "tweet", :author => "pea53", :message => "just a test tweet" }
     params = { :id => 2 }
 
     response = @index.bulk do |b|
       b.index document, params
     end
 
-    assert_instance_of Fixnum, response['took']
+    assert_instance_of Fixnum, response["took"]
 
-    items = response['items']
+    items = response["items"]
     assert_bulk_index(items[0])
 
-    refute_found @index.docs('tweet').get(:id => 1)
-    assert_equal 'just a test tweet', @index.docs('tweet').get(:id => 2)['_source']['message']
+    refute_found @index.docs("tweet").get(:id => 1)
+    assert_equal "just a test tweet", @index.docs("tweet").get(:id => 2)["_source"]["message"]
   end
 
   it 'doesn\'t upgrade non-prefixed keys to parameters' do
-    document = { :id => 1, :type => 'book', :version => 5, :author => 'pea53', :message => 'just a test tweet' }
-    params = { :id => 2, :type => 'tweet' }
+    document = { :id => 1, :type => "book", :version => 5, :author => "pea53", :message => "just a test tweet" }
+    params = { :id => 2, :type => "tweet" }
 
     response = @index.bulk do |b|
       b.index document, params
     end
 
-    assert_instance_of Fixnum, response['took']
+    assert_instance_of Fixnum, response["took"]
 
-    items = response['items']
+    items = response["items"]
     assert_bulk_index(items[0])
 
-    assert_equal '2', items[0]['index']['_id']
-    assert_equal 'tweet', items[0]['index']['_type']
-    assert_equal 1, items[0]['index']['_version']
+    assert_equal "2", items[0]["index"]["_id"]
+    assert_equal "tweet", items[0]["index"]["_type"]
+    assert_equal 1, items[0]["index"]["_version"]
   end
 
-  it 'streams bulk responses' do
+  it "streams bulk responses" do
     ops = [
-      [:index, { :message => 'tweet 1' }, { :_id => 1, :_type => 'book', :_index => @index.name }],
-      [:index, { :message => 'tweet 2' }, { :_id => 2, :_type => 'book', :_index => @index.name }],
-      [:index, { :message => 'tweet 3' }, { :_id => 3, :_type => 'book', :_index => @index.name }]
+      [:index, { :message => "tweet 1" }, { :_id => 1, :_type => "book", :_index => @index.name }],
+      [:index, { :message => "tweet 2" }, { :_id => 2, :_type => "book", :_index => @index.name }],
+      [:index, { :message => "tweet 3" }, { :_id => 3, :_type => "book", :_index => @index.name }]
     ]
     responses = $client.bulk_stream_responses(ops, { :action_count => 2 }).to_a
     assert_equal(2, responses.length)
@@ -307,11 +307,11 @@ describe Elastomer::Client::Bulk do
     assert_bulk_index(responses[1]["items"][0])
   end
 
-  it 'streams bulk items' do
+  it "streams bulk items" do
     ops = [
-      [:index, { :message => 'tweet 1' }, { :_id => 1, :_type => 'book', :_index => @index.name }],
-      [:index, { :message => 'tweet 2' }, { :_id => 2, :_type => 'book', :_index => @index.name }],
-      [:index, { :message => 'tweet 3' }, { :_id => 3, :_type => 'book', :_index => @index.name }]
+      [:index, { :message => "tweet 1" }, { :_id => 1, :_type => "book", :_index => @index.name }],
+      [:index, { :message => "tweet 2" }, { :_id => 2, :_type => "book", :_index => @index.name }],
+      [:index, { :message => "tweet 3" }, { :_id => 3, :_type => "book", :_index => @index.name }]
     ]
     items = []
     stats = $client.bulk_stream_items(ops, { :action_count => 2 }) { |item| items << item }