Commit 2fe1ebd0f75c7a4224341d5b0f8ad6cada165f04
1 parent
9e0087a1
Exists in
master
and in
27 other branches
rails3: fix external_feed tests
Showing
1 changed file
with
6 additions
and
5 deletions
Show diff stats
test/unit/external_feed_test.rb
| @@ -5,10 +5,10 @@ class ExternalFeedTest < ActiveSupport::TestCase | @@ -5,10 +5,10 @@ class ExternalFeedTest < ActiveSupport::TestCase | ||
| 5 | should 'require blog' do | 5 | should 'require blog' do |
| 6 | e = ExternalFeed.new | 6 | e = ExternalFeed.new |
| 7 | e.valid? | 7 | e.valid? |
| 8 | - assert e.errors[:blog_id] | 8 | + assert e.errors[:blog_id].present? |
| 9 | e.blog = create_blog | 9 | e.blog = create_blog |
| 10 | e.valid? | 10 | e.valid? |
| 11 | - assert !e.errors[:blog_id] | 11 | + assert e.errors[:blog_id].blank? |
| 12 | end | 12 | end |
| 13 | 13 | ||
| 14 | should 'belong to blog' do | 14 | should 'belong to blog' do |
| @@ -58,15 +58,15 @@ class ExternalFeedTest < ActiveSupport::TestCase | @@ -58,15 +58,15 @@ class ExternalFeedTest < ActiveSupport::TestCase | ||
| 58 | end | 58 | end |
| 59 | 59 | ||
| 60 | should 'require address if enabled' do | 60 | should 'require address if enabled' do |
| 61 | - e = build(ExternalFeed, :enabled => true) | 61 | + e = build(ExternalFeed, :enabled => true, :address => nil) |
| 62 | assert !e.valid? | 62 | assert !e.valid? |
| 63 | - assert e.errors[:address] | 63 | + assert e.errors[:address].present? |
| 64 | end | 64 | end |
| 65 | 65 | ||
| 66 | should 'not require address if disabled' do | 66 | should 'not require address if disabled' do |
| 67 | e = build(ExternalFeed, :enabled => false, :address => nil) | 67 | e = build(ExternalFeed, :enabled => false, :address => nil) |
| 68 | e.valid? | 68 | e.valid? |
| 69 | - assert !e.errors[:address] | 69 | + assert e.errors[:address].blank? |
| 70 | end | 70 | end |
| 71 | 71 | ||
| 72 | should 'list enabled external feeds' do | 72 | should 'list enabled external feeds' do |
| @@ -161,6 +161,7 @@ class ExternalFeedTest < ActiveSupport::TestCase | @@ -161,6 +161,7 @@ class ExternalFeedTest < ActiveSupport::TestCase | ||
| 161 | 161 | ||
| 162 | dd = [] | 162 | dd = [] |
| 163 | Article.where(['parent_id = ?', blog.id]).all.each do |a| | 163 | Article.where(['parent_id = ?', blog.id]).all.each do |a| |
| 164 | + next if a.kind_of?(RssFeed) | ||
| 164 | dd << a.body.to_s.strip.gsub(/\s+/, ' ') | 165 | dd << a.body.to_s.strip.gsub(/\s+/, ' ') |
| 165 | end | 166 | end |
| 166 | assert_equal '<img src="noosfero.png" /><p>Html content 1.</p><p>Html content 2.</p>', dd.sort.join | 167 | assert_equal '<img src="noosfero.png" /><p>Html content 1.</p><p>Html content 2.</p>', dd.sort.join |