Skip unnecessary :context|:all hooks

This makes that previous test pass, and the test suite is still green,
so I'm _assuming_ that this works and doesn't break other stuff!
However, I'd be happy to make any alterations if this doesn't look like
it's the right way of tackling this problem.

Author: devonestes

lib/rspec/core/hooks.rb

@@ -456,7 +456,9 @@ def run(position, scope, example_or_group)
           return if RSpec.configuration.dry_run?
 
           if scope == :context
-            run_owned_hooks_for(position, :context, example_or_group)
+            unless example_or_group.class.metadata[:skip]
+              run_owned_hooks_for(position, :context, example_or_group)
+            end
           else
             case position
             when :before then run_example_hooks_for(example_or_group, :before, :reverse_each)

spec/rspec/core/hooks_filtering_spec.rb

@@ -319,7 +319,24 @@ module RSpec::Core
         expect(filters).to eq([])
       end
 
-      it "does not run :all|:context hooks if the entire context is skipped" do
+      it "runs :all|:context hooks even if there are no unskipped examples in that context" do
+        filters = []
+        group = RSpec.describe("skipped describe") do
+          before(:all) { filters << "before all in group"}
+          after(:all) { filters << "after all in group"}
+
+          xcontext("skipped context") do
+            before(:context) { filters << "before context in group"}
+            after(:context) { filters << "after context in group"}
+
+            it("is skipped") {}
+          end
+        end
+        group.run
+        expect(filters).to eq(["before all in group", "after all in group"])
+      end
+
+      it "does not run :all|:context hooks in global config if the entire context is skipped" do
         filters = []
         RSpec.configure do |c|
           c.before(:all) { filters << "before all in config"}
@@ -328,7 +345,24 @@ module RSpec::Core
           c.after(:context) { filters << "after context in config"}
         end
         group = RSpec.xdescribe("skipped describe") do
-          xcontext("skipped context") do
+          context("skipped context") do
+            it("is skipped") {}
+          end
+        end
+        group.run
+        expect(filters).to eq([])
+      end
+
+      it "does not run local :all|:context hooks if the entire context is skipped" do
+        filters = []
+        group = RSpec.xdescribe("skipped describe") do
+          before(:all) { filters << "before all in group"}
+          after(:all) { filters << "after all in group"}
+
+          context("skipped context") do
+            before(:context) { filters << "before context in group"}
+            after(:context) { filters << "after context in group"}
+
             it("is skipped") {}
           end
         end