dispatch-rider 1.4.0 → 1.4.2

data/spec/lib/dispatch-rider/publisher/configuration/destination_spec.rb

@@ -0,0 +1,100 @@
+require 'spec_helper'
+require 'rspec/its'
+
+describe DispatchRider::Publisher::Configuration::Destination do
+
+  let(:attributes) do
+    {
+      "service" => "aws_sns",
+      "channel" => "employee_updates",
+      "options" => options
+    }
+  end
+
+  let(:options) do
+    {
+      "account" => "123456789",
+      "region" => "us-east",
+      "topic" => "employee-updates"
+    }
+  end
+
+  subject{ described_class.new("employee", attributes) }
+
+  describe "#name" do
+    its(:name) { is_expected.to eq("employee") }
+  end
+
+  describe "#service" do
+    its(:service) { is_expected.to eq("aws_sns") }
+  end
+
+  describe "#channel" do
+    its(:channel) { is_expected.to eq("employee_updates") }
+  end
+
+  describe "#options" do
+    its(:options) { is_expected.to eq(options) }
+  end
+
+  describe "#==" do
+    let(:other){ described_class.new(name, other_attributes) }
+
+    context "when the destinations' name, service, channel and options are the same" do
+      let(:name){ subject.name }
+      let(:other_attributes){ attributes }
+
+      it{ should eq other }
+    end
+
+    context "when the destinations' name is different" do
+      let(:name){ "account" }
+      let(:other_attributes){ attributes }
+
+      it{ should_not eq other }
+    end
+
+    context "when the destinations' service is different" do
+      let(:name){ subject.name }
+
+      let(:other_attributes) do
+        {
+          "service" => "file_system",
+          "channel" => "employee_updates",
+          "options" => options
+        }
+      end
+
+      it{ should_not eq other }
+    end
+
+    context "when the destinations' channel is different" do
+      let(:name){ subject.name }
+
+      let(:other_attributes) do
+        {
+          "service" => "aws_sns",
+          "channel" => "account_updates",
+          "options" => options
+        }
+      end
+
+      it{ should_not eq other }
+    end
+
+    context "when the destinations' options are different" do
+      let(:name){ subject.name }
+
+      let(:other_attributes) do
+        {
+          "service" => "aws_sns",
+          "channel" => "employee_updates",
+          "options" => {}
+        }
+      end
+
+      it{ should_not eq other }
+    end
+  end
+
+end

data/spec/lib/dispatch-rider/publisher/configuration/notification_service_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+describe DispatchRider::Publisher::Configuration::NotificationService do
+
+  let(:options) do
+    {
+      "default_folder" => "/tmp/dispatch_rider"
+    }
+  end
+
+  subject{ described_class.new("file_system", options) }
+
+  describe "#name" do
+    its(:name){ should == "file_system" }
+  end
+
+  describe "#options" do
+    its(:options){ should == options }
+  end
+
+  describe "#==" do
+    let(:other){ described_class.new(name, other_options) }
+
+    context "two notification services with the same name and options" do
+      let(:name){ subject.name }
+      let(:other_options){ options }
+
+      it{ should eq other }
+    end
+
+    context "two notification services with different names but the same options" do
+      let(:name){ "aws_sns" }
+      let(:other_options){ options }
+
+      it{ should_not eq other }
+    end
+
+    context "two notificaiton services with the same name but different options" do
+      let(:name){ subject.name }
+      let(:other_options){ { "topic" => "employee_updates" } }
+
+      it{ should_not eq other }
+    end
+
+    context "two notification services with different names and options" do
+      let(:name){ "aws_sns" }
+      let(:other_options){ { "topic" => "employee_updates" } }
+
+      it{ should_not eq other }
+    end
+  end
+
+end

data/spec/lib/dispatch-rider/publisher/configuration_reader_spec.rb

@@ -0,0 +1,129 @@
+require 'spec_helper'
+
+describe DispatchRider::Publisher::ConfigurationReader do
+
+  let :publisher do
+    double(:publisher)
+  end
+
+  describe ".load_config" do
+
+    subject { described_class }
+
+    it "responds to :load_config" do
+      subject.should respond_to :load_config
+    end
+
+    it "requires 2 paramaters" do
+      expect {
+        subject.load_config
+      }.to raise_exception(ArgumentError, /0 for 2/)
+    end
+
+    it "deals with an empty configuration hash" do
+      expect {
+        subject.load_config(DispatchRider::Publisher::Configuration.new, publisher)
+      }.to_not raise_exception
+    end
+
+    describe "notification_services parsing" do
+
+      let(:configuration){ DispatchRider::Publisher::Configuration.new(configuration_hash) }
+
+      context "when notification_services has no items in it" do
+
+        let :configuration_hash do
+          {
+            notification_services: {
+            }
+          }
+        end
+
+        it "doesn't call register_notification_service" do
+          publisher.should_not_receive(:register_notification_service)
+          subject.load_config(configuration, publisher)
+        end
+      end
+
+      context "when notification_services has an item in it" do
+        let :configuration_hash do
+          {
+            notification_services: {
+              file_system: {}
+            }
+          }
+        end
+
+        it "calls register_notification_service with :file_system and {}" do
+          publisher.should_receive(:register_notification_service).with("file_system", {})
+          subject.load_config(configuration, publisher)
+        end
+      end
+
+      context "when notification_services has 2 items in it" do
+        let :configuration_hash do
+          {
+            notification_services: {
+              file_system: {},
+              foo: {bar: "123"},
+            }
+          }
+        end
+
+        it "calls register_notification_service with :file_system and {}, as well as :foo, {bar: '123'}" do
+          publisher.should_receive(:register_notification_service).with("file_system", {})
+          publisher.should_receive(:register_notification_service).with("foo", {"bar" => "123"})
+          subject.load_config(configuration, publisher)
+        end
+      end
+
+    end
+
+    describe "destinations" do
+
+      let(:configuration){ DispatchRider::Publisher::Configuration.new(configuration_hash) }
+
+      context "when destinations has no items in it" do
+
+        let :configuration_hash do
+          {
+            destinations: {
+            }
+          }
+        end
+
+        it "doesn't call register_destination" do
+          publisher.should_not_receive(:register_destination)
+          subject.load_config(configuration, publisher)
+        end
+
+      end
+
+      context "when destinations has items in it" do
+
+        let :configuration_hash do
+          {
+            destinations: {
+              out1:{
+                service: :file_system,
+                channel: :foo,
+                options: {
+                  path: "tmp/test/channel"
+                }
+              }
+            }
+          }
+        end
+
+        it "should call register_destination with the right parameters" do
+          publisher.should_receive(:register_destination).exactly(1).times.with("out1", :file_system, :foo, "path" => "tmp/test/channel")
+          subject.load_config(configuration, publisher)
+        end
+
+      end
+
+    end
+
+  end # .load_config
+
+end

data/spec/lib/dispatch-rider/publisher/configuration_spec.rb

@@ -0,0 +1,149 @@
+require 'spec_helper'
+
+describe DispatchRider::Publisher::Configuration do
+
+  let(:configuration_hash) do
+    {
+      "notification_services" => {
+        "file_system" => {
+          "default_folder" => "tmp/dispatch_rider"
+        },
+        "aws_sns" => {}
+      },
+      "destinations" => {
+        "employee_updates" => {
+          "service" => "file_system",
+          "channel" => "employee_updates",
+          "options" => {
+            "file_prefix" => "employee_"
+          }
+        },
+        "account_updates" => {
+          "service" => "aws_sns",
+          "channel" => "account_updates",
+          "options" => {
+            "account" => "123456789",
+            "region" => "us-east",
+            "topic" => "account-updates"
+          }
+        }
+      }
+    }
+  end
+
+  subject{ described_class.new(configuration_hash) }
+
+  describe "#notification services" do
+    let(:file_system) do
+      DispatchRider::Publisher::Configuration::NotificationService.new(
+        "file_system",
+        { "default_folder" => "tmp/dispatch_rider" }
+      )
+    end
+
+    let(:sns) do
+      DispatchRider::Publisher::Configuration::NotificationService.new("aws_sns", {})
+    end
+
+    it "contains both notification services" do
+      subject.notification_services.count.should == 2
+      subject.notification_services.should =~ [file_system, sns]
+    end
+  end
+
+  describe "#destinations" do
+    let(:employee_updates) do
+      DispatchRider::Publisher::Configuration::Destination.new(
+        "employee_updates",
+        {
+          "service" => "file_system",
+          "channel" => "employee_updates",
+          "options" => {
+            "file_prefix" => "employee_"
+          }
+        }
+      )
+    end
+
+    let(:account_updates) do
+      DispatchRider::Publisher::Configuration::Destination.new(
+        "account_updates",
+        {
+          "service" => "aws_sns",
+          "channel" => "account_updates",
+          "options" => {
+            "account" => "123456789",
+            "region" => "us-east",
+            "topic" => "account-updates"
+          }
+        }
+      )
+    end
+
+    it "contains both destinations" do
+      subject.destinations.count.should == 2
+      subject.destinations.should =~ [employee_updates, account_updates]
+    end
+  end
+
+  describe "#clear" do
+    example do
+      expect{
+        subject.clear
+      }.to change(subject.notification_services, :count).by(-2)
+    end
+
+    example do
+      expect{
+        subject.clear
+      }.to change(subject.destinations, :count).by(-2)
+    end
+  end
+
+  describe "#parse" do
+    let(:new_configuration_hash) do
+      {
+        "notification_services" => {
+          "aws_sqs" => {}
+        },
+        "destinations" => {
+          "user_deletion" => {
+            "service" => "aws_sqs",
+            "channel" => "user_deletion",
+            "options" => {}
+          }
+        }
+      }
+    end
+
+    let(:notification_service) do
+      DispatchRider::Publisher::Configuration::NotificationService.new("aws_sqs", {})
+    end
+
+    let(:destination) do
+      DispatchRider::Publisher::Configuration::Destination.new(
+        "user_deletion",
+        {
+          "service" => "aws_sqs",
+          "channel" => "user_deletion",
+          "options" => {}
+        }
+      )
+    end
+
+    before :each do
+      subject.parse(new_configuration_hash)
+    end
+
+    it "replaces the current notification services with the new notification service" do
+      subject.notification_services.count.should == 1
+      subject.notification_services.should =~ [notification_service]
+    end
+
+    it "replaces the current destinations with the new destination" do
+      subject.destinations.count.should == 1
+      subject.destinations.should =~ [destination]
+    end
+  end
+
+end

data/spec/lib/dispatch-rider/publisher/configuration_support_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+
+describe DispatchRider::Publisher::ConfigurationSupport do
+
+  subject{ Object.new.extend(described_class) }
+
+  describe ".configuration" do
+    example do
+      subject.configuration.should be_a(DispatchRider::Publisher::Configuration)
+    end
+  end
+
+  describe ".config" do
+    example do
+      subject.method(:config).should == subject.method(:configuration)
+    end
+  end
+
+  describe ".configure" do
+    let :configuration_hash do
+      {
+        notification_services: {
+          file_system: {}
+        },
+        destinations: {
+          file_foo: {
+            service: :file_system,
+            channel: :foo,
+            options: {
+              path: "tmp/test/channel",
+            }
+          }
+        }
+      }
+    end
+
+    let(:notification_service) do
+      DispatchRider::Publisher::Configuration::NotificationService.new("file_system", {})
+    end
+
+    let(:destination) do
+      DispatchRider::Publisher::Configuration::Destination.new(
+        "file_foo",
+        {
+          service: :file_system,
+          channel: :foo,
+          options: {
+            path: "tmp/test/channel"
+          }
+        }
+      )
+    end
+
+    context "when configuring with a hash" do
+      before :each do
+        subject.configure(configuration_hash)
+      end
+
+      it "sets the configuration's notification services correctly" do
+        subject.configuration.notification_services.count.should == 1
+        subject.configuration.notification_services.should =~ [notification_service]
+      end
+
+      it "sets the configuration's destinations correctly" do
+        subject.configuration.destinations.count.should == 1
+        subject.configuration.destinations.should =~ [destination]
+      end
+    end
+
+    context "when configuring with a block" do
+      before :each do
+        subject.configure do |config|
+          config.parse(configuration_hash)
+        end
+      end
+
+      it "sets the configuration's notification services correctly" do
+        subject.configuration.notification_services.count.should == 1
+        subject.configuration.notification_services.should =~ [notification_service]
+      end
+
+      it "sets the configuration's destinations correctly" do
+        subject.configuration.destinations.count.should == 1
+        subject.configuration.destinations.should =~ [destination]
+      end
+    end
+  end
+
+end