diff options
Diffstat (limited to 'spec/graphql/mutations/commits/create_spec.rb')
-rw-r--r-- | spec/graphql/mutations/commits/create_spec.rb | 180 |
1 files changed, 180 insertions, 0 deletions
diff --git a/spec/graphql/mutations/commits/create_spec.rb b/spec/graphql/mutations/commits/create_spec.rb new file mode 100644 index 00000000000..bb0b8c577b0 --- /dev/null +++ b/spec/graphql/mutations/commits/create_spec.rb @@ -0,0 +1,180 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe Mutations::Commits::Create do + subject(:mutation) { described_class.new(object: nil, context: context, field: nil) } + + let_it_be(:project) { create(:project, :public, :repository) } + let_it_be(:user) { create(:user) } + let(:context) do + GraphQL::Query::Context.new( + query: OpenStruct.new(schema: nil), + values: { current_user: user }, + object: nil + ) + end + + specify { expect(described_class).to require_graphql_authorizations(:push_code) } + + describe '#resolve' do + subject { mutation.resolve(project_path: project.full_path, branch: branch, message: message, actions: actions) } + + let(:branch) { 'master' } + let(:message) { 'Commit message' } + let(:actions) do + [ + { + action: 'create', + file_path: 'NEW_FILE.md', + content: 'Hello' + } + ] + end + + let(:mutated_commit) { subject[:commit] } + + it 'raises an error if the resource is not accessible to the user' do + expect { subject }.to raise_error(Gitlab::Graphql::Errors::ResourceNotAvailable) + end + + context 'when user does not have enough permissions' do + before do + project.add_guest(user) + end + + it 'raises an error' do + expect { subject }.to raise_error(Gitlab::Graphql::Errors::ResourceNotAvailable) + end + end + + context 'when user is a maintainer of a different project' do + before do + create(:project_empty_repo).add_maintainer(user) + end + + it 'raises an error' do + expect { subject }.to raise_error(Gitlab::Graphql::Errors::ResourceNotAvailable) + end + end + + context 'when the user can create a commit' do + let(:deltas) { mutated_commit.raw_deltas } + + before_all do + project.add_developer(user) + end + + context 'when service successfully creates a new commit' do + it 'returns a new commit' do + expect(mutated_commit).to have_attributes(message: message, project: project) + expect(subject[:errors]).to be_empty + + expect_to_contain_deltas([ + a_hash_including(a_mode: '0', b_mode: '100644', new_file: true, new_path: 'NEW_FILE.md') + ]) + end + end + + context 'when request has multiple actions' do + let(:actions) do + [ + { + action: 'create', + file_path: 'foo/foobar', + content: 'some content' + }, + { + action: 'delete', + file_path: 'README.md' + }, + { + action: 'move', + file_path: "LICENSE.md", + previous_path: "LICENSE", + content: "some content" + }, + { + action: 'update', + file_path: 'VERSION', + content: 'new content' + }, + { + action: 'chmod', + file_path: 'CHANGELOG', + execute_filemode: true + } + ] + end + + it 'returns a new commit' do + expect(mutated_commit).to have_attributes(message: message, project: project) + expect(subject[:errors]).to be_empty + + expect_to_contain_deltas([ + a_hash_including(a_mode: '0', b_mode: '100644', new_path: 'foo/foobar'), + a_hash_including(deleted_file: true, new_path: 'README.md'), + a_hash_including(deleted_file: true, new_path: 'LICENSE'), + a_hash_including(new_file: true, new_path: 'LICENSE.md'), + a_hash_including(new_file: false, new_path: 'VERSION'), + a_hash_including(a_mode: '100644', b_mode: '100755', new_path: 'CHANGELOG') + ]) + end + end + + context 'when actions are not defined' do + let(:actions) { [] } + + it 'returns a new commit' do + expect(mutated_commit).to have_attributes(message: message, project: project) + expect(subject[:errors]).to be_empty + + expect_to_contain_deltas([]) + end + end + + context 'when branch does not exist' do + let(:branch) { 'unknown' } + + it 'returns errors' do + expect(mutated_commit).to be_nil + expect(subject[:errors]).to eq(['You can only create or edit files when you are on a branch']) + end + end + + context 'when message is not set' do + let(:message) { nil } + + it 'returns errors' do + expect(mutated_commit).to be_nil + expect(subject[:errors]).to eq(['3:UserCommitFiles: empty CommitMessage']) + end + end + + context 'when actions are incorrect' do + let(:actions) { [{ action: 'unknown', file_path: 'test.md', content: '' }] } + + it 'returns errors' do + expect(mutated_commit).to be_nil + expect(subject[:errors]).to eq(['Unknown action \'unknown\'']) + end + end + + context 'when branch is protected' do + before do + create(:protected_branch, project: project, name: branch) + end + + it 'returns errors' do + expect(mutated_commit).to be_nil + expect(subject[:errors]).to eq(['You are not allowed to push into this branch']) + end + end + end + end + + def expect_to_contain_deltas(expected_deltas) + expect(deltas.count).to eq(expected_deltas.count) + expect(deltas).to include(*expected_deltas) + end +end |