diff options
Diffstat (limited to 'spec/rubocop/cop/gitlab/event_store_subscriber_spec.rb')
-rw-r--r-- | spec/rubocop/cop/gitlab/event_store_subscriber_spec.rb | 82 |
1 files changed, 82 insertions, 0 deletions
diff --git a/spec/rubocop/cop/gitlab/event_store_subscriber_spec.rb b/spec/rubocop/cop/gitlab/event_store_subscriber_spec.rb new file mode 100644 index 00000000000..e17fb71f9bc --- /dev/null +++ b/spec/rubocop/cop/gitlab/event_store_subscriber_spec.rb @@ -0,0 +1,82 @@ +# frozen_string_literal: true + +require 'fast_spec_helper' + +require_relative '../../../../rubocop/cop/gitlab/event_store_subscriber' + +RSpec.describe RuboCop::Cop::Gitlab::EventStoreSubscriber do + subject(:cop) { described_class.new } + + context 'when an event store subscriber overrides #perform' do + it 'registers an offense' do + expect_offense(<<~WORKER) + class SomeWorker + include Gitlab::EventStore::Subscriber + + def perform(*args) + ^^^^^^^^^^^^^^^^^^ Do not override `perform` in a `Gitlab::EventStore::Subscriber`. + end + + def handle_event(event); end + end + WORKER + end + end + + context 'when an event store subscriber does not override #perform' do + it 'does not register an offense' do + expect_no_offenses(<<~WORKER) + class SomeWorker + include Gitlab::EventStore::Subscriber + + def handle_event(event); end + end + WORKER + end + end + + context 'when an event store subscriber does not implement #handle_event' do + it 'registers an offense' do + expect_offense(<<~WORKER) + class SomeWorker + include Gitlab::EventStore::Subscriber + ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ A `Gitlab::EventStore::Subscriber` must implement `#handle_event(event)`. + end + WORKER + end + end + + context 'when a Sidekiq worker overrides #perform' do + it 'does not register an offense' do + expect_no_offenses(<<~WORKER) + class SomeWorker + include ApplicationWorker + + def perform(*args); end + end + WORKER + end + end + + context 'when a Sidekiq worker implements #handle_event' do + it 'does not register an offense' do + expect_no_offenses(<<~WORKER) + class SomeWorker + include ApplicationWorker + + def handle_event(event); end + end + WORKER + end + end + + context 'a non worker class' do + it 'does not register an offense' do + expect_no_offenses(<<~MODEL) + class Model < ApplicationRecord + include ActiveSupport::Concern + end + MODEL + end + end +end |