aboutsummaryrefslogtreecommitdiffstats
path: root/spec
diff options
context:
space:
mode:
authorJoe Sak <joe@joesak.com>2011-04-22 15:05:55 -0500
committerJoe Sak <joe@joesak.com>2011-04-22 15:05:55 -0500
commitfe3bb08fcf7455bf446734e380e68bfc410886c3 (patch)
tree7928c3a13ed728accf8a50bbce0614d047ac25b2 /spec
parentb0d7a687b7dcf2bc70fe83b8b4b191f8861d3c0f (diff)
downloadrefinerycms-blog-fe3bb08fcf7455bf446734e380e68bfc410886c3.tar.gz
refinerycms-blog-fe3bb08fcf7455bf446734e380e68bfc410886c3.tar.bz2
refinerycms-blog-fe3bb08fcf7455bf446734e380e68bfc410886c3.zip
tests were failing all over the place
Diffstat (limited to 'spec')
-rw-r--r--spec/models/blog_categories_spec.rb24
-rw-r--r--spec/models/blog_posts_spec.rb72
2 files changed, 44 insertions, 52 deletions
diff --git a/spec/models/blog_categories_spec.rb b/spec/models/blog_categories_spec.rb
index 53bc345..8ddec46 100644
--- a/spec/models/blog_categories_spec.rb
+++ b/spec/models/blog_categories_spec.rb
@@ -3,39 +3,39 @@ Dir[File.expand_path('../../../features/support/factories/*.rb', __FILE__)].each
describe BlogCategory do
before(:each) do
- @attr = { :title => "RefineryCMS" }
+ @blog_category = Factory(:blog_category)
end
describe "validations" do
it "requires title" do
- BlogCategory.new(@attr.merge(:title => "")).should_not be_valid
+ Factory.build(:blog_category, :title => "").should_not be_valid
end
it "won't allow duplicate titles" do
- BlogCategory.create!(@attr)
- BlogCategory.new(@attr).should_not be_valid
+ Factory.build(:blog_category, :title => @blog_category.title).should_not be_valid
end
end
describe "blog posts association" do
it "has a posts attribute" do
- BlogCategory.new.should respond_to(:posts)
+ @blog_category.should respond_to(:posts)
end
it "returns posts by published_at date in descending order" do
- @category = BlogCategory.create!(@attr)
- @first_post = @category.posts.create!({ :title => "Breaking News: Joe Sak is hot stuff you guys!!", :body => "True story.", :published_at => Time.now.yesterday })
- @latest_post = @category.posts.create!({ :title => "parndt is p. okay", :body => "For a kiwi.", :published_at => Time.now })
- @category.posts.first.should == @latest_post
+ first_post = @blog_category.posts.create!({ :title => "Breaking News: Joe Sak is hot stuff you guys!!", :body => "True story.", :published_at => Time.now.yesterday })
+ latest_post = @blog_category.posts.create!({ :title => "parndt is p. okay", :body => "For a kiwi.", :published_at => Time.now })
+
+ @blog_category.posts.first.should == latest_post
end
end
describe "#post_count" do
it "returns post count in category" do
- Factory(:post, :categories => [Factory(:blog_category)])
- Factory(:post, :categories => [Factory(:blog_category)])
- BlogCategory.first.post_count.should == 2
+ 2.times do
+ @blog_category.posts << Factory(:post)
+ end
+ @blog_category.post_count.should == 2
end
end
end
diff --git a/spec/models/blog_posts_spec.rb b/spec/models/blog_posts_spec.rb
index 1b96ece..66947a9 100644
--- a/spec/models/blog_posts_spec.rb
+++ b/spec/models/blog_posts_spec.rb
@@ -2,46 +2,38 @@ require 'spec_helper'
Dir[File.expand_path('../../../features/support/factories/*.rb', __FILE__)].each{|factory| require factory}
describe BlogPost do
+ before(:each) do
+ @blog_post = Factory(:post)
+ end
+
describe "validations" do
- before(:each) do
- @attr = { :title => "RefineryCMS", :body => "Some random text ..." }
- end
-
it "requires title" do
- BlogPost.new(@attr.merge(:title => "")).should_not be_valid
+ Factory.build(:post, :title => "").should_not be_valid
end
it "won't allow duplicate titles" do
- BlogPost.create!(@attr)
- BlogPost.new(@attr).should_not be_valid
+ Factory.build(:post, :title => @blog_post.title).should_not be_valid
end
it "requires body" do
- BlogPost.new(@attr.merge(:body => nil)).should_not be_valid
+ Factory.build(:post, :body => nil).should_not be_valid
end
end
describe "comments association" do
- before(:each) do
- @blog_post = Factory(:post)
- end
it "have a comments attribute" do
@blog_post.should respond_to(:comments)
end
it "destroys associated comments" do
- Factory(:blog_comment, :blog_post_id => @blog_post)
+ Factory(:blog_comment, :blog_post_id => @blog_post.id)
@blog_post.destroy
- BlogComment.find_by_blog_post_id(@blog_post).should be_nil
+ BlogComment.find_by_blog_post_id(@blog_post.id).should == nil
end
end
describe "categories association" do
- before(:each) do
- @blog_post = Factory(:post)
- end
-
it "have categories attribute" do
@blog_post.should respond_to(:categories)
end
@@ -49,8 +41,10 @@ describe BlogPost do
describe "tags" do
it "acts as taggable" do
- (post = Factory(:post)).should respond_to(:tag_list)
- post.tag_list.should include("chicago")
+ @blog_post.should respond_to(:tag_list)
+
+ #the factory has default tags, including 'chicago'
+ @blog_post.tag_list.should include("chicago")
end
end
@@ -62,9 +56,16 @@ describe BlogPost do
describe "by_archive scope" do
it "returns all posts from specified month" do
- blog_post1 = Factory(:post, :published_at => Time.now.advance(:minutes => -2))
- blog_post2 = Factory(:post, :published_at => Time.now.advance(:minutes => -1))
+ BlogPost.delete_all
+
+ #this month
+ blog_post1 = Factory(:post, :published_at => Time.now - 2.days)
+ blog_post2 = Factory(:post, :published_at => Time.now - 1.day)
+
+ #2 months ago
Factory(:post, :published_at => Time.now - 2.months)
+
+ #check for this month
date = "#{Time.now.month}/#{Time.now.year}"
BlogPost.by_archive(Time.parse(date)).count.should == 2
BlogPost.by_archive(Time.parse(date)).should == [blog_post2, blog_post1]
@@ -73,8 +74,10 @@ describe BlogPost do
describe "all_previous scope" do
it "returns all posts from previous months" do
- blog_post1 = Factory(:post, :published_at => Time.now.advance(:months => -2))
- blog_post2 = Factory(:post, :published_at => Time.now.advance(:months => -1))
+ BlogPost.delete_all
+
+ blog_post1 = Factory(:post, :published_at => Time.now - 2.months)
+ blog_post2 = Factory(:post, :published_at => Time.now - 1.month)
Factory(:post, :published_at => Time.now)
BlogPost.all_previous.count.should == 2
BlogPost.all_previous.should == [blog_post2, blog_post1]
@@ -83,6 +86,8 @@ describe BlogPost do
describe "live scope" do
it "returns all posts which aren't in draft and pub date isn't in future" do
+ BlogPost.delete_all
+
blog_post1 = Factory(:post, :published_at => Time.now.advance(:minutes => -2))
blog_post2 = Factory(:post, :published_at => Time.now.advance(:minutes => -1))
Factory(:post, :draft => true)
@@ -92,22 +97,6 @@ describe BlogPost do
end
end
- describe "next scope" do
- it "returns next article based on given article" do
- blog_post1 = Factory(:post, :published_at => Time.now.advance(:minutes => -1))
- blog_post2 = Factory(:post)
- BlogPost.next(blog_post1).should == [blog_post2]
- end
- end
-
- describe "previous scope" do
- it "returns previous article based on given article" do
- blog_post1 = Factory(:post, :published_at => Time.now.advance(:minutes => -1))
- blog_post2 = Factory(:post)
- BlogPost.previous(blog_post2).should == [blog_post1]
- end
- end
-
describe "uncategorized scope" do
it "returns uncategorized posts if they exist" do
uncategorized_blog_post = Factory(:post)
@@ -136,6 +125,8 @@ describe BlogPost do
describe "#next" do
it "returns next article when called on current article" do
+ BlogPost.delete_all
+
Factory(:post, :published_at => Time.now.advance(:minutes => -1))
blog_post = Factory(:post)
blog_posts = BlogPost.all
@@ -145,6 +136,8 @@ describe BlogPost do
describe "#prev" do
it "returns previous article when called on current article" do
+ BlogPost.delete_all
+
Factory(:post)
blog_post = Factory(:post, :published_at => Time.now.advance(:minutes => -1))
blog_posts = BlogPost.all
@@ -154,7 +147,6 @@ describe BlogPost do
describe "#category_ids=" do
before(:each) do
- @blog_post = Factory(:post)
@cat1 = Factory(:blog_category, :id => 1)
@cat2 = Factory(:blog_category, :id => 2)
@cat3 = Factory(:blog_category, :id => 3)