Chris@441
|
1 # Redmine - project management software
|
Chris@1494
|
2 # Copyright (C) 2006-2014 Jean-Philippe Lang
|
Chris@0
|
3 #
|
Chris@0
|
4 # This program is free software; you can redistribute it and/or
|
Chris@0
|
5 # modify it under the terms of the GNU General Public License
|
Chris@0
|
6 # as published by the Free Software Foundation; either version 2
|
Chris@0
|
7 # of the License, or (at your option) any later version.
|
Chris@441
|
8 #
|
Chris@0
|
9 # This program is distributed in the hope that it will be useful,
|
Chris@0
|
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
|
Chris@0
|
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
Chris@0
|
12 # GNU General Public License for more details.
|
Chris@441
|
13 #
|
Chris@0
|
14 # You should have received a copy of the GNU General Public License
|
Chris@0
|
15 # along with this program; if not, write to the Free Software
|
Chris@0
|
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
Chris@0
|
17
|
Chris@119
|
18 require File.expand_path('../../test_helper', __FILE__)
|
Chris@0
|
19
|
Chris@0
|
20 class JournalTest < ActiveSupport::TestCase
|
Chris@1115
|
21 fixtures :projects, :issues, :issue_statuses, :journals, :journal_details,
|
Chris@1115
|
22 :users, :members, :member_roles, :roles, :enabled_modules,
|
Chris@1115
|
23 :projects_trackers, :trackers
|
Chris@0
|
24
|
Chris@0
|
25 def setup
|
Chris@0
|
26 @journal = Journal.find 1
|
Chris@0
|
27 end
|
Chris@0
|
28
|
Chris@0
|
29 def test_journalized_is_an_issue
|
Chris@0
|
30 issue = @journal.issue
|
Chris@0
|
31 assert_kind_of Issue, issue
|
Chris@0
|
32 assert_equal 1, issue.id
|
Chris@0
|
33 end
|
Chris@0
|
34
|
Chris@0
|
35 def test_new_status
|
Chris@0
|
36 status = @journal.new_status
|
Chris@0
|
37 assert_not_nil status
|
Chris@0
|
38 assert_kind_of IssueStatus, status
|
Chris@441
|
39 assert_equal 2, status.id
|
Chris@0
|
40 end
|
Chris@441
|
41
|
Chris@0
|
42 def test_create_should_send_email_notification
|
Chris@0
|
43 ActionMailer::Base.deliveries.clear
|
Chris@1464
|
44 issue = Issue.first
|
Chris@1464
|
45 user = User.first
|
Chris@0
|
46 journal = issue.init_journal(user, issue)
|
Chris@0
|
47
|
Chris@0
|
48 assert journal.save
|
Chris@0
|
49 assert_equal 1, ActionMailer::Base.deliveries.size
|
Chris@0
|
50 end
|
Chris@0
|
51
|
Chris@1115
|
52 def test_should_not_save_journal_with_blank_notes_and_no_details
|
Chris@1115
|
53 journal = Journal.new(:journalized => Issue.first, :user => User.first)
|
Chris@1115
|
54
|
Chris@1115
|
55 assert_no_difference 'Journal.count' do
|
Chris@1115
|
56 assert_equal false, journal.save
|
Chris@1115
|
57 end
|
Chris@1115
|
58 end
|
Chris@1115
|
59
|
Chris@1115
|
60 def test_create_should_not_split_non_private_notes
|
Chris@1115
|
61 assert_difference 'Journal.count' do
|
Chris@1115
|
62 assert_no_difference 'JournalDetail.count' do
|
Chris@1115
|
63 journal = Journal.generate!(:notes => 'Notes')
|
Chris@1115
|
64 end
|
Chris@1115
|
65 end
|
Chris@1115
|
66
|
Chris@1115
|
67 assert_difference 'Journal.count' do
|
Chris@1115
|
68 assert_difference 'JournalDetail.count' do
|
Chris@1115
|
69 journal = Journal.generate!(:notes => 'Notes', :details => [JournalDetail.new])
|
Chris@1115
|
70 end
|
Chris@1115
|
71 end
|
Chris@1115
|
72
|
Chris@1115
|
73 assert_difference 'Journal.count' do
|
Chris@1115
|
74 assert_difference 'JournalDetail.count' do
|
Chris@1115
|
75 journal = Journal.generate!(:notes => '', :details => [JournalDetail.new])
|
Chris@1115
|
76 end
|
Chris@1115
|
77 end
|
Chris@1115
|
78 end
|
Chris@1115
|
79
|
Chris@1115
|
80 def test_create_should_split_private_notes
|
Chris@1115
|
81 assert_difference 'Journal.count' do
|
Chris@1115
|
82 assert_no_difference 'JournalDetail.count' do
|
Chris@1115
|
83 journal = Journal.generate!(:notes => 'Notes', :private_notes => true)
|
Chris@1115
|
84 journal.reload
|
Chris@1115
|
85 assert_equal true, journal.private_notes
|
Chris@1115
|
86 assert_equal 'Notes', journal.notes
|
Chris@1115
|
87 end
|
Chris@1115
|
88 end
|
Chris@1115
|
89
|
Chris@1115
|
90 assert_difference 'Journal.count', 2 do
|
Chris@1115
|
91 assert_difference 'JournalDetail.count' do
|
Chris@1115
|
92 journal = Journal.generate!(:notes => 'Notes', :private_notes => true, :details => [JournalDetail.new])
|
Chris@1115
|
93 journal.reload
|
Chris@1115
|
94 assert_equal true, journal.private_notes
|
Chris@1115
|
95 assert_equal 'Notes', journal.notes
|
Chris@1115
|
96 assert_equal 0, journal.details.size
|
Chris@1115
|
97
|
Chris@1115
|
98 journal_with_changes = Journal.order('id DESC').offset(1).first
|
Chris@1115
|
99 assert_equal false, journal_with_changes.private_notes
|
Chris@1115
|
100 assert_nil journal_with_changes.notes
|
Chris@1115
|
101 assert_equal 1, journal_with_changes.details.size
|
Chris@1115
|
102 assert_equal journal.created_on, journal_with_changes.created_on
|
Chris@1115
|
103 end
|
Chris@1115
|
104 end
|
Chris@1115
|
105
|
Chris@1115
|
106 assert_difference 'Journal.count' do
|
Chris@1115
|
107 assert_difference 'JournalDetail.count' do
|
Chris@1115
|
108 journal = Journal.generate!(:notes => '', :private_notes => true, :details => [JournalDetail.new])
|
Chris@1115
|
109 journal.reload
|
Chris@1115
|
110 assert_equal false, journal.private_notes
|
Chris@1115
|
111 assert_equal '', journal.notes
|
Chris@1115
|
112 assert_equal 1, journal.details.size
|
Chris@1115
|
113 end
|
Chris@1115
|
114 end
|
Chris@1115
|
115 end
|
Chris@1115
|
116
|
Chris@441
|
117 def test_visible_scope_for_anonymous
|
Chris@441
|
118 # Anonymous user should see issues of public projects only
|
Chris@441
|
119 journals = Journal.visible(User.anonymous).all
|
Chris@441
|
120 assert journals.any?
|
Chris@441
|
121 assert_nil journals.detect {|journal| !journal.issue.project.is_public?}
|
Chris@441
|
122 # Anonymous user should not see issues without permission
|
Chris@441
|
123 Role.anonymous.remove_permission!(:view_issues)
|
Chris@441
|
124 journals = Journal.visible(User.anonymous).all
|
Chris@441
|
125 assert journals.empty?
|
Chris@441
|
126 end
|
Chris@441
|
127
|
Chris@441
|
128 def test_visible_scope_for_user
|
Chris@441
|
129 user = User.find(9)
|
Chris@441
|
130 assert user.projects.empty?
|
Chris@441
|
131 # Non member user should see issues of public projects only
|
Chris@441
|
132 journals = Journal.visible(user).all
|
Chris@441
|
133 assert journals.any?
|
Chris@441
|
134 assert_nil journals.detect {|journal| !journal.issue.project.is_public?}
|
Chris@441
|
135 # Non member user should not see issues without permission
|
Chris@441
|
136 Role.non_member.remove_permission!(:view_issues)
|
Chris@441
|
137 user.reload
|
Chris@441
|
138 journals = Journal.visible(user).all
|
Chris@441
|
139 assert journals.empty?
|
Chris@1464
|
140 # User should see issues of projects for which user has view_issues permissions only
|
Chris@441
|
141 Member.create!(:principal => user, :project_id => 1, :role_ids => [1])
|
Chris@441
|
142 user.reload
|
Chris@441
|
143 journals = Journal.visible(user).all
|
Chris@441
|
144 assert journals.any?
|
Chris@441
|
145 assert_nil journals.detect {|journal| journal.issue.project_id != 1}
|
Chris@441
|
146 end
|
Chris@441
|
147
|
Chris@441
|
148 def test_visible_scope_for_admin
|
Chris@441
|
149 user = User.find(1)
|
Chris@441
|
150 user.members.each(&:destroy)
|
Chris@441
|
151 assert user.projects.empty?
|
Chris@441
|
152 journals = Journal.visible(user).all
|
Chris@441
|
153 assert journals.any?
|
Chris@1464
|
154 # Admin should see issues on private projects that admin does not belong to
|
Chris@441
|
155 assert journals.detect {|journal| !journal.issue.project.is_public?}
|
Chris@441
|
156 end
|
Chris@1464
|
157
|
Chris@1464
|
158 def test_preload_journals_details_custom_fields_should_set_custom_field_instance_variable
|
Chris@1464
|
159 d = JournalDetail.new(:property => 'cf', :prop_key => '2')
|
Chris@1464
|
160 journals = [Journal.new(:details => [d])]
|
Chris@1464
|
161
|
Chris@1464
|
162 d.expects(:instance_variable_set).with("@custom_field", CustomField.find(2)).once
|
Chris@1464
|
163 Journal.preload_journals_details_custom_fields(journals)
|
Chris@1464
|
164 end
|
Chris@1464
|
165
|
Chris@1464
|
166 def test_preload_journals_details_custom_fields_with_empty_set
|
Chris@1464
|
167 assert_nothing_raised do
|
Chris@1464
|
168 Journal.preload_journals_details_custom_fields([])
|
Chris@1464
|
169 end
|
Chris@1464
|
170 end
|
Chris@1464
|
171
|
Chris@1464
|
172 def test_details_should_normalize_dates
|
Chris@1464
|
173 j = JournalDetail.create!(:old_value => Date.parse('2012-11-03'), :value => Date.parse('2013-01-02'))
|
Chris@1464
|
174 j.reload
|
Chris@1464
|
175 assert_equal '2012-11-03', j.old_value
|
Chris@1464
|
176 assert_equal '2013-01-02', j.value
|
Chris@1464
|
177 end
|
Chris@1464
|
178
|
Chris@1464
|
179 def test_details_should_normalize_true_values
|
Chris@1464
|
180 j = JournalDetail.create!(:old_value => true, :value => true)
|
Chris@1464
|
181 j.reload
|
Chris@1464
|
182 assert_equal '1', j.old_value
|
Chris@1464
|
183 assert_equal '1', j.value
|
Chris@1464
|
184 end
|
Chris@1464
|
185
|
Chris@1464
|
186 def test_details_should_normalize_false_values
|
Chris@1464
|
187 j = JournalDetail.create!(:old_value => false, :value => false)
|
Chris@1464
|
188 j.reload
|
Chris@1464
|
189 assert_equal '0', j.old_value
|
Chris@1464
|
190 assert_equal '0', j.value
|
Chris@1464
|
191 end
|
Chris@1464
|
192
|
Chris@1464
|
193 def test_custom_field_should_return_custom_field_for_cf_detail
|
Chris@1464
|
194 d = JournalDetail.new(:property => 'cf', :prop_key => '2')
|
Chris@1464
|
195 assert_equal CustomField.find(2), d.custom_field
|
Chris@1464
|
196 end
|
Chris@1464
|
197
|
Chris@1464
|
198 def test_custom_field_should_return_nil_for_non_cf_detail
|
Chris@1464
|
199 d = JournalDetail.new(:property => 'subject')
|
Chris@1464
|
200 assert_equal nil, d.custom_field
|
Chris@1464
|
201 end
|
Chris@1464
|
202
|
Chris@1464
|
203 def test_visible_details_should_include_relations_to_visible_issues_only
|
Chris@1464
|
204 issue = Issue.generate!
|
Chris@1464
|
205 visible_issue = Issue.generate!
|
Chris@1464
|
206 IssueRelation.create!(:issue_from => issue, :issue_to => visible_issue, :relation_type => 'relates')
|
Chris@1464
|
207 hidden_issue = Issue.generate!(:is_private => true)
|
Chris@1464
|
208 IssueRelation.create!(:issue_from => issue, :issue_to => hidden_issue, :relation_type => 'relates')
|
Chris@1464
|
209 issue.reload
|
Chris@1464
|
210 assert_equal 1, issue.journals.size
|
Chris@1464
|
211 journal = issue.journals.first
|
Chris@1464
|
212 assert_equal 2, journal.details.size
|
Chris@1464
|
213
|
Chris@1464
|
214 visible_details = journal.visible_details(User.anonymous)
|
Chris@1464
|
215 assert_equal 1, visible_details.size
|
Chris@1464
|
216 assert_equal visible_issue.id.to_s, visible_details.first.value
|
Chris@1464
|
217
|
Chris@1464
|
218 visible_details = journal.visible_details(User.find(2))
|
Chris@1464
|
219 assert_equal 2, visible_details.size
|
Chris@1464
|
220 end
|
Chris@0
|
221 end
|