diff --git a/test/unit/external_feed_test.rb b/test/unit/external_feed_test.rb index 836f4b6..e9f389b 100644 --- a/test/unit/external_feed_test.rb +++ b/test/unit/external_feed_test.rb @@ -5,10 +5,10 @@ class ExternalFeedTest < ActiveSupport::TestCase should 'require blog' do e = ExternalFeed.new e.valid? - assert e.errors[:blog_id] + assert e.errors[:blog_id].present? e.blog = create_blog e.valid? - assert !e.errors[:blog_id] + assert e.errors[:blog_id].blank? end should 'belong to blog' do @@ -58,15 +58,15 @@ class ExternalFeedTest < ActiveSupport::TestCase end should 'require address if enabled' do - e = build(ExternalFeed, :enabled => true) + e = build(ExternalFeed, :enabled => true, :address => nil) assert !e.valid? - assert e.errors[:address] + assert e.errors[:address].present? end should 'not require address if disabled' do e = build(ExternalFeed, :enabled => false, :address => nil) e.valid? - assert !e.errors[:address] + assert e.errors[:address].blank? end should 'list enabled external feeds' do @@ -161,6 +161,7 @@ class ExternalFeedTest < ActiveSupport::TestCase dd = [] Article.where(['parent_id = ?', blog.id]).all.each do |a| + next if a.kind_of?(RssFeed) dd << a.body.to_s.strip.gsub(/\s+/, ' ') end assert_equal '

Html content 1.

Html content 2.

', dd.sort.join -- libgit2 0.21.2