Merge pull request #15 from SuperGoodSoft/logging

Logging!

4 files changed, 18 insertions, 1 deletions

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 9d4033f..778a4e0 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 ## master
 
+- Add support for request/response logging.
+
 ## v0.15.0
 
 - Made sure cache key is a string, instead of a giant nested hash/array structure that probably won't get interpreted by Rails well. Still not happy with the caching behaviour, but it's configurable.
diff --git a/lib/super_good/solidus_taxjar.rb b/lib/super_good/solidus_taxjar.rb
index 1016397..9318181 100644
--- a/lib/super_good/solidus_taxjar.rb
+++ b/lib/super_good/solidus_taxjar.rb
@@ -16,6 +16,7 @@ module SuperGood
       attr_accessor :discount_calculator
       attr_accessor :exception_handler
       attr_accessor :line_item_tax_label_maker
+      attr_accessor :logging_enabled
       attr_accessor :shipping_calculator
       attr_accessor :shipping_tax_label_maker
       attr_accessor :taxable_address_check
@@ -30,6 +31,7 @@ module SuperGood
       Rails.logger.error "An error occurred while fetching TaxJar tax rates - #{e}: #{e.message}"
     }
     self.line_item_tax_label_maker = ->(taxjar_line_item, spree_line_item) { "Sales Tax" }
+    self.logging_enabled = false
     self.shipping_calculator = ->(order) { order.shipment_total }
     self.shipping_tax_label_maker = ->(shipment, shipping_tax) { "Sales Tax" }
     self.taxable_address_check = ->(address) { true }
diff --git a/lib/super_good/solidus_taxjar/api.rb b/lib/super_good/solidus_taxjar/api.rb
index 7309a64..0d182ce 100644
--- a/lib/super_good/solidus_taxjar/api.rb
+++ b/lib/super_good/solidus_taxjar/api.rb
@@ -13,7 +13,13 @@ module SuperGood
       end
 
       def tax_for(order)
-        taxjar_client.tax_for_order APIParams.order_params(order)
+        taxjar_client.tax_for_order(APIParams.order_params(order)).tap do |taxes|
+          next unless SuperGood::SolidusTaxJar.logging_enabled
+
+          Rails.logger.info(
+            "TaxJar response for #{order.number}: #{taxes.to_h.inspect}"
+          )
+        end
       end
 
       def tax_rates_for(address)
diff --git a/lib/super_good/solidus_taxjar/api_params.rb b/lib/super_good/solidus_taxjar/api_params.rb
index 7c6942f..5412232 100644
--- a/lib/super_good/solidus_taxjar/api_params.rb
+++ b/lib/super_good/solidus_taxjar/api_params.rb
@@ -8,6 +8,13 @@ module SuperGood
             .merge(order_address_params(order.tax_address))
             .merge(line_items_params(order.line_items))
             .merge(shipping: order.shipment_total)
+            .tap do |params|
+              next unless SuperGood::SolidusTaxJar.logging_enabled
+
+              Rails.logger.info(
+                "TaxJar params: #{params.inspect}"
+              )
+            end
         end
 
         def address_params(address)