improve test that users.all.count uses api response total_count rather than paging all and counting

darragh · darragh · commit 0756b957c361 · 2012-11-30T13:08:46.000Z
diff --git a/spec/unit/intercom/user_collection_proxy_spec.rb b/spec/unit/intercom/user_collection_proxy_spec.rb
@@ -18,8 +18,10 @@
     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("users", {:per_page => 1}).returns(page_of_users(1,1))
+  it "counts users based on total_count in paged_response" do
+    paged_response = page_of_users(1, 1)
+    Intercom.expects(:get).with("users", {:per_page => 1}).returns(paged_response)
+    paged_response.expects(:[]).with("total_count").returns(3)
     Intercom::User.all.count.must_equal 3
   end
 
