gitlab-org--gitlab-foss/spec/graphql/mutations/commits/create_spec.rb

# 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(:user) { create(:user) }
  let_it_be(:project) { create(:project, :public, :repository) }

  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, start_branch: start_branch, message: message, actions: actions) }

    let(:branch) { 'master' }
    let(:start_branch) { nil }
    let(:message) { 'Commit message' }
    let(:file_path) { "#{SecureRandom.uuid}.md" }
    let(:actions) do
      [
        {
          action: 'create',
          file_path: file_path,
          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 the ETag path for the commit's pipeline" do
          commit_pipeline_path = subject[:commit_pipeline_path]
          expect(commit_pipeline_path).to match(%r(pipelines/sha/\w+))
        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_file: true, new_path: file_path)
          ])
        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 branch does not exist and a start branch is provided' do
        let(:branch) { 'my-branch' }
        let(:start_branch) { 'master' }
        let(:actions) do
          [
            {
              action: 'create',
              file_path: 'ANOTHER_FILE.md',
              content: 'Bye'
            }
          ]
        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_file: true, new_path: 'ANOTHER_FILE.md')
          ])
        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_s).to match(/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