Refactor customer deletion logic to use customer.balance.zero? instea…

…d of balance? method
fabianoflorentino · Mar 25, 2024 · b27d5c6 · b27d5c6
1 parent eef1176
commit b27d5c6
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/app/use_cases/customer_use_case/delete.rb b/app/use_cases/customer_use_case/delete.rb
@@ -8,7 +8,7 @@ def initialize(id)
     end
 
     def call
-      return customer.destroy! if balance?
+      return customer.destroy! if customer.balance.zero?
 
       raise SharedErrors::BalanceEmpty
     end
@@ -18,11 +18,7 @@ def call
     attr_reader :id
 
     def customer
-      @customer = Customer.includes(:transactions).find(@id)
-    end
-
-    def balance?
-      customer.balance.zero?
+      Customer.includes(:transactions).find(@id)
     end
   end
 end