elastomer-client 0.7.0 → 0.8.1

data/test/client/scroller_test.rb

@@ -1,29 +1,29 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 describe Elastomer::Client::Scroller do
 
   before do
-    @name  = 'elastomer-scroller-test'
+    @name  = "elastomer-scroller-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' },
-              :sorter  => { :type => 'integer' }
+              :message => { :type => "string", :analyzer => "standard" },
+              :author  => { :type => "string", :index => "not_analyzed" },
+              :sorter  => { :type => "integer" }
             }
           },
           :book => {
             :_source => { :enabled => true }, :_all => { :enabled => false },
             :properties => {
-              :title  => { :type => 'string', :analyzer => 'standard' },
-              :author => { :type => 'string', :index => 'not_analyzed' },
-              :sorter  => { :type => 'integer' }
+              :title  => { :type => "string", :analyzer => "standard" },
+              :author => { :type => "string", :index => "not_analyzed" },
+              :sorter  => { :type => "integer" }
             }
           }
         }
@@ -37,76 +37,76 @@ describe Elastomer::Client::Scroller do
     @index.delete if @index.exists?
   end
 
-  it 'scans over all documents in an index' do
+  it "scans over all documents in an index" do
     scan = @index.scan '{"query":{"match_all":{}}}', :size => 10
 
-    counts = {'tweet' => 0, 'book' => 0}
-    scan.each_document { |h| counts[h['_type']] += 1 }
+    counts = {"tweet" => 0, "book" => 0}
+    scan.each_document { |h| counts[h["_type"]] += 1 }
 
-    assert_equal 50, counts['tweet']
-    assert_equal 25, counts['book']
+    assert_equal 50, counts["tweet"]
+    assert_equal 25, counts["book"]
   end
 
-  it 'restricts the scan to a single document type' do
-    scan = @index.scan '{"query":{"match_all":{}}}', :type => 'book'
+  it "restricts the scan to a single document type" do
+    scan = @index.scan '{"query":{"match_all":{}}}', :type => "book"
 
-    counts = {'tweet' => 0, 'book' => 0}
-    scan.each_document { |h| counts[h['_type']] += 1 }
+    counts = {"tweet" => 0, "book" => 0}
+    scan.each_document { |h| counts[h["_type"]] += 1 }
 
-    assert_equal 0, counts['tweet']
-    assert_equal 25, counts['book']
+    assert_equal 0, counts["tweet"]
+    assert_equal 25, counts["book"]
   end
 
-  it 'limits results by query' do
+  it "limits results by query" do
     scan = @index.scan :query => { :bool => { :should => [
-      {:match => {:author => 'pea53'}},
-      {:match => {:title => '17'}}
+      {:match => {:author => "pea53"}},
+      {:match => {:title => "17"}}
     ]}}
 
-    counts = {'tweet' => 0, 'book' => 0}
-    scan.each_document { |h| counts[h['_type']] += 1 }
+    counts = {"tweet" => 0, "book" => 0}
+    scan.each_document { |h| counts[h["_type"]] += 1 }
 
-    assert_equal 50, counts['tweet']
-    assert_equal 1, counts['book']
+    assert_equal 50, counts["tweet"]
+    assert_equal 1, counts["book"]
   end
 
-  it 'scrolls and sorts over all documents' do
+  it "scrolls and sorts over all documents" do
     scroll = @index.scroll({
       :query => {:match_all => {}},
       :sort => {:sorter => {:order => :asc}}
-    }, :type => 'tweet')
+    }, :type => "tweet")
 
     tweets = []
-    scroll.each_document { |h| tweets << h['_id'].to_i }
+    scroll.each_document { |h| tweets << h["_id"].to_i }
 
     expected = (0...50).to_a.reverse
     assert_equal expected, tweets
   end
 
-  it 'propagates URL query strings' do
-    scan = @index.scan(nil, { :q => 'author:pea53 || title:17' })
+  it "propagates URL query strings" do
+    scan = @index.scan(nil, { :q => "author:pea53 || title:17" })
 
-    counts = {'tweet' => 0, 'book' => 0}
-    scan.each_document { |h| counts[h['_type']] += 1 }
+    counts = {"tweet" => 0, "book" => 0}
+    scan.each_document { |h| counts[h["_type"]] += 1 }
 
-    assert_equal 50, counts['tweet']
-    assert_equal 1, counts['book']
+    assert_equal 50, counts["tweet"]
+    assert_equal 1, counts["book"]
   end
 
   def populate!
     h = @index.bulk do |b|
       50.times { |num|
-        b.index %Q({"author":"pea53","message":"this is tweet number #{num}","sorter":#{50-num}}), :_id => num, :_type => 'tweet'
+        b.index %Q({"author":"pea53","message":"this is tweet number #{num}","sorter":#{50-num}}), :_id => num, :_type => "tweet"
       }
     end
-    h['items'].each {|item| assert_bulk_index(item) }
+    h["items"].each {|item| assert_bulk_index(item) }
 
     h = @index.bulk do |b|
       25.times { |num|
-        b.index %Q({"author":"Pratchett","title":"DiscWorld Book #{num}","sorter":#{25-num}}), :_id => num, :_type => 'book'
+        b.index %Q({"author":"Pratchett","title":"DiscWorld Book #{num}","sorter":#{25-num}}), :_id => num, :_type => "book"
       }
     end
-    h['items'].each {|item| assert_bulk_index(item) }
+    h["items"].each {|item| assert_bulk_index(item) }
 
     @index.refresh
   end

data/test/client/snapshot_test.rb

@@ -1,5 +1,5 @@
 
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 describe Elastomer::Client::Snapshot do
   if es_version_1_x?
@@ -8,16 +8,16 @@ describe Elastomer::Client::Snapshot do
         skip "To enable snapshot tests, add a path.repo setting to your elasticsearch.yml file."
       end
 
-      @index_name = 'elastomer-snapshot-test-index'
+      @index_name = "elastomer-snapshot-test-index"
       @index = $client.index(@index_name)
-      @name = 'elastomer-test'
+      @name = "elastomer-test"
     end
 
     after do
       @index.delete if @index && @index.exists?
     end
 
-    it 'determines if a snapshot exists' do
+    it "determines if a snapshot exists" do
       with_tmp_repo do |repo|
         snapshot = repo.snapshot(@name)
         assert_equal false, snapshot.exists?
@@ -27,14 +27,14 @@ describe Elastomer::Client::Snapshot do
       end
     end
 
-    it 'creates snapshots' do
+    it "creates snapshots" do
       with_tmp_repo do |repo|
         response = repo.snapshot(@name).create({}, :wait_for_completion => true)
         assert_equal @name, response["snapshot"]["snapshot"]
       end
     end
 
-    it 'creates snapshots with options' do
+    it "creates snapshots with options" do
       @index.create(:number_of_shards => 1, :number_of_replicas => 0)
       with_tmp_repo do |repo|
         response = repo.snapshot(@name).create({:indices => [@index_name]}, :wait_for_completion => true)
@@ -43,7 +43,7 @@ describe Elastomer::Client::Snapshot do
       end
     end
 
-    it 'gets snapshot info for one and all' do
+    it "gets snapshot info for one and all" do
       with_tmp_snapshot do |snapshot, repo|
         response = snapshot.get
         assert_equal snapshot.name, response["snapshots"][0]["snapshot"]
@@ -52,7 +52,7 @@ describe Elastomer::Client::Snapshot do
       end
     end
 
-    it 'gets snapshot status for one and all' do
+    it "gets snapshot status for one and all" do
       @index.create(:number_of_shards => 1, :number_of_replicas => 0)
       with_tmp_repo do |repo|
         repo.snapshot(@name).create({:indices => [@index_name]}, :wait_for_completion => true)
@@ -61,7 +61,7 @@ describe Elastomer::Client::Snapshot do
       end
     end
 
-    it 'gets status of snapshots in progress' do
+    it "gets status of snapshots in progress" do
       # we can't reliably get status of an in-progress snapshot in tests, so
       # check for an empty result instead
       with_tmp_repo do |repo|
@@ -72,19 +72,19 @@ describe Elastomer::Client::Snapshot do
       end
     end
 
-    it 'disallows nil repo name with non-nil snapshot name' do
-      assert_raises(ArgumentError) { $client.repository.snapshot('snapshot') }
-      assert_raises(ArgumentError) { $client.snapshot(nil, 'snapshot') }
+    it "disallows nil repo name with non-nil snapshot name" do
+      assert_raises(ArgumentError) { $client.repository.snapshot("snapshot") }
+      assert_raises(ArgumentError) { $client.snapshot(nil, "snapshot") }
     end
 
-    it 'deletes snapshots' do
+    it "deletes snapshots" do
       with_tmp_snapshot do |snapshot|
         response = snapshot.delete
         assert_equal true, response["acknowledged"]
       end
     end
 
-    it 'restores snapshots' do
+    it "restores snapshots" do
       @index.create(:number_of_shards => 1, :number_of_replicas => 0)
       wait_for_index(@index_name)
       with_tmp_repo do |repo|
@@ -106,7 +106,7 @@ describe Elastomer::Client::Snapshot do
         @restored_index.delete if @restored_index && @restored_index.exists?
       end
 
-      it 'restores snapshots with options' do
+      it "restores snapshots with options" do
         @index.create(:number_of_shards => 1, :number_of_replicas => 0)
         wait_for_index(@index_name)
         with_tmp_repo do |repo|

data/test/client/stubbed_client_test.rb

@@ -1,40 +1,40 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
-describe 'stubbed client tests' do
+describe "stubbed client tests" do
   before do
     @stubs  = Faraday::Adapter.lookup_middleware(:test)::Stubs.new
     @client = Elastomer::Client.new :adapter => [:test, @stubs]
   end
 
   describe Elastomer::Client::Cluster do
-    it 'reroutes shards' do
-      @stubs.post '/_cluster/reroute?dry_run=true' do |env|
+    it "reroutes shards" do
+      @stubs.post "/_cluster/reroute?dry_run=true" do |env|
         assert_match %r/^\{"commands":\[\{"move":\{[^\{\}]+\}\}\]\}$/, env[:body]
-        [200, {'Content-Type' => 'application/json'}, '{"acknowledged" : true}']
+        [200, {"Content-Type" => "application/json"}, '{"acknowledged" : true}']
       end
 
-      commands = { :move => { :index => 'test', :shard => 0, :from_node => 'node1', :to_node => 'node2' }}
+      commands = { :move => { :index => "test", :shard => 0, :from_node => "node1", :to_node => "node2" }}
       h = @client.cluster.reroute commands, :dry_run => true
       assert_acknowledged h
     end
 
-    it 'performs a shutdown of the cluster' do
-      @stubs.post('/_shutdown') { [200, {'Content-Type' => 'application/json'}, '{"cluster_name":"elasticsearch"}'] }
+    it "performs a shutdown of the cluster" do
+      @stubs.post("/_shutdown") { [200, {"Content-Type" => "application/json"}, '{"cluster_name":"elasticsearch"}'] }
       h = @client.cluster.shutdown
-      assert_equal "elasticsearch", h['cluster_name']
+      assert_equal "elasticsearch", h["cluster_name"]
     end
   end
 
   describe Elastomer::Client::Nodes do
-    it 'performs a shutdown of the node(s)' do
-      @stubs.post('/_cluster/nodes/_all/_shutdown')  { [200, {'Content-Type' => 'application/json'}, '{"nodes":{"1":{"name":"Node1"}}}'] }
-      @stubs.post('/_cluster/nodes/node2/_shutdown') { [200, {'Content-Type' => 'application/json'}, '{"nodes":{"2":{"name":"Node2"}}}'] }
+    it "performs a shutdown of the node(s)" do
+      @stubs.post("/_cluster/nodes/_all/_shutdown")  { [200, {"Content-Type" => "application/json"}, '{"nodes":{"1":{"name":"Node1"}}}'] }
+      @stubs.post("/_cluster/nodes/node2/_shutdown") { [200, {"Content-Type" => "application/json"}, '{"nodes":{"2":{"name":"Node2"}}}'] }
 
       h = @client.nodes("_all").shutdown
-      assert_equal "Node1", h['nodes']['1']['name']
+      assert_equal "Node1", h["nodes"]["1"]["name"]
 
-      h = @client.nodes('node2').shutdown
-      assert_equal 'Node2', h['nodes']['2']['name']
+      h = @client.nodes("node2").shutdown
+      assert_equal "Node2", h["nodes"]["2"]["name"]
     end
   end
 end

data/test/client/template_test.rb

@@ -1,9 +1,9 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 describe Elastomer::Client::Cluster do
 
   before do
-    @name = 'elastomer-template-test'
+    @name = "elastomer-template-test"
     @template = $client.template @name
   end
 
@@ -11,29 +11,29 @@ describe Elastomer::Client::Cluster do
     @template.delete if @template.exists?
   end
 
-  it 'lists templates in the cluster' do
-    @template.create({:template => 'test-elastomer*'})
+  it "lists templates in the cluster" do
+    @template.create({:template => "test-elastomer*"})
     templates = $client.cluster.templates
     assert !templates.empty?, "expected to see a template"
   end
 
-  it 'creates a template' do
-    assert !@template.exists?, 'the template should not exist'
+  it "creates a template" do
+    assert !@template.exists?, "the template should not exist"
 
     @template.create({
-      :template => 'test-elastomer*',
+      :template => "test-elastomer*",
       :settings => { :number_of_shards => 3 },
       :mappings => {
         :doco => { :_source => { :enabled => false }}
       }
     })
 
-    assert @template.exists?, ' we now have a cluster-test template'
+    assert @template.exists?, " we now have a cluster-test template"
 
     template = @template.get
     assert_equal [@name], template.keys
-    assert_equal 'test-elastomer*', template[@name]['template']
-    assert_equal '3', template[@name]['settings']['index.number_of_shards']
+    assert_equal "test-elastomer*", template[@name]["template"]
+    assert_equal "3", template[@name]["settings"]["index.number_of_shards"]
   end
 
 end

data/test/client/warmer_test.rb

@@ -1,19 +1,19 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 describe Elastomer::Client::Warmer do
   before do
-    @name  = 'elastomer-warmer-test'
+    @name  = "elastomer-warmer-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" }
             }
           }
         }
@@ -26,31 +26,31 @@ describe Elastomer::Client::Warmer do
     @index.delete if @index.exists?
   end
 
-  it 'creates warmers' do
-    h = @index.warmer('test1').create(:query => { :match_all => {}})
+  it "creates warmers" do
+    h = @index.warmer("test1").create(:query => { :match_all => {}})
     assert_acknowledged h
   end
 
-  it 'deletes warmers' do
-    @index.warmer('test1').create(:query => { :match_all => {}})
+  it "deletes warmers" do
+    @index.warmer("test1").create(:query => { :match_all => {}})
 
-    h = @index.warmer('test1').delete
+    h = @index.warmer("test1").delete
     assert_acknowledged h
   end
 
-  it 'gets warmers' do
+  it "gets warmers" do
     body = { "query" => {"match_all" => {}}}
-    @index.warmer('test1').create(body)
+    @index.warmer("test1").create(body)
 
-    h = @index.warmer('test1').get
+    h = @index.warmer("test1").get
     assert_equal body, h[@name]["warmers"]["test1"]["source"]
   end
 
-  it 'knows when warmers exist' do
-    assert_equal false, @index.warmer('test1').exists?
-    assert_equal false, @index.warmer('test1').exist?
+  it "knows when warmers exist" do
+    assert_equal false, @index.warmer("test1").exists?
+    assert_equal false, @index.warmer("test1").exist?
 
-    h = @index.warmer('test1').create(:query => { :match_all => {}})
-    assert_equal true, @index.warmer('test1').exists?
+    h = @index.warmer("test1").create(:query => { :match_all => {}})
+    assert_equal true, @index.warmer("test1").exists?
   end
 end

data/test/client_test.rb

@@ -1,8 +1,8 @@
-require File.expand_path('../test_helper', __FILE__)
+require File.expand_path("../test_helper", __FILE__)
 
 describe Elastomer::Client do
 
-  it 'uses the adapter specified at creation' do
+  it "uses the adapter specified at creation" do
     c = Elastomer::Client.new(:adapter => :test)
     assert_includes c.connection.builder.handlers, Faraday::Adapter::Test
   end
@@ -13,50 +13,50 @@ describe Elastomer::Client do
     assert_includes c.connection.builder.handlers, adapter
   end
 
-  it 'uses the same connection for all requests' do
+  it "uses the same connection for all requests" do
     c = $client.connection
     assert_same c, $client.connection
   end
 
-  it 'raises an error for unknown HTTP request methods' do
-    assert_raises(ArgumentError) { $client.request :foo, '/', {} }
+  it "raises an error for unknown HTTP request methods" do
+    assert_raises(ArgumentError) { $client.request :foo, "/", {} }
   end
 
-  it 'raises an error on 4XX responses with an `error` field' do
+  it "raises an error on 4XX responses with an `error` field" do
     begin
-      $client.get '/non-existent-index/_search?q=*:*'
-      assert false, 'exception was not raised when it should have been'
+      $client.get "/non-existent-index/_search?q=*:*"
+      assert false, "exception was not raised when it should have been"
     rescue Elastomer::Client::Error => err
       assert_equal 404, err.status
-      assert_equal 'IndexMissingException[[non-existent-index] missing]', err.message
+      assert_equal "IndexMissingException[[non-existent-index] missing]", err.message
     end
   end
 
-  it 'handles path expansions' do
-    uri = $client.expand_path '/{foo}/{bar}', :foo => '_cluster', :bar => 'health'
-    assert_equal '/_cluster/health', uri
+  it "handles path expansions" do
+    uri = $client.expand_path "/{foo}/{bar}", :foo => "_cluster", :bar => "health"
+    assert_equal "/_cluster/health", uri
 
-    uri = $client.expand_path '{/foo}{/baz}{/bar}', :foo => '_cluster', :bar => 'state'
-    assert_equal '/_cluster/state', uri
+    uri = $client.expand_path "{/foo}{/baz}{/bar}", :foo => "_cluster", :bar => "state"
+    assert_equal "/_cluster/state", uri
   end
 
-  it 'handles query parameters' do
-    uri = $client.expand_path '/_cluster/health', :level => 'shards'
-    assert_equal '/_cluster/health?level=shards', uri
+  it "handles query parameters" do
+    uri = $client.expand_path "/_cluster/health", :level => "shards"
+    assert_equal "/_cluster/health?level=shards", uri
   end
 
-  it 'validates path expansions' do
+  it "validates path expansions" do
     assert_raises(ArgumentError) {
-      $client.expand_path '/{foo}/{bar}', :foo => '_cluster', :bar => nil
+      $client.expand_path "/{foo}/{bar}", :foo => "_cluster", :bar => nil
     }
 
     assert_raises(ArgumentError) {
-      $client.expand_path '/{foo}/{bar}', :foo => '_cluster', :bar => ''
+      $client.expand_path "/{foo}/{bar}", :foo => "_cluster", :bar => ""
     }
   end
 
-  describe 'when extracting and converting :body params' do
-    it 'deletes the :body from the params (or it gets the hose)' do
+  describe "when extracting and converting :body params" do
+    it "deletes the :body from the params (or it gets the hose)" do
       params = { :body => nil, :q => "what what?" }
       body = $client.extract_body params
 
@@ -64,27 +64,27 @@ describe Elastomer::Client do
       assert_equal({:q => "what what?"}, params)
     end
 
-    it 'leaves String values unchanged' do
+    it "leaves String values unchanged" do
       body = $client.extract_body :body => '{"query":{"match_all":{}}}'
       assert_equal '{"query":{"match_all":{}}}', body
 
-      body = $client.extract_body :body => 'not a JSON string, but who cares!'
-      assert_equal 'not a JSON string, but who cares!', body
+      body = $client.extract_body :body => "not a JSON string, but who cares!"
+      assert_equal "not a JSON string, but who cares!", body
     end
 
-    it 'joins Array values' do
+    it "joins Array values" do
       body = $client.extract_body :body => %w[foo bar baz]
       assert_equal "foo\nbar\nbaz\n", body
 
       body = $client.extract_body :body => [
-        'the first entry',
-        'the second entry',
+        "the first entry",
+        "the second entry",
         nil
       ]
       assert_equal "the first entry\nthe second entry\n", body
     end
 
-    it 'converts values to JSON' do
+    it "converts values to JSON" do
       body = $client.extract_body :body => true
       assert_equal "true", body
 
@@ -93,61 +93,61 @@ describe Elastomer::Client do
     end
   end
 
-  describe 'when validating parameters' do
-    it 'rejects nil values' do
+  describe "when validating parameters" do
+    it "rejects nil values" do
       assert_raises(ArgumentError) { $client.assert_param_presence nil }
     end
 
-    it 'rejects empty strings' do
+    it "rejects empty strings" do
       assert_raises(ArgumentError) { $client.assert_param_presence "" }
       assert_raises(ArgumentError) { $client.assert_param_presence " " }
       assert_raises(ArgumentError) { $client.assert_param_presence " \t \r \n " }
     end
 
-    it 'rejects empty strings and nil values found in arrays' do
-      assert_raises(ArgumentError) { $client.assert_param_presence ['foo', nil, 'bar'] }
-      assert_raises(ArgumentError) { $client.assert_param_presence ['baz', " \t \r \n "] }
+    it "rejects empty strings and nil values found in arrays" do
+      assert_raises(ArgumentError) { $client.assert_param_presence ["foo", nil, "bar"] }
+      assert_raises(ArgumentError) { $client.assert_param_presence ["baz", " \t \r \n "] }
     end
 
-    it 'strips whitespace from strings' do
-      assert_equal 'foo', $client.assert_param_presence("  foo  \t")
+    it "strips whitespace from strings" do
+      assert_equal "foo", $client.assert_param_presence("  foo  \t")
     end
 
-    it 'joins array values into a string' do
-      assert_equal 'foo,bar', $client.assert_param_presence(%w[foo bar])
+    it "joins array values into a string" do
+      assert_equal "foo,bar", $client.assert_param_presence(%w[foo bar])
     end
 
-    it 'flattens arrays' do
-      assert_equal 'foo,bar,baz,buz', $client.assert_param_presence(["  foo  \t", %w[bar baz buz]])
+    it "flattens arrays" do
+      assert_equal "foo,bar,baz,buz", $client.assert_param_presence(["  foo  \t", %w[bar baz buz]])
     end
 
-    it 'allows strings' do
-      assert_equal 'foo', $client.assert_param_presence("foo")
+    it "allows strings" do
+      assert_equal "foo", $client.assert_param_presence("foo")
     end
 
-    it 'converts numbers and symbols to strings' do
-      assert_equal 'foo', $client.assert_param_presence(:foo)
-      assert_equal '9', $client.assert_param_presence(9)
+    it "converts numbers and symbols to strings" do
+      assert_equal "foo", $client.assert_param_presence(:foo)
+      assert_equal "9", $client.assert_param_presence(9)
     end
   end
 
-  describe 'top level actions' do
-    it 'pings the cluster' do
+  describe "top level actions" do
+    it "pings the cluster" do
       assert_equal true, $client.ping
       assert_equal true, $client.available?
     end
 
-    it 'gets cluster info' do
+    it "gets cluster info" do
       h = $client.info
-      assert h.key?('name'), 'expected cluster name to be returned'
-      assert h.key?('status'), 'expected cluster info status to be returned'
+      assert h.key?("name"), "expected cluster name to be returned"
+      assert h.key?("status"), "expected cluster info status to be returned"
     end
 
-    it 'gets cluster version' do
+    it "gets cluster version" do
       assert_match /[\d\.]+/, $client.version
     end
 
-    it 'gets semantic version' do
+    it "gets semantic version" do
       version_string = $client.version
       assert_equal Semantic::Version.new(version_string), $client.semantic_version
     end