未验证 提交 74a9a292 编写于 作者: B bogdanvlviv

Include `ActiveSupport::Testing::MethodCallAssertions` to `ActionCable::TestCase`

Remove all `include ActiveSupport::Testing::MethodCallAssertions`
in actioncable's tests since we can do it only in `ActionCable::TestCase`
in order to prevent code duplication.
We use the same approach for other modules of Rails.
上级 d97f1670
# frozen_string_literal: true
require "test_helper"
require "active_support/testing/method_call_assertions"
require "stubs/test_connection"
require "stubs/room"
class ActionCable::Channel::BroadcastingTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
class ChatChannel < ActionCable::Channel::Base
end
......
......@@ -4,11 +4,8 @@
require "stubs/test_connection"
require "stubs/room"
require "active_support/time"
require "active_support/testing/method_call_assertions"
class ActionCable::Channel::PeriodicTimersTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
class ChatChannel < ActionCable::Channel::Base
# Method name arg
periodically :send_updates, every: 1
......
# frozen_string_literal: true
require "test_helper"
require "active_support/testing/method_call_assertions"
require "minitest/mock"
require "stubs/test_connection"
require "stubs/room"
......@@ -164,8 +163,6 @@ def subscribed
end
class StreamFromTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
setup do
@server = TestServer.new(subscription_adapter: ActionCable::SubscriptionAdapter::Async)
@server.config.allowed_request_origins = %w( http://rubyonrails.com )
......
......@@ -7,7 +7,6 @@
require "json"
require "active_support/hash_with_indifferent_access"
require "active_support/testing/method_call_assertions"
####
# 😷 Warning suppression 😷
......@@ -28,8 +27,6 @@ def initialize(*)
####
class ClientTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
WAIT_WHEN_EXPECTING_EVENT = 2
WAIT_WHEN_NOT_EXPECTING_EVENT = 0.5
......
# frozen_string_literal: true
require "test_helper"
require "active_support/testing/method_call_assertions"
require "stubs/test_server"
class ActionCable::Connection::AuthorizationTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
class Connection < ActionCable::Connection::Base
attr_reader :websocket
......
......@@ -3,11 +3,8 @@
require "test_helper"
require "stubs/test_server"
require "active_support/core_ext/object/json"
require "active_support/testing/method_call_assertions"
class ActionCable::Connection::BaseTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
class Connection < ActionCable::Connection::Base
attr_reader :websocket, :subscriptions, :message_buffer, :connected
......
......@@ -2,11 +2,8 @@
require "test_helper"
require "stubs/test_server"
require "active_support/testing/method_call_assertions"
class ActionCable::Connection::ClientSocketTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
class Connection < ActionCable::Connection::Base
attr_reader :connected, :websocket, :errors
......
# frozen_string_literal: true
require "test_helper"
require "active_support/testing/method_call_assertions"
require "stubs/test_server"
require "stubs/user"
class ActionCable::Connection::IdentifierTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
class Connection < ActionCable::Connection::Base
identified_by :current_user
attr_reader :websocket
......
# frozen_string_literal: true
require "test_helper"
require "active_support/testing/method_call_assertions"
require "minitest/mock"
require "stubs/test_server"
class ActionCable::Connection::StreamTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
class Connection < ActionCable::Connection::Base
attr_reader :connected, :websocket, :errors
......
# frozen_string_literal: true
require "test_helper"
require "active_support/testing/method_call_assertions"
class ActionCable::Connection::SubscriptionsTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
class Connection < ActionCable::Connection::Base
attr_reader :websocket
......
......@@ -3,11 +3,8 @@
require "test_helper"
require "stubs/test_server"
require "active_support/core_ext/hash/indifferent_access"
require "active_support/testing/method_call_assertions"
class BaseTest < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
def setup
@server = ActionCable::Server::Base.new
@server.config.cable = { adapter: "async" }.with_indifferent_access
......
......@@ -4,7 +4,6 @@
require_relative "common"
require_relative "channel_prefix"
require "active_support/testing/method_call_assertions"
require "action_cable/subscription_adapter/redis"
class RedisAdapterTest < ActionCable::TestCase
......@@ -31,8 +30,6 @@ def cable_config
end
class RedisAdapterTest::Connector < ActionCable::TestCase
include ActiveSupport::Testing::MethodCallAssertions
test "slices url, host, port, db, and password from config" do
config = { url: 1, host: 2, port: 3, db: 4, password: 5 }
......
......@@ -2,6 +2,7 @@
require "action_cable"
require "active_support/testing/autorun"
require "active_support/testing/method_call_assertions"
require "puma"
require "rack/mock"
......@@ -15,6 +16,8 @@
Dir[File.expand_path("stubs/*.rb", __dir__)].each { |file| require file }
class ActionCable::TestCase < ActiveSupport::TestCase
include ActiveSupport::Testing::MethodCallAssertions
def wait_for_async
wait_for_executor Concurrent.global_io_executor
end
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册