Removing redundant except method

Author: dmgarland

lib/bitbucket_rest_api/core_ext/array.rb

@@ -1,13 +1,5 @@
 class Array # :nodoc:
 
-  def except(*keys) # :nodoc:
-    self.dup.except!(*keys)
-  end unless method_defined?(:except)
-
-  def except!(*items) # :nodoc:
-    self.reject! { |item| items.include? item }
-  end unless method_defined?(:except!)
-
   def extract_options!
     last.is_a?(::Hash) ? pop : {}
   end

lib/bitbucket_rest_api/core_ext/hash.rb

@@ -1,14 +1,5 @@
 class Hash # :nodoc:
 
-  def except(*items) # :nodoc:
-    self.dup.except!(*items)
-  end unless method_defined?(:except)
-
-  def except!(*keys) # :nodoc:
-    keys.each { |key| self.delete(key) }
-    self
-  end unless method_defined?(:except!)
-
   def symbolize_keys  # :nodoc:
     inject({}) do |hash, (key, value)|
       hash[(key.to_sym rescue key) || key] = value

spec/bitbucket_rest_api/core_ext/array_spec.rb

@@ -3,24 +3,6 @@
 
 describe Array do
   let(:array) { [:a, :b, :c, :d, { key: :value }] }
-  describe '#except' do
-    it 'removes the keys' do
-      new_array = array.except(:a, :b)
-
-      expect(new_array).to_not include(:a)
-      expect(new_array).to_not include(:b)
-    end
-  end
-
-  describe '#except!' do
-    it 'removes the keys from the self' do
-      array = [:a, :b, :c, :d]
-      array.except!(:a, :b)
-
-      expect(array).to_not include(:a)
-      expect(array).to_not include(:b)
-    end
-  end
 
   describe '#extract_options!' do
     it 'selects a hash from the arguments list' do

spec/bitbucket_rest_api/core_ext/hash_spec.rb

@@ -12,25 +12,6 @@
     @symbols = { :a => { :b => { :c => 1 } } }
   end
 
-  context '#except!' do
-    it 'should respond to except!' do
-      @nested_hash.should respond_to :except!
-      copy = @nested_hash.dup
-      copy.except!('b', 'a', 'c')
-      expect(copy).to be_empty
-    end
-  end
-
-  context '#except' do
-    it 'should respond to except' do
-      @nested_hash.should respond_to :except
-    end
-
-    it 'should remove key from the hash' do
-      expect(@nested_hash.except('a')).to be_empty
-    end
-  end
-
   context '#symbolize_keys' do
     it 'should respond to symbolize_keys' do
       expect(@nested_hash).to respond_to :symbolize_keys