# frozen_string_literal: true

require 'spec_helper'

describe RedmineService do
  describe 'Associations' do
    it { is_expected.to belong_to :project }
    it { is_expected.to have_one :service_hook }
  end

  describe 'Validations' do
    # if redmine is set in setting the urls are set to defaults
    # therefore the validation passes as the values are not nil
    before do
      settings = {
        'redmine' => {}
      }
      allow(Gitlab.config).to receive(:issues_tracker).and_return(settings)
    end

    context 'when service is active' do
      before do
        subject.active = true
      end

      it { is_expected.to validate_presence_of(:project_url) }
      it { is_expected.to validate_presence_of(:issues_url) }
      it { is_expected.to validate_presence_of(:new_issue_url) }

      it_behaves_like 'issue tracker service URL attribute', :project_url
      it_behaves_like 'issue tracker service URL attribute', :issues_url
      it_behaves_like 'issue tracker service URL attribute', :new_issue_url
    end

    context 'when service is inactive' do
      before do
        subject.active = false
      end

      it { is_expected.not_to validate_presence_of(:project_url) }
      it { is_expected.not_to validate_presence_of(:issues_url) }
      it { is_expected.not_to validate_presence_of(:new_issue_url) }
    end
  end

  describe '.reference_pattern' do
    it_behaves_like 'allows project key on reference pattern'

    it 'does allow # on the reference' do
      expect(described_class.reference_pattern.match('#123')[:issue]).to eq('123')
    end
  end

  context 'overriding properties' do
    let(:url) { 'http://redmine.example.com' }
    let(:access_params) do
      { project_url: url, issues_url: url, new_issue_url: url }
    end

    # this  will be removed as part of https://gitlab.com/gitlab-org/gitlab/issues/29404
    context 'when data are stored in properties' do
      let(:properties) { access_params.merge(title: title, description: description) }
      let(:service) do
        create(:redmine_service, :without_properties_callback, properties: properties)
      end

      it_behaves_like 'issue tracker fields'
    end

    context 'when data are stored in separated fields' do
      let(:service) do
        create(:redmine_service, title: title, description: description, properties: access_params)
      end

      it_behaves_like 'issue tracker fields'
    end

    context 'when data are stored in both properties and separated fields' do
      let(:properties) { access_params.merge(title: 'wrong title', description: 'wrong description') }
      let(:service) do
        create(:redmine_service, :without_properties_callback, title: title, description: description, properties: properties)
      end

      it_behaves_like 'issue tracker fields'
    end

    context 'when no title & description are set' do
      let(:service) do
        create(:redmine_service, properties: access_params)
      end

      it 'returns default values' do
        expect(service.title).to eq('Redmine')
        expect(service.description).to eq('Redmine issue tracker')
      end
    end
  end
end
