aboutsummaryrefslogtreecommitdiffstats
path: root/spec/models/refinery/blog
diff options
context:
space:
mode:
authorPhilip Arndt <git@arndt.io>2014-12-04 08:50:09 -0800
committerPhilip Arndt <git@arndt.io>2014-12-04 13:44:21 -0800
commit494e38d2f14cd3549d10bafd382d1ab71e5266d1 (patch)
treefecd13cb9daa9edb96623f5d6357def1b7c8417c /spec/models/refinery/blog
parent1cec6e540daa2a5971290dc33bae77ed35f5429a (diff)
downloadrefinerycms-blog-494e38d2f14cd3549d10bafd382d1ab71e5266d1.tar.gz
refinerycms-blog-494e38d2f14cd3549d10bafd382d1ab71e5266d1.tar.bz2
refinerycms-blog-494e38d2f14cd3549d10bafd382d1ab71e5266d1.zip
Upgraded to rspec 3.x
Diffstat (limited to 'spec/models/refinery/blog')
-rw-r--r--spec/models/refinery/blog/category_spec.rb10
-rw-r--r--spec/models/refinery/blog/comment_spec.rb4
-rw-r--r--spec/models/refinery/blog/post_spec.rb66
3 files changed, 40 insertions, 40 deletions
diff --git a/spec/models/refinery/blog/category_spec.rb b/spec/models/refinery/blog/category_spec.rb
index 0292991..577cb9d 100644
--- a/spec/models/refinery/blog/category_spec.rb
+++ b/spec/models/refinery/blog/category_spec.rb
@@ -8,17 +8,17 @@ module Refinery
describe "validations" do
it "requires title" do
- FactoryGirl.build(:blog_category, :title => "").should_not be_valid
+ expect(FactoryGirl.build(:blog_category, :title => "")).not_to be_valid
end
it "won't allow duplicate titles" do
- FactoryGirl.build(:blog_category, :title => category.title).should_not be_valid
+ expect(FactoryGirl.build(:blog_category, :title => category.title)).not_to be_valid
end
end
describe "blog posts association" do
it "has a posts attribute" do
- category.should respond_to(:posts)
+ expect(category).to respond_to(:posts)
end
it "returns posts by published_at date in descending order" do
@@ -32,7 +32,7 @@ module Refinery
:published_at => Time.now,
:author => refinery_user })
- category.posts.newest_first.first.should == latest_post
+ expect(category.posts.newest_first.first).to eq(latest_post)
end
end
@@ -42,7 +42,7 @@ module Refinery
2.times do
category.posts << FactoryGirl.create(:blog_post)
end
- category.post_count.should == 2
+ expect(category.post_count).to eq(2)
end
end
end
diff --git a/spec/models/refinery/blog/comment_spec.rb b/spec/models/refinery/blog/comment_spec.rb
index 7a35a59..69430ac 100644
--- a/spec/models/refinery/blog/comment_spec.rb
+++ b/spec/models/refinery/blog/comment_spec.rb
@@ -7,11 +7,11 @@ module Refinery
let(:comment) { FactoryGirl.create(:blog_comment) }
it "saves" do
- comment.should_not be_nil
+ expect(comment).not_to be_nil
end
it "has a blog post" do
- comment.post.should_not be_nil
+ expect(comment.post).not_to be_nil
end
end
end
diff --git a/spec/models/refinery/blog/post_spec.rb b/spec/models/refinery/blog/post_spec.rb
index 73668e4..a56a33b 100644
--- a/spec/models/refinery/blog/post_spec.rb
+++ b/spec/models/refinery/blog/post_spec.rb
@@ -7,51 +7,51 @@ module Refinery
describe "validations" do
it "requires title" do
- FactoryGirl.build(:blog_post, :title => "").should_not be_valid
+ expect(FactoryGirl.build(:blog_post, :title => "")).not_to be_valid
end
it "won't allow duplicate titles" do
- FactoryGirl.build(:blog_post, :title => post.title).should_not be_valid
+ expect(FactoryGirl.build(:blog_post, :title => post.title)).not_to be_valid
end
it "requires body" do
- FactoryGirl.build(:blog_post, :body => nil).should_not be_valid
+ expect(FactoryGirl.build(:blog_post, :body => nil)).not_to be_valid
end
end
describe "comments association" do
it "have a comments attribute" do
- post.should respond_to(:comments)
+ expect(post).to respond_to(:comments)
end
it "destroys associated comments" do
FactoryGirl.create(:blog_comment, :blog_post_id => post.id)
post.destroy
- Blog::Comment.where(:blog_post_id => post.id).should be_empty
+ expect(Blog::Comment.where(:blog_post_id => post.id)).to be_empty
end
end
describe "categories association" do
it "have categories attribute" do
- post.should respond_to(:categories)
+ expect(post).to respond_to(:categories)
end
end
describe "tags" do
it "acts as taggable" do
- post.should respond_to(:tag_list)
+ expect(post).to respond_to(:tag_list)
post.tag_list = "refinery, cms"
post.save!
- post.tag_list.should include("refinery")
+ expect(post.tag_list).to include("refinery")
end
end
describe "authors" do
it "are authored" do
- described_class.instance_methods.map(&:to_sym).should include(:author)
+ expect(described_class.instance_methods.map(&:to_sym)).to include(:author)
end
end
@@ -67,8 +67,8 @@ module Refinery
it "returns all posts from specified month" do
#check for this month
date = "03/2011"
- described_class.by_month(Time.parse(date)).count.should be == 2
- described_class.by_month(Time.parse(date)).should == [@post2, @post1]
+ expect(described_class.by_month(Time.parse(date)).count).to eq(2)
+ expect(described_class.by_month(Time.parse(date))).to eq([@post2, @post1])
end
end
@@ -82,7 +82,7 @@ module Refinery
it "returns all published dates older than the argument" do
expected = [@post2.published_at, @post1.published_at]
- described_class.published_dates_older_than(5.minutes.ago).should eq(expected)
+ expect(described_class.published_dates_older_than(5.minutes.ago)).to eq(expected)
end
end
@@ -96,9 +96,9 @@ module Refinery
it "returns all posts which aren't in draft and pub date isn't in future" do
live_posts = described_class.live
- live_posts.count.should be == 2
- live_posts.should include(@post2)
- live_posts.should include(@post1)
+ expect(live_posts.count).to eq(2)
+ expect(live_posts).to include(@post2)
+ expect(live_posts).to include(@post1)
end
end
@@ -111,22 +111,22 @@ module Refinery
end
it "returns uncategorized posts if they exist" do
- described_class.uncategorized.should include @uncategorized_post
- described_class.uncategorized.should_not include @categorized_post
+ expect(described_class.uncategorized).to include @uncategorized_post
+ expect(described_class.uncategorized).not_to include @categorized_post
end
end
describe "#live?" do
it "returns true if post is not in draft and it's published" do
- FactoryGirl.build(:blog_post).should be_live
+ expect(FactoryGirl.build(:blog_post)).to be_live
end
it "returns false if post is in draft" do
- FactoryGirl.build(:blog_post, :draft => true).should_not be_live
+ expect(FactoryGirl.build(:blog_post, :draft => true)).not_to be_live
end
it "returns false if post pub date is in future" do
- FactoryGirl.build(:blog_post, :published_at => Time.now.advance(:minutes => 1)).should_not be_live
+ expect(FactoryGirl.build(:blog_post, :published_at => Time.now.advance(:minutes => 1))).not_to be_live
end
end
@@ -137,7 +137,7 @@ module Refinery
end
it "returns next article when called on current article" do
- described_class.newest_first.last.next.should == @post
+ expect(described_class.newest_first.last.next).to eq(@post)
end
end
@@ -148,7 +148,7 @@ module Refinery
end
it "returns previous article when called on current article" do
- described_class.first.prev.should == @post
+ expect(described_class.first.prev).to eq(@post)
end
end
@@ -159,7 +159,7 @@ module Refinery
end
it "should be true" do
- described_class.comments_allowed?.should be_truthy
+ expect(described_class.comments_allowed?).to be_truthy
end
end
@@ -169,14 +169,14 @@ module Refinery
end
it "should be false" do
- described_class.comments_allowed?.should be_falsey
+ expect(described_class.comments_allowed?).to be_falsey
end
end
end
describe "custom teasers" do
it "should allow a custom teaser" do
- FactoryGirl.create(:blog_post, :custom_teaser => 'This is some custom content').should be_valid
+ expect(FactoryGirl.create(:blog_post, :custom_teaser => 'This is some custom content')).to be_valid
end
end
@@ -187,7 +187,7 @@ module Refinery
end
it "should be true" do
- described_class.teasers_enabled?.should be_truthy
+ expect(described_class.teasers_enabled?).to be_truthy
end
end
@@ -197,7 +197,7 @@ module Refinery
end
it "should be false" do
- described_class.teasers_enabled?.should be_falsey
+ expect(described_class.teasers_enabled?).to be_falsey
end
end
end
@@ -205,9 +205,9 @@ module Refinery
describe "source url" do
it "should allow a source url and title" do
p = FactoryGirl.create(:blog_post, :source_url => 'google.com', :source_url_title => 'author')
- p.should be_valid
- p.source_url.should include('google')
- p.source_url_title.should include('author')
+ expect(p).to be_valid
+ expect(p.source_url).to include('google')
+ expect(p.source_url_title).to include('author')
end
end
@@ -217,10 +217,10 @@ module Refinery
Refinery::Blog.validate_source_url = true
end
it "should have canonical url" do
- UrlValidator.any_instance.should_receive(:resolve_redirects_verify_url).
+ expect_any_instance_of(UrlValidator).to receive(:resolve_redirects_verify_url).
and_return('http://www.google.com')
p = FactoryGirl.create(:blog_post, :source_url => 'google.com', :source_url_title => 'google')
- p.source_url.should include('www')
+ expect(p.source_url).to include('www')
end
end
context "with Refinery::Blog.validate_source_url set to false" do
@@ -229,7 +229,7 @@ module Refinery
end
it "should have original url" do
p = FactoryGirl.create(:blog_post, :source_url => 'google.com', :source_url_title => 'google')
- p.source_url.should_not include('www')
+ expect(p.source_url).not_to include('www')
end
end
end