2019-07-07 11:18:12 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2023-01-13 00:05:48 +05:30
|
|
|
RSpec.describe GitlabShellWorker, :sidekiq_inline do
|
2020-04-08 14:13:33 +05:30
|
|
|
describe '#perform' do
|
2023-01-13 00:05:48 +05:30
|
|
|
Gitlab::Shell::PERMITTED_ACTIONS.each do |action|
|
|
|
|
describe "with the #{action} action" do
|
|
|
|
it 'forwards the message to Gitlab::Shell' do
|
|
|
|
expect_next_instance_of(Gitlab::Shell) do |instance|
|
|
|
|
expect(instance).to respond_to(action)
|
|
|
|
expect(instance).to receive(action).with('foo', 'bar')
|
|
|
|
end
|
|
|
|
|
|
|
|
described_class.perform_async(action, 'foo', 'bar')
|
2020-04-08 14:13:33 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2023-01-13 00:05:48 +05:30
|
|
|
describe 'all other commands' do
|
|
|
|
context 'when verify_gitlab_shell_worker_method_names is enabled' do
|
|
|
|
it 'raises ArgumentError' do
|
|
|
|
allow_next_instance_of(described_class) do |job_instance|
|
|
|
|
expect(job_instance).not_to receive(:gitlab_shell)
|
|
|
|
end
|
|
|
|
|
|
|
|
expect { described_class.perform_async('foo', 'bar', 'baz') }
|
|
|
|
.to raise_error(ArgumentError, 'foo not allowed for GitlabShellWorker')
|
2020-04-08 14:13:33 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2023-01-13 00:05:48 +05:30
|
|
|
context 'when verify_gitlab_shell_worker_method_names is disabled' do
|
|
|
|
before do
|
|
|
|
stub_feature_flags(verify_gitlab_shell_worker_method_names: false)
|
2020-04-08 14:13:33 +05:30
|
|
|
end
|
|
|
|
|
2023-01-13 00:05:48 +05:30
|
|
|
it 'forwards the message to Gitlab::Shell' do
|
|
|
|
expect_next_instance_of(Gitlab::Shell) do |instance|
|
|
|
|
expect(instance).to receive('foo').with('bar', 'baz')
|
|
|
|
end
|
|
|
|
|
|
|
|
described_class.perform_async('foo', 'bar', 'baz')
|
|
|
|
end
|
2020-03-09 13:42:32 +05:30
|
|
|
end
|
2018-03-17 18:26:18 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|