Merge pull request rails#50589 from sato11/assert-has-no-stream

Rename `#assert_not_has_stream` `#assert_has_no_stream`

Author: rafaelfranca

actioncable/CHANGELOG.md

@@ -1,13 +1,13 @@
-*   Add two new assertion methods for ActionCable test cases: `assert_not_has_stream`
-    and `assert_not_has_stream_for`. These methods can be used to assert that a
+*   Add two new assertion methods for ActionCable test cases: `assert_has_no_stream`
+    and `assert_has_no_stream_for`. These methods can be used to assert that a
     stream has been stopped, e.g. via `stop_stream` or `stop_stream_for`. They complement
     the already existing `assert_has_stream` and `assert_has_stream_for` methods.
 
     ```ruby
-    assert_not_has_stream "messages"
-    assert_not_has_stream_for User.find(42)
+    assert_has_no_stream "messages"
+    assert_has_no_stream_for User.find(42)
     ```
 
-    *Sebastian Pöll*
+    *Sebastian Pöll*, *Junichi Sato*
 
 Please check [7-1-stable](https://github.com/rails/rails/blob/7-1-stable/actioncable/CHANGELOG.md) for previous changes.

actioncable/lib/action_cable/channel/test_case.rb

@@ -314,24 +314,24 @@ def assert_has_stream_for(object)
 
         # Asserts that the specified stream has not been started.
         #
-        #   def test_assert_not_started_stream
+        #   def test_assert_no_started_stream
         #     subscribe
-        #     assert_not_has_stream 'messages'
+        #     assert_has_no_stream 'messages'
         #   end
         #
-        def assert_not_has_stream(stream)
+        def assert_has_no_stream(stream)
           assert subscription.streams.exclude?(stream), "Stream #{stream} has been started"
         end
 
         # Asserts that the specified stream for a model has not started.
         #
-        #   def test_assert_not_started_stream_for
+        #   def test_assert_no_started_stream_for
         #     subscribe id: 41
-        #     assert_not_has_stream_for User.find(42)
+        #     assert_has_no_stream_for User.find(42)
         #   end
         #
-        def assert_not_has_stream_for(object)
-          assert_not_has_stream(broadcasting_for(object))
+        def assert_has_no_stream_for(object)
+          assert_has_no_stream(broadcasting_for(object))
         end
 
         private

actioncable/test/channel/test_case_test.rb

@@ -111,14 +111,14 @@ def test_not_stream_without_params
     subscribe
     unsubscribe
 
-    assert_not_has_stream "test_0"
+    assert_has_no_stream "test_0"
   end
 
   def test_not_stream_with_params
     subscribe id: 42
     perform :unsubscribed, id: 42
 
-    assert_not_has_stream "test_42"
+    assert_has_no_stream "test_42"
   end
 
   def test_unsubscribe_from_stream
@@ -150,7 +150,7 @@ def test_not_stream_with_params
     subscribe id: 42
     perform :unsubscribed, id: 42
 
-    assert_not_has_stream_for User.new(42)
+    assert_has_no_stream_for User.new(42)
   end
 end