Improve document for RSpec/BeforeAfterAll

rubocop · Nov 14, 2023 · c38be61 · c38be61
1 parent fe95441
commit c38be61
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 20 deletions.
diff --git a/docs/modules/ROOT/pages/cops_rspec.adoc b/docs/modules/ROOT/pages/cops_rspec.adoc
@@ -381,25 +381,19 @@ expect(foo).to be(nil)
 | 2.23
 |===
 
-Check that before/after(:all) isn't being used.
+Check that before/after(:all/:context) isn't being used.
 
 === Examples
 
 [source,ruby]
 ----
-# bad
-#
-# Faster but risk of state leaking between examples
-#
+# bad - Faster but risk of state leaking between examples
 describe MyClass do
   before(:all) { Widget.create }
-  after(:all) { Widget.delete_all }
+  after(:context) { Widget.delete_all }
 end
 
-# good
-#
-# Slower but examples are properly isolated
-#
+# good - Slower but examples are properly isolated
 describe MyClass do
   before(:each) { Widget.create }
   after(:each) { Widget.delete_all }

diff --git a/lib/rubocop/cop/rspec/before_after_all.rb b/lib/rubocop/cop/rspec/before_after_all.rb
@@ -3,22 +3,16 @@
 module RuboCop
   module Cop
     module RSpec
-      # Check that before/after(:all) isn't being used.
+      # Check that before/after(:all/:context) isn't being used.
       #
       # @example
-      #   # bad
-      #   #
-      #   # Faster but risk of state leaking between examples
-      #   #
+      #   # bad - Faster but risk of state leaking between examples
       #   describe MyClass do
       #     before(:all) { Widget.create }
-      #     after(:all) { Widget.delete_all }
+      #     after(:context) { Widget.delete_all }
       #   end
       #
-      #   # good
-      #   #
-      #   # Slower but examples are properly isolated
-      #   #
+      #   # good - Slower but examples are properly isolated
       #   describe MyClass do
       #     before(:each) { Widget.create }
       #     after(:each) { Widget.delete_all }