Revert "(#7220) Add the ability to "inherit" options."

This reverts commit 9329a1f33b4d7df81ad8661de74f8a3656428570.

Conflicts:

	spec/unit/interface/action_builder_spec.rb
	spec/unit/interface/action_spec.rb

2 files changed, 0 insertions, 121 deletions

diff --git a/spec/unit/interface/action_builder_spec.rb b/spec/unit/interface/action_builder_spec.rb
index e9f10a1a6..c39860591 100755
--- a/spec/unit/interface/action_builder_spec.rb
+++ b/spec/unit/interface/action_builder_spec.rb
@@ -62,68 +62,6 @@ describe Puppet::Interface::ActionBuilder do
     end
   end
 
-  describe "#inherit_options_from" do
-    let :face do
-      Puppet::Interface.new(:face_with_some_options, '0.0.1') do
-        option '-w'
-
-        action(:foo) do
-          when_invoked do true end
-          option '-x', '--ex'
-          option '-y', '--why'
-        end
-
-        action(:bar) do
-          when_invoked do true end
-          option '-z', '--zee'
-        end
-
-        action(:baz) do
-          when_invoked do true end
-          option '-z', '--zed'
-        end
-      end
-    end
-
-    it 'should add the options from the specified action' do
-      foo = face.get_action(:foo)
-      action = Puppet::Interface::ActionBuilder.build(face, :inherit_options) do
-        when_invoked do true end
-        inherit_options_from foo
-      end
-      action.options.should == foo.options
-    end
-
-    it 'should add the options from multiple actions' do
-      foo = face.get_action(:foo)
-      bar = face.get_action(:bar)
-      action = Puppet::Interface::ActionBuilder.build(face, :inherit_options) do
-        when_invoked do true end
-        inherit_options_from foo
-        inherit_options_from bar
-      end
-      action.options.should == (foo.options + bar.options).uniq
-    end
-
-    it 'should permit symbolic names for actions in the same face' do
-      foo = face.get_action(:foo)
-      action = Puppet::Interface::ActionBuilder.build(face, :inherit_options) do
-        when_invoked do true end
-        inherit_options_from :foo
-      end
-      action.options.should == foo.options
-    end
-
-    it 'should raise a useful error if you supply a bad action name' do
-      expect do
-        Puppet::Interface::ActionBuilder.build(face, :inherit_options) do
-          when_invoked do true end
-          inherit_options_from :nowhere
-        end
-      end.to raise_error /nowhere/
-    end
-  end
-
   context "inline documentation" do
     it "should set the summary" do
       action = Puppet::Interface::ActionBuilder.build(face, :foo) do
diff --git a/spec/unit/interface/action_spec.rb b/spec/unit/interface/action_spec.rb
index f43968709..9e539c68e 100755
--- a/spec/unit/interface/action_spec.rb
+++ b/spec/unit/interface/action_spec.rb
@@ -68,65 +68,6 @@ describe Puppet::Interface::Action do
     end
   end
 
-  describe "#inherit_options_from" do
-    let :face do
-      Puppet::Interface.new(:face_with_some_options, '0.0.1') do
-        option '-w'
-
-        action(:foo) do
-          when_invoked do true end
-          option '-x', '--ex'
-          option '-y', '--why'
-        end
-
-        action(:bar) do
-          when_invoked do true end
-          option '-z', '--zee'
-        end
-
-        action(:baz) do
-          when_invoked do true end
-          option '-z', '--zed'
-        end
-
-        action(:noopts) do
-          # no options declared
-          when_invoked do true end
-        end
-      end
-    end
-
-    subject { action = face.action(:new_action) { when_invoked do true end } }
-
-    it 'should add the options from the specified action' do
-      subject.inherit_options_from(foo = face.get_action(:foo))
-      subject.options.should == foo.options
-    end
-
-    it 'should not die when the specified action has no options' do
-      original_options = subject.options
-      subject.inherit_options_from(face.get_action(:noopts))
-      subject.options.should == original_options
-    end
-
-    it 'should add the options from multiple actions' do
-      subject.inherit_options_from(foo = face.get_action(:foo))
-      subject.inherit_options_from(bar = face.get_action(:bar))
-      subject.options.should == (foo.options + bar.options).uniq
-    end
-
-    it 'should not inherit face options' do
-      subject.expects(:add_option)
-      subject.expects(:add_option).with(face.get_option(:w)).never
-      subject.inherit_options_from(face.get_action(:bar))
-    end
-
-    it 'should raise an error if inheritance would duplicate options' do
-      subject.inherit_options_from(face.get_action(:bar))
-      expect { subject.inherit_options_from(face.get_action(:baz)) }.to raise_error
-    end
-  end
-
   describe "when invoking" do
     it "should be able to call other actions on the same object" do
       face = Puppet::Interface.new(:my_face, '0.0.1') do