From 22dcadedb495d2e1279b834a624710d34daee6ad Mon Sep 17 00:00:00 2001
From: Kurtis Rainbolt-Greene <kurtis@rainbolt-greene.online>
Date: Tue, 4 Apr 2017 21:14:37 -0700
Subject: [PATCH] We're going to want these nice helper methods, lets share
them with a parent class that matches Rails 5 practices (application level
abstraction)
---
app/workers/application_worker.rb | 5 +++++
app/workers/distribution_worker.rb | 4 ++--
2 files changed, 7 insertions(+), 2 deletions(-)
create mode 100644 app/workers/application_worker.rb
diff --git a/app/workers/application_worker.rb b/app/workers/application_worker.rb
new file mode 100644
index 0000000000..f2d7c10622
--- /dev/null
+++ b/app/workers/application_worker.rb
@@ -0,0 +1,5 @@
+class ApplicationWorker
+ def info(message)
+ Rails.logger.info("#{self.class.name} - #{message}")
+ end
+end
diff --git a/app/workers/distribution_worker.rb b/app/workers/distribution_worker.rb
index f4e738d804..9a2867ea66 100644
--- a/app/workers/distribution_worker.rb
+++ b/app/workers/distribution_worker.rb
@@ -1,6 +1,6 @@
# frozen_string_literal: true
-class DistributionWorker
+class DistributionWorker < ApplicationWorker
include Sidekiq::Worker
def perform(status_id)
@@ -9,6 +9,6 @@ class DistributionWorker
FanOutOnWriteService.new.call(status)
WarmCacheService.new.call(status)
rescue ActiveRecord::RecordNotFound
- true
+ info("Couldn't find the status")
end
end
--
GitLab