Index: test/unit/news_test.rb
===================================================================
--- test/unit/news_test.rb	(revision 0)
+++ test/unit/news_test.rb	(revision 0)
@@ -0,0 +1,67 @@
+# redMine - project management software
+# Copyright (C) 2006-2008  Jean-Philippe Lang
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License
+# as published by the Free Software Foundation; either version 2
+# of the License, or (at your option) any later version.
+# 
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+# 
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+
+require File.dirname(__FILE__) + '/../test_helper'
+
+class NewsTest < Test::Unit::TestCase
+  fixtures :projects, :versions, :users, :roles, :members, :issues, :journals, :journal_details,
+           :trackers, :projects_trackers, :issue_statuses, :enabled_modules, :enumerations, :boards, :messages
+
+  def setup
+    @project = Project.find(1)
+  end
+  
+  def test_should_include_news_for_projects_with_news_enabled
+    project = projects(:projects_001)
+    assert project.enabled_modules.any?{ |em| em.name == 'news' }
+
+    # News.latest should return news from projects_001
+    assert News.latest.any? { |news| news.project == project }
+  end
+  
+  def test_should_not_include_news_for_projects_with_news_disabled
+    # The projects_002 (OnlineStore) doesn't have the news module enabled, use that project for this test
+    project = projects(:projects_002)
+    assert ! project.enabled_modules.any?{ |em| em.name == 'news' }
+
+    # Add a piece of news to the project
+    news = project.news.create(:title => 'Test news', :description => 'This should not be returned by News.latest')
+
+    # News.latest should not return that new piece of news
+    assert News.latest.include?(news) == false
+  end
+  
+  def test_should_only_include_news_from_projects_visibly_to_the_user
+    # users_001 has no memberships so can only get news from public project
+    assert News.latest(users(:users_001)).all? { |news| news.project.is_public? } 
+  end
+  
+  def test_should_limit_the_amount_of_returned_news
+    # Make sure we have a bunch of news stories
+    10.times { projects(:projects_001).news.create(:title => 'Test news', :description => 'Lorem ipsum etc') }
+    assert_equal 2, News.latest(users(:users_002), 2).size
+    assert_equal 6, News.latest(users(:users_002), 6).size
+  end
+
+  def test_should_return_5_news_stories_by_default
+    # Make sure we have a bunch of news stories
+    10.times { projects(:projects_001).news.create(:title => 'Test news', :description => 'Lorem ipsum etc') }
+    assert_equal 5, News.latest(users(:users_004)).size
+  end
+
+
+end
Index: app/models/news.rb
===================================================================
--- app/models/news.rb	(revision 1901)
+++ app/models/news.rb	(working copy)
@@ -28,8 +28,11 @@
   acts_as_event :url => Proc.new {|o| {:controller => 'news', :action => 'show', :id => o.id}}
   acts_as_activity_provider :find_options => {:include => [:project, :author]}
   
-  # returns latest news for projects visible by user
-  def self.latest(user=nil, count=5)
-    find(:all, :limit => count, :conditions => Project.visible_by(user), :include => [ :author, :project ], :order => "#{News.table_name}.created_on DESC")	
+  # Returns latest news for projects that have the news module enabled and are visible by user
+  def self.latest(user=nil, count=5)    
+    result = find(:all, :limit => count, :conditions => Project.visible_by(user), :include => [ :author, :project ], :order => "#{News.table_name}.created_on DESC")	
+    
+    # Only returns news from projects that have the news module enabled
+    result.select { |news| news.project.enabled_modules.any? { |em| em.name == 'news' } }
   end
 end