diff options
Diffstat (limited to 'spec/requests/users/pins_spec.rb')
-rw-r--r-- | spec/requests/users/pins_spec.rb | 67 |
1 files changed, 67 insertions, 0 deletions
diff --git a/spec/requests/users/pins_spec.rb b/spec/requests/users/pins_spec.rb new file mode 100644 index 00000000000..9a32d7e9d76 --- /dev/null +++ b/spec/requests/users/pins_spec.rb @@ -0,0 +1,67 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe 'Pinning navigation menu items', feature_category: :navigation do + let(:user) { create(:user) } + let(:menu_item_ids) { %w[item4 item7] } + let(:other_panel_data) { { 'group' => ['some_item_id'] } } + + before do + user.update!(pinned_nav_items: other_panel_data) + sign_in(user) + end + + describe 'PUT /-/users/pins' do + before do + put pins_path, params: params, headers: { 'ACCEPT' => 'application/json' } + end + + context 'with valid params' do + let(:panel) { 'project' } + let(:params) { { menu_item_ids: menu_item_ids, panel: panel } } + + it 'saves the menu_item_ids for the correct panel' do + expect(user.pinned_nav_items).to include(panel => menu_item_ids) + end + + it 'does not change menu_item_ids of other panels' do + expect(user.pinned_nav_items).to include(other_panel_data) + end + + it 'responds OK' do + expect(response).to have_gitlab_http_status(:ok) + end + end + + context 'with invalid params' do + shared_examples 'unchanged data and error response' do + it 'does not modify existing panel data' do + expect(user.reload.pinned_nav_items).to eq(other_panel_data) + end + + it 'responds with error' do + expect(response).to have_gitlab_http_status(:bad_request) + end + end + + context 'when panel name is unknown' do + let(:params) { { menu_item_ids: menu_item_ids, panel: 'something_else' } } + + it_behaves_like 'unchanged data and error response' + end + + context 'when menu_item_ids is not array of strings' do + let(:params) { { menu_item_ids: 'not_an_array', panel: 'project' } } + + it_behaves_like 'unchanged data and error response' + end + + context 'when params are not permitted' do + let(:params) { { random_param: 'random_value' } } + + it_behaves_like 'unchanged data and error response' + end + end + end +end |