Remove Intercom::UserCollectionProxy#count (clashes with Enumerable#count)

timcraft · timcraft · commit 9cccbedfa197 · 2013-03-04T14:34:55.000Z
diff --git a/lib/intercom/user_collection_proxy.rb b/lib/intercom/user_collection_proxy.rb
@@ -8,21 +8,12 @@ module Intercom
   #
   # == Examples:
   #
-  # Fetching a count of all Users tracked on Intercom
-  #    Intercom::User.all.count
-  #
   # Iterating over each user
   #    Intercom::User.all.each do |user|
   #      puts user.inspect
   #    end
   #
   class UserCollectionProxy
-    # @return [Integer] number of users tracked on Intercom for this application
-    def count
-      response = Intercom.get("/v1/users", {:per_page => 1})
-      response["total_count"]
-    end
-
     # yields each {User} to the block provided
     # @return [void]
     def each(&block)
diff --git a/spec/unit/intercom/user_collection_proxy_spec.rb b/spec/unit/intercom/user_collection_proxy_spec.rb
@@ -18,11 +18,6 @@
     Intercom::User.all.collect { |user| user.email }.must_equal %W(user1@example.com user2@example.com user3@example.com)
   end
 
-  it "yields each user to the block" do
-    Intercom.expects(:get).with("/v1/users", {:per_page => 1}).returns(page_of_users(1,1))
-    Intercom::User.all.count.must_equal 3
-  end
-
   it "loads multiple pages" do
     Intercom.expects(:get).with("/v1/users", {:page => 1}).returns(page_of_users(1, 1))
     Intercom.expects(:get).with("/v1/users", {:page => 2}).returns(page_of_users(2, 1))
