esearch 0.2.1 → 0.2.2

data/spec/unit/esearch/command/document/index/update/run_spec.rb

@@ -1,3 +1,7 @@
+# # encoding: UTF-8
+
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Command::Document::Index::Update, '.run' do
@@ -5,10 +9,10 @@ describe Esearch::Command::Document::Index::Update, '.run' do
 
   let(:object)   { described_class    }
   let(:document) { double('Document') }
-  let(:options)  { { :foo => :bar }   }
+  let(:options)  { { foo: :bar }   }
 
   let(:expected_request) do
-    Esearch::Request.new(:post, '/foo', document, { :foo => :bar, :op_type => :index })
+    Esearch::Request.new(:post, '/foo', document, foo: :bar, op_type: :index)
   end
 
   expect_presenter(Esearch::Presenter::Document::Operation::Index)

data/spec/unit/esearch/command/exist/result_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Command::Exist, '.run' do
@@ -13,11 +15,11 @@ describe Esearch::Command::Exist, '.run' do
     connection.should_receive(:run).with(expected_request).and_return(response)
   end
 
-  let(:context)    { double('Context', :connection => connection, :path => Pathname.new('/foo')) }
+  let(:context)    { double('Context', connection: connection, path: Pathname.new('/foo')) }
   let(:connection) { double('Connection')                                                        }
 
-  let(:headers)    { {'content-type' => 'application/json; charset=UTF-8'} }
-  let(:response)   { double('Response', :frozen? => true, :status => status, :headers => headers, :body => '{}') }
+  let(:headers)    { { 'content-type' => 'application/json; charset=UTF-8' } }
+  let(:response)   { double('Response', :frozen? => true, status: status, headers: headers, body: '{}') }
 
   context 'with status 200' do
     let(:status)     { 200 }

data/spec/unit/esearch/command/index/create/run_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Command::Index::Create, '.run' do

data/spec/unit/esearch/command/index/delete/run_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Command::Index::Delete, '.run' do
@@ -8,6 +10,6 @@ describe Esearch::Command::Index::Delete, '.run' do
   let(:expected_request) do
     Esearch::Request.new(:delete, '/foo', {}, {})
   end
- 
+
   expect_presenter(Esearch::Presenter::Index::Delete)
 end

data/spec/unit/esearch/command/index/refresh/run_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Command::Index::Refresh, '.run' do
@@ -8,6 +10,6 @@ describe Esearch::Command::Index::Refresh, '.run' do
   let(:expected_request) do
     Esearch::Request.new(:post, '/foo/_refresh', {}, {})
   end
- 
+
   expect_presenter(Esearch::Presenter::Index::Refresh)
 end

data/spec/unit/esearch/command/result_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Command, '#result' do
@@ -18,13 +20,12 @@ describe Esearch::Command, '#result' do
     end
   end
 
-
-  let(:context)    { double('Context', :connection => connection, :path => Pathname.new('/foo')) }
+  let(:context)    { double('Context', connection: connection, path: Pathname.new('/foo')) }
   let(:connection) { double('Connection')                                                        }
 
-  let(:headers)    { {'content-type' => content_type } }
+  let(:headers)    { { 'content-type' => content_type } }
   let(:content_type) { 'application/json; charset=UTF-8' }
-  let(:response)   { double('Response', :frozen? => true, :status => status, :headers => headers, :body => body) }
+  let(:response)   { double('Response', :frozen? => true, status: status, headers: headers, body: body) }
   let(:body)       { '{}' }
 
   let(:status)     { 200 }
@@ -43,7 +44,10 @@ describe Esearch::Command, '#result' do
     let(:content_type) { 'text/plain' }
 
     it 'should raise error' do
-      expect { subject }.to raise_error(Esearch::ProtocolError, 'Expected json content type, but got: "text/plain"')
+      expect { subject }.to raise_error(
+        Esearch::ProtocolError,
+        'Expected json content type, but got: "text/plain"'
+      )
     end
   end
 
@@ -52,7 +56,10 @@ describe Esearch::Command, '#result' do
 
     context 'with json body' do
       it 'should raise error' do
-        expect { subject }.to raise_error(Esearch::ProtocolError, 'expected response stati: [200] but got: 201, remote message: {}')
+        expect { subject }.to raise_error(
+          Esearch::ProtocolError,
+          'expected response stati: [200] but got: 201, remote message: {}'
+        )
       end
     end
 
@@ -61,7 +68,10 @@ describe Esearch::Command, '#result' do
       let(:body)         { 'message'   }
 
       it 'should raise error' do
-        expect { subject }.to raise_error(Esearch::ProtocolError, 'expected response stati: [200] but got: 201, remote message: "message"')
+        expect { subject }.to raise_error(
+          Esearch::ProtocolError,
+          'expected response stati: [200] but got: 201, remote message: "message"'
+        )
       end
     end
   end

data/spec/unit/esearch/command/search/run_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Command::Search, '.run' do

data/spec/unit/esearch/command/status/run_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Command::Status, '.run' do
@@ -8,6 +10,6 @@ describe Esearch::Command::Status, '.run' do
   let(:expected_request) do
     Esearch::Request.new(:get, '/foo/_status', {}, {})
   end
- 
+
   expect_presenter(Esearch::Presenter::Status)
 end

data/spec/unit/esearch/connection/class_methods/build_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Connection, '.build' do

data/spec/unit/esearch/connection/run_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Connection, '#run' do
@@ -5,10 +7,10 @@ describe Esearch::Connection, '#run' do
 
   let(:object) { described_class.new(connection, logger) }
 
-  let(:connection) { double('Connection')                                  }
-  let(:logger)     { DummyLogger.new                                       }
-  let(:request)    { double('Request', :log_string => 'the-logged-string') }
-  let(:response)   { double('Response', :status => 200)                    }
+  let(:connection) { double('Connection')                               }
+  let(:logger)     { DummyLogger.new                                    }
+  let(:request)    { double('Request', log_string: 'the-logged-string') }
+  let(:response)   { double('Response', status: 200)                    }
 
   class DummyLogger
     attr_reader :messages

data/spec/unit/esearch/document_spec.rb

@@ -0,0 +1,44 @@
+# encoding: UTF-8
+
+require 'spec_helper'
+
+describe Esearch::Document do
+  let(:object)     { described_class.new(type, id)          }
+  let(:type)       { double('Type', connection: connection) }
+  let(:connection) { double('Connection')                   }
+  let(:id)         { '1'                                    }
+
+  describe '#connection' do
+    subject { object.connection }
+
+    it { should be(connection) }
+  end
+
+  describe '#delete' do
+    subject { object.delete }
+
+    let(:expected_arguments) { [] }
+
+    expect_to_run_command(Esearch::Command::Document::Delete)
+  end
+
+  describe '#get' do
+    subject { object.get }
+
+    let(:expected_arguments) { [] }
+
+    expect_to_run_command(Esearch::Command::Document::Get)
+  end
+
+  describe '#path' do
+    subject { object.path }
+
+    let(:type) { double('Type', connection: connection, path: Pathname.new('a-index/b-type')) }
+
+    let(:id)   { 'c-id' }
+
+    it { should eql(Pathname.new('a-index/b-type/c-id')) }
+
+    it_should_behave_like 'an idempotent method'
+  end
+end

data/spec/unit/esearch/index_spec.rb

@@ -0,0 +1,42 @@
+# encoding: UTF-8
+
+require 'spec_helper'
+
+describe Esearch::Index do
+  let(:object)     { described_class.new(connection, index_name) }
+  let(:connection) { double('Connection')                        }
+  let(:index_name) { 'foo'                                       }
+
+  describe '#create' do
+    subject { object.create(settings) }
+
+    let(:settings)           { { my: :settings } }
+    let(:expected_arguments) { [settings]        }
+
+    expect_to_run_command(Esearch::Command::Index::Create)
+  end
+
+  describe '#delete' do
+    subject { object.delete }
+
+    let(:expected_arguments) { [] }
+
+    expect_to_run_command(Esearch::Command::Index::Delete)
+  end
+
+  describe '#path' do
+    subject { object.path }
+
+    it { should eql(Pathname.new('/foo')) }
+
+    it_should_behave_like 'an idempotent method'
+  end
+
+  describe '#type' do
+    subject { object.type(name) }
+
+    let(:name) { 'bar' }
+
+    it { should eql(Esearch::Type.new(object, name)) }
+  end
+end

data/spec/unit/esearch/indices/all_spec.rb

@@ -0,0 +1,16 @@
+# encoding: UTF-8
+
+require 'spec_helper'
+
+describe Esearch::Indices::All do
+  let(:object)     { described_class.new(connection) }
+  let(:connection) { double('Connection')            }
+
+  describe '#path' do
+    subject { object.path }
+
+    it { should eql(Pathname.new('/_all')) }
+
+    it_should_behave_like 'an idempotent method'
+  end
+end

data/spec/unit/esearch/indices_spec.rb

@@ -0,0 +1,18 @@
+# encoding: UTF-8
+
+require 'spec_helper'
+
+describe Esearch::Indices do
+  let(:object)     { described_class.new(connection, names) }
+  let(:connection) { double('Connection')                   }
+
+  let(:names) { %w(foo bar) }
+
+  describe '#path' do
+    subject { object.path }
+
+    it { should eql(Pathname.new('/foo,bar')) }
+
+    it_should_behave_like 'an idempotent method'
+  end
+end

data/spec/unit/esearch/mixin/document/index_create_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Mixin::Document, '#index_create' do

data/spec/unit/esearch/mixin/document/index_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Mixin::Document, '#index' do

data/spec/unit/esearch/mixin/document/index_update_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Mixin::Document, '#index_update' do

data/spec/unit/esearch/mixin/exist/exist_predicate_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Mixin::Exist, '#exist?' do

data/spec/unit/esearch/mixin/index/refresh_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Mixin::Index, '#refresh' do

data/spec/unit/esearch/mixin/index/status_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Mixin::Index, '#status' do

data/spec/unit/esearch/mixin/search/search_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Mixin::Search, '#search' do

data/spec/unit/esearch/presenter/aspect/range/from_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Presenter::Aspect::Range, '#from' do

data/spec/unit/esearch/presenter/aspect/range/to_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Presenter::Aspect::Range, '#to' do

data/spec/unit/esearch/presenter/class_methods/new_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Presenter, '.new' do

data/spec/unit/esearch/presenter/facet_spec.rb

@@ -0,0 +1,60 @@
+# encoding: UTF-8
+
+require 'spec_helper'
+
+describe Esearch::Presenter::Facet do
+  describe '.build' do
+    subject { object.build(raw) }
+
+    let(:object) { described_class }
+
+    context 'when type is "terms"' do
+      let(:raw) { { '_type' => 'terms' } }
+      it { should eql(Esearch::Presenter::Facet::Terms.new(raw)) }
+    end
+
+    context 'when type is "range"' do
+      let(:raw) { { '_type' => 'range' } }
+      it { should eql(Esearch::Presenter::Facet::Range.new(raw)) }
+    end
+
+    context 'with unknown type' do
+      let(:raw) { { '_type' => 'foo' } }
+
+      it 'should raise error' do
+        expect { subject }.to raise_error('Facet with type "foo" is not known')
+      end
+    end
+
+  end
+
+  describe '#aspects' do
+    subject { object.aspects }
+
+    class DummyAspect
+      include Concord.new(:value)
+    end
+
+    class DummyFacet < Esearch::Presenter::Facet
+      ASPECT_CLASS = DummyAspect
+      FACET_KEY = 'foo'.freeze
+    end
+
+    let(:object) do
+      DummyFacet.new(raw)
+    end
+
+    let(:raw) do
+      {
+        'foo' => [
+          'a',
+          'b'
+        ]
+      }
+    end
+
+    it { should eql([ DummyAspect.new('a'), DummyAspect.new('b') ]) }
+
+    it_should_behave_like 'an idempotent method'
+  end
+end

data/spec/unit/esearch/presenter/hit/fields_spec.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe Esearch::Presenter::Hit, '#fields' do