Remove String#mongoid_id? and Symbol#mongoid_id?

Author: johnnyshields

lib/mongoid/extensions/string.rb

@@ -63,16 +63,6 @@ def collectionize
         tableize.gsub("/", "_")
       end
 
-      # Is the string a valid value for a Mongoid id?
-      #
-      # @example Is the string an id value?
-      #   "_id".mongoid_id?
-      #
-      # @return [ true | false ] If the string is id or _id.
-      def mongoid_id?
-        self =~ /\A(|_)id\z/
-      end
-
       # Is the string a number? The literals "NaN", "Infinity", and "-Infinity"
       # are counted as numbers.
       #

lib/mongoid/extensions/symbol.rb

@@ -7,16 +7,6 @@ module Extensions
     # Adds type-casting behavior to Symbol class.
     module Symbol
 
-      # Is the symbol a valid value for a Mongoid id?
-      #
-      # @example Is the string an id value?
-      #   :_id.mongoid_id?
-      #
-      # @return [ true | false ] If the symbol is :id or :_id.
-      def mongoid_id?
-        to_s.mongoid_id?
-      end
-
       module ClassMethods
 
         # Turn the object from the ruby type we deal with to a Mongo friendly

spec/mongoid/extensions/string_spec.rb

@@ -172,37 +172,6 @@ class Patient
     end
   end
 
-  describe "#mongoid_id?" do
-
-    context "when the string is id" do
-
-      it "returns true" do
-        expect("id").to be_mongoid_id
-      end
-    end
-
-    context "when the string is _id" do
-
-      it "returns true" do
-        expect("_id").to be_mongoid_id
-      end
-    end
-
-    context "when the string contains id" do
-
-      it "returns false" do
-        expect("identity").to_not be_mongoid_id
-      end
-    end
-
-    context "when the string contains _id" do
-
-      it "returns false" do
-        expect("something_id").to_not be_mongoid_id
-      end
-    end
-  end
-
   [ :mongoize, :demongoize ].each do |method|
 
     describe ".#{method}" do

spec/mongoid/extensions/symbol_spec.rb

@@ -5,37 +5,6 @@
 
 describe Mongoid::Extensions::Symbol do
 
-  describe "#mongoid_id?" do
-
-    context "when the string is id" do
-
-      it "returns true" do
-        expect(:id).to be_mongoid_id
-      end
-    end
-
-    context "when the string is _id" do
-
-      it "returns true" do
-        expect(:_id).to be_mongoid_id
-      end
-    end
-
-    context "when the string contains id" do
-
-      it "returns false" do
-        expect(:identity).to_not be_mongoid_id
-      end
-    end
-
-    context "when the string contains _id" do
-
-      it "returns false" do
-        expect(:something_id).to_not be_mongoid_id
-      end
-    end
-  end
-
   [ :mongoize, :demongoize ].each do |method|
 
     describe ".mongoize" do