about summary refs log tree commit diff
path: root/app
diff options
context:
space:
mode:
authorMatt Jankowski <mjankowski@thoughtbot.com>2017-05-07 08:44:28 -0400
committerEugen Rochko <eugen@zeonfederated.com>2017-05-07 14:44:28 +0200
commitb8ba719f73ec0a2ec7c11b96a740aad8132a5580 (patch)
treee2ba5151ff488dda1c04520bd2163a1c05421be7 /app
parentada8a6cb77af2a142a04319f9064609313f80028 (diff)
Unblock domain service specs/refactor (#2867)
* Add spec for unblock domain service

* Refactor UnblockDomainService
Diffstat (limited to 'app')
-rw-r--r--app/services/unblock_domain_service.rb28
1 files changed, 19 insertions, 9 deletions
diff --git a/app/services/unblock_domain_service.rb b/app/services/unblock_domain_service.rb
index 77460f76b..169d2f781 100644
--- a/app/services/unblock_domain_service.rb
+++ b/app/services/unblock_domain_service.rb
@@ -1,17 +1,27 @@
 # frozen_string_literal: true
 
 class UnblockDomainService < BaseService
+  attr_accessor :domain_block
+
   def call(domain_block, retroactive)
-    if retroactive
-      accounts = Account.where(domain: domain_block.domain).in_batches
+    @domain_block = domain_block
+    process_retroactive_updates if retroactive
+    domain_block.destroy
+  end
 
-      if domain_block.silence?
-        accounts.update_all(silenced: false)
-      else
-        accounts.update_all(suspended: false)
-      end
-    end
+  def process_retroactive_updates
+    blocked_accounts.in_batches.update_all(update_options)
+  end
 
-    domain_block.destroy
+  def blocked_accounts
+    Account.where(domain: domain_block.domain)
+  end
+
+  def update_options
+    { domain_block_impact => false }
+  end
+
+  def domain_block_impact
+    domain_block.silence? ? :silenced : :suspended
   end
 end