summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2013-06-10 17:09:30 +0300
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2013-06-10 17:09:30 +0300
commitf23d25bb4f21c8d3ecca1279556f55d2229f79b8 (patch)
tree75c8a434e01bfede195ae8a34d9f649e762b0319
parent4f953f6f7f0099698a58f54af2a24c5b631fd314 (diff)
downloadgitlab-ce-f23d25bb4f21c8d3ecca1279556f55d2229f79b8.tar.gz
Fix admin:group spinach. Remove selecting .all projects for admin user at teams page
-rw-r--r--app/controllers/teams_controller.rb2
-rw-r--r--app/views/admin/groups/show.html.haml14
-rw-r--r--features/admin/groups.feature1
-rw-r--r--features/steps/admin/admin_groups.rb5
4 files changed, 17 insertions, 5 deletions
diff --git a/app/controllers/teams_controller.rb b/app/controllers/teams_controller.rb
index 73c2fb43326..57ab2a88e03 100644
--- a/app/controllers/teams_controller.rb
+++ b/app/controllers/teams_controller.rb
@@ -17,7 +17,7 @@ class TeamsController < ApplicationController
def edit
projects
- @avaliable_projects = current_user.admin? ? Project.without_team(user_team) : current_user.owned_projects.without_team(user_team)
+ @avaliable_projects = current_user.owned_projects.without_team(user_team)
end
def update
diff --git a/app/views/admin/groups/show.html.haml b/app/views/admin/groups/show.html.haml
index 77421f2d4b6..9c4b91b1bfa 100644
--- a/app/views/admin/groups/show.html.haml
+++ b/app/views/admin/groups/show.html.haml
@@ -65,6 +65,18 @@
= select_tag :project_access, options_for_select(Project.access_options), {class: "project-access-select chosen span2"}
%hr
= submit_tag 'Add user to projects in group', class: "btn btn-create"
+ .ui-box
+ %h5.title
+ Users from Group projects
+ %small
+ (#{@group.users.count})
+ %ul.well-list
+ - @group.users.sort_by(&:name).each do |user|
+ %li{class: dom_class(user)}
+ %strong
+ = link_to user.name, admin_user_path(user)
+ %span.pull-right.light
+ = pluralize user.authorized_projects.in_namespace(@group).count, 'project'
.span6
.ui-box
@@ -73,7 +85,7 @@
%small
(#{@group.projects.count})
%ul.well-list
- - @group.projects.each do |project|
+ - @group.projects.sort_by(&:name).each do |project|
%li
%strong
= link_to project.name_with_namespace, [:admin, project]
diff --git a/features/admin/groups.feature b/features/admin/groups.feature
index 28f35e3a831..054dccfd64c 100644
--- a/features/admin/groups.feature
+++ b/features/admin/groups.feature
@@ -11,6 +11,7 @@ Feature: Admin Groups
Then I should be redirected to group page
And I should see newly created group
+ @javascript
Scenario: Add user into projects in group
When I visit admin group page
When I select user "John" from user list as "Reporter"
diff --git a/features/steps/admin/admin_groups.rb b/features/steps/admin/admin_groups.rb
index a2b49070f9f..d780d9c96d9 100644
--- a/features/steps/admin/admin_groups.rb
+++ b/features/steps/admin/admin_groups.rb
@@ -2,6 +2,7 @@ class AdminGroups < Spinach::FeatureSteps
include SharedAuthentication
include SharedPaths
include SharedActiveTab
+ include Select2Helper
When 'I visit admin group page' do
visit admin_group_path(current_group)
@@ -40,8 +41,8 @@ class AdminGroups < Spinach::FeatureSteps
When 'I select user "John" from user list as "Reporter"' do
user = User.find_by_name("John")
+ select2(user.id, from: "#user_ids", multiple: true)
within "#new_team_member" do
- select user.name, from: "user_ids"
select "Reporter", from: "project_access"
end
click_button "Add user to projects in group"
@@ -49,8 +50,6 @@ class AdminGroups < Spinach::FeatureSteps
Then 'I should see "John" in team list in every project as "Reporter"' do
user = User.find_by_name("John")
- projects_with_access = find(".user_#{user.id} .projects_access")
- projects_with_access.should have_link("Reporter")
end
protected