bps 0.1.2 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ddb7d0cf9385205fd1faa31471ba98be7d5c59f26fd0742cf1cb6b5f9f1e2951
-  data.tar.gz: ee82c473da8be43ecc9183c521036b9c97a6d3df179ea4e3b6333e0578e2521c
+  metadata.gz: a7ea055a7a6f92d9464ec2a7142a94c414cea104ab54684b561c6dfebf5107f9
+  data.tar.gz: 97c3f004f66bdeca362e188511ff44bbc118ac852b65f362510cf87e9410f93e
 SHA512:
-  metadata.gz: a1e851fb6716a04faaefccd79089a328de2972342ab81840f5a1fd58453d03b9c518a09d21c725d52b3ae92db3db27c6d6255d0e1de08484bcfe847878c1015c
-  data.tar.gz: a87404ee2e004cf821177fd2e69129326ffc7aff8655817e8bdeb93f09c8cd36cf1249edd9459796539e57919d581a9d603d6c7170e03e52f4dc87137ddae328
+  metadata.gz: 28a8bf949c811ca202265494dc7ae1c2277b53b953f6786d74b425b252ce59a98c6f6ff1cf0fff8c8cc7e711710b075b0f9807332c1b89eb42eb1ab5a166d0e9
+  data.tar.gz: 64d9f267c210fc2be7901563050d7f30901f8473a8c3c616b88620edf9aac8e41c13a6295d861b5547fd087a9a24f905805662a7ae4135ecc3a4d9a5d1ccd06f

data/lib/bps/publisher/abstract.rb

@@ -1,3 +1,5 @@
+require 'securerandom'
+
 module BPS
   module Publisher
     class Abstract
@@ -12,7 +14,8 @@ module BPS
       end
 
       def initialize
-        ObjectSpace.define_finalizer(self, proc { close })
+        @uuid = SecureRandom.uuid
+        ObjectSpace.define_finalizer(@uuid, proc { close })
       end
 
       # Retrieve a topic handle.
@@ -22,7 +25,9 @@ module BPS
       end
 
       # Close the publisher.
-      def close; end
+      def close
+        ObjectSpace.undefine_finalizer(@uuid)
+      end
     end
   end
 end

data/lib/bps/subscriber/abstract.rb

@@ -2,7 +2,8 @@ module BPS
   module Subscriber
     class Abstract
       def initialize
-        ObjectSpace.define_finalizer(self, proc { close })
+        @uuid = SecureRandom.uuid
+        ObjectSpace.define_finalizer(@uuid, proc { close })
       end
 
       # Subscribe to a topic
@@ -12,7 +13,9 @@ module BPS
       end
 
       # Close the subscriber.
-      def close; end
+      def close
+        ObjectSpace.undefine_finalizer(@uuid)
+      end
     end
   end
 end

data/spec/bps/coercer_spec.rb

@@ -1,7 +1,9 @@
 require 'spec_helper'
 
 RSpec.describe BPS::Coercer do
-  subject do
+  subject { coercer }
+
+  let(:coercer) do
     described_class.new(
       name: :string,
       codec: :symbol,
@@ -11,7 +13,6 @@ RSpec.describe BPS::Coercer do
       tags: [:string],
     )
   end
-
   let :options do
     {
       name: 123,
@@ -24,13 +25,13 @@ RSpec.describe BPS::Coercer do
     }
   end
 
-  it 'should validate' do
+  it 'validates' do
     expect { described_class.new(name: :unknown) }.to raise_error(ArgumentError, /Unknown type :unknown/)
     expect { described_class.new(bad: []) }.to raise_error(ArgumentError, /Array types must have exactly one entry/)
   end
 
-  it 'should coerce options' do
-    expect(subject.coerce(options)).to eq(
+  it 'coerces options' do
+    expect(coercer.coerce(options)).to eq(
       name: '123',
       codec: :snappy,
       retries: 4,

data/spec/bps/publisher/in_mem_spec.rb

@@ -1,22 +1,26 @@
 require 'spec_helper'
 
 RSpec.describe BPS::Publisher::InMem do
-  it 'should maintain topics' do
-    expect(subject.topic('x')).to be_a(described_class::Topic)
-    expect(subject.topic_names).to eq(%w[x])
-    expect(subject.topic('z')).to be_a(described_class::Topic)
-    expect(subject.topic_names).to eq(%w[x z])
-    expect(subject.topic('y')).to be_a(described_class::Topic)
-    expect(subject.topic_names).to eq(%w[x y z])
+  subject { publisher }
+
+  let(:publisher) { described_class.new }
+
+  it 'maintains topics' do
+    expect(publisher.topic('x')).to be_a(described_class::Topic)
+    expect(publisher.topic_names).to eq(%w[x])
+    expect(publisher.topic('z')).to be_a(described_class::Topic)
+    expect(publisher.topic_names).to eq(%w[x z])
+    expect(publisher.topic('y')).to be_a(described_class::Topic)
+    expect(publisher.topic_names).to eq(%w[x y z])
   end
 
-  it 'should publish' do
-    topic = subject.topic('x')
+  it 'publishes' do
+    topic = publisher.topic('x')
     topic.publish('foo')
     topic.publish('bar')
     expect(topic.messages).to eq(%w[foo bar])
 
     expect { topic.flush }.not_to raise_error
-    expect(subject.topic('x').messages).to eq(%w[foo bar])
+    expect(publisher.topic('x').messages).to eq(%w[foo bar])
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bps
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.2
 platform: ruby
 authors:
 - Black Square Media
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-11-09 00:00:00.000000000 Z
+date: 2021-09-16 00:00:00.000000000 Z
 dependencies: []
 description: Minimalist abstraction for publish-subscribe
 email: info@blacksquaremedia.com
@@ -43,7 +43,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.2.15
 signing_key: 
 specification_version: 4
 summary: Multi-platform pubsub adapter