Refactored duplicated test code to a setup.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2932 e93f8b46-1217-0410-a6f0-8f06a7374b81

Refactored duplicated test code to a setup.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2932 e93f8b46-1217-0410-a6f0-8f06a7374b81
945ea9b0 · Eric Davis · 548d5a21 · 945ea9b0
Commit 945ea9b0 authored Oct 19, 2009 by Eric Davis
Hide whitespace changes
Inline Side-by-side

Showing with 26 additions and 42 deletions

project_test.rb test/unit/project_test.rb +26 -42

No files found.
--- a/test/unit/project_test.rb
+++ b/test/unit/project_test.rb
@@ -291,65 +291,49 @@ class ProjectTest < ActiveSupport::TestCase
  end

  context "#copy" do
-
-    should "copy issues" do
-      # Setup
+    setup do
      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)
+      @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
+    end
+
+    should "copy issues" do
+      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_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
+        assert_equal @project, issue.project
      end
    end

    should "copy members" do
-      # 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)
+      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_equal @source_project.members.size, @project.members.size
+      @project.members.each do |member|
        assert member
-        assert_equal project, member.project
+        assert_equal @project, member.project
      end
    end

    should "copy project specific queries" do
-      # 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)
+      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_equal @source_project.queries.size, @project.queries.size
+      @project.queries.each do |query|
        assert query
-        assert_equal project, query.project
+        assert_equal @project, query.project
      end
    end