646 lines
20 KiB
Ruby
646 lines
20 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'spec_helper'
|
|
|
|
RSpec.describe Service do
|
|
describe "Associations" do
|
|
it { is_expected.to belong_to :project }
|
|
it { is_expected.to have_one :service_hook }
|
|
it { is_expected.to have_one :jira_tracker_data }
|
|
it { is_expected.to have_one :issue_tracker_data }
|
|
end
|
|
|
|
describe 'validations' do
|
|
it { is_expected.to validate_presence_of(:type) }
|
|
|
|
it 'validates presence of project_id if not template', :aggregate_failures do
|
|
expect(build(:service, project_id: nil, template: true)).to be_valid
|
|
expect(build(:service, project_id: nil, template: false)).to be_invalid
|
|
end
|
|
|
|
it 'validates presence of project_id if not instance', :aggregate_failures do
|
|
expect(build(:service, project_id: nil, instance: true)).to be_valid
|
|
expect(build(:service, project_id: nil, instance: false)).to be_invalid
|
|
end
|
|
|
|
it 'validates absence of project_id if instance', :aggregate_failures do
|
|
expect(build(:service, project_id: nil, instance: true)).to be_valid
|
|
expect(build(:service, instance: true)).to be_invalid
|
|
end
|
|
|
|
it 'validates absence of project_id if template', :aggregate_failures do
|
|
expect(build(:service, template: true)).to validate_absence_of(:project_id)
|
|
expect(build(:service, template: false)).not_to validate_absence_of(:project_id)
|
|
end
|
|
|
|
it 'validates service is template or instance' do
|
|
expect(build(:service, project_id: nil, template: true, instance: true)).to be_invalid
|
|
end
|
|
|
|
context 'with an existing service template' do
|
|
before do
|
|
create(:service, :template)
|
|
end
|
|
|
|
it 'validates only one service template per type' do
|
|
expect(build(:service, :template)).to be_invalid
|
|
end
|
|
end
|
|
|
|
context 'with an existing instance service' do
|
|
before do
|
|
create(:service, :instance)
|
|
end
|
|
|
|
it 'validates only one service instance per type' do
|
|
expect(build(:service, :instance)).to be_invalid
|
|
end
|
|
end
|
|
|
|
it 'validates uniqueness of type and project_id on create' do
|
|
project = create(:project)
|
|
|
|
expect(create(:service, project: project, type: 'Service')).to be_valid
|
|
expect(build(:service, project: project, type: 'Service').valid?(:create)).to eq(false)
|
|
expect(build(:service, project: project, type: 'Service').valid?(:update)).to eq(true)
|
|
end
|
|
end
|
|
|
|
describe 'Scopes' do
|
|
describe '.by_type' do
|
|
let!(:service1) { create(:jira_service) }
|
|
let!(:service2) { create(:jira_service) }
|
|
let!(:service3) { create(:redmine_service) }
|
|
|
|
subject { described_class.by_type(type) }
|
|
|
|
context 'when type is "JiraService"' do
|
|
let(:type) { 'JiraService' }
|
|
|
|
it { is_expected.to match_array([service1, service2]) }
|
|
end
|
|
|
|
context 'when type is "RedmineService"' do
|
|
let(:type) { 'RedmineService' }
|
|
|
|
it { is_expected.to match_array([service3]) }
|
|
end
|
|
end
|
|
|
|
describe '#operating?' do
|
|
it 'is false when the service is not active' do
|
|
expect(build(:service).operating?).to eq(false)
|
|
end
|
|
|
|
it 'is false when the service is not persisted' do
|
|
expect(build(:service, active: true).operating?).to eq(false)
|
|
end
|
|
|
|
it 'is true when the service is active and persisted' do
|
|
expect(create(:service, active: true).operating?).to eq(true)
|
|
end
|
|
end
|
|
|
|
describe '.confidential_note_hooks' do
|
|
it 'includes services where confidential_note_events is true' do
|
|
create(:service, active: true, confidential_note_events: true)
|
|
|
|
expect(described_class.confidential_note_hooks.count).to eq 1
|
|
end
|
|
|
|
it 'excludes services where confidential_note_events is false' do
|
|
create(:service, active: true, confidential_note_events: false)
|
|
|
|
expect(described_class.confidential_note_hooks.count).to eq 0
|
|
end
|
|
end
|
|
|
|
describe '.alert_hooks' do
|
|
it 'includes services where alert_events is true' do
|
|
create(:service, active: true, alert_events: true)
|
|
|
|
expect(described_class.alert_hooks.count).to eq 1
|
|
end
|
|
|
|
it 'excludes services where alert_events is false' do
|
|
create(:service, active: true, alert_events: false)
|
|
|
|
expect(described_class.alert_hooks.count).to eq 0
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "Test Button" do
|
|
describe '#can_test?' do
|
|
subject { service.can_test? }
|
|
|
|
let(:service) { create(:service, project: project) }
|
|
|
|
context 'when repository is not empty' do
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
it { is_expected.to be true }
|
|
end
|
|
|
|
context 'when repository is empty' do
|
|
let(:project) { create(:project) }
|
|
|
|
it { is_expected.to be true }
|
|
end
|
|
|
|
context 'when instance-level service' do
|
|
Service.available_services_types.each do |service_type|
|
|
let(:service) do
|
|
service_type.constantize.new(instance: true)
|
|
end
|
|
|
|
it { is_expected.to be_falsey }
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '#test' do
|
|
let(:data) { 'test' }
|
|
let(:service) { create(:service, project: project) }
|
|
|
|
context 'when repository is not empty' do
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
it 'test runs execute' do
|
|
expect(service).to receive(:execute).with(data)
|
|
|
|
service.test(data)
|
|
end
|
|
end
|
|
|
|
context 'when repository is empty' do
|
|
let(:project) { create(:project) }
|
|
|
|
it 'test runs execute' do
|
|
expect(service).to receive(:execute).with(data)
|
|
|
|
service.test(data)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '.find_or_initialize_instances' do
|
|
shared_examples 'service instances' do
|
|
it 'returns the available service instances' do
|
|
expect(Service.find_or_initialize_instances.pluck(:type)).to match_array(Service.available_services_types)
|
|
end
|
|
|
|
it 'does not create service instances' do
|
|
expect { Service.find_or_initialize_instances }.not_to change { Service.count }
|
|
end
|
|
end
|
|
|
|
it_behaves_like 'service instances'
|
|
|
|
context 'with all existing instances' do
|
|
before do
|
|
Service.insert_all(
|
|
Service.available_services_types.map { |type| { instance: true, type: type } }
|
|
)
|
|
end
|
|
|
|
it_behaves_like 'service instances'
|
|
|
|
context 'with a previous existing service (Previous) and a new service (Asana)' do
|
|
before do
|
|
Service.insert(type: 'PreviousService', instance: true)
|
|
Service.delete_by(type: 'AsanaService', instance: true)
|
|
end
|
|
|
|
it_behaves_like 'service instances'
|
|
end
|
|
end
|
|
|
|
context 'with a few existing instances' do
|
|
before do
|
|
create(:jira_service, :instance)
|
|
end
|
|
|
|
it_behaves_like 'service instances'
|
|
end
|
|
end
|
|
|
|
describe 'template' do
|
|
let(:project) { create(:project) }
|
|
|
|
shared_examples 'retrieves service templates' do
|
|
it 'returns the available service templates' do
|
|
expect(Service.find_or_create_templates.pluck(:type)).to match_array(Service.available_services_types)
|
|
end
|
|
end
|
|
|
|
describe '.find_or_create_templates' do
|
|
it 'creates service templates' do
|
|
expect { Service.find_or_create_templates }.to change { Service.count }.from(0).to(Service.available_services_names.size)
|
|
end
|
|
|
|
it_behaves_like 'retrieves service templates'
|
|
|
|
context 'with all existing templates' do
|
|
before do
|
|
Service.insert_all(
|
|
Service.available_services_types.map { |type| { template: true, type: type } }
|
|
)
|
|
end
|
|
|
|
it 'does not create service templates' do
|
|
expect { Service.find_or_create_templates }.not_to change { Service.count }
|
|
end
|
|
|
|
it_behaves_like 'retrieves service templates'
|
|
|
|
context 'with a previous existing service (Previous) and a new service (Asana)' do
|
|
before do
|
|
Service.insert(type: 'PreviousService', template: true)
|
|
Service.delete_by(type: 'AsanaService', template: true)
|
|
end
|
|
|
|
it_behaves_like 'retrieves service templates'
|
|
end
|
|
end
|
|
|
|
context 'with a few existing templates' do
|
|
before do
|
|
create(:jira_service, :template)
|
|
end
|
|
|
|
it 'creates the rest of the service templates' do
|
|
expect { Service.find_or_create_templates }.to change { Service.count }.from(1).to(Service.available_services_names.size)
|
|
end
|
|
|
|
it_behaves_like 'retrieves service templates'
|
|
end
|
|
end
|
|
|
|
describe '.build_from_integration' do
|
|
context 'when integration is invalid' do
|
|
let(:integration) do
|
|
build(:prometheus_service, :template, active: true, properties: {})
|
|
.tap { |integration| integration.save(validate: false) }
|
|
end
|
|
|
|
it 'sets service to inactive' do
|
|
service = described_class.build_from_integration(project.id, integration)
|
|
|
|
expect(service).to be_valid
|
|
expect(service.active).to be false
|
|
end
|
|
end
|
|
|
|
context 'when integration is an instance' do
|
|
let(:integration) { create(:jira_service, :instance) }
|
|
|
|
it 'sets inherit_from_id from integration' do
|
|
service = described_class.build_from_integration(project.id, integration)
|
|
|
|
expect(service.inherit_from_id).to eq(integration.id)
|
|
end
|
|
end
|
|
|
|
describe 'build issue tracker from an integration' do
|
|
let(:url) { 'http://jira.example.com' }
|
|
let(:api_url) { 'http://api-jira.example.com' }
|
|
let(:username) { 'jira-username' }
|
|
let(:password) { 'jira-password' }
|
|
let(:data_params) do
|
|
{
|
|
url: url, api_url: api_url,
|
|
username: username, password: password
|
|
}
|
|
end
|
|
|
|
shared_examples 'service creation from an integration' do
|
|
it 'creates a correct service' do
|
|
service = described_class.build_from_integration(project.id, integration)
|
|
|
|
expect(service).to be_active
|
|
expect(service.url).to eq(url)
|
|
expect(service.api_url).to eq(api_url)
|
|
expect(service.username).to eq(username)
|
|
expect(service.password).to eq(password)
|
|
expect(service.template).to eq(false)
|
|
expect(service.instance).to eq(false)
|
|
end
|
|
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) { data_params }
|
|
let!(:integration) do
|
|
create(:jira_service, :without_properties_callback, template: true, properties: properties.merge(additional: 'something'))
|
|
end
|
|
|
|
it_behaves_like 'service creation from an integration'
|
|
end
|
|
|
|
context 'when data are stored in separated fields' do
|
|
let(:integration) do
|
|
create(:jira_service, :template, data_params.merge(properties: {}))
|
|
end
|
|
|
|
it_behaves_like 'service creation from an integration'
|
|
end
|
|
|
|
context 'when data are stored in both properties and separated fields' do
|
|
let(:properties) { data_params }
|
|
let(:integration) do
|
|
create(:jira_service, :without_properties_callback, active: true, template: true, properties: properties).tap do |service|
|
|
create(:jira_tracker_data, data_params.merge(service: service))
|
|
end
|
|
end
|
|
|
|
it_behaves_like 'service creation from an integration'
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "for pushover service" do
|
|
let!(:service_template) do
|
|
PushoverService.create(
|
|
template: true,
|
|
properties: {
|
|
device: 'MyDevice',
|
|
sound: 'mic',
|
|
priority: 4,
|
|
api_key: '123456789'
|
|
})
|
|
end
|
|
|
|
describe 'is prefilled for projects pushover service' do
|
|
it "has all fields prefilled" do
|
|
service = project.find_or_initialize_service('pushover')
|
|
|
|
expect(service.template).to eq(false)
|
|
expect(service.device).to eq('MyDevice')
|
|
expect(service.sound).to eq('mic')
|
|
expect(service.priority).to eq(4)
|
|
expect(service.api_key).to eq('123456789')
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'instance' do
|
|
describe '.instance_for' do
|
|
let_it_be(:jira_service) { create(:jira_service, :instance) }
|
|
let_it_be(:slack_service) { create(:slack_service, :instance) }
|
|
|
|
subject { described_class.instance_for(type) }
|
|
|
|
context 'Hipchat serivce' do
|
|
let(:type) { 'HipchatService' }
|
|
|
|
it { is_expected.to eq(nil) }
|
|
end
|
|
|
|
context 'Jira serivce' do
|
|
let(:type) { 'JiraService' }
|
|
|
|
it { is_expected.to eq(jira_service) }
|
|
end
|
|
|
|
context 'Slack serivce' do
|
|
let(:type) { 'SlackService' }
|
|
|
|
it { is_expected.to eq(slack_service) }
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "{property}_changed?" do
|
|
let(:service) do
|
|
BambooService.create(
|
|
project: create(:project),
|
|
properties: {
|
|
bamboo_url: 'http://gitlab.com',
|
|
username: 'mic',
|
|
password: "password"
|
|
}
|
|
)
|
|
end
|
|
|
|
it "returns false when the property has not been assigned a new value" do
|
|
service.username = "key_changed"
|
|
expect(service.bamboo_url_changed?).to be_falsy
|
|
end
|
|
|
|
it "returns true when the property has been assigned a different value" do
|
|
service.bamboo_url = "http://example.com"
|
|
expect(service.bamboo_url_changed?).to be_truthy
|
|
end
|
|
|
|
it "returns true when the property has been assigned a different value twice" do
|
|
service.bamboo_url = "http://example.com"
|
|
service.bamboo_url = "http://example.com"
|
|
expect(service.bamboo_url_changed?).to be_truthy
|
|
end
|
|
|
|
it "returns false when the property has been re-assigned the same value" do
|
|
service.bamboo_url = 'http://gitlab.com'
|
|
expect(service.bamboo_url_changed?).to be_falsy
|
|
end
|
|
|
|
it "returns false when the property has been assigned a new value then saved" do
|
|
service.bamboo_url = 'http://example.com'
|
|
service.save
|
|
expect(service.bamboo_url_changed?).to be_falsy
|
|
end
|
|
end
|
|
|
|
describe "{property}_touched?" do
|
|
let(:service) do
|
|
BambooService.create(
|
|
project: create(:project),
|
|
properties: {
|
|
bamboo_url: 'http://gitlab.com',
|
|
username: 'mic',
|
|
password: "password"
|
|
}
|
|
)
|
|
end
|
|
|
|
it "returns false when the property has not been assigned a new value" do
|
|
service.username = "key_changed"
|
|
expect(service.bamboo_url_touched?).to be_falsy
|
|
end
|
|
|
|
it "returns true when the property has been assigned a different value" do
|
|
service.bamboo_url = "http://example.com"
|
|
expect(service.bamboo_url_touched?).to be_truthy
|
|
end
|
|
|
|
it "returns true when the property has been assigned a different value twice" do
|
|
service.bamboo_url = "http://example.com"
|
|
service.bamboo_url = "http://example.com"
|
|
expect(service.bamboo_url_touched?).to be_truthy
|
|
end
|
|
|
|
it "returns true when the property has been re-assigned the same value" do
|
|
service.bamboo_url = 'http://gitlab.com'
|
|
expect(service.bamboo_url_touched?).to be_truthy
|
|
end
|
|
|
|
it "returns false when the property has been assigned a new value then saved" do
|
|
service.bamboo_url = 'http://example.com'
|
|
service.save
|
|
expect(service.bamboo_url_changed?).to be_falsy
|
|
end
|
|
end
|
|
|
|
describe "{property}_was" do
|
|
let(:service) do
|
|
BambooService.create(
|
|
project: create(:project),
|
|
properties: {
|
|
bamboo_url: 'http://gitlab.com',
|
|
username: 'mic',
|
|
password: "password"
|
|
}
|
|
)
|
|
end
|
|
|
|
it "returns nil when the property has not been assigned a new value" do
|
|
service.username = "key_changed"
|
|
expect(service.bamboo_url_was).to be_nil
|
|
end
|
|
|
|
it "returns the previous value when the property has been assigned a different value" do
|
|
service.bamboo_url = "http://example.com"
|
|
expect(service.bamboo_url_was).to eq('http://gitlab.com')
|
|
end
|
|
|
|
it "returns initial value when the property has been re-assigned the same value" do
|
|
service.bamboo_url = 'http://gitlab.com'
|
|
expect(service.bamboo_url_was).to eq('http://gitlab.com')
|
|
end
|
|
|
|
it "returns initial value when the property has been assigned multiple values" do
|
|
service.bamboo_url = "http://example.com"
|
|
service.bamboo_url = "http://example2.com"
|
|
expect(service.bamboo_url_was).to eq('http://gitlab.com')
|
|
end
|
|
|
|
it "returns nil when the property has been assigned a new value then saved" do
|
|
service.bamboo_url = 'http://example.com'
|
|
service.save
|
|
expect(service.bamboo_url_was).to be_nil
|
|
end
|
|
end
|
|
|
|
describe 'initialize service with no properties' do
|
|
let(:service) do
|
|
GitlabIssueTrackerService.create(
|
|
project: create(:project),
|
|
project_url: 'http://gitlab.example.com'
|
|
)
|
|
end
|
|
|
|
it 'does not raise error' do
|
|
expect { service }.not_to raise_error
|
|
end
|
|
|
|
it 'sets data correctly' do
|
|
expect(service.data_fields.project_url).to eq('http://gitlab.example.com')
|
|
end
|
|
end
|
|
|
|
describe "callbacks" do
|
|
let(:project) { create(:project) }
|
|
let!(:service) do
|
|
RedmineService.new(
|
|
project: project,
|
|
active: true,
|
|
properties: {
|
|
project_url: 'http://redmine/projects/project_name_in_redmine',
|
|
issues_url: "http://redmine/#{project.id}/project_name_in_redmine/:id",
|
|
new_issue_url: 'http://redmine/projects/project_name_in_redmine/issues/new'
|
|
}
|
|
)
|
|
end
|
|
|
|
describe "on create" do
|
|
it "updates the has_external_issue_tracker boolean" do
|
|
expect do
|
|
service.save!
|
|
end.to change { service.project.has_external_issue_tracker }.from(false).to(true)
|
|
end
|
|
end
|
|
|
|
describe "on update" do
|
|
it "updates the has_external_issue_tracker boolean" do
|
|
service.save!
|
|
|
|
expect do
|
|
service.update(active: false)
|
|
end.to change { service.project.has_external_issue_tracker }.from(true).to(false)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '#api_field_names' do
|
|
let(:fake_service) do
|
|
Class.new(Service) do
|
|
def fields
|
|
[
|
|
{ name: 'token' },
|
|
{ name: 'api_token' },
|
|
{ name: 'key' },
|
|
{ name: 'api_key' },
|
|
{ name: 'password' },
|
|
{ name: 'password_field' },
|
|
{ name: 'safe_field' }
|
|
]
|
|
end
|
|
end
|
|
end
|
|
|
|
let(:service) do
|
|
fake_service.new(properties: [
|
|
{ token: 'token-value' },
|
|
{ api_token: 'api_token-value' },
|
|
{ key: 'key-value' },
|
|
{ api_key: 'api_key-value' },
|
|
{ password: 'password-value' },
|
|
{ password_field: 'password_field-value' },
|
|
{ safe_field: 'safe_field-value' }
|
|
])
|
|
end
|
|
|
|
it 'filters out sensitive fields' do
|
|
expect(service.api_field_names).to eq(['safe_field'])
|
|
end
|
|
end
|
|
|
|
context 'logging' do
|
|
let(:project) { create(:project) }
|
|
let(:service) { create(:service, project: project) }
|
|
let(:test_message) { "test message" }
|
|
let(:arguments) do
|
|
{
|
|
service_class: service.class.name,
|
|
project_path: project.full_path,
|
|
project_id: project.id,
|
|
message: test_message,
|
|
additional_argument: 'some argument'
|
|
}
|
|
end
|
|
|
|
it 'logs info messages using json logger' do
|
|
expect(Gitlab::JsonLogger).to receive(:info).with(arguments)
|
|
|
|
service.log_info(test_message, additional_argument: 'some argument')
|
|
end
|
|
|
|
it 'logs error messages using json logger' do
|
|
expect(Gitlab::JsonLogger).to receive(:error).with(arguments)
|
|
|
|
service.log_error(test_message, additional_argument: 'some argument')
|
|
end
|
|
end
|
|
end
|