Merge pull request #150 from AuthorizeNet/revert-131-bug-order-class-missing-attrs

Revert "adds missing attr_accessors to AuthorizeNet::Order class"

Author: anuragg29

lib/authorize_net/order.rb

@@ -1,13 +1,12 @@
 module AuthorizeNet
-
+  
   # Models an order.
   class Order
-
+    
     include AuthorizeNet::Model
-
-    attr_accessor :invoice_num, :description, :tax, :tax_amount, :tax_name, :tax_description, :freight, :freight_name, :freight_description, :duty, :duty_amount, :duty_name, :duty_description, :tax_exempt, :po_num, :line_items
-    attr_accessor :shipping_amount, :shipping_name, :shipping_description
-
+    
+    attr_accessor :invoice_num, :description, :tax, :tax_name, :tax_description, :freight, :freight_name, :freight_description, :duty, :duty_name, :duty_description, :tax_exempt, :po_num, :line_items
+    
     def add_line_item(id = nil, name = nil, description = nil, quantity = nil, price = nil, taxable = nil)
       if id.kind_of?(AuthorizeNet::LineItem)
         line_item = id
@@ -16,33 +15,28 @@ def add_line_item(id = nil, name = nil, description = nil, quantity = nil, price
       end
       @line_items = @line_items.to_a << line_item
     end
-
+    
     def to_hash
       hash = {
         :invoice_num => @invoice_num,
         :description => @description,
         :tax => @tax,
-        :tax_amount => @tax_amount,
         :tax_name => @tax_name,
         :tax_description => @tax_description,
         :freight => @freight,
         :freight_name => @freight_name,
         :freight_description => @freight_description,
         :duty => @duty,
-        :duty_amount => @duty_amount,
         :duty_name => @duty_name,
         :duty_description => @duty_description,
-        :shipping_amount => @shipping_amount,
-        :shipping_name => @shipping_name,
-        :shipping_description => @shipping_description,
         :tax_exempt => @tax_exempt,
         :po_num => @po_num,
         :line_items => handle_multivalue_hashing(@line_items)
       }
       hash.delete_if {|k, v| v.nil?}
       hash
     end
-
+    
   end
-
-end
+  
+end

spec/reporting_spec.rb

@@ -368,21 +368,6 @@
           </order>
           <authAmount>10.00</authAmount>
           <settleAmount>10.00</settleAmount>
-          <tax>
-            <amount>0.00</amount>
-            <name>tax name</name>
-            <description>tax description</description>
-          </tax>
-          <shipping>
-            <amount>0.00</amount>
-            <name>shipping name</name>
-            <description>shipping description</description>
-          </shipping>
-          <duty>
-            <amount>0.00</amount>
-            <name>duty name</name>
-            <description>duty description</description>
-          </duty>
           <lineItems>
             <lineItem>
               <itemId>ITEM00001</itemId>
@@ -453,18 +438,6 @@
       transaction.order.line_items[0].should be_kind_of(AuthorizeNet::LineItem)
       transaction.order.tax_exempt.should be_falsey
 
-      transaction.order.tax_amount.should == 0.00
-      transaction.order.tax_name.should == 'tax name'
-      transaction.order.tax_description.should == 'tax description'
-
-      transaction.order.duty_amount.should == 0.00
-      transaction.order.duty_name.should == 'duty name'
-      transaction.order.duty_description.should == 'duty description'
-
-      transaction.order.shipping_amount.should == 0.00
-      transaction.order.shipping_name.should == 'shipping name'
-      transaction.order.shipping_description.should == 'shipping description'
-
       transaction.payment_method.should be_kind_of(AuthorizeNet::CreditCard)
       transaction.payment_method.card_number.should == 'XXXX1111'