message_bus 4.4.0 → 4.5.0

data/spec/fixtures/test/config.ru

@@ -14,5 +14,5 @@ run ->(env) do
     end
   end
 
-  [200, { "Content-Type" => "text/html" }, ["Howdy"]]
+  [200, { "content-type" => "text/html" }, ["Howdy"]]
 end

data/spec/integration/http_client_spec.rb

@@ -65,7 +65,7 @@ describe MessageBus::HTTPClient do
           # that we sleep for the right interval after failure
           sleep 0.5
 
-          assert_match(/Errno::ECONNREFUSED|SocketError/, fake.string)
+          assert_match(/Errno::ECONNREFUSED|SocketError|Socket::ResolutionError/, fake.string)
         ensure
           $stderr = original_stderr
         end

data/spec/lib/message_bus/client_spec.rb

@@ -59,7 +59,7 @@ describe MessageBus::Client do
       @client.use_chunked = true
       r, w = IO.pipe
       @client.io = w
-      @client.headers = { "Content-Type" => "application/json; charset=utf-8" }
+      @client.headers = { "content-type" => "application/json; charset=utf-8" }
       @client << MessageBus::Message.new(1, 1, '/test', 'test')
       @client << MessageBus::Message.new(2, 2, '/test', "a|\r\n|\r\n|b")
 

data/spec/lib/message_bus/connection_manager_spec.rb

@@ -90,6 +90,29 @@ describe MessageBus::ConnectionManager do
     @manager.notify_clients(m)
     assert_nil @resp.sent
   end
+
+  it "should prune empty channel and site_id entries from subscriptions after removing clients" do
+    client1 = MessageBus::Client.new(client_id: "abc", user_id: 1, site_id: 20)
+    client1.subscribe('test', -1)
+    @manager.add_client(client1)
+
+    client2 = MessageBus::Client.new(client_id: "def", user_id: 2, site_id: 20)
+    client2.subscribe('test', -1)
+    @manager.add_client(client2)
+
+    @manager.remove_client(client1)
+
+    subscriptions = @manager.instance_variable_get(:@subscriptions)
+    assert subscriptions[client1.site_id].key?('test')
+    assert subscriptions[client1.site_id]['test'].include?(client2.client_id)
+    refute subscriptions[client1.site_id]['test'].include?(client1.client_id)
+
+    @manager.remove_client(client2)
+
+    subscriptions = @manager.instance_variable_get(:@subscriptions)
+    assert_nil subscriptions.dig(client1.site_id, 'test')
+    assert_nil subscriptions[client1.site_id]
+  end
 end
 
 describe MessageBus::ConnectionManager, "notifying and subscribing concurrently" do

metadata

@@ -1,28 +1,28 @@
 --- !ruby/object:Gem::Specification
 name: message_bus
 version: !ruby/object:Gem::Version
-  version: 4.4.0
+  version: 4.5.0
 platform: ruby
 authors:
 - Sam Saffron
 bindir: bin
 cert_chain: []
-date: 2025-03-14 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.3
+        version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.3
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
@@ -164,7 +164,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: puma
+  name: pitchfork
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -205,6 +205,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: method_source
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: oj
   requirement: !ruby/object:Gem::Requirement
@@ -260,7 +274,6 @@ files:
 - ".prettierrc"
 - ".rubocop.yml"
 - CHANGELOG
-- DEV.md
 - Dockerfile
 - Gemfile
 - Guardfile
@@ -311,8 +324,8 @@ files:
 - lib/message_bus/timer_thread.rb
 - lib/message_bus/version.rb
 - message_bus.gemspec
-- package-lock.json
 - package.json
+- pnpm-lock.yaml
 - spec/assets/SpecHelper.js
 - spec/assets/message-bus.spec.js
 - spec/fixtures/test/Gemfile
@@ -354,7 +367,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.6.9
 specification_version: 4
 summary: ''
 test_files: []

data/DEV.md

@@ -1,7 +0,0 @@
-### How to Publish to NPM
-
-1. First, edit `package.json` and bump the version.
-
-2. Log in to npm `yarn login`
-
-3. Publish: `yarn publish`