Raise on unsupported hook scope usage

Author: pirj

Changelog.md

@@ -9,6 +9,7 @@ Breaking Changes:
 * Change the default `shared_context_metadata_behavior` to `apply_to_host_groups`
   and remove the configuration option. (Phil Pirozhkov, #2834)
 * Remove `run_all_when_everything_filtered` configuration option. (Phil Pirozhkov, #2845)
+* Raise on unsupported hook scope usage. (Phil Pirozhkov, #2849)
 
 Enhancements:
 

lib/rspec/core/hooks.rb

@@ -442,18 +442,13 @@ def register(prepend_or_append, position, *args, &block)
           scope, options = scope_and_options_from(*args)
 
           if scope == :suite
-            # TODO: consider making this an error in RSpec 4. For SemVer reasons,
-            # we are only warning in RSpec 3.
-            RSpec.warn_with "WARNING: `#{position}(:suite)` hooks are only supported on " \
-                            "the RSpec configuration object. This " \
-                            "`#{position}(:suite)` hook, registered on an example " \
-                            "group, will be ignored."
-            return
+            raise ArgumentError, "`#{position}(:suite)` hooks are only " \
+              "supported on the RSpec configuration object. This " \
+              "`#{position}(:suite)` hook, registered on an example " \
+              "group, will be ignored."
           elsif scope == :context && position == :around
-            # TODO: consider making this an error in RSpec 4. For SemVer reasons,
-            # we are only warning in RSpec 3.
-            RSpec.warn_with "WARNING: `around(:context)` hooks are not supported and " \
-                            "behave like `around(:example)."
+            raise ArgumentError, "`around(:context)` hooks are not supported " \
+              "and behave like `around(:example)`."
           end
 
           hook = HOOK_TYPES[position][scope].new(block, options)

spec/rspec/core/hooks_spec.rb

@@ -484,32 +484,30 @@ def yielder
       ])
     end
 
-    it 'emits a warning for `around(:context)`' do
-      expect(RSpec).to receive(:warn_with).with(a_string_including(
-        '`around(:context)` hooks are not supported'
-      ))
-      RSpec.describe do
-        around(:context) { }
-      end
+    it 'raises an error for `around(:context)`' do
+      expect {
+        RSpec.describe do
+          around(:context) { }
+        end
+      }.to raise_error(ArgumentError, a_string_including("`around(:context)` hooks are not supported"))
     end
 
-    it 'emits a warning for `around(:context)` defined in `configure`' do
-      expect(RSpec).to receive(:warn_with).with(a_string_including(
-        '`around(:context)` hooks are not supported'
-      ))
-      RSpec.configure do |c|
-        c.around(:context) { }
-      end
+    it 'raises an error for `around(:context)` defined in `configure`' do
+      expect {
+        RSpec.configure do |c|
+          c.around(:context) { }
+        end
+      }.to raise_error(ArgumentError, a_string_including("`around(:context)` hooks are not supported"))
     end
 
     [:before, :around, :after].each do |type|
       it "emits a warning for `#{type}(:suite)` hooks" do
-        expect(RSpec).to receive(:warn_with).with(a_string_including(
-          "`#{type}(:suite)` hooks are only supported on the RSpec configuration object."
-        ))
-        RSpec.describe do
-          send(type, :suite) { }
-        end
+        expect {
+          RSpec.describe do
+            send(type, :suite) { }
+          end
+        }.to raise_error(ArgumentError, a_string_including(
+          "`#{type}(:suite)` hooks are only supported on the RSpec configuration object"))
       end
     end
   end

spec/rspec/core/suite_hooks_spec.rb

@@ -66,16 +66,12 @@ module RSpec::Core
         end
 
         context "registered on an example group" do
-          it "is ignored with a clear warning" do
-            sequence = []
-
+          it "raises an error with a clear message" do
             expect {
               RSpec.describe "Group" do
-                __send__(registration_method, :suite) { sequence << :suite_hook }
-                example { sequence << :example }
-              end.run
-            }.to change { sequence }.to([:example]).
-              and output(a_string_including("#{type}(:suite)")).to_stderr
+                __send__(registration_method, :suite) { }
+              end
+            }.to raise_error(a_string_including("#{type}(:suite)"))
           end
         end