]> source.dussan.org Git - redmine.git/commitdiff
remove trailing white-spaces from test/unit/project_test.rb.
authorToshi MARUYAMA <marutosijp2@yahoo.co.jp>
Sun, 21 Aug 2011 11:50:39 +0000 (11:50 +0000)
committerToshi MARUYAMA <marutosijp2@yahoo.co.jp>
Sun, 21 Aug 2011 11:50:39 +0000 (11:50 +0000)
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@6520 e93f8b46-1217-0410-a6f0-8f06a7374b81

test/unit/project_test.rb

index 1fe6f15067610444738f27fd27967b08aa9c9c34..e45111db1c2aa527964131685a1a13b5c47870e7 100644 (file)
@@ -5,12 +5,12 @@
 # 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.
@@ -25,7 +25,7 @@ class ProjectTest < ActiveSupport::TestCase
     @ecookbook_sub1 = Project.find(3)
     User.current = nil
   end
-  
+
   should_validate_presence_of :name
   should_validate_presence_of :identifier
 
@@ -59,7 +59,7 @@ class ProjectTest < ActiveSupport::TestCase
     assert_kind_of Project, @ecookbook
     assert_equal "eCookbook", @ecookbook.name
   end
-  
+
   def test_default_attributes
     with_settings :default_projects_public => '1' do
       assert_equal true, Project.new.is_public
@@ -84,11 +84,11 @@ class ProjectTest < ActiveSupport::TestCase
     with_settings :default_projects_modules => ['issue_tracking', 'repository'] do
       assert_equal ['issue_tracking', 'repository'], Project.new.enabled_module_names
     end
-    
+
     assert_equal Tracker.all, Project.new.trackers
     assert_equal Tracker.find(1, 3), Project.new(:tracker_ids => [1, 3]).trackers
   end
-  
+
   def test_update
     assert_equal "eCookbook", @ecookbook.name
     @ecookbook.name = "eCook"
@@ -96,14 +96,14 @@ class ProjectTest < ActiveSupport::TestCase
     @ecookbook.reload
     assert_equal "eCook", @ecookbook.name
   end
-  
+
   def test_validate_identifier
     to_test = {"abc" => true,
                "ab12" => true,
                "ab-12" => true,
                "12" => false,
                "new" => false}
-               
+
     to_test.each do |identifier, valid|
       p = Project.new
       p.identifier = identifier
@@ -117,18 +117,18 @@ class ProjectTest < ActiveSupport::TestCase
       assert_nil project.members.detect {|m| !(m.user.is_a?(User) && m.user.active?) }
     end
   end
-  
+
   def test_users_should_be_active_users
     Project.all.each do |project|
       assert_nil project.users.detect {|u| !(u.is_a?(User) && u.active?) }
     end
   end
-  
+
   def test_archive
     user = @ecookbook.members.first.user
     @ecookbook.archive
     @ecookbook.reload
-    
+
     assert !@ecookbook.active?
     assert @ecookbook.archived?
     assert !user.projects.include?(@ecookbook)
@@ -136,24 +136,24 @@ class ProjectTest < ActiveSupport::TestCase
     assert !@ecookbook.children.empty?
     assert @ecookbook.descendants.active.empty?
   end
-  
+
   def test_archive_should_fail_if_versions_are_used_by_non_descendant_projects
     # Assign an issue of a project to a version of a child project
     Issue.find(4).update_attribute :fixed_version_id, 4
-    
+
     assert_no_difference "Project.count(:all, :conditions => 'status = #{Project::STATUS_ARCHIVED}')" do
       assert_equal false, @ecookbook.archive
     end
     @ecookbook.reload
     assert @ecookbook.active?
   end
-  
+
   def test_unarchive
     user = @ecookbook.members.first.user
     @ecookbook.archive
     # A subproject of an archived project can not be unarchived
     assert !@ecookbook_sub1.unarchive
-    
+
     # Unarchive project
     assert @ecookbook.unarchive
     @ecookbook.reload
@@ -164,7 +164,7 @@ class ProjectTest < ActiveSupport::TestCase
     @ecookbook_sub1.reload
     assert @ecookbook_sub1.unarchive
   end
-  
+
   def test_destroy
     # 2 active members
     assert_equal 2, @ecookbook.members.size
@@ -172,7 +172,7 @@ class ProjectTest < ActiveSupport::TestCase
     assert_equal 3, Member.find(:all, :conditions => ['project_id = ?', @ecookbook.id]).size
     # some boards
     assert @ecookbook.boards.any?
-    
+
     @ecookbook.destroy
     # make sure that the project non longer exists
     assert_raise(ActiveRecord::RecordNotFound) { Project.find(@ecookbook.id) }
@@ -181,12 +181,12 @@ class ProjectTest < ActiveSupport::TestCase
     assert_nil Board.first(:conditions => {:project_id => @ecookbook.id})
     assert_nil Issue.first(:conditions => {:project_id => @ecookbook.id})
   end
-  
+
   def test_destroying_root_projects_should_clear_data
     Project.roots.each do |root|
       root.destroy
     end
-    
+
     assert_equal 0, Project.count, "Projects were not deleted: #{Project.all.inspect}"
     assert_equal 0, Member.count, "Members were not deleted: #{Member.all.inspect}"
     assert_equal 0, MemberRole.count
@@ -216,7 +216,7 @@ class ProjectTest < ActiveSupport::TestCase
     assert_equal 0, Project.connection.select_all("SELECT * FROM custom_fields_projects").size
     assert_equal 0, CustomValue.count(:conditions => {:customized_type => ['Project', 'Issue', 'TimeEntry', 'Version']})
   end
-  
+
   def test_move_an_orphan_project_to_a_root_project
     sub = Project.find(2)
     sub.set_parent! @ecookbook
@@ -224,22 +224,22 @@ class ProjectTest < ActiveSupport::TestCase
     @ecookbook.reload
     assert_equal 4, @ecookbook.children.size
   end
-  
+
   def test_move_an_orphan_project_to_a_subproject
     sub = Project.find(2)
     assert sub.set_parent!(@ecookbook_sub1)
   end
-  
+
   def test_move_a_root_project_to_a_project
     sub = @ecookbook
     assert sub.set_parent!(Project.find(2))
   end
-  
+
   def test_should_not_move_a_project_to_its_children
     sub = @ecookbook
     assert !(sub.set_parent!(Project.find(3)))
   end
-  
+
   def test_set_parent_should_add_roots_in_alphabetical_order
     ProjectCustomField.delete_all
     Project.delete_all
@@ -247,11 +247,11 @@ class ProjectTest < ActiveSupport::TestCase
     Project.create!(:name => 'Project B', :identifier => 'project-b').set_parent!(nil)
     Project.create!(:name => 'Project D', :identifier => 'project-d').set_parent!(nil)
     Project.create!(:name => 'Project A', :identifier => 'project-a').set_parent!(nil)
-    
+
     assert_equal 4, Project.count
     assert_equal Project.all.sort_by(&:name), Project.all.sort_by(&:lft)
   end
-  
+
   def test_set_parent_should_add_children_in_alphabetical_order
     ProjectCustomField.delete_all
     parent = Project.create!(:name => 'Parent', :identifier => 'parent')
@@ -259,12 +259,12 @@ class ProjectTest < ActiveSupport::TestCase
     Project.create!(:name => 'Project B', :identifier => 'project-b').set_parent!(parent)
     Project.create!(:name => 'Project D', :identifier => 'project-d').set_parent!(parent)
     Project.create!(:name => 'Project A', :identifier => 'project-a').set_parent!(parent)
-    
+
     parent.reload
     assert_equal 4, parent.children.size
     assert_equal parent.children.sort_by(&:name), parent.children
   end
-  
+
   def test_rebuild_should_sort_children_alphabetically
     ProjectCustomField.delete_all
     parent = Project.create!(:name => 'Parent', :identifier => 'parent')
@@ -272,10 +272,10 @@ class ProjectTest < ActiveSupport::TestCase
     Project.create!(:name => 'Project B', :identifier => 'project-b').move_to_child_of(parent)
     Project.create!(:name => 'Project D', :identifier => 'project-d').move_to_child_of(parent)
     Project.create!(:name => 'Project A', :identifier => 'project-a').move_to_child_of(parent)
-    
+
     Project.update_all("lft = NULL, rgt = NULL")
     Project.rebuild!
-    
+
     parent.reload
     assert_equal 4, parent.children.size
     assert_equal parent.children.sort_by(&:name), parent.children
@@ -300,49 +300,49 @@ class ProjectTest < ActiveSupport::TestCase
     issue_with_hierarchy_fixed_version.update_attribute(:fixed_version_id, 6)
     issue_with_hierarchy_fixed_version.reload
     assert_equal 6, issue_with_hierarchy_fixed_version.fixed_version_id
-    
+
     # Move project out of the issue's hierarchy
     moved_project = Project.find(3)
     moved_project.set_parent!(Project.find(2))
     parent_issue.reload
     issue_with_local_fixed_version.reload
     issue_with_hierarchy_fixed_version.reload
-    
+
     assert_equal 4, issue_with_local_fixed_version.fixed_version_id, "Fixed version was not keep on an issue local to the moved project"
     assert_equal nil, issue_with_hierarchy_fixed_version.fixed_version_id, "Fixed version is still set after moving the Project out of the hierarchy where the version is defined in"
     assert_equal nil, parent_issue.fixed_version_id, "Fixed version is still set after moving the Version out of the hierarchy for the issue."
   end
-  
+
   def test_parent
     p = Project.find(6).parent
     assert p.is_a?(Project)
     assert_equal 5, p.id
   end
-  
+
   def test_ancestors
     a = Project.find(6).ancestors
     assert a.first.is_a?(Project)
     assert_equal [1, 5], a.collect(&:id)
   end
-  
+
   def test_root
     r = Project.find(6).root
     assert r.is_a?(Project)
     assert_equal 1, r.id
   end
-  
+
   def test_children
     c = Project.find(1).children
     assert c.first.is_a?(Project)
     assert_equal [5, 3, 4], c.collect(&:id)
   end
-  
+
   def test_descendants
     d = Project.find(1).descendants
     assert d.first.is_a?(Project)
     assert_equal [5, 6, 3, 4], d.collect(&:id)
   end
-  
+
   def test_allowed_parents_should_be_empty_for_non_member_user
     Role.non_member.add_permission!(:add_project)
     user = User.find(9)
@@ -350,7 +350,7 @@ class ProjectTest < ActiveSupport::TestCase
     User.current = user
     assert Project.new.allowed_parents.compact.empty?
   end
-  
+
   def test_allowed_parents_with_add_subprojects_permission
     Role.find(1).remove_permission!(:add_project)
     Role.find(1).add_permission!(:add_subprojects)
@@ -392,7 +392,7 @@ class ProjectTest < ActiveSupport::TestCase
     assert Project.find(3).allowed_parents.include?(Project.find(1))
     assert Project.find(3).allowed_parents.include?(nil)
   end
-  
+
   def test_users_by_role
     users_by_role = Project.find(1).users_by_role
     assert_kind_of Hash, users_by_role
@@ -400,29 +400,29 @@ class ProjectTest < ActiveSupport::TestCase
     assert_kind_of Array, users_by_role[role]
     assert users_by_role[role].include?(User.find(2))
   end
-  
+
   def test_rolled_up_trackers
     parent = Project.find(1)
     parent.trackers = Tracker.find([1,2])
     child = parent.children.find(3)
-  
+
     assert_equal [1, 2], parent.tracker_ids
     assert_equal [2, 3], child.trackers.collect(&:id)
-    
+
     assert_kind_of Tracker, parent.rolled_up_trackers.first
     assert_equal Tracker.find(1), parent.rolled_up_trackers.first
-    
+
     assert_equal [1, 2, 3], parent.rolled_up_trackers.collect(&:id)
     assert_equal [2, 3], child.rolled_up_trackers.collect(&:id)
   end
-  
+
   def test_rolled_up_trackers_should_ignore_archived_subprojects
     parent = Project.find(1)
     parent.trackers = Tracker.find([1,2])
     child = parent.children.find(3)
     child.trackers = Tracker.find([1,3])
     parent.children.each(&:archive)
-    
+
     assert_equal [1,2], parent.rolled_up_trackers.collect(&:id)
   end
 
@@ -432,11 +432,11 @@ class ProjectTest < ActiveSupport::TestCase
       @parent_version_1 = Version.generate!(:project => @project)
       @parent_version_2 = Version.generate!(:project => @project)
     end
-    
+
     should "include the versions for the current project" do
       assert_same_elements [@parent_version_1, @parent_version_2], @project.rolled_up_versions
     end
-    
+
     should "include versions for a subproject" do
       @subproject = Project.generate!
       @subproject.set_parent!(@project)
@@ -448,7 +448,7 @@ class ProjectTest < ActiveSupport::TestCase
                             @subproject_version
                            ], @project.rolled_up_versions
     end
-    
+
     should "include versions for a sub-subproject" do
       @subproject = Project.generate!
       @subproject.set_parent!(@project)
@@ -465,7 +465,6 @@ class ProjectTest < ActiveSupport::TestCase
                            ], @project.rolled_up_versions
     end
 
-    
     should "only check active projects" do
       @subproject = Project.generate!
       @subproject.set_parent!(@project)
@@ -478,7 +477,7 @@ class ProjectTest < ActiveSupport::TestCase
       assert_same_elements [@parent_version_1, @parent_version_2], @project.rolled_up_versions
     end
   end
-  
+
   def test_shared_versions_none_sharing
     p = Project.find(5)
     v = Version.create!(:name => 'none_sharing', :project => p, :sharing => 'none')
@@ -498,7 +497,7 @@ class ProjectTest < ActiveSupport::TestCase
     assert !p.siblings.first.shared_versions.include?(v)
     assert !p.root.siblings.first.shared_versions.include?(v)
   end
-  
+
   def test_shared_versions_hierarchy_sharing
     p = Project.find(5)
     v = Version.create!(:name => 'hierarchy_sharing', :project => p, :sharing => 'hierarchy')
@@ -533,7 +532,7 @@ class ProjectTest < ActiveSupport::TestCase
     parent = Project.find(1)
     child = parent.children.find(3)
     private_child = parent.children.find(5)
-    
+
     assert_equal [1,2,3], parent.version_ids.sort
     assert_equal [4], child.version_ids
     assert_equal [6], private_child.version_ids
@@ -552,7 +551,7 @@ class ProjectTest < ActiveSupport::TestCase
     child = parent.children.find(3)
     child.archive
     parent.reload
-    
+
     assert_equal [1,2,3], parent.version_ids.sort
     assert_equal [4], child.version_ids
     assert !parent.shared_versions.collect(&:id).include?(4)
@@ -562,12 +561,12 @@ class ProjectTest < ActiveSupport::TestCase
     user = User.find(3)
     parent = Project.find(1)
     child = parent.children.find(5)
-    
+
     assert_equal [1,2,3], parent.version_ids.sort
     assert_equal [6], child.version_ids
 
     versions = parent.shared_versions.visible(user)
-    
+
     assert_equal 4, versions.size
     versions.each do |version|
       assert_kind_of Version, version
@@ -576,7 +575,6 @@ class ProjectTest < ActiveSupport::TestCase
     assert !versions.collect(&:id).include?(6)
   end
 
-  
   def test_next_identifier
     ProjectCustomField.delete_all
     Project.create!(:name => 'last', :identifier => 'p2008040')
@@ -587,11 +585,11 @@ class ProjectTest < ActiveSupport::TestCase
     Project.delete_all
     assert_nil Project.next_identifier
   end
-  
+
   def test_enabled_module_names
     with_settings :default_projects_modules => ['issue_tracking', 'repository'] do
       project = Project.new
-      
+
       project.enabled_module_names = %w(issue_tracking news)
       assert_equal %w(issue_tracking news), project.enabled_module_names.sort
     end
@@ -667,7 +665,7 @@ class ProjectTest < ActiveSupport::TestCase
     assert copied_project.id.blank?
     assert copied_project.name.blank?
     assert copied_project.identifier.blank?
-    
+
     # Duplicated attributes
     assert_equal source_project.description, copied_project.description
     assert_equal source_project.enabled_modules, copied_project.enabled_modules
@@ -745,11 +743,11 @@ class ProjectTest < ActiveSupport::TestCase
     assert system_activity.active?
     overridden_activity = TimeEntryActivity.generate!(:project => project, :parent => system_activity, :active => false)
     assert overridden_activity.save!
-    
+
     assert !project.activities.include?(overridden_activity), "Inactive Project specific Activity not found"
     assert !project.activities.include?(system_activity), "System activity found when the project has an inactive override"
   end
-  
+
   def test_close_completed_versions
     Version.update_all("status = 'open'")
     project = Project.find(1)
@@ -787,7 +785,7 @@ class ProjectTest < ActiveSupport::TestCase
         assert ! issue.assigned_to.blank?
         assert_equal @project, issue.project
       end
-      
+
       copied_issue = @project.issues.first(:conditions => {:subject => "copy issue status"})
       assert copied_issue
       assert copied_issue.status
@@ -804,7 +802,7 @@ class ProjectTest < ActiveSupport::TestCase
                                   :subject => "change the new issues to use the copied version",
                                   :tracker_id => 1,
                                   :project_id => @source_project.id)
-      
+
       assert @project.copy(@source_project)
       @project.reload
       copied_issue = @project.issues.first(:conditions => {:subject => "change the new issues to use the copied version"})
@@ -861,10 +859,10 @@ class ProjectTest < ActiveSupport::TestCase
         assert_equal @project, membership.project
       end
     end
-    
+
     should "copy memberships with groups and additional roles" do
       group = Group.create!(:lastname => "Copy group")
-      user = User.find(7) 
+      user = User.find(7)
       group.users << user
       # group role
       Member.create!(:project_id => @source_project.id, :principal => group, :role_ids => [2])
@@ -918,7 +916,7 @@ class ProjectTest < ActiveSupport::TestCase
       assert_difference 'WikiPage.count', @source_project.wiki.pages.size do
         assert @project.copy(@source_project)
       end
-      
+
       assert @project.wiki
       assert_equal @source_project.wiki.pages.size, @project.wiki.pages.size
 
@@ -926,7 +924,7 @@ class ProjectTest < ActiveSupport::TestCase
         assert wiki_page.content
         assert !@source_project.wiki.pages.include?(wiki_page)
       end
-      
+
       parent = @project.wiki.find_page('Parent_page')
       child1 = @project.wiki.find_page('Child_page_1')
       child2 = @project.wiki.find_page('Child_page_2')
@@ -964,19 +962,19 @@ class ProjectTest < ActiveSupport::TestCase
         assert_not_equal IssueCategory.find(3), issue.category # Different record
       end
     end
-    
+
     should "limit copy with :only option" do
       assert @project.members.empty?
       assert @project.issue_categories.empty?
       assert @source_project.issues.any?
-    
+
       assert @project.copy(@source_project, :only => ['members', 'issue_categories'])
 
       assert @project.members.any?
       assert @project.issue_categories.any?
       assert @project.issues.empty?
     end
-    
+
   end
 
   context "#start_date" do
@@ -985,11 +983,11 @@ class ProjectTest < ActiveSupport::TestCase
       @project = Project.generate!(:identifier => 'test0')
       @project.trackers << Tracker.generate!
     end
-    
+
     should "be nil if there are no issues on the project" do
       assert_nil @project.start_date
     end
-    
+
     should "be tested when issues have no start date"
 
     should "be the earliest start date of it's issues" do
@@ -1008,11 +1006,11 @@ class ProjectTest < ActiveSupport::TestCase
       @project = Project.generate!(:identifier => 'test0')
       @project.trackers << Tracker.generate!
     end
-    
+
     should "be nil if there are no issues on the project" do
       assert_nil @project.due_date
     end
-    
+
     should "be tested when issues have no due date"
 
     should "be the latest due date of it's issues" do
@@ -1027,7 +1025,7 @@ class ProjectTest < ActiveSupport::TestCase
       future = 7.days.from_now.to_date
       @project.versions << Version.generate!(:effective_date => future)
       @project.versions << Version.generate!(:effective_date => Date.today)
-      
+
 
       assert_equal future, @project.due_date
 
@@ -1038,7 +1036,7 @@ class ProjectTest < ActiveSupport::TestCase
       far_future = 14.days.from_now.to_date
       Issue.generate_for_project!(@project, :due_date => far_future)
       @project.versions << Version.generate!(:effective_date => future)
-      
+
       assert_equal far_future, @project.due_date
     end
 
@@ -1090,7 +1088,7 @@ class ProjectTest < ActiveSupport::TestCase
     setup do
       @project = Project.generate!
       @role = Role.generate!
-      
+
       @user_with_membership_notification = User.generate!(:mail_notification => 'selected')
       Member.generate!(:project => @project, :roles => [@role], :principal => @user_with_membership_notification, :mail_notification => true)
 
@@ -1109,30 +1107,30 @@ class ProjectTest < ActiveSupport::TestCase
       @only_owned_user = User.generate!(:mail_notification => 'only_owner')
       Member.generate!(:project => @project, :roles => [@role], :principal => @only_owned_user)
     end
-    
+
     should "include members with a mail notification" do
       assert @project.notified_users.include?(@user_with_membership_notification)
     end
-    
+
     should "include users with the 'all' notification option" do
       assert @project.notified_users.include?(@all_events_user)
     end
-    
+
     should "not include users with the 'none' notification option" do
       assert !@project.notified_users.include?(@no_events_user)
     end
-    
+
     should "not include users with the 'only_my_events' notification option" do
       assert !@project.notified_users.include?(@only_my_events_user)
     end
-    
+
     should "not include users with the 'only_assigned' notification option" do
       assert !@project.notified_users.include?(@only_assigned_user)
     end
-    
+
     should "not include users with the 'only_owner' notification option" do
       assert !@project.notified_users.include?(@only_owned_user)
     end
   end
-  
+
 end