solidus_bactracs 3.0.0 → 3.1.0

data/lib/generators/{solidus_backtracs → solidus_bactracs}/install/templates/initializer.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-SolidusBacktracs.configure do |config|
+SolidusBactracs.configure do |config|
   # Choose between Grams, Ounces or Pounds.
   config.weight_units = "Grams"
 
-  # Capture payment when Backtracs notifies a shipping label creation.
+  # Capture payment when Bactracs notifies a shipping label creation.
   # Set this to `true` and `Spree::Config.require_payment_to_ship` to `false` if you
   # want to charge your customers at the time of shipment.
   config.capture_at_notification = false
@@ -13,8 +13,8 @@ SolidusBacktracs.configure do |config|
   config.api_base = ENV['BACKTRACS_API_BASE'] || 'https://bactracstest.andlor.com'
   config.api_retries = ENV['BACKTRACS_API_RETRIES'] || 3
 
-  # Backtracs expects the endpoint to be protected by HTTP Basic Auth.
-  # Set the username and password you desire for Backtracs to use.
+  # Bactracs expects the endpoint to be protected by HTTP Basic Auth.
+  # Set the username and password you desire for Bactracs to use.
   config.webhook_username = "smoking_jay_cutler"
   config.webhook_password = "my-awesome-password"
 
@@ -42,14 +42,14 @@ SolidusBacktracs.configure do |config|
   ####### XML integration
   # Only uncomment these lines if you're going to use the XML integration.
 
-  # Export canceled shipments to Backtracs
+  # Export canceled shipments to Bactracs
   # Set this to `true` if you want canceled shipments included in the endpoint.
   # config.export_canceled_shipments = false
 
   # You can customize the class used to receive notifications from the POST request
   # Make sure it has a class method `from_payload` which receives the notification hash
   # and an instance method `apply`
-  # config.shipment_notice_class = 'SolidusBacktracs::ShipmentNotice'
+  # config.shipment_notice_class = 'SolidusBactracs::ShipmentNotice'
 
   ####### API integration
   # Only uncomment these lines if you're going to use the API integration.
@@ -58,21 +58,21 @@ SolidusBacktracs.configure do |config|
   # that responds to `#call`. At the very least, you'll need to uncomment the
   # following lines and customize your store ID.
   # config.api_shipment_serializer = proc do |shipment|
-  #   SolidusBacktracs::Api::ShipmentSerializer.new(store_id: '12345678').call(shipment)
+  #   SolidusBactracs::Api::ShipmentSerializer.new(store_id: '12345678').call(shipment)
   # end
 
-  # Override the logic used to match a Backtracs order to a shipment from a
+  # Override the logic used to match a Bactracs order to a shipment from a
   # given collection. This can be useful when you override the default serializer
   # and change the logic used to generate the order number.
-  # config.api_shipment_matcher = proc do |backtracs_order, shipments|
-  #   shipments.find { |shipment| shipment.number == backtracs_order['orderNumber'] }
+  # config.api_shipment_matcher = proc do |bactracs_order, shipments|
+  #   shipments.find { |shipment| shipment.number == bactracs_order['orderNumber'] }
   # end
 
-  # API key and secret for accessing the Backtracs API.
+  # API key and secret for accessing the Bactracs API.
   # config.api_key = "api-key"
   # config.api_secret = "api-secret"
 
-  # Number of shipments to import into Backtracs at once.
+  # Number of shipments to import into Bactracs at once.
   # If unsure, leave this set to 100, which is the maximum
   # number of shipments that can be imported at once.
   config.api_batch_size = 100
@@ -90,3 +90,33 @@ SolidusBacktracs.configure do |config|
     Sentry.capture_exception(error, extra: context)
   }
 end
+
+class Spree
+  class Shipment
+    def normalize_valid_variants_from_bundles
+      self.line_items.reduce([]) do |variants, line_item|
+        if line_item.product.respond_to?(:assembly?) && line_item.product.assembly?
+          variants += line_item.product.parts.reject(&:invalid_part?)
+        else
+          variants << line_item.variant
+        end
+        variants
+      end
+    end
+
+    private
+
+    def invalid_part?(part)
+      part.product
+        .product_properties
+        .where(property_id: default_property_id)
+        .present?
+    end
+
+    def default_property_id
+      ::Spree::Property.find_by(
+        name: SolidusBactracs.config.default_property_name,
+      )&.id
+    end
+  end
+end

data/lib/solidus_backtracs/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SolidusBacktracs
-  VERSION = '3.0.0'
+  VERSION = '3.1.0'
 end

data/lib/{solidus_backtracs → solidus_bactracs}/api/batch_syncer.rb

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   module Api
     class BatchSyncer
       class << self
         def from_config
           new(
-            client: SolidusBacktracs::Api::Client.from_config,
-            shipment_matcher: SolidusBacktracs.config.api_shipment_matcher,
+            client: SolidusBactracs::Api::Client.from_config,
+            shipment_matcher: SolidusBactracs.config.api_shipment_matcher,
           )
         end
       end
@@ -24,7 +24,7 @@ module SolidusBacktracs
           response = client.bulk_create_orders(shipments)
         rescue RateLimitedError => e
           ::Spree::Event.fire(
-            'solidus_backtracs.api.rate_limited',
+            'solidus_bactracs.api.rate_limited',
             shipments: shipments,
             error: e,
           )
@@ -32,7 +32,7 @@ module SolidusBacktracs
           raise e
         rescue RequestError => e
           ::Spree::Event.fire(
-            'solidus_backtracs.api.sync_errored',
+            'solidus_bactracs.api.sync_errored',
             shipments: shipments,
             error: e,
           )

data/lib/{solidus_backtracs → solidus_bactracs}/api/client.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   module Api
     class Client
       class << self
         def from_config
           new(
             request_runner: RequestRunner.new,
-            error_handler: SolidusBacktracs.config.error_handler,
-            shipment_serializer: SolidusBacktracs.config.api_shipment_serializer,
+            error_handler: SolidusBactracs.config.error_handler,
+            shipment_serializer: SolidusBactracs.config.api_shipment_serializer,
           )
         end
       end
@@ -23,7 +23,7 @@ module SolidusBacktracs
 
       def bulk_create_orders(shipments)
         shipments.each do |shipment|
-          SolidusBacktracs::Api::SyncShipmentJob.perform_now(
+          SolidusBactracs::Api::SyncShipmentJob.perform_now(
             shipment_id: shipment.id,
             error_handler: @error_handler,
             shipment_serializer: @shipment_serializer,

data/lib/{solidus_backtracs → solidus_bactracs}/api/rate_limited_error.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   module Api
     class RateLimitedError < RequestError
       attr_reader :retry_in

data/lib/{solidus_backtracs → solidus_bactracs}/api/request_error.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   module Api
     class RequestError < RuntimeError
       attr_reader :response_code, :response_body, :response_headers

data/lib/solidus_bactracs/api/request_runner.rb

@@ -0,0 +1,135 @@
+# frozen_string_literal: true
+require "uri"
+require "net/http"
+
+module SolidusBactracs
+  module Api
+    class RequestRunner
+
+      def initialize
+        @username = SolidusBactracs.configuration.authentication_username
+        @password = SolidusBactracs.configuration.authentication_password
+        @api_base = SolidusBactracs.configuration.api_base
+        @retries  = SolidusBactracs.configuration.api_retries
+      end
+
+      def authenticated_call(method: nil, path: nil, serializer: nil, shipment: nil, count: 0)
+        if count <= @retries
+          sguid = authenticate! rescue nil
+
+          if !sguid.presence
+            clear_cache
+            count += 1
+            self.authenticated_call(method: :post, path: '/webservices/rma/rmaservice.asmx', serializer: serializer, shipment: shipment, count: count)
+          else
+            params = serializer.call(shipment, sguid)
+
+            rma_response = call(method: :post, path: path, params: params)
+            if create_rma_success?(rma_response)
+              Rails.logger.info({ event: 'success CreateRMA', rma: shipment.number, response: parse_rma_creation_response(rma_response, "Message")})
+              shipment_synced(shipment)
+              return true
+            elsif rma_exists?(rma_response)
+              return false
+            else
+              clear_cache
+              count += 1
+              Rails.logger.warn({ event: 'bactracs failed CreateRMA', error: parse_rma_creation_response(rma_response, "Message")})
+              self.authenticated_call(method: :post, path: '/webservices/rma/rmaservice.asmx', serializer: serializer, shipment: shipment, count: count)
+            end
+          end
+        else
+          shipment_sync_failed(shipment)
+        end
+      end
+
+
+      def call(method: nil, path: nil, params: {})
+        doc = {}
+        if params.present?
+          doc = Nokogiri::XML(params.to_s)
+        end
+        response = HTTParty.send(
+          method,
+          URI.join(@api_base, path),
+          body: doc.to_xml,
+          http_proxyaddr: SolidusBactracs.configuration.proxy_address,
+          http_proxyport: SolidusBactracs.configuration.proxy_port,
+          http_proxyuser: SolidusBactracs.configuration.proxy_username,
+          http_proxypass: SolidusBactracs.configuration.proxy_password,
+          headers: {
+            'Content-Type' => 'text/xml',
+          },
+        )
+
+        case response.code.to_s
+        when /2\d{2}/
+          response
+        when '429'
+          raise RateLimitedError.from_response(response)
+        else
+          raise RequestError.from_response(response)
+        end
+      end
+
+      def clear_cache
+        Rails.cache.delete('bactracks_cache_key')
+        @response = nil
+      end
+
+      def authenticate!()
+        unless @username.present? || @password.present? || @api_base.present?
+          raise "Credentials not defined for Authentication"
+        end
+
+        @response = Rails.cache.fetch("backtracks_cache_key", expires_in: 1.hour, skip_nil: true) do
+          self.call(method: :get, path: "/webservices/user/Authentication.asmx/Login?sUserName=#{@username}&sPassword=#{@password}")
+        end
+
+        raise RequestError.from_response(@response) unless @response # just try again for @retries?
+        if "false" == parse_authentication_response(@response, "Result")
+          Rails.logger.warn({ event: 'bactracs auth failed', error: parse_authentication_response(@response, "Message")})
+          raise RequestError.from_response(@response)
+        end
+        sguid = parse_authentication_response(@response, "Message")
+
+        return sguid
+      end
+
+      def parse_authentication_response(response, field)
+        response.dig("AuthenticationResponse", field)
+      end
+
+      def parse_rma_creation_response(response, field = "Result")
+        response.dig("Envelope", "Body", "CreateNewResponse", "CreateNewResult", field).to_s.downcase
+      end
+
+      def create_rma_success?(response)
+        parse_rma_creation_response(response) == 'true' && parse_rma_creation_response(response, "Message") == "ok"
+      end
+
+      def rma_exists?(response)
+        if parse_rma_creation_response(response, "Message").match(/rma .* already exists/)
+          Rails.logger.warn({ event: 'bactracs failed CreateRMA', error: parse_rma_creation_response(rma_response, "Message")})
+          return true
+        end
+      end
+
+      def shipment_synced(shipment)
+        shipment.update_column(:bactracs_synced_at, Time.zone.now)
+
+        ::Spree::Event.fire(
+          'solidus_bactracs.api.sync_completed',
+          shipment: shipment
+        )
+      end
+
+      def shipment_sync_failed(shipment)
+        ::Spree::Event.fire(
+          'solidus_bactracs.api.sync_failed',
+          shipment: shipment
+        )
+      end
+    end
+  end
+end

data/lib/{solidus_backtracs → solidus_bactracs}/api/shipment_serializer.rb

@@ -1,20 +1,20 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   module Api
     class ShipmentSerializer
 
       def initialize(shipment:)
         @shipment = shipment
-        @config = SolidusBacktracs.config
-        @property_id = ::Spree::Property.find_by(name: SolidusBacktracs.config.default_property_name)&.id
+        @config = SolidusBactracs.config
+        @property_id = ::Spree::Property.find_by(name: SolidusBactracs.config.default_property_name)&.id
       end
 
       def call(sguid: nil)
         order = @shipment.order
         user = @shipment.user
 
-        xml = Builder::XmlMarkup.new 
+        xml = Builder::XmlMarkup.new
         xml.instruct!(:xml, :encoding => "UTF-8")
 
         xml.soap(:Envelope, {"xmlns:xsi" => "http://www.w3.org/2001/XMLSchema-instance", "xmlns:xsd" => "http://www.w3.org/2001/XMLSchema", "xmlns:soap" => "http://schemas.xmlsoap.org/soap/envelope/"}) do
@@ -24,28 +24,28 @@ module SolidusBacktracs
               xml.NewRMA {
                 xml.RMANumber                 @shipment.number
                 xml.RMATypeName               @config.default_rma_type
-                xml.RMASubTypeName            
-                xml.CustomerRef               
-                xml.InboundShippingPriority   
+                xml.RMASubTypeName
+                xml.CustomerRef
+                xml.InboundShippingPriority
                 xml.InboundTrackingNumber     @shipment.tracking
 
                 xml.Ship {
                   xml.Carrier                 @config.default_carrier
                   xml.ShipMethod              @config.default_ship_method
-                  xml.ShipDate                @shipment.created_at.strftime(SolidusBacktracs::ExportHelper::BACTRACS_DATE_FORMAT)
+                  xml.ShipDate                @shipment.created_at.strftime(SolidusBactracs::ExportHelper::BACTRACS_DATE_FORMAT)
                   xml.TrackingNumber          @shipment.tracking
                   xml.SerialNumber            @shipment.number
-                  xml.Ud1                     
+                  xml.Ud1
                 }
                 xml.Customer {
-                  SolidusBacktracs::ExportHelper.backtracs_address(xml, order, :ship)
-                  SolidusBacktracs::ExportHelper.backtracs_address(xml, order, :bill)
+                  SolidusBactracs::ExportHelper.bactracs_address(xml, order, :ship)
+                  SolidusBactracs::ExportHelper.bactracs_address(xml, order, :bill)
                 }
                 xml.Rep {
-                  xml.Code                    
+                  xml.Code
                   xml.Name                    user.full_name
                   xml.Email                   user.email
-                }        
+                }
 
                 xml.RMALines {
                   @shipment.line_items.each do |line|
@@ -60,39 +60,40 @@ module SolidusBacktracs
                     end
                   end
                 }
-                xml.OrderDate     order.completed_at.strftime(SolidusBacktracs::ExportHelper::BACTRACS_DATE_FORMAT)
-                xml.CreateDate    @shipment.created_at.strftime(SolidusBacktracs::ExportHelper::BACTRACS_DATE_FORMAT)
+                xml.OrderDate     order.completed_at.strftime(SolidusBactracs::ExportHelper::BACTRACS_DATE_FORMAT)
+                xml.CreateDate    @shipment.created_at.strftime(SolidusBactracs::ExportHelper::BACTRACS_DATE_FORMAT)
                 xml.Status        @config.default_status
                 xml.RMAId         @shipment.id
                 xml.ClientGuid
               }
             end
-          end 
+          end
         end
+        Rails.logger.info(xml.to_s)
         xml
       end
 
       def line_items_xml(xml: nil, line_item: nil, variant: nil, order: nil)
-        shipment_notice = @shipment.shipment_notice        
+        shipment_notice = @shipment.shipment_notice
         xml.RMALine {
           xml.DFItem                  find_sku_variant(variant)
           xml.DFModelNum              find_sku_variant(variant)
-          xml.DFCategory              
-          xml.DFCategoryDescription   
+          xml.DFCategory
+          xml.DFCategoryDescription
           xml.DFQuantity              line_item.quantity
           xml.DFUnitPrice             line_item.price
-          xml.DFSerialNumbers         
-          xml.Ud1s                    
-          xml.CurrentWarranties       
-          xml.DFComments              
+          xml.DFSerialNumbers
+          xml.Ud1s
+          xml.CurrentWarranties
+          xml.DFComments
           xml.DFStatus                @shipment.state
-          xml.PurchaseDate            order.completed_at.strftime(SolidusBacktracs::ExportHelper::BACTRACS_DATE_FORMAT)
+          xml.PurchaseDate            order.completed_at.strftime(SolidusBactracs::ExportHelper::BACTRACS_DATE_FORMAT)
           xml.ServiceProvider         shipment_notice&.service
-          xml.WarrantyRepair          
-          xml.RMALineTest             
+          xml.WarrantyRepair
+          xml.RMALineTest
           xml.InboundShipWeight       variant.weight.to_f
           xml.RPLocation              @config.default_rp_location
-        } 
+        }
       end
 
       def find_sku_variant(variant)

data/lib/{solidus_backtracs → solidus_bactracs}/api/threshold_verifier.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   module Api
     class ThresholdVerifier
       class << self
@@ -13,14 +13,14 @@ module SolidusBacktracs
         private
 
         def shipment_requires_creation?(shipment)
-          shipment.backtracs_synced_at.nil? &&
-            Time.zone.now - shipment.order.updated_at < SolidusBacktracs.config.api_sync_threshold
+          shipment.bactracs_synced_at.nil? &&
+            Time.zone.now - shipment.order.updated_at < SolidusBactracs.config.api_sync_threshold
         end
 
         def shipment_requires_update?(shipment)
-          shipment.backtracs_synced_at &&
-            shipment.backtracs_synced_at < shipment.order.updated_at &&
-            Time.zone.now - shipment.order.updated_at < SolidusBacktracs.config.api_sync_threshold
+          shipment.bactracs_synced_at &&
+            shipment.bactracs_synced_at < shipment.order.updated_at &&
+            Time.zone.now - shipment.order.updated_at < SolidusBactracs.config.api_sync_threshold
         end
       end
     end

data/lib/{solidus_backtracs → solidus_bactracs}/configuration.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   class Configuration
     attr_accessor(
       :webhook_username,
@@ -41,11 +41,11 @@ module SolidusBacktracs
       @error_handler = ->(_error, _extra = {}) {
         Rails.logger.error "#{error.inspect} (#{extra.inspect})"
       }
-      @api_shipment_matcher = proc do |backtracs_order, shipments|
-        shipments.find { |shipment| shipment.number == backtracs_order['orderNumber'] }
+      @api_shipment_matcher = proc do |bactracs_order, shipments|
+        shipments.find { |shipment| shipment.number == bactracs_order['orderNumber'] }
       end
 
-      @shipment_notice_class = 'SolidusBacktracs::ShipmentNotice'
+      @shipment_notice_class = 'SolidusBactracs::ShipmentNotice'
     end
   end
 

data/lib/{solidus_backtracs → solidus_bactracs}/engine.rb

@@ -3,13 +3,13 @@
 require 'solidus_core'
 require 'solidus_support'
 
-module SolidusBacktracs
+module SolidusBactracs
   class Engine < Rails::Engine
     include SolidusSupport::EngineExtensions
 
     isolate_namespace ::Spree
 
-    engine_name 'solidus_backtracs'
+    engine_name 'solidus_bactracs'
 
     # use rspec for tests
     config.generators do |g|

data/lib/{solidus_backtracs → solidus_bactracs}/errors.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   class Error < StandardError; end
 
   class OrderNotPaidError < Error

data/lib/{solidus_backtracs → solidus_bactracs}/shipment_notice.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-module SolidusBacktracs
+module SolidusBactracs
   class ShipmentNotice
     attr_reader :shipment_number, :shipment_tracking
 
@@ -38,7 +38,7 @@ module SolidusBacktracs
     def process_payment
       return if shipment.order.paid?
 
-      unless SolidusBacktracs.configuration.capture_at_notification
+      unless SolidusBactracs.configuration.capture_at_notification
         raise OrderNotPaidError, shipment.order
       end
 

data/lib/solidus_bactracs/version.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+module SolidusBactracs
+  VERSION = '3.1.0'
+end

data/lib/solidus_bactracs.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require 'httparty'
+
+require 'solidus_bactracs/api/batch_syncer'
+require 'solidus_bactracs/api/request_runner'
+require 'solidus_bactracs/api/client'
+require 'solidus_bactracs/api/request_error'
+require 'solidus_bactracs/api/rate_limited_error'
+require 'solidus_bactracs/api/shipment_serializer'
+require 'solidus_bactracs/api/threshold_verifier'
+require 'solidus_bactracs/configuration'
+require 'solidus_bactracs/errors'
+require 'solidus_bactracs/shipment_notice'
+require 'solidus_bactracs/version'
+require 'solidus_bactracs/engine'

data/solidus_bactracs.gemspec

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require_relative 'lib/solidus_backtracs/version'
+require_relative 'lib/solidus_bactracs/version'
 
 Gem::Specification.new do |spec|
   spec.name = 'solidus_bactracs'
-  spec.version = SolidusBacktracs::VERSION
+  spec.version = SolidusBactracs::VERSION
   spec.authors = ['Zeryab Ali', 'Zachary Jones']
   spec.email = 'techbilling@suvie.com'
 
@@ -15,6 +15,7 @@ Gem::Specification.new do |spec|
   spec.metadata['homepage_uri'] = spec.homepage
   spec.metadata['source_code_uri'] = 'https://github.com/suvie-eng/solidus_bactracs/'
   spec.metadata['changelog_uri'] = 'https://github.com/suvie-eng/solidus_bactracs/blob/master/CHANGELOG.md'
+  spec.metadata['allowed_push_host'] = "https://rubygems.org"
 
   spec.required_ruby_version = Gem::Requirement.new('>= 2.5')
 

data/spec/controllers/spree/{backtracs_controller_spec.rb → bactracs_controller_spec.rb}

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-RSpec.describe Spree::BacktracsController do
+RSpec.describe Spree::BactracsController do
   render_views
 
   describe '#export' do
@@ -14,7 +14,7 @@ RSpec.describe Spree::BacktracsController do
 
     context 'when the authentication is valid' do
       it 'responds with 200 OK' do
-        stub_backtracs_auth
+        stub_bactracs_auth
         create(:order_ready_to_ship)
 
         get :export,
@@ -27,8 +27,8 @@ RSpec.describe Spree::BacktracsController do
         expect(response.status).to eq(200)
       end
 
-      it 'generates Backtracs-compliant XML' do
-        stub_backtracs_auth
+      it 'generates Bactracs-compliant XML' do
+        stub_bactracs_auth
         create(:order_ready_to_ship)
 
         get :export, params: {
@@ -37,7 +37,7 @@ RSpec.describe Spree::BacktracsController do
           format: 'xml'
         }
 
-        expect(response.body).to pass_validation('spec/fixtures/backtracs_xml_schema.xsd')
+        expect(response.body).to pass_validation('spec/fixtures/bactracs_xml_schema.xsd')
       end
     end
   end
@@ -46,7 +46,7 @@ RSpec.describe Spree::BacktracsController do
     context 'when the authentication is valid' do
       context 'when the shipment can be found' do
         it 'responds with 200 OK' do
-          stub_backtracs_auth
+          stub_bactracs_auth
           shipment = create(:order_ready_to_ship).shipments.first
 
           post :shipnotify, params: {
@@ -60,7 +60,7 @@ RSpec.describe Spree::BacktracsController do
         end
 
         it 'updates the shipment' do
-          stub_backtracs_auth
+          stub_bactracs_auth
           shipment = create(:order_ready_to_ship).shipments.first
 
           post :shipnotify, params: {
@@ -80,7 +80,7 @@ RSpec.describe Spree::BacktracsController do
 
       context 'when the shipment cannot be found' do
         it 'responds with 400 Bad Request' do
-          stub_backtracs_auth
+          stub_bactracs_auth
           shipment = create(:order_ready_to_ship).shipments.first
 
           post :shipnotify, params: {
@@ -96,7 +96,7 @@ RSpec.describe Spree::BacktracsController do
     end
   end
 
-  def stub_backtracs_auth(username = 'mario', password = 'lemieux')
+  def stub_bactracs_auth(username = 'mario', password = 'lemieux')
     stub_configuration(username: username, password: password)
     request.headers['Authorization'] = ActionController::HttpAuthentication::Basic.encode_credentials(username, password)
   end