##// END OF EJS Templates
Use :joins instead of :include....
Jean-Philippe Lang -
r8956:c30a6dffe051
parent child
Show More
@@ -1,321 +1,321
1 # Redmine - project management software
1 # Redmine - project management software
2 # Copyright (C) 2006-2011 Jean-Philippe Lang
2 # Copyright (C) 2006-2011 Jean-Philippe Lang
3 #
3 #
4 # This program is free software; you can redistribute it and/or
4 # This program is free software; you can redistribute it and/or
5 # modify it under the terms of the GNU General Public License
5 # modify it under the terms of the GNU General Public License
6 # as published by the Free Software Foundation; either version 2
6 # as published by the Free Software Foundation; either version 2
7 # of the License, or (at your option) any later version.
7 # of the License, or (at your option) any later version.
8 #
8 #
9 # This program is distributed in the hope that it will be useful,
9 # This program is distributed in the hope that it will be useful,
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 # GNU General Public License for more details.
12 # GNU General Public License for more details.
13 #
13 #
14 # You should have received a copy of the GNU General Public License
14 # You should have received a copy of the GNU General Public License
15 # along with this program; if not, write to the Free Software
15 # along with this program; if not, write to the Free Software
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17
17
18 require File.expand_path('../../test_helper', __FILE__)
18 require File.expand_path('../../test_helper', __FILE__)
19
19
20 class RepositoryTest < ActiveSupport::TestCase
20 class RepositoryTest < ActiveSupport::TestCase
21 fixtures :projects,
21 fixtures :projects,
22 :trackers,
22 :trackers,
23 :projects_trackers,
23 :projects_trackers,
24 :enabled_modules,
24 :enabled_modules,
25 :repositories,
25 :repositories,
26 :issues,
26 :issues,
27 :issue_statuses,
27 :issue_statuses,
28 :issue_categories,
28 :issue_categories,
29 :changesets,
29 :changesets,
30 :changes,
30 :changes,
31 :users,
31 :users,
32 :members,
32 :members,
33 :member_roles,
33 :member_roles,
34 :roles,
34 :roles,
35 :enumerations
35 :enumerations
36
36
37 include Redmine::I18n
37 include Redmine::I18n
38
38
39 def setup
39 def setup
40 @repository = Project.find(1).repository
40 @repository = Project.find(1).repository
41 end
41 end
42
42
43 def test_blank_log_encoding_error_message
43 def test_blank_log_encoding_error_message
44 set_language_if_valid 'en'
44 set_language_if_valid 'en'
45 repo = Repository::Bazaar.new(
45 repo = Repository::Bazaar.new(
46 :project => Project.find(3),
46 :project => Project.find(3),
47 :url => "/test",
47 :url => "/test",
48 :log_encoding => ''
48 :log_encoding => ''
49 )
49 )
50 assert !repo.save
50 assert !repo.save
51 assert_include "Commit messages encoding can't be blank",
51 assert_include "Commit messages encoding can't be blank",
52 repo.errors.full_messages
52 repo.errors.full_messages
53 end
53 end
54
54
55 def test_blank_log_encoding_error_message_fr
55 def test_blank_log_encoding_error_message_fr
56 set_language_if_valid 'fr'
56 set_language_if_valid 'fr'
57 str = "Encodage des messages de commit doit \xc3\xaatre renseign\xc3\xa9(e)"
57 str = "Encodage des messages de commit doit \xc3\xaatre renseign\xc3\xa9(e)"
58 str.force_encoding('UTF-8') if str.respond_to?(:force_encoding)
58 str.force_encoding('UTF-8') if str.respond_to?(:force_encoding)
59 repo = Repository::Bazaar.new(
59 repo = Repository::Bazaar.new(
60 :project => Project.find(3),
60 :project => Project.find(3),
61 :url => "/test"
61 :url => "/test"
62 )
62 )
63 assert !repo.save
63 assert !repo.save
64 assert_include str, repo.errors.full_messages
64 assert_include str, repo.errors.full_messages
65 end
65 end
66
66
67 def test_create
67 def test_create
68 repository = Repository::Subversion.new(:project => Project.find(3))
68 repository = Repository::Subversion.new(:project => Project.find(3))
69 assert !repository.save
69 assert !repository.save
70
70
71 repository.url = "svn://localhost"
71 repository.url = "svn://localhost"
72 assert repository.save
72 assert repository.save
73 repository.reload
73 repository.reload
74
74
75 project = Project.find(3)
75 project = Project.find(3)
76 assert_equal repository, project.repository
76 assert_equal repository, project.repository
77 end
77 end
78
78
79 def test_first_repository_should_be_set_as_default
79 def test_first_repository_should_be_set_as_default
80 repository1 = Repository::Subversion.new(
80 repository1 = Repository::Subversion.new(
81 :project => Project.find(3),
81 :project => Project.find(3),
82 :identifier => 'svn1',
82 :identifier => 'svn1',
83 :url => 'file:///svn1'
83 :url => 'file:///svn1'
84 )
84 )
85 assert repository1.save
85 assert repository1.save
86 assert repository1.is_default?
86 assert repository1.is_default?
87
87
88 repository2 = Repository::Subversion.new(
88 repository2 = Repository::Subversion.new(
89 :project => Project.find(3),
89 :project => Project.find(3),
90 :identifier => 'svn2',
90 :identifier => 'svn2',
91 :url => 'file:///svn2'
91 :url => 'file:///svn2'
92 )
92 )
93 assert repository2.save
93 assert repository2.save
94 assert !repository2.is_default?
94 assert !repository2.is_default?
95
95
96 assert_equal repository1, Project.find(3).repository
96 assert_equal repository1, Project.find(3).repository
97 assert_equal [repository1, repository2], Project.find(3).repositories.sort
97 assert_equal [repository1, repository2], Project.find(3).repositories.sort
98 end
98 end
99
99
100 def test_destroy
100 def test_destroy
101 changesets = Changeset.count(:all, :conditions => "repository_id = 10")
101 changesets = Changeset.count(:all, :conditions => "repository_id = 10")
102 changes = Change.count(:all, :conditions => "repository_id = 10",
102 changes = Change.count(:all, :conditions => "repository_id = 10",
103 :include => :changeset)
103 :joins => :changeset)
104 assert_difference 'Changeset.count', -changesets do
104 assert_difference 'Changeset.count', -changesets do
105 assert_difference 'Change.count', -changes do
105 assert_difference 'Change.count', -changes do
106 Repository.find(10).destroy
106 Repository.find(10).destroy
107 end
107 end
108 end
108 end
109 end
109 end
110
110
111 def test_destroy_should_delete_parents_associations
111 def test_destroy_should_delete_parents_associations
112 changeset = Changeset.find(102)
112 changeset = Changeset.find(102)
113 changeset.parents = Changeset.find_all_by_id([100, 101])
113 changeset.parents = Changeset.find_all_by_id([100, 101])
114
114
115 assert_difference 'Changeset.connection.select_all("select * from changeset_parents").size', -2 do
115 assert_difference 'Changeset.connection.select_all("select * from changeset_parents").size', -2 do
116 Repository.find(10).destroy
116 Repository.find(10).destroy
117 end
117 end
118 end
118 end
119
119
120 def test_destroy_should_delete_issues_associations
120 def test_destroy_should_delete_issues_associations
121 changeset = Changeset.find(102)
121 changeset = Changeset.find(102)
122 changeset.issues = Issue.find_all_by_id([1, 2])
122 changeset.issues = Issue.find_all_by_id([1, 2])
123
123
124 assert_difference 'Changeset.connection.select_all("select * from changesets_issues").size', -2 do
124 assert_difference 'Changeset.connection.select_all("select * from changesets_issues").size', -2 do
125 Repository.find(10).destroy
125 Repository.find(10).destroy
126 end
126 end
127 end
127 end
128
128
129 def test_should_not_create_with_disabled_scm
129 def test_should_not_create_with_disabled_scm
130 # disable Subversion
130 # disable Subversion
131 with_settings :enabled_scm => ['Darcs', 'Git'] do
131 with_settings :enabled_scm => ['Darcs', 'Git'] do
132 repository = Repository::Subversion.new(
132 repository = Repository::Subversion.new(
133 :project => Project.find(3), :url => "svn://localhost")
133 :project => Project.find(3), :url => "svn://localhost")
134 assert !repository.save
134 assert !repository.save
135 assert_equal I18n.translate('activerecord.errors.messages.invalid'),
135 assert_equal I18n.translate('activerecord.errors.messages.invalid'),
136 repository.errors[:type].to_s
136 repository.errors[:type].to_s
137 end
137 end
138 end
138 end
139
139
140 def test_scan_changesets_for_issue_ids
140 def test_scan_changesets_for_issue_ids
141 Setting.default_language = 'en'
141 Setting.default_language = 'en'
142 Setting.notified_events = ['issue_added','issue_updated']
142 Setting.notified_events = ['issue_added','issue_updated']
143
143
144 # choosing a status to apply to fix issues
144 # choosing a status to apply to fix issues
145 Setting.commit_fix_status_id = IssueStatus.find(
145 Setting.commit_fix_status_id = IssueStatus.find(
146 :first,
146 :first,
147 :conditions => ["is_closed = ?", true]).id
147 :conditions => ["is_closed = ?", true]).id
148 Setting.commit_fix_done_ratio = "90"
148 Setting.commit_fix_done_ratio = "90"
149 Setting.commit_ref_keywords = 'refs , references, IssueID'
149 Setting.commit_ref_keywords = 'refs , references, IssueID'
150 Setting.commit_fix_keywords = 'fixes , closes'
150 Setting.commit_fix_keywords = 'fixes , closes'
151 Setting.default_language = 'en'
151 Setting.default_language = 'en'
152 ActionMailer::Base.deliveries.clear
152 ActionMailer::Base.deliveries.clear
153
153
154 # make sure issue 1 is not already closed
154 # make sure issue 1 is not already closed
155 fixed_issue = Issue.find(1)
155 fixed_issue = Issue.find(1)
156 assert !fixed_issue.status.is_closed?
156 assert !fixed_issue.status.is_closed?
157 old_status = fixed_issue.status
157 old_status = fixed_issue.status
158
158
159 Repository.scan_changesets_for_issue_ids
159 Repository.scan_changesets_for_issue_ids
160 assert_equal [101, 102], Issue.find(3).changeset_ids
160 assert_equal [101, 102], Issue.find(3).changeset_ids
161
161
162 # fixed issues
162 # fixed issues
163 fixed_issue.reload
163 fixed_issue.reload
164 assert fixed_issue.status.is_closed?
164 assert fixed_issue.status.is_closed?
165 assert_equal 90, fixed_issue.done_ratio
165 assert_equal 90, fixed_issue.done_ratio
166 assert_equal [101], fixed_issue.changeset_ids
166 assert_equal [101], fixed_issue.changeset_ids
167
167
168 # issue change
168 # issue change
169 journal = fixed_issue.journals.find(:first, :order => 'created_on desc')
169 journal = fixed_issue.journals.find(:first, :order => 'created_on desc')
170 assert_equal User.find_by_login('dlopper'), journal.user
170 assert_equal User.find_by_login('dlopper'), journal.user
171 assert_equal 'Applied in changeset r2.', journal.notes
171 assert_equal 'Applied in changeset r2.', journal.notes
172
172
173 # 2 email notifications
173 # 2 email notifications
174 assert_equal 2, ActionMailer::Base.deliveries.size
174 assert_equal 2, ActionMailer::Base.deliveries.size
175 mail = ActionMailer::Base.deliveries.first
175 mail = ActionMailer::Base.deliveries.first
176 assert_not_nil mail
176 assert_not_nil mail
177 assert mail.subject.starts_with?(
177 assert mail.subject.starts_with?(
178 "[#{fixed_issue.project.name} - #{fixed_issue.tracker.name} ##{fixed_issue.id}]")
178 "[#{fixed_issue.project.name} - #{fixed_issue.tracker.name} ##{fixed_issue.id}]")
179 assert mail.body.include?(
179 assert mail.body.include?(
180 "Status changed from #{old_status} to #{fixed_issue.status}")
180 "Status changed from #{old_status} to #{fixed_issue.status}")
181
181
182 # ignoring commits referencing an issue of another project
182 # ignoring commits referencing an issue of another project
183 assert_equal [], Issue.find(4).changesets
183 assert_equal [], Issue.find(4).changesets
184 end
184 end
185
185
186 def test_for_changeset_comments_strip
186 def test_for_changeset_comments_strip
187 repository = Repository::Mercurial.create(
187 repository = Repository::Mercurial.create(
188 :project => Project.find( 4 ),
188 :project => Project.find( 4 ),
189 :url => '/foo/bar/baz' )
189 :url => '/foo/bar/baz' )
190 comment = <<-COMMENT
190 comment = <<-COMMENT
191 This is a loooooooooooooooooooooooooooong comment
191 This is a loooooooooooooooooooooooooooong comment
192
192
193
193
194 COMMENT
194 COMMENT
195 changeset = Changeset.new(
195 changeset = Changeset.new(
196 :comments => comment, :commit_date => Time.now,
196 :comments => comment, :commit_date => Time.now,
197 :revision => 0, :scmid => 'f39b7922fb3c',
197 :revision => 0, :scmid => 'f39b7922fb3c',
198 :committer => 'foo <foo@example.com>',
198 :committer => 'foo <foo@example.com>',
199 :committed_on => Time.now, :repository => repository )
199 :committed_on => Time.now, :repository => repository )
200 assert( changeset.save )
200 assert( changeset.save )
201 assert_not_equal( comment, changeset.comments )
201 assert_not_equal( comment, changeset.comments )
202 assert_equal( 'This is a loooooooooooooooooooooooooooong comment',
202 assert_equal( 'This is a loooooooooooooooooooooooooooong comment',
203 changeset.comments )
203 changeset.comments )
204 end
204 end
205
205
206 def test_for_urls_strip_cvs
206 def test_for_urls_strip_cvs
207 repository = Repository::Cvs.create(
207 repository = Repository::Cvs.create(
208 :project => Project.find(4),
208 :project => Project.find(4),
209 :url => ' :pserver:login:password@host:/path/to/the/repository',
209 :url => ' :pserver:login:password@host:/path/to/the/repository',
210 :root_url => 'foo ',
210 :root_url => 'foo ',
211 :log_encoding => 'UTF-8')
211 :log_encoding => 'UTF-8')
212 assert repository.save
212 assert repository.save
213 repository.reload
213 repository.reload
214 assert_equal ':pserver:login:password@host:/path/to/the/repository',
214 assert_equal ':pserver:login:password@host:/path/to/the/repository',
215 repository.url
215 repository.url
216 assert_equal 'foo', repository.root_url
216 assert_equal 'foo', repository.root_url
217 end
217 end
218
218
219 def test_for_urls_strip_subversion
219 def test_for_urls_strip_subversion
220 repository = Repository::Subversion.create(
220 repository = Repository::Subversion.create(
221 :project => Project.find(4),
221 :project => Project.find(4),
222 :url => ' file:///dummy ')
222 :url => ' file:///dummy ')
223 assert repository.save
223 assert repository.save
224 repository.reload
224 repository.reload
225 assert_equal 'file:///dummy', repository.url
225 assert_equal 'file:///dummy', repository.url
226 end
226 end
227
227
228 def test_for_urls_strip_git
228 def test_for_urls_strip_git
229 repository = Repository::Git.create(
229 repository = Repository::Git.create(
230 :project => Project.find(4),
230 :project => Project.find(4),
231 :url => ' c:\dummy ')
231 :url => ' c:\dummy ')
232 assert repository.save
232 assert repository.save
233 repository.reload
233 repository.reload
234 assert_equal 'c:\dummy', repository.url
234 assert_equal 'c:\dummy', repository.url
235 end
235 end
236
236
237 def test_manual_user_mapping
237 def test_manual_user_mapping
238 assert_no_difference "Changeset.count(:conditions => 'user_id <> 2')" do
238 assert_no_difference "Changeset.count(:conditions => 'user_id <> 2')" do
239 c = Changeset.create!(
239 c = Changeset.create!(
240 :repository => @repository,
240 :repository => @repository,
241 :committer => 'foo',
241 :committer => 'foo',
242 :committed_on => Time.now,
242 :committed_on => Time.now,
243 :revision => 100,
243 :revision => 100,
244 :comments => 'Committed by foo.'
244 :comments => 'Committed by foo.'
245 )
245 )
246 assert_nil c.user
246 assert_nil c.user
247 @repository.committer_ids = {'foo' => '2'}
247 @repository.committer_ids = {'foo' => '2'}
248 assert_equal User.find(2), c.reload.user
248 assert_equal User.find(2), c.reload.user
249 # committer is now mapped
249 # committer is now mapped
250 c = Changeset.create!(
250 c = Changeset.create!(
251 :repository => @repository,
251 :repository => @repository,
252 :committer => 'foo',
252 :committer => 'foo',
253 :committed_on => Time.now,
253 :committed_on => Time.now,
254 :revision => 101,
254 :revision => 101,
255 :comments => 'Another commit by foo.'
255 :comments => 'Another commit by foo.'
256 )
256 )
257 assert_equal User.find(2), c.user
257 assert_equal User.find(2), c.user
258 end
258 end
259 end
259 end
260
260
261 def test_auto_user_mapping_by_username
261 def test_auto_user_mapping_by_username
262 c = Changeset.create!(
262 c = Changeset.create!(
263 :repository => @repository,
263 :repository => @repository,
264 :committer => 'jsmith',
264 :committer => 'jsmith',
265 :committed_on => Time.now,
265 :committed_on => Time.now,
266 :revision => 100,
266 :revision => 100,
267 :comments => 'Committed by john.'
267 :comments => 'Committed by john.'
268 )
268 )
269 assert_equal User.find(2), c.user
269 assert_equal User.find(2), c.user
270 end
270 end
271
271
272 def test_auto_user_mapping_by_email
272 def test_auto_user_mapping_by_email
273 c = Changeset.create!(
273 c = Changeset.create!(
274 :repository => @repository,
274 :repository => @repository,
275 :committer => 'john <jsmith@somenet.foo>',
275 :committer => 'john <jsmith@somenet.foo>',
276 :committed_on => Time.now,
276 :committed_on => Time.now,
277 :revision => 100,
277 :revision => 100,
278 :comments => 'Committed by john.'
278 :comments => 'Committed by john.'
279 )
279 )
280 assert_equal User.find(2), c.user
280 assert_equal User.find(2), c.user
281 end
281 end
282
282
283 def test_filesystem_avaialbe
283 def test_filesystem_avaialbe
284 klass = Repository::Filesystem
284 klass = Repository::Filesystem
285 assert klass.scm_adapter_class
285 assert klass.scm_adapter_class
286 assert_equal true, klass.scm_available
286 assert_equal true, klass.scm_available
287 end
287 end
288
288
289 def test_merge_extra_info
289 def test_merge_extra_info
290 repo = Repository::Subversion.new(:project => Project.find(3))
290 repo = Repository::Subversion.new(:project => Project.find(3))
291 assert !repo.save
291 assert !repo.save
292 repo.url = "svn://localhost"
292 repo.url = "svn://localhost"
293 assert repo.save
293 assert repo.save
294 repo.reload
294 repo.reload
295 project = Project.find(3)
295 project = Project.find(3)
296 assert_equal repo, project.repository
296 assert_equal repo, project.repository
297 assert_nil repo.extra_info
297 assert_nil repo.extra_info
298 h1 = {"test_1" => {"test_11" => "test_value_11"}}
298 h1 = {"test_1" => {"test_11" => "test_value_11"}}
299 repo.merge_extra_info(h1)
299 repo.merge_extra_info(h1)
300 assert_equal h1, repo.extra_info
300 assert_equal h1, repo.extra_info
301 h2 = {"test_2" => {
301 h2 = {"test_2" => {
302 "test_21" => "test_value_21",
302 "test_21" => "test_value_21",
303 "test_22" => "test_value_22",
303 "test_22" => "test_value_22",
304 }}
304 }}
305 repo.merge_extra_info(h2)
305 repo.merge_extra_info(h2)
306 assert_equal (h = {"test_11" => "test_value_11"}),
306 assert_equal (h = {"test_11" => "test_value_11"}),
307 repo.extra_info["test_1"]
307 repo.extra_info["test_1"]
308 assert_equal "test_value_21",
308 assert_equal "test_value_21",
309 repo.extra_info["test_2"]["test_21"]
309 repo.extra_info["test_2"]["test_21"]
310 h3 = {"test_2" => {
310 h3 = {"test_2" => {
311 "test_23" => "test_value_23",
311 "test_23" => "test_value_23",
312 "test_24" => "test_value_24",
312 "test_24" => "test_value_24",
313 }}
313 }}
314 repo.merge_extra_info(h3)
314 repo.merge_extra_info(h3)
315 assert_equal (h = {"test_11" => "test_value_11"}),
315 assert_equal (h = {"test_11" => "test_value_11"}),
316 repo.extra_info["test_1"]
316 repo.extra_info["test_1"]
317 assert_nil repo.extra_info["test_2"]["test_21"]
317 assert_nil repo.extra_info["test_2"]["test_21"]
318 assert_equal "test_value_23",
318 assert_equal "test_value_23",
319 repo.extra_info["test_2"]["test_23"]
319 repo.extra_info["test_2"]["test_23"]
320 end
320 end
321 end
321 end
General Comments 0
You need to be logged in to leave comments. Login now