flying-sphinx 1.3.1 → 2.0.0

data/spec/flying_sphinx/commands/configure_spec.rb

@@ -0,0 +1,38 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Configure do
+  let(:subject) do
+    FlyingSphinx::Commands::Configure.new configuration_double, :api => api
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+
+    stub_const 'FlyingSphinx::Configurer', double(:call => "/foo")
+  end
+
+  it "sends through gzipped configuration archive" do
+    expect(api).to receive(:post).with "/perform",
+      :action => "configure",
+      :path   => "/foo"
+
+    subject.call
+  end
+
+  it "raises an exception if an invalid path error is returned" do
+    allow(api).to receive(:post).and_return('status' => 'INVALID PATH')
+
+    expect { subject.call }.to raise_error(FlyingSphinx::Error)
+  end
+
+  it "raises an exception if an unknown error is returned" do
+    allow(api).to receive(:post).and_return('status' => nil)
+
+    expect { subject.call }.to raise_error(FlyingSphinx::Error)
+  end
+end

data/spec/flying_sphinx/commands/index_sql_spec.rb

@@ -0,0 +1,69 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::IndexSQL do
+  let(:subject) do
+    FlyingSphinx::Commands::IndexSQL.new configuration_double, :api => api
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+  end
+
+  context "synchronous" do
+    it "sends the action" do
+      expect(api).to receive(:post).with "/perform",
+        :action  => "index",
+        :indices => ""
+
+      subject.call
+    end
+
+    it "waits for the action to finish" do
+      expect(action_class).to receive(:perform) do |identifier, &block|
+        block.call
+      end
+
+      subject.call
+    end
+  end
+
+  context "asynchronous" do
+    let(:subject) do
+      FlyingSphinx::Commands::IndexSQL.new(
+        double, :api => api, :indices => ["foo_delta"]
+      )
+    end
+
+    it "sends the action" do
+      expect(api).to receive(:post).with "/perform",
+        :action  => "index",
+        :indices => "foo_delta",
+        :unique  => "true"
+
+      subject.call
+    end
+
+    it "does not wait for the action to finish" do
+      expect(action_class).not_to receive(:perform)
+
+      subject.call
+    end
+
+    it "raises an exception if a blocked error is returned" do
+      allow(api).to receive(:post).and_return('status' => 'BLOCKED')
+
+      expect { subject.call }.to raise_error(FlyingSphinx::Error)
+    end
+
+    it "raises an exception if an unconfigured error is returned" do
+      allow(api).to receive(:post).and_return('status' => 'UNCONFIGURED')
+
+      expect { subject.call }.to raise_error(FlyingSphinx::Error)
+    end
+  end
+end

data/spec/flying_sphinx/commands/merge_spec.rb

@@ -0,0 +1,30 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Merge do
+  let(:subject) do
+    FlyingSphinx::Commands::Merge.new configuration_double, :api => api,
+      :core_index => core_index, :delta_index => delta_index,
+      :filters => {:deleted => 0}
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:core_index)   { double "index", :name => "core" }
+  let(:delta_index)  { double "index", :name => "delta" }
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+  end
+
+  it "sends through action" do
+    expect(api).to receive(:post).with "/perform",
+      :action      => "merge",
+      :core_index  => "core",
+      :delta_index => "delta",
+      :filters     => '{"deleted":0}'
+
+    subject.call
+  end
+end

data/spec/flying_sphinx/commands/rebuild_spec.rb

@@ -0,0 +1,26 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Rebuild do
+  let(:subject) do
+    FlyingSphinx::Commands::Rebuild.new configuration_double, :api => api
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+
+    stub_const 'FlyingSphinx::Configurer', double(:call => "/foo")
+  end
+
+  it "sends through gzipped configuration archive" do
+    expect(api).to receive(:post).with "/perform",
+      :action => "rebuild",
+      :path   => "/foo"
+
+    subject.call
+  end
+end

data/spec/flying_sphinx/commands/reset_spec.rb

@@ -0,0 +1,26 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Reset do
+  let(:subject) do
+    FlyingSphinx::Commands::Reset.new configuration_double, :api => api
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+
+    stub_const 'FlyingSphinx::Configurer', double(:call => "/foo")
+  end
+
+  it "sends through gzipped configuration archive" do
+    expect(api).to receive(:post).with "/perform",
+      :action => "reset",
+      :path   => "/foo"
+
+    subject.call
+  end
+end

data/spec/flying_sphinx/commands/restart_spec.rb

@@ -0,0 +1,23 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Restart do
+  let(:subject) do
+    FlyingSphinx::Commands::Restart.new configuration_double, :api => api
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+  end
+
+  it "sends through action" do
+    expect(api).to receive(:post).with "/perform",
+      :action => "restart"
+
+    subject.call
+  end
+end

data/spec/flying_sphinx/commands/rotate_spec.rb

@@ -0,0 +1,22 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Rotate do
+  let(:subject) do
+    FlyingSphinx::Commands::Rotate.new configuration_double, :api => api
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+  end
+
+  it "sends through action" do
+    expect(api).to receive(:post).with "/perform", :action => "rotate"
+
+    subject.call
+  end
+end

data/spec/flying_sphinx/commands/running_spec.rb

@@ -0,0 +1,24 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Running do
+  let(:subject) do
+    FlyingSphinx::Commands::Running.new configuration_double, :api => api
+  end
+  let(:api) { double 'API', :identifier => 'foo', :get => {"running" => true} }
+
+  it "sends through the API request" do
+    expect(api).to receive(:get).with("/running").and_return("running" => true)
+
+    subject.call
+  end
+
+  it "returns true when the API does" do
+    expect(subject.call).to eq(true)
+  end
+
+  it "returns false when the API does" do
+    allow(api).to receive(:get).with("/running").and_return("running" => false)
+
+    expect(subject.call).to eq(false)
+  end
+end

data/spec/flying_sphinx/commands/start_attached_spec.rb

@@ -0,0 +1,14 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::StartAttached do
+  let(:subject) do
+    FlyingSphinx::Commands::StartAttached.new(configuration_double, {}, stream)
+  end
+  let(:stream) { double 'Stream', :puts => nil }
+
+  it "prints a warning" do
+    expect(stream).to receive(:puts)
+
+    subject.call
+  end
+end

data/spec/flying_sphinx/commands/start_spec.rb

@@ -0,0 +1,23 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Start do
+  let(:subject) do
+    FlyingSphinx::Commands::Start.new configuration_double, :api => api
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+  end
+
+  it "sends through action" do
+    expect(api).to receive(:post).with "/perform",
+      :action => "start"
+
+    subject.call
+  end
+end

data/spec/flying_sphinx/commands/stop_spec.rb

@@ -0,0 +1,23 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Commands::Stop do
+  let(:subject) do
+    FlyingSphinx::Commands::Stop.new configuration_double, :api => api
+  end
+  let(:api) do
+    double 'API', :identifier => 'foo', :post => {'status' => 'OK'}
+  end
+  let(:action_class) { double }
+
+  before :each do
+    stub_const 'FlyingSphinx::Action', action_class
+    action_class.stub(:perform) { |identifier, &block| block.call }
+  end
+
+  it "sends through action" do
+    expect(api).to receive(:post).with "/perform",
+      :action => "stop"
+
+    subject.call
+  end
+end

data/spec/flying_sphinx/configurer_spec.rb

@@ -0,0 +1,147 @@
+require "spec_helper"
+
+RSpec.describe FlyingSphinx::Configurer do
+  let(:subject) { FlyingSphinx::Configurer.new api }
+  let(:api)     { double :get => presignature }
+  let(:presignature) { {
+    "status" => "OK",
+    "path"   => "a/path/of/my/own",
+    "url"    => "https://confserver",
+    "fields" => {"message" => "something"}
+  } }
+  let(:configuration_options) { double "conf options", :version => "2.2.3",
+    :raw => "indexer ...", :settings => settings, :engine => "manticore" }
+  let(:settings) { {
+    "extra"             => "wordforms/txt.txt",
+    "wordforms/txt.txt" => "something"
+  } }
+
+  before :each do
+    stub_request(:post, "https://confserver").to_return(:status => 200)
+
+    allow(FlyingSphinx::ConfigurationOptions).to receive(:new).
+      and_return(configuration_options)
+  end
+
+  it "requests a presignature path" do
+    expect(api).to receive(:get).with("/presignature").and_return(presignature)
+
+    subject.call
+  end
+
+  it "uploads a file to the presignature path" do
+    subject.call
+
+    expect(
+      a_multipart_request(:post, 'https://confserver')
+    ).to have_been_made
+  end
+
+  it "includes the configuration in the file" do
+    subject.call
+
+    expect(
+      a_multipart_request(:post, "https://confserver").
+      with_file { |contents|
+        reader = GZippedTar::Reader.new contents
+        reader.read("sphinx/raw.conf") == "indexer ..."
+      }
+    ).to have_been_made
+  end
+
+  it "includes the Sphinx version in the file" do
+    subject.call
+
+    expect(
+      a_multipart_request(:post, "https://confserver").
+      with_file { |contents|
+        reader = GZippedTar::Reader.new contents
+        reader.read("sphinx/version.txt") == "2.2.3"
+      }
+    ).to have_been_made
+  end
+
+  it "includes the Sphinx engine in the file" do
+    subject.call
+
+    expect(
+      a_multipart_request(:post, "https://confserver").
+      with_file { |contents|
+        reader = GZippedTar::Reader.new contents
+        reader.read("sphinx/engine.txt") == "manticore"
+      }
+    ).to have_been_made
+  end
+
+  it "includes the extra settings in the file" do
+    subject.call
+
+    expect(
+      a_multipart_request(:post, "https://confserver").
+      with_file { |contents|
+        reader = GZippedTar::Reader.new contents
+        reader.read("wordforms/txt.txt") == "something"
+      }
+    ).to have_been_made
+  end
+
+  it "includes the extra summary in the file" do
+    subject.call
+
+    expect(
+      a_multipart_request(:post, "https://confserver").
+      with_file { |contents|
+        reader = GZippedTar::Reader.new contents
+        reader.read("sphinx/extra.txt") == "wordforms/txt.txt"
+      }
+    ).to have_been_made
+  end
+
+  it "includes the provided fields in the request" do
+    subject.call
+
+    expect(
+      a_multipart_request(:post, "https://confserver").
+      with_params { |params| params["message"] == "something" }
+    ).to have_been_made
+  end
+
+  it "returns the presignature path" do
+    expect(subject.call).to eq("a/path/of/my/own")
+  end
+
+  context "presignature failure" do
+    before :each do
+      presignature["status"] = "failure"
+    end
+
+    it "raises a PresignatureError exception" do
+      expect { subject.call }.to raise_error(
+        FlyingSphinx::Configurer::PresignatureError
+      )
+    end
+
+    it "does not attempt to upload" do
+      begin
+        subject.call
+      rescue FlyingSphinx::Configurer::PresignatureError
+      end
+
+      expect(
+        a_multipart_request(:post, "https://confserver")
+      ).not_to have_been_made
+    end
+  end
+
+  context "upload failure" do
+    before :each do
+      stub_request(:post, "https://confserver").to_return(:status => 400)
+    end
+
+    it "raises an UploadError exception" do
+      expect { subject.call }.to raise_error(
+        FlyingSphinx::Configurer::UploadError
+      )
+    end
+  end
+end