Add id group api sorting option
With introduction of subgroups, many groups may have the same name and/or path, thus further sorting options are necessary. Signed-off-by: Marko, Peter <peter.marko@siemens.com>
This commit is contained in:
parent
a8da4f3fa8
commit
79033a5f67
|
@ -0,0 +1,5 @@
|
||||||
|
---
|
||||||
|
title: Added id sorting option to GET groups and subgroups API
|
||||||
|
merge_request: 19665
|
||||||
|
author: Marko, Peter
|
||||||
|
type: added
|
|
@ -12,7 +12,7 @@ Parameters:
|
||||||
| `skip_groups` | array of integers | no | Skip the group IDs passed |
|
| `skip_groups` | array of integers | no | Skip the group IDs passed |
|
||||||
| `all_available` | boolean | no | Show all the groups you have access to (defaults to `false` for authenticated users, `true` for admin) |
|
| `all_available` | boolean | no | Show all the groups you have access to (defaults to `false` for authenticated users, `true` for admin) |
|
||||||
| `search` | string | no | Return the list of authorized groups matching the search criteria |
|
| `search` | string | no | Return the list of authorized groups matching the search criteria |
|
||||||
| `order_by` | string | no | Order groups by `name` or `path`. Default is `name` |
|
| `order_by` | string | no | Order groups by `name`, `path` or `id`. Default is `name` |
|
||||||
| `sort` | string | no | Order groups in `asc` or `desc` order. Default is `asc` |
|
| `sort` | string | no | Order groups in `asc` or `desc` order. Default is `asc` |
|
||||||
| `statistics` | boolean | no | Include group statistics (admins only) |
|
| `statistics` | boolean | no | Include group statistics (admins only) |
|
||||||
| `with_custom_attributes` | boolean | no | Include [custom attributes](custom_attributes.md) in response (admins only) |
|
| `with_custom_attributes` | boolean | no | Include [custom attributes](custom_attributes.md) in response (admins only) |
|
||||||
|
@ -96,7 +96,7 @@ Parameters:
|
||||||
| `skip_groups` | array of integers | no | Skip the group IDs passed |
|
| `skip_groups` | array of integers | no | Skip the group IDs passed |
|
||||||
| `all_available` | boolean | no | Show all the groups you have access to (defaults to `false` for authenticated users, `true` for admin) |
|
| `all_available` | boolean | no | Show all the groups you have access to (defaults to `false` for authenticated users, `true` for admin) |
|
||||||
| `search` | string | no | Return the list of authorized groups matching the search criteria |
|
| `search` | string | no | Return the list of authorized groups matching the search criteria |
|
||||||
| `order_by` | string | no | Order groups by `name` or `path`. Default is `name` |
|
| `order_by` | string | no | Order groups by `name`, `path` or `id`. Default is `name` |
|
||||||
| `sort` | string | no | Order groups in `asc` or `desc` order. Default is `asc` |
|
| `sort` | string | no | Order groups in `asc` or `desc` order. Default is `asc` |
|
||||||
| `statistics` | boolean | no | Include group statistics (admins only) |
|
| `statistics` | boolean | no | Include group statistics (admins only) |
|
||||||
| `with_custom_attributes` | boolean | no | Include [custom attributes](custom_attributes.md) in response (admins only) |
|
| `with_custom_attributes` | boolean | no | Include [custom attributes](custom_attributes.md) in response (admins only) |
|
||||||
|
|
|
@ -32,7 +32,7 @@ module API
|
||||||
optional :all_available, type: Boolean, desc: 'Show all group that you have access to'
|
optional :all_available, type: Boolean, desc: 'Show all group that you have access to'
|
||||||
optional :search, type: String, desc: 'Search for a specific group'
|
optional :search, type: String, desc: 'Search for a specific group'
|
||||||
optional :owned, type: Boolean, default: false, desc: 'Limit by owned by authenticated user'
|
optional :owned, type: Boolean, default: false, desc: 'Limit by owned by authenticated user'
|
||||||
optional :order_by, type: String, values: %w[name path], default: 'name', desc: 'Order by name or path'
|
optional :order_by, type: String, values: %w[name path id], default: 'name', desc: 'Order by name, path or id'
|
||||||
optional :sort, type: String, values: %w[asc desc], default: 'asc', desc: 'Sort by asc (ascending) or desc (descending)'
|
optional :sort, type: String, values: %w[asc desc], default: 'asc', desc: 'Sort by asc (ascending) or desc (descending)'
|
||||||
use :pagination
|
use :pagination
|
||||||
end
|
end
|
||||||
|
|
|
@ -138,10 +138,12 @@ describe API::Groups do
|
||||||
|
|
||||||
context "when using sorting" do
|
context "when using sorting" do
|
||||||
let(:group3) { create(:group, name: "a#{group1.name}", path: "z#{group1.path}") }
|
let(:group3) { create(:group, name: "a#{group1.name}", path: "z#{group1.path}") }
|
||||||
|
let(:group4) { create(:group, name: "z#{group1.name}", path: "y#{group1.path}") }
|
||||||
let(:response_groups) { json_response.map { |group| group['name'] } }
|
let(:response_groups) { json_response.map { |group| group['name'] } }
|
||||||
|
|
||||||
before do
|
before do
|
||||||
group3.add_owner(user1)
|
group3.add_owner(user1)
|
||||||
|
group4.add_owner(user1)
|
||||||
end
|
end
|
||||||
|
|
||||||
it "sorts by name ascending by default" do
|
it "sorts by name ascending by default" do
|
||||||
|
@ -150,7 +152,7 @@ describe API::Groups do
|
||||||
expect(response).to have_gitlab_http_status(200)
|
expect(response).to have_gitlab_http_status(200)
|
||||||
expect(response).to include_pagination_headers
|
expect(response).to include_pagination_headers
|
||||||
expect(json_response).to be_an Array
|
expect(json_response).to be_an Array
|
||||||
expect(response_groups).to eq([group3.name, group1.name])
|
expect(response_groups).to eq([group3.name, group1.name, group4.name])
|
||||||
end
|
end
|
||||||
|
|
||||||
it "sorts in descending order when passed" do
|
it "sorts in descending order when passed" do
|
||||||
|
@ -159,16 +161,25 @@ describe API::Groups do
|
||||||
expect(response).to have_gitlab_http_status(200)
|
expect(response).to have_gitlab_http_status(200)
|
||||||
expect(response).to include_pagination_headers
|
expect(response).to include_pagination_headers
|
||||||
expect(json_response).to be_an Array
|
expect(json_response).to be_an Array
|
||||||
expect(response_groups).to eq([group1.name, group3.name])
|
expect(response_groups).to eq([group4.name, group1.name, group3.name])
|
||||||
end
|
end
|
||||||
|
|
||||||
it "sorts by the order_by param" do
|
it "sorts by path in order_by param" do
|
||||||
get api("/groups", user1), order_by: "path"
|
get api("/groups", user1), order_by: "path"
|
||||||
|
|
||||||
expect(response).to have_gitlab_http_status(200)
|
expect(response).to have_gitlab_http_status(200)
|
||||||
expect(response).to include_pagination_headers
|
expect(response).to include_pagination_headers
|
||||||
expect(json_response).to be_an Array
|
expect(json_response).to be_an Array
|
||||||
expect(response_groups).to eq([group1.name, group3.name])
|
expect(response_groups).to eq([group1.name, group4.name, group3.name])
|
||||||
|
end
|
||||||
|
|
||||||
|
it "sorts by id in the order_by param" do
|
||||||
|
get api("/groups", user1), order_by: "id"
|
||||||
|
|
||||||
|
expect(response).to have_gitlab_http_status(200)
|
||||||
|
expect(response).to include_pagination_headers
|
||||||
|
expect(json_response).to be_an Array
|
||||||
|
expect(response_groups).to eq([group1.name, group3.name, group4.name])
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue