| @@ -48,10 +48,18 class SysController < ActionController::Base | |||
|
|
48 | 48 | |
|
|
49 | 49 | def fetch_changesets |
|
|
50 | 50 | projects = [] |
|
|
51 | scope = Project.active.has_module(:repository) | |
|
|
51 | 52 | if params[:id] |
|
|
52 | projects << Project.active.has_module(:repository).find(params[:id]) | |
|
|
53 | project = nil | |
|
|
54 | if params[:id].to_s =~ /^\d*$/ | |
|
|
55 | project = scope.find(params[:id]) | |
|
|
53 | 56 | else |
|
|
54 | projects = Project.active.has_module(:repository).all | |
|
|
57 | project = scope.find_by_identifier(params[:id]) | |
|
|
58 | end | |
|
|
59 | raise ActiveRecord::RecordNotFound unless project | |
|
|
60 | projects << project | |
|
|
61 | else | |
|
|
62 | projects = scope.all | |
|
|
55 | 63 | end |
|
|
56 | 64 | projects.each do |project| |
|
|
57 | 65 | project.repositories.each do |repository| |
| @@ -89,12 +89,18 class SysControllerTest < ActionController::TestCase | |||
|
|
89 | 89 | assert_response :success |
|
|
90 | 90 | end |
|
|
91 | 91 | |
|
|
92 | def test_fetch_changesets_one_project | |
|
|
92 | def test_fetch_changesets_one_project_by_identifier | |
|
|
93 | 93 | Repository::Subversion.any_instance.expects(:fetch_changesets).once.returns(true) |
|
|
94 | 94 | get :fetch_changesets, :id => 'ecookbook' |
|
|
95 | 95 | assert_response :success |
|
|
96 | 96 | end |
|
|
97 | 97 | |
|
|
98 | def test_fetch_changesets_one_project_by_id | |
|
|
99 | Repository::Subversion.any_instance.expects(:fetch_changesets).once.returns(true) | |
|
|
100 | get :fetch_changesets, :id => '1' | |
|
|
101 | assert_response :success | |
|
|
102 | end | |
|
|
103 | ||
|
|
98 | 104 | def test_fetch_changesets_unknown_project |
|
|
99 | 105 | get :fetch_changesets, :id => 'unknown' |
|
|
100 | 106 | assert_response 404 |
General Comments 0
You need to be logged in to leave comments.
Login now