Skip to content

Commit 3110f86

Browse files
committed
Add issue status check tests
1 parent eb703a0 commit 3110f86

File tree

3 files changed

+54
-7
lines changed

3 files changed

+54
-7
lines changed

test/integration/text_blocks_admin_test.rb

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
require_relative '../test_helper'
22

33
class TextBlocksAdminTest < Redmine::IntegrationTest
4-
fixtures :users, :email_addresses, :user_preferences
4+
fixtures :users, :email_addresses, :user_preferences, :issue_statuses
55

66
def setup
77
super
@@ -23,14 +23,17 @@ def test_textblock_crud
2323
assert_response :success
2424

2525
assert_difference 'TextBlock.count' do
26-
post '/text_blocks', params: { text_block: { name: 'test', text: 'lorem ipsum'}}
26+
post '/text_blocks', params: {
27+
text_block: { name: 'test', text: 'lorem ipsum', issue_status_ids: [1, 2] }
28+
}
2729
end
2830
assert_redirected_to '/text_blocks'
2931

3032
follow_redirect!
3133

3234
assert b = TextBlock.find_by_name('test')
3335
assert_equal 'lorem ipsum', b.text
36+
assert_equal [1, 2], b.issue_statuses.map(&:id).sort
3437
assert_equal 1, b.position
3538

3639
get "/text_blocks/#{b.id}/edit"
@@ -42,14 +45,17 @@ def test_textblock_crud
4245
assert_equal 'new', b.name
4346

4447
assert_difference 'TextBlock.count' do
45-
post '/text_blocks', params: { text_block: { name: 'test2', text: 'lorem ipsum2'}}
48+
post '/text_blocks', params: {
49+
text_block: { name: 'test2', text: 'lorem ipsum2', issue_status_ids: [1, 2] }
50+
}
4651
end
4752
assert_redirected_to '/text_blocks'
4853

4954
follow_redirect!
5055

5156
assert b = TextBlock.find_by_name('test2')
5257
assert_equal 'lorem ipsum2', b.text
58+
assert_equal [1, 2], b.issue_statuses.map(&:id).sort
5359
assert_equal 2, b.position
5460

5561
assert_difference 'TextBlock.count', -1 do

test/integration/text_blocks_project_test.rb

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
class TextBlocksProjectTest < Redmine::IntegrationTest
44
fixtures :users, :email_addresses, :user_preferences,
5-
:roles, :projects, :members, :member_roles
5+
:roles, :projects, :members, :member_roles, :issue_statuses
66

77
def setup
88
super
@@ -37,14 +37,17 @@ def test_textblock_crud
3737
assert_response :success
3838

3939
assert_difference 'TextBlock.count' do
40-
post '/projects/ecookbook/text_blocks', params: { text_block: { name: 'test', text: 'lorem ipsum'}}
40+
post '/projects/ecookbook/text_blocks', params: {
41+
text_block: { name: 'test', text: 'lorem ipsum', issue_status_ids: [1, 2] }
42+
}
4143
end
4244
assert_redirected_to '/projects/ecookbook/settings/text_blocks'
4345

4446
follow_redirect!
4547

4648
assert b = TextBlock.find_by_name('test')
4749
assert_equal 'lorem ipsum', b.text
50+
assert_equal [1, 2], b.issue_statuses.map(&:id).sort
4851
assert_equal 1, b.position
4952

5053
get "/projects/ecookbook/text_blocks/#{b.id}/edit"
@@ -56,14 +59,17 @@ def test_textblock_crud
5659
assert_equal 'new', b.name
5760

5861
assert_difference 'TextBlock.count' do
59-
post '/projects/ecookbook/text_blocks', params: { text_block: { name: 'test2', text: 'lorem ipsum2'}}
62+
post '/projects/ecookbook/text_blocks', params: {
63+
text_block: { name: 'test2', text: 'lorem ipsum2', issue_status_ids: [1, 2] }
64+
}
6065
end
6166
assert_redirected_to '/projects/ecookbook/settings/text_blocks'
6267

6368
follow_redirect!
6469

6570
assert b = TextBlock.find_by_name('test2')
6671
assert_equal 'lorem ipsum2', b.text
72+
assert_equal [1, 2], b.issue_statuses.map(&:id).sort
6773
assert_equal 2, b.position
6874

6975
assert_difference 'TextBlock.count', -1 do

test/unit/text_block_test.rb

Lines changed: 36 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
require_relative '../test_helper'
22

33
class TextBlockTest < ActiveSupport::TestCase
4-
fixtures :projects
4+
fixtures :projects, :issue_statuses
55

66
setup do
77
@project = Project.find 'ecookbook'
@@ -55,6 +55,41 @@ class TextBlockTest < ActiveSupport::TestCase
5555
end
5656
end
5757

58+
test 'should save params in global text block' do
59+
assert_difference 'TextBlock.count' do
60+
r = RedmineTextBlocks::SaveTextBlock.(
61+
{
62+
name: 'test',
63+
text: 'lorem ipsum',
64+
issue_status_ids: [1, 2]
65+
}
66+
)
67+
assert r.text_block_saved?
68+
assert_equal 'test', r.text_block.name
69+
assert_equal 'lorem ipsum', r.text_block.text
70+
assert_equal [1, 2], r.text_block.issue_status_ids.sort
71+
assert_equal 1, r.text_block.position
72+
end
73+
end
74+
75+
test 'should save params in local text block' do
76+
assert_difference 'TextBlock.count' do
77+
r = RedmineTextBlocks::SaveTextBlock.(
78+
{
79+
name: 'test',
80+
text: 'lorem ipsum',
81+
issue_status_ids: [1, 2]
82+
},
83+
project: @project
84+
)
85+
assert r.text_block_saved?
86+
assert_equal 'test', r.text_block.name
87+
assert_equal 'lorem ipsum', r.text_block.text
88+
assert_equal [1, 2], r.text_block.issue_status_ids.sort
89+
assert_equal 1, r.text_block.position
90+
end
91+
end
92+
5893
test 'deletion of project should delete textblocks' do
5994
RedmineTextBlocks::SaveTextBlock.({name: 'test'}, project: @project)
6095
assert_difference 'TextBlock.count', -1 do

0 commit comments

Comments
 (0)