Don't rely on ordering in news_test
The tests would fail because not all storage backends have a default ordering
This commit is contained in:
parent
6a26543887
commit
d4f0542e74
@ -17,7 +17,7 @@ class NewsTest < ActiveSupport::TestCase
|
|||||||
fixtures :projects, :users, :roles, :members, :member_roles, :enabled_modules, :news
|
fixtures :projects, :users, :roles, :members, :member_roles, :enabled_modules, :news
|
||||||
|
|
||||||
def valid_news
|
def valid_news
|
||||||
{ :title => 'Test news', :description => 'Lorem ipsum etc', :author => User.find(:first) }
|
{ :title => 'Test news', :description => 'Lorem ipsum etc', :author => users(:users_001) }
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
||||||
@ -28,7 +28,7 @@ class NewsTest < ActiveSupport::TestCase
|
|||||||
def test_create_should_send_email_notification
|
def test_create_should_send_email_notification
|
||||||
ActionMailer::Base.deliveries.clear
|
ActionMailer::Base.deliveries.clear
|
||||||
Setting.notified_events = Setting.notified_events.dup << 'news_added'
|
Setting.notified_events = Setting.notified_events.dup << 'news_added'
|
||||||
news = Project.find(:first).news.new(valid_news)
|
news = projects(:projects_001).news.new(valid_news)
|
||||||
|
|
||||||
assert news.save
|
assert news.save
|
||||||
assert_equal 2, ActionMailer::Base.deliveries.size
|
assert_equal 2, ActionMailer::Base.deliveries.size
|
||||||
|
Loading…
x
Reference in New Issue
Block a user