Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat (limited to 'spec/rubocop/cop/gitlab/event_store_subscriber_spec.rb')
-rw-r--r--spec/rubocop/cop/gitlab/event_store_subscriber_spec.rb82
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