Commit 0b1028c7 authored by Uģis Ozols's avatar Uģis Ozols
Browse files

Use 'count' to get item count because it performs SELECT COUNT(*) instead of...

Use 'count' to get item count because it performs SELECT COUNT(*) instead of instantiating objects and then counting them.
parent e3390d29
...@@ -16,7 +16,7 @@ module Refinery ...@@ -16,7 +16,7 @@ module Refinery
5.times { Factory(:news_item, :publish_date => publish_date) } 5.times { Factory(:news_item, :publish_date => publish_date) }
5.times { Factory(:news_item, :publish_date => future_date) } 5.times { Factory(:news_item, :publish_date => future_date) }
Refinery::News::Item.by_archive(archive_range).length.should == 5 Refinery::News::Item.by_archive(archive_range).count.should == 5
end end
end end
...@@ -84,12 +84,12 @@ module Refinery ...@@ -84,12 +84,12 @@ module Refinery
5.times { Factory(:news_item, :publish_date => Time.now + 1.hour) } 5.times { Factory(:news_item, :publish_date => Time.now + 1.hour) }
Refinery::News::Item.latest.count.should == 5 Refinery::News::Item.latest.count.should == 5
7.times { Factory(:news_item) } 7.times { Factory(:news_item) }
Refinery::News::Item.latest.length.should == 10 Refinery::News::Item.latest.count.should == 10
end end
it "returns latest n news items" do it "returns latest n news items" do
4.times { Factory(:news_item) } 4.times { Factory(:news_item) }
Refinery::News::Item.latest(3).length.should == 3 Refinery::News::Item.latest(3).count.should == 3
end end
end end
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment