diff options
Diffstat (limited to 'spec/requests/api/graphql/mutations/boards/lists/destroy_spec.rb')
-rw-r--r-- | spec/requests/api/graphql/mutations/boards/lists/destroy_spec.rb | 73 |
1 files changed, 12 insertions, 61 deletions
diff --git a/spec/requests/api/graphql/mutations/boards/lists/destroy_spec.rb b/spec/requests/api/graphql/mutations/boards/lists/destroy_spec.rb index 42f690f53ed..83309ead352 100644 --- a/spec/requests/api/graphql/mutations/boards/lists/destroy_spec.rb +++ b/spec/requests/api/graphql/mutations/boards/lists/destroy_spec.rb @@ -6,72 +6,23 @@ RSpec.describe Mutations::Boards::Lists::Destroy do include GraphqlHelpers let_it_be(:current_user, reload: true) { create(:user) } - let_it_be(:project, reload: true) { create(:project) } - let_it_be(:board) { create(:board, project: project) } - let_it_be(:list) { create(:list, board: board) } - let(:mutation) do - variables = { - list_id: GitlabSchema.id_from_object(list).to_s - } - graphql_mutation(:destroy_board_list, variables) - end - - subject { post_graphql_mutation(mutation, current_user: current_user) } - - def mutation_response - graphql_mutation_response(:destroy_board_list) - end - - context 'when the user does not have permission' do - it_behaves_like 'a mutation that returns a top-level access error' - - it 'does not destroy the list' do - expect { subject }.not_to change { List.count } - end - end - - context 'when the user has permission' do - before do - project.add_maintainer(current_user) - end - - context 'when given id is not for a list' do - let_it_be(:list) { build_stubbed(:issue, project: project) } - - it 'returns an error' do - subject + it_behaves_like 'board lists destroy request' do + let_it_be(:group, reload: true) { create(:group) } + let_it_be(:board) { create(:board, group: group) } + let_it_be(:list, refind: true) { create(:list, board: board) } - expect(graphql_errors.first['message']).to include('does not represent an instance of List') - end + let(:variables) do + { + list_id: GitlabSchema.id_from_object(list).to_s + } end - context 'when everything is ok' do - it 'destroys the list' do - expect { subject }.to change { List.count }.from(2).to(1) - end - - it 'returns an empty list' do - post_graphql_mutation(mutation, current_user: current_user) - - expect(mutation_response).to have_key('list') - expect(mutation_response['list']).to be_nil - end + let(:mutation) do + graphql_mutation(:destroy_board_list, variables) end - context 'when the list is not destroyable' do - let_it_be(:list) { create(:list, board: board, list_type: :backlog) } - - it 'does not destroy the list' do - expect { subject }.not_to change { List.count }.from(3) - end - - it 'returns an error and not nil list' do - subject - - expect(mutation_response['errors']).not_to be_empty - expect(mutation_response['list']).not_to be_nil - end - end + let(:mutation_response) { graphql_mutation_response(:destroy_board_list) } + let(:klass) { List } end end |