multiple_man 1.3.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d92ba04f6fe9404dd98ce8c89addb6bbd11a3011
-  data.tar.gz: f5cd19e11c8cb204e0be5da36c0ac58fbeb6dad0
+  metadata.gz: 546b5d63fbc985f621b1b5c891aa7cdbc93bab8b
+  data.tar.gz: 0201b5a619231469deb7dc5213be502dda2c0fe8
 SHA512:
-  metadata.gz: 3718e17f9e5fc958ef18d30d870319569a909508980de9922dfdb076a304390fc2c6e2caaa13ce32555579effa937c2fff8ad12f18a2b9e1b3533a4fb423ac96
-  data.tar.gz: fe756d81f72831b3db89064ed7e0f3749823bf47a76d210f53c0f05c8da8b4bf782f2e6e611e3b1fcc59a327a561f9da86b1804fb5a82d8e376baf7d7b0751c6
+  metadata.gz: 175d5be3b1a35cc18447d0be840d6a7615f5992f62650d62a054eed9288201629bf6646b8e257fc5bcd1549cf2ddc19cba7a17accb7d9b283679358e301b1bae
+  data.tar.gz: 230e1bedf174de89182ab379e4e1ec93b1f1d8863bd42b214dbd43d23001337d815a03b0276bd97a3220beb12c570cb8bead859d2a1a4361e335579cc4625d1f

data/lib/multiple_man/channel_maintenance/reaper.rb

@@ -9,9 +9,9 @@ module MultipleMan
           loop do
             channel = queue.pop
             begin
-              channel.close unless closed?
+              channel.close unless channel.closed?
               puts "Channel #{channel.number} closed!"
-            rescue Exception
+            rescue Bunny::Exception, Timeout::Error
               sleep config.connection_recovery[:time_between_retries]
               retry
             end

data/lib/multiple_man/version.rb

@@ -1,3 +1,3 @@
 module MultipleMan
-  VERSION = "1.3.0"
+  VERSION = "1.4.0"
 end

data/spec/channel_maintenance/reaper_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe MultipleMan::ChannelMaintenance::Reaper do
+  let(:mock_channel) { double(Bunny::Channel, close: nil, closed?: false, number: 1) }
+  let(:mock_queue) { double(Queue, pop: mock_channel) }
+  let(:mock_config_hash) { { time_between_retries: 1 } }
+  let(:mock_mm_config) { double(MultipleMan, connection_recovery: mock_config_hash) }
+
+  before do
+    expect(Queue).to receive(:new).and_return(mock_queue)
+    expect(Thread).to receive(:new).and_yield
+    expect_any_instance_of(
+      MultipleMan::ChannelMaintenance::Reaper
+    ).to receive(:loop).and_yield
+  end
+
+  it "reaps an open connection" do
+    expect(mock_channel).to receive(:close)
+
+    described_class.new(mock_mm_config)
+  end
+
+  it "raises errors" do
+    expect(mock_channel).to receive(:closed?).and_raise(NoMethodError)
+
+    expect { described_class.new(mock_mm_config) }.to raise_error(NoMethodError)
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: multiple_man
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Ryan Brunner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-03 00:00:00.000000000 Z
+date: 2017-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bunny
@@ -138,6 +138,7 @@ files:
 - lib/multiple_man/version.rb
 - multiple_man.gemspec
 - spec/attribute_extractor_spec.rb
+- spec/channel_maintenance/reaper_spec.rb
 - spec/connection_spec.rb
 - spec/consumers/general_spec.rb
 - spec/consumers/seed_spec.rb
@@ -176,12 +177,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.5.2
 signing_key: 
 specification_version: 4
 summary: MultipleMan syncs changes to ActiveRecord models via AMQP
 test_files:
 - spec/attribute_extractor_spec.rb
+- spec/channel_maintenance/reaper_spec.rb
 - spec/connection_spec.rb
 - spec/consumers/general_spec.rb
 - spec/consumers/seed_spec.rb