From 7dd7337d8e2f11ce3575a75b2f7d4a46f61a1dfc Mon Sep 17 00:00:00 2001 From: Gregor Schmidt Date: Tue, 26 Apr 2016 14:51:54 +0200 Subject: [PATCH 1/3] Extend test data for issue imported Adding two new rows to test issue hierarchies which are out of order within the CSV file --- test/fixtures/files/import_issues.csv | 2 ++ test/unit/issue_import_test.rb | 16 ++++++++-------- 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/test/fixtures/files/import_issues.csv b/test/fixtures/files/import_issues.csv index 918f9fc..49b2f04 100644 --- a/test/fixtures/files/import_issues.csv +++ b/test/fixtures/files/import_issues.csv @@ -2,3 +2,5 @@ priority;subject;description;start_date;due_date;parent;private;progress;custom; High;First;First description;2015-07-08;2015-08-25;;no;;PostgreSQL;;New category;dlopper;1 Normal;Child 1;Child description;;;1;yes;10;MySQL;2.0;New category;;2 Normal;Child of existing issue;Child description;;;#2;no;20;;2.1;Printing;;3 +;Child of following issue;;;;5;;;;;;; +;Parent of previous issue;;;;;;;;;;; diff --git a/test/unit/issue_import_test.rb b/test/unit/issue_import_test.rb index 61e1007..f30ccab 100644 --- a/test/unit/issue_import_test.rb +++ b/test/unit/issue_import_test.rb @@ -38,7 +38,7 @@ class IssueImportTest < ActiveSupport::TestCase import.save! version = new_record(Version) do - assert_difference 'Issue.count', 3 do + assert_difference 'Issue.count', 5 do import.run end end @@ -51,7 +51,7 @@ class IssueImportTest < ActiveSupport::TestCase import.save! category = new_record(IssueCategory) do - assert_difference 'Issue.count', 3 do + assert_difference 'Issue.count', 5 do import.run end end @@ -63,7 +63,7 @@ class IssueImportTest < ActiveSupport::TestCase import.mapping.merge!('parent_issue_id' => '5') import.save! - issues = new_records(Issue, 3) { import.run } + issues = new_records(Issue, 5) { import.run } assert_nil issues[0].parent assert_equal issues[0].id, issues[1].parent_id assert_equal 2, issues[2].parent_id @@ -74,8 +74,8 @@ class IssueImportTest < ActiveSupport::TestCase import.mapping.merge!('assigned_to' => '11') import.save! - issues = new_records(Issue, 3) { import.run } - assert_equal [User.find(3), nil, nil], issues.map(&:assigned_to) + issues = new_records(Issue, 5) { import.run } + assert_equal [User.find(3), nil, nil], issues[0..2].map(&:assigned_to) end def test_user_custom_field_should_be_set @@ -84,7 +84,7 @@ class IssueImportTest < ActiveSupport::TestCase import.mapping.merge!("cf_#{field.id}" => '11') import.save! - issues = new_records(Issue, 3) { import.run } + issues = new_records(Issue, 5) { import.run } assert_equal '3', issues.first.custom_field_value(field) end @@ -93,8 +93,8 @@ class IssueImportTest < ActiveSupport::TestCase import.mapping.merge!('is_private' => '6') import.save! - issues = new_records(Issue, 3) { import.run } - assert_equal [false, true, false], issues.map(&:is_private) + issues = new_records(Issue, 5) { import.run } + assert_equal [false, true, false], issues[0..2].map(&:is_private) end def test_dates_should_be_parsed_using_date_format_setting -- 2.8.0