raise sanity exception with more info

rawliving-germany · Mar 3, 2023 · f59e998 · f59e998
1 parent 45048d7
commit f59e998
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/lib/shoplex.rb b/lib/shoplex.rb
@@ -20,7 +20,7 @@ def self.process file_content
     bookings = result.invoices.map do |invoice|
       begin
         Shoplex::ShippingSplitter::apply!(invoice:)
-        #Shoplex::SanityCheck::check!(invoice:)
+        Shoplex::SanityCheck::check!(invoice:)
         Shoplex::InvoiceBookingConverter.convert(invoice:)
       rescue => e
         result.mark_error(maker: self, error: :unknown, obj: [e, invoice])

diff --git a/lib/shoplex/sanity_check.rb b/lib/shoplex/sanity_check.rb
@@ -2,8 +2,13 @@ module Shoplex
   class SanityCheck
     class GrossDoesNotMatchNet < StandardError ; end
     def self.check!(invoice:)
-      if invoice.invoice_amount != invoice.invoice_amount_net + invoice.tax07_amount + invoice.tax19_amount
-        raise GrossDoesNotMatchNet
+      net_tax_sum = (invoice.invoice_amount_net + invoice.tax07_amount + invoice.tax19_amount).round(2)
+      if invoice.invoice_amount != net_tax_sum
+        raise GrossDoesNotMatchNet.new(
+          "Gross amount #{invoice.invoice_amount} does not equal net + tax sums "\
+          "(#{invoice.invoice_amount_net} "\
+          "+ #{invoice.tax07_amount} "\
+          "+ #{invoice.tax19_amount} == #{net_tax_sum})")
       end
     end
   end