From fa7bd1c71dca1e9c74e6d83277336321393dac9f Mon Sep 17 00:00:00 2001 From: Eric Davis Date: Sun, 3 May 2009 21:25:37 +0000 Subject: Added the ability to copy a project in the Project Administration panel. * Added Copy project button. * Added Project#copy_from to duplicate a project to be modified and saved by the user * Added a ProjectsController#copy based off the add method ** Used Project#copy_from to create a duplicate project in memory * Implemented Project#copy to copy data for a project from another and save it. ** Members ** Project level queries ** Project custom fields * Added a plugin hook for Project#copy. #1125 #1556 #886 #309 git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2704 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- test/fixtures/issues.yml | 4 +- test/fixtures/queries.yml | 30 +++++++++- test/functional/projects_controller_test.rb | 20 ++++++- test/unit/project_test.rb | 86 ++++++++++++++++++++++++++++- 4 files changed, 134 insertions(+), 6 deletions(-) (limited to 'test') diff --git a/test/fixtures/issues.yml b/test/fixtures/issues.yml index 921ba40c4..856f80289 100644 --- a/test/fixtures/issues.yml +++ b/test/fixtures/issues.yml @@ -58,7 +58,7 @@ issues_004: category_id: description: Issue on project 2 tracker_id: 1 - assigned_to_id: + assigned_to_id: 2 author_id: 2 status_id: 1 issues_005: @@ -125,4 +125,4 @@ issues_008: start_date: due_date: lock_version: 0 - \ No newline at end of file + diff --git a/test/fixtures/queries.yml b/test/fixtures/queries.yml index a1bb08eff..563bf583a 100644 --- a/test/fixtures/queries.yml +++ b/test/fixtures/queries.yml @@ -106,4 +106,32 @@ queries_006: --- - - priority - desc - \ No newline at end of file +queries_007: + id: 7 + project_id: 2 + is_public: true + name: Public query for project 2 + filters: | + --- + tracker_id: + :values: + - "3" + :operator: "=" + + user_id: 2 + column_names: +queries_008: + id: 8 + project_id: 2 + is_public: false + name: Private query for project 2 + filters: | + --- + tracker_id: + :values: + - "3" + :operator: "=" + + user_id: 2 + column_names: + diff --git a/test/functional/projects_controller_test.rb b/test/functional/projects_controller_test.rb index 1aded6429..4393ac075 100644 --- a/test/functional/projects_controller_test.rb +++ b/test/functional/projects_controller_test.rb @@ -453,7 +453,6 @@ class ProjectsControllerTest < Test::Unit::TestCase 6.times do |i| p = Project.create!(:name => "Breadcrumbs #{i}", :identifier => "breadcrumbs-#{i}") p.set_parent!(parent) - get :show, :id => p assert_tag :h1, :parent => { :attributes => {:id => 'header'}}, :children => { :count => [i, 3].min, @@ -462,7 +461,24 @@ class ProjectsControllerTest < Test::Unit::TestCase parent = p end end - + + def test_copy_with_project + @request.session[:user_id] = 1 # admin + get :copy, :id => 1 + assert_response :success + assert_template 'copy' + assert assigns(:project) + assert_equal Project.find(1).description, assigns(:project).description + assert_nil assigns(:project).id + end + + def test_copy_without_project + @request.session[:user_id] = 1 # admin + get :copy + assert_response :redirect + assert_redirected_to :controller => 'admin', :action => 'projects' + end + def test_jump_should_redirect_to_active_tab get :show, :id => 1, :jump => 'issues' assert_redirected_to 'projects/ecookbook/issues' diff --git a/test/unit/project_test.rb b/test/unit/project_test.rb index f579e14ff..f9a17e2ec 100644 --- a/test/unit/project_test.rb +++ b/test/unit/project_test.rb @@ -20,7 +20,8 @@ require File.dirname(__FILE__) + '/../test_helper' class ProjectTest < Test::Unit::TestCase fixtures :projects, :enabled_modules, :issues, :issue_statuses, :journals, :journal_details, - :users, :members, :roles, :projects_trackers, :trackers, :boards + :users, :members, :roles, :projects_trackers, :trackers, :boards, + :queries def setup @ecookbook = Project.find(1) @@ -221,6 +222,7 @@ class ProjectTest < Test::Unit::TestCase assert_nil Project.next_identifier end + def test_enabled_module_names_should_not_recreate_enabled_modules project = Project.find(1) # Remove one module @@ -233,4 +235,86 @@ class ProjectTest < Test::Unit::TestCase # Ids should be preserved assert_equal project.enabled_module_ids.sort, modules.collect(&:id).sort end + + def test_copy_from_existing_project + source_project = Project.find(1) + copied_project = Project.copy_from(1) + + assert copied_project + # Cleared attributes + 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 + assert_equal source_project.trackers, copied_project.trackers + + # Default attributes + assert_equal 1, copied_project.status + end + + # Context: Project#copy + def test_copy_should_copy_issues + # Setup + ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests + source_project = Project.find(2) + Project.destroy_all :identifier => "copy-test" + project = Project.new(:name => 'Copy Test', :identifier => 'copy-test') + project.trackers = source_project.trackers + assert project.valid? + + assert project.issues.empty? + assert project.copy(source_project) + + # Tests + assert_equal source_project.issues.size, project.issues.size + project.issues.each do |issue| + assert issue.valid? + assert ! issue.assigned_to.blank? + assert_equal project, issue.project + end + end + + def test_copy_should_copy_members + # Setup + ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests + source_project = Project.find(2) + project = Project.new(:name => 'Copy Test', :identifier => 'copy-test') + project.trackers = source_project.trackers + project.enabled_modules = source_project.enabled_modules + assert project.valid? + + assert project.members.empty? + assert project.copy(source_project) + + # Tests + assert_equal source_project.members.size, project.members.size + project.members.each do |member| + assert member + assert_equal project, member.project + end + end + + def test_copy_should_copy_project_level_queries + # Setup + ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests + source_project = Project.find(2) + project = Project.new(:name => 'Copy Test', :identifier => 'copy-test') + project.trackers = source_project.trackers + project.enabled_modules = source_project.enabled_modules + assert project.valid? + + assert project.queries.empty? + assert project.copy(source_project) + + # Tests + assert_equal source_project.queries.size, project.queries.size + project.queries.each do |query| + assert query + assert_equal project, query.project + end + end + end -- cgit v1.2.3