actioncable 7.0.8.7 → 7.1.5.1

data/lib/action_cable/test_helper.rb

@@ -44,15 +44,14 @@ module ActionCable
     #
     def assert_broadcasts(stream, number, &block)
       if block_given?
-        original_count = broadcasts_size(stream)
-        _assert_nothing_raised_or_warn("assert_broadcasts", &block)
-        new_count = broadcasts_size(stream)
-        actual_count = new_count - original_count
+        new_messages = new_broadcasts_from(broadcasts(stream), stream, "assert_broadcasts", &block)
+
+        actual_count = new_messages.size
+        assert_equal number, actual_count, "#{number} broadcasts to #{stream} expected, but #{actual_count} were sent"
       else
-        actual_count = broadcasts_size(stream)
+        actual_count = broadcasts(stream).size
+        assert_equal number, actual_count, "#{number} broadcasts to #{stream} expected, but #{actual_count} were sent"
       end
-
-      assert_equal number, actual_count, "#{number} broadcasts to #{stream} expected, but #{actual_count} were sent"
     end
 
     # Asserts that no messages have been sent to the stream.
@@ -79,6 +78,22 @@ module ActionCable
       assert_broadcasts stream, 0, &block
     end
 
+    # Returns the messages that are broadcasted in the block.
+    #
+    #   def test_broadcasts
+    #     messages = capture_broadcasts('messages') do
+    #       ActionCable.server.broadcast 'messages', { text: 'hi' }
+    #       ActionCable.server.broadcast 'messages', { text: 'how are you?' }
+    #     end
+    #     assert_equal 2, messages.length
+    #     assert_equal({ text: 'hi' }, messages.first)
+    #     assert_equal({ text: 'how are you?' }, messages.last)
+    #   end
+    #
+    def capture_broadcasts(stream, &block)
+      new_broadcasts_from(broadcasts(stream), stream, "capture_broadcasts", &block).map { |m| ActiveSupport::JSON.decode(m) }
+    end
+
     # Asserts that the specified message has been sent to the stream.
     #
     #   def test_assert_transmitted_message
@@ -103,20 +118,22 @@ module ActionCable
 
       new_messages = broadcasts(stream)
       if block_given?
-        old_messages = new_messages
-        clear_messages(stream)
-
-        _assert_nothing_raised_or_warn("assert_broadcast_on", &block)
-        new_messages = broadcasts(stream)
-        clear_messages(stream)
-
-        # Restore all sent messages
-        (old_messages + new_messages).each { |m| pubsub_adapter.broadcast(stream, m) }
+        new_messages = new_broadcasts_from(new_messages, stream, "assert_broadcast_on", &block)
       end
 
       message = new_messages.find { |msg| ActiveSupport::JSON.decode(msg) == serialized_msg }
 
-      assert message, "No messages sent with #{data} to #{stream}"
+      error_message = "No messages sent with #{data} to #{stream}"
+
+      if new_messages.any?
+        error_message = new_messages.inject("#{error_message}\nMessage(s) found:\n") do |error_message, new_message|
+          error_message + "#{ActiveSupport::JSON.decode(new_message)}\n"
+        end
+      else
+        error_message = "#{error_message}\nNo message found for #{stream}"
+      end
+
+      assert message, error_message
     end
 
     def pubsub_adapter # :nodoc:
@@ -126,8 +143,18 @@ module ActionCable
     delegate :broadcasts, :clear_messages, to: :pubsub_adapter
 
     private
-      def broadcasts_size(channel)
-        broadcasts(channel).size
+      def new_broadcasts_from(current_messages, stream, assertion, &block)
+        old_messages = current_messages
+        clear_messages(stream)
+
+        _assert_nothing_raised_or_warn(assertion, &block)
+        new_messages = broadcasts(stream)
+        clear_messages(stream)
+
+        # Restore all sent messages
+        (old_messages + new_messages).each { |m| pubsub_adapter.broadcast(stream, m) }
+
+        new_messages
       end
   end
 end

data/lib/action_cable/version.rb

@@ -3,7 +3,7 @@
 require_relative "gem_version"
 
 module ActionCable
-  # Returns the currently loaded version of Action Cable as a <tt>Gem::Version</tt>.
+  # Returns the currently loaded version of Action Cable as a +Gem::Version+.
   def self.version
     gem_version
   end

data/lib/action_cable.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 #--
-# Copyright (c) 2015-2022 Basecamp, LLC
+# Copyright (c) 37signals LLC
 #
 # Permission is hereby granted, free of charge, to any person obtaining
 # a copy of this software and associated documentation files (the
@@ -25,10 +25,29 @@
 
 require "active_support"
 require "active_support/rails"
-require "action_cable/version"
+require "zeitwerk"
 
+Zeitwerk::Loader.for_gem.tap do |loader|
+  loader.ignore(
+    "#{__dir__}/rails", # Contains generators, templates, docs, etc.
+    "#{__dir__}/action_cable/gem_version.rb",
+    "#{__dir__}/action_cable/deprecator.rb",
+  )
+
+  loader.do_not_eager_load(
+    "#{__dir__}/action_cable/subscription_adapter", # Adapters are required and loaded on demand.
+    "#{__dir__}/action_cable/test_helper.rb",
+    Dir["#{__dir__}/action_cable/**/test_case.rb"]
+  )
+
+  loader.inflector.inflect("postgresql" => "PostgreSQL")
+end.setup
+
+# :markup: markdown
+# :include: ../README.md
 module ActionCable
-  extend ActiveSupport::Autoload
+  require_relative "action_cable/version"
+  require_relative "action_cable/deprecator"
 
   INTERNAL = {
     message_types: {
@@ -41,7 +60,8 @@ module ActionCable
     disconnect_reasons: {
       unauthorized: "unauthorized",
       invalid_request: "invalid_request",
-      server_restart: "server_restart"
+      server_restart: "server_restart",
+      remote: "remote"
     },
     default_mount_path: "/cable",
     protocols: ["actioncable-v1-json", "actioncable-unsupported"].freeze
@@ -51,12 +71,4 @@ module ActionCable
   module_function def server
     @server ||= ActionCable::Server::Base.new
   end
-
-  autoload :Server
-  autoload :Connection
-  autoload :Channel
-  autoload :RemoteConnections
-  autoload :SubscriptionAdapter
-  autoload :TestHelper
-  autoload :TestCase
 end

data/lib/rails/generators/channel/USAGE

@@ -1,13 +1,19 @@
 Description:
-============
     Generates a new cable channel for the server (in Ruby) and client (in JavaScript).
     Pass the channel name, either CamelCased or under_scored, and an optional list of channel actions as arguments.
 
-Example:
-========
-    bin/rails generate channel Chat speak
+Examples:
+    `bin/rails generate channel notification`
 
-    creates a Chat channel class, test and JavaScript asset:
-        Channel:    app/channels/chat_channel.rb
-        Test:       test/channels/chat_channel_test.rb
-        Assets:     $JAVASCRIPT_PATH/channels/chat_channel.js
+    creates a notification channel class, test and JavaScript asset:
+        Channel:    app/channels/notification_channel.rb
+        Test:       test/channels/notification_channel_test.rb
+        Assets:     $JAVASCRIPT_PATH/channels/notification_channel.js
+
+    `bin/rails generate channel chat speak`
+
+    creates a chat channel with a speak action.
+
+    `bin/rails generate channel comments --no-assets`
+
+    creates a comments channel without JavaScript assets.

data/lib/rails/generators/channel/channel_generator.rb

@@ -24,7 +24,7 @@ module Rails
 
             if using_importmap?
               pin_javascript_dependencies
-            elsif using_node?
+            elsif using_js_runtime?
               install_javascript_dependencies
             end
           end
@@ -57,22 +57,26 @@ module Rails
         def create_channel_javascript_file
           channel_js_path = File.join("app/javascript/channels", class_path, "#{file_name}_channel")
           js_template "javascript/channel", channel_js_path
-          gsub_file "#{channel_js_path}.js", /\.\/consumer/, "channels/consumer" unless using_node?
+          gsub_file "#{channel_js_path}.js", /\.\/consumer/, "channels/consumer" unless using_js_runtime?
         end
 
         def import_channels_in_javascript_entrypoint
           append_to_file "app/javascript/application.js",
-            using_node? ? %(import "./channels"\n) : %(import "channels"\n)
+            using_js_runtime? ? %(import "./channels"\n) : %(import "channels"\n)
         end
 
         def import_channel_in_javascript_entrypoint
           append_to_file "app/javascript/channels/index.js",
-            using_node? ? %(import "./#{file_name}_channel"\n) : %(import "channels/#{file_name}_channel"\n)
+            using_js_runtime? ? %(import "./#{file_name}_channel"\n) : %(import "channels/#{file_name}_channel"\n)
         end
 
         def install_javascript_dependencies
           say "Installing JavaScript dependencies", :green
-          run "yarn add @rails/actioncable"
+          if using_bun?
+            run "bun add @rails/actioncable"
+          elsif using_node?
+            run "yarn add @rails/actioncable"
+          end
         end
 
         def pin_javascript_dependencies
@@ -82,7 +86,6 @@ pin_all_from "app/javascript/channels", under: "channels"
           RUBY
         end
 
-
         def file_name
           @_file_name ||= super.sub(/_channel\z/i, "")
         end
@@ -95,8 +98,19 @@ pin_all_from "app/javascript/channels", under: "channels"
           @using_javascript ||= options[:assets] && root.join("app/javascript").exist?
         end
 
+        def using_js_runtime?
+          @using_js_runtime ||= root.join("package.json").exist?
+        end
+
+        def using_bun?
+          # Cannot assume bun.lockb has been generated yet so we look for
+          # a file known to be generated by the jsbundling-rails gem
+          @using_bun ||= using_js_runtime? && root.join("bun.config.js").exist?
+        end
+
         def using_node?
-          @using_node ||= root.join("package.json").exist?
+          # Bun is the only runtime that _isn't_ node.
+          @using_node ||= using_js_runtime? && !root.join("bun.config.js").exist?
         end
 
         def using_importmap?

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: actioncable
 version: !ruby/object:Gem::Version
-  version: 7.0.8.7
+  version: 7.1.5.1
 platform: ruby
 authors:
 - Pratik Naik
@@ -17,28 +17,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 7.0.8.7
+        version: 7.1.5.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 7.0.8.7
+        version: 7.1.5.1
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 7.0.8.7
+        version: 7.1.5.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 7.0.8.7
+        version: 7.1.5.1
 - !ruby/object:Gem::Dependency
   name: nio4r
   requirement: !ruby/object:Gem::Requirement
@@ -67,6 +67,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.6.1
+- !ruby/object:Gem::Dependency
+  name: zeitwerk
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.6'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.6'
 description: Structure many real-time application concerns into channels over a single
   WebSocket connection.
 email:
@@ -83,7 +97,6 @@ files:
 - app/assets/javascripts/actioncable.esm.js
 - app/assets/javascripts/actioncable.js
 - lib/action_cable.rb
-- lib/action_cable/channel.rb
 - lib/action_cable/channel/base.rb
 - lib/action_cable/channel/broadcasting.rb
 - lib/action_cable/channel/callbacks.rb
@@ -91,9 +104,9 @@ files:
 - lib/action_cable/channel/periodic_timers.rb
 - lib/action_cable/channel/streams.rb
 - lib/action_cable/channel/test_case.rb
-- lib/action_cable/connection.rb
 - lib/action_cable/connection/authorization.rb
 - lib/action_cable/connection/base.rb
+- lib/action_cable/connection/callbacks.rb
 - lib/action_cable/connection/client_socket.rb
 - lib/action_cable/connection/identification.rb
 - lib/action_cable/connection/internal_channel.rb
@@ -104,18 +117,17 @@ files:
 - lib/action_cable/connection/tagged_logger_proxy.rb
 - lib/action_cable/connection/test_case.rb
 - lib/action_cable/connection/web_socket.rb
+- lib/action_cable/deprecator.rb
 - lib/action_cable/engine.rb
 - lib/action_cable/gem_version.rb
 - lib/action_cable/helpers/action_cable_helper.rb
 - lib/action_cable/remote_connections.rb
-- lib/action_cable/server.rb
 - lib/action_cable/server/base.rb
 - lib/action_cable/server/broadcasting.rb
 - lib/action_cable/server/configuration.rb
 - lib/action_cable/server/connections.rb
 - lib/action_cable/server/worker.rb
 - lib/action_cable/server/worker/active_record_connection_management.rb
-- lib/action_cable/subscription_adapter.rb
 - lib/action_cable/subscription_adapter/async.rb
 - lib/action_cable/subscription_adapter/base.rb
 - lib/action_cable/subscription_adapter/channel_prefix.rb
@@ -142,10 +154,10 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/rails/rails/issues
-  changelog_uri: https://github.com/rails/rails/blob/v7.0.8.7/actioncable/CHANGELOG.md
-  documentation_uri: https://api.rubyonrails.org/v7.0.8.7/
+  changelog_uri: https://github.com/rails/rails/blob/v7.1.5.1/actioncable/CHANGELOG.md
+  documentation_uri: https://api.rubyonrails.org/v7.1.5.1/
   mailing_list_uri: https://discuss.rubyonrails.org/c/rubyonrails-talk
-  source_code_uri: https://github.com/rails/rails/tree/v7.0.8.7/actioncable
+  source_code_uri: https://github.com/rails/rails/tree/v7.1.5.1/actioncable
   rubygems_mfa_required: 'true'
 post_install_message: 
 rdoc_options: []

data/lib/action_cable/channel.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-module ActionCable
-  module Channel
-    extend ActiveSupport::Autoload
-
-    eager_autoload do
-      autoload :Base
-      autoload :Broadcasting
-      autoload :Callbacks
-      autoload :Naming
-      autoload :PeriodicTimers
-      autoload :Streams
-      autoload :TestCase
-    end
-  end
-end

data/lib/action_cable/connection.rb

@@ -1,22 +0,0 @@
-# frozen_string_literal: true
-
-module ActionCable
-  module Connection
-    extend ActiveSupport::Autoload
-
-    eager_autoload do
-      autoload :Authorization
-      autoload :Base
-      autoload :ClientSocket
-      autoload :Identification
-      autoload :InternalChannel
-      autoload :MessageBuffer
-      autoload :Stream
-      autoload :StreamEventLoop
-      autoload :Subscriptions
-      autoload :TaggedLoggerProxy
-      autoload :TestCase
-      autoload :WebSocket
-    end
-  end
-end

data/lib/action_cable/server.rb

@@ -1,16 +0,0 @@
-# frozen_string_literal: true
-
-module ActionCable
-  module Server
-    extend ActiveSupport::Autoload
-
-    eager_autoload do
-      autoload :Base
-      autoload :Broadcasting
-      autoload :Connections
-      autoload :Configuration
-
-      autoload :Worker
-    end
-  end
-end

data/lib/action_cable/subscription_adapter.rb

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-
-module ActionCable
-  module SubscriptionAdapter
-    extend ActiveSupport::Autoload
-
-    autoload :Base
-    autoload :Test
-    autoload :SubscriberMap
-    autoload :ChannelPrefix
-  end
-end