netsuite 0.2.4 → 0.2.5

data/spec/netsuite/records/customer_refund_spec.rb

@@ -87,7 +87,7 @@ describe NetSuite::Records::CustomerRefund do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :is_person => true }) }
 
       it 'returns an CustomerRefund instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::CustomerRefund, :external_id => 10).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::CustomerRefund, {:external_id => 10}], {}).and_return(response)
         refund = NetSuite::Records::CustomerRefund.get(:external_id => 10)
         refund.should be_kind_of(NetSuite::Records::CustomerRefund)
       end
@@ -97,7 +97,7 @@ describe NetSuite::Records::CustomerRefund do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::CustomerRefund, :external_id => 10).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::CustomerRefund, {:external_id => 10}], {}).and_return(response)
         lambda {
           NetSuite::Records::CustomerRefund.get(:external_id => 10)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -109,14 +109,14 @@ describe NetSuite::Records::CustomerRefund do
   describe '.initialize' do
     context 'when the request is successful' do
       it 'returns an initialized invoice from the customer entity' do
-        NetSuite::Actions::Initialize.should_receive(:call).with(NetSuite::Records::CustomerRefund, memo).and_return(response)
+        NetSuite::Actions::Initialize.should_receive(:call).with([NetSuite::Records::CustomerRefund, memo], {}).and_return(response)
         refund = NetSuite::Records::CustomerRefund.initialize(memo)
         refund.should be_kind_of(NetSuite::Records::CustomerRefund)
       end
     end
 
     context 'when the response is unsuccessful' do
-      pending
+      skip
     end
   end
 
@@ -129,7 +129,7 @@ describe NetSuite::Records::CustomerRefund do
       it 'returns true' do
         refund = NetSuite::Records::CustomerRefund.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(refund).
+            with([refund], {}).
             and_return(response)
         refund.add.should be_true
       end
@@ -141,7 +141,7 @@ describe NetSuite::Records::CustomerRefund do
       it 'returns false' do
         refund = NetSuite::Records::CustomerRefund.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(refund).
+            with([refund], {}).
             and_return(response)
         refund.add.should be_false
       end
@@ -155,7 +155,7 @@ describe NetSuite::Records::CustomerRefund do
       it 'returns true' do
         refund = NetSuite::Records::CustomerRefund.new
         NetSuite::Actions::Delete.should_receive(:call).
-            with(refund).
+            with([refund], {}).
             and_return(response)
         refund.delete.should be_true
       end
@@ -167,7 +167,7 @@ describe NetSuite::Records::CustomerRefund do
       it 'returns false' do
         refund = NetSuite::Records::CustomerRefund.new
         NetSuite::Actions::Delete.should_receive(:call).
-            with(refund).
+            with([refund], {}).
             and_return(response)
         refund.delete.should be_false
       end

data/spec/netsuite/records/customer_spec.rb

@@ -6,19 +6,19 @@ describe NetSuite::Records::Customer do
   it 'has all the right fields' do
     [
       :access_role, :account_number, :aging, :alt_email, :alt_name, :alt_phone, :balance, :bill_pay,
-      :buying_reason, :buying_time_frame, :campaign_category, :category, :click_stream, :comments, :company_name,
+      :buying_reason, :buying_time_frame, :campaign_category, :click_stream, :comments, :company_name,
       :consol_aging, :consol_balance, :consol_days_overdue, :consol_deposit_balance, :consol_overdue_balance,
       :consol_unbilled_orders, :contact_roles_list, :contrib_pct, :credit_cards_list, :credit_hold_override, :credit_limit,
       :currency, :currency_list, :date_created, :days_overdue, :default_address,
       :deposit_balance, :download_list, :email, :email_preference, :email_transactions, :end_date, :entity_id,
       :estimated_budget, :fax, :fax_transactions, :first_name, :first_visit, :give_access, :global_subscription_status,
       :group_pricing_list, :home_phone, :image, :is_budget_approved, :is_inactive, :is_person, :item_pricing_list, :keywords,
-      :language, :last_modified_date, :last_name, :last_page_visited, :last_visit, :lead_source, :middle_name, :mobile_phone,
+      :language, :last_modified_date, :last_name, :last_page_visited, :last_visit, :middle_name, :mobile_phone,
       :opening_balance, :opening_balance_account, :opening_balance_date, :overdue_balance, :parent, :partners_list,
-      :password, :password2, :phone, :phonetic_name, :pref_cc_processor, :price_level, :print_on_check_as,
+      :password, :password2, :phone, :phonetic_name, :pref_cc_processor,:print_on_check_as,
       :print_transactions, :referrer, :reminder_days, :representing_subsidiary, :require_pwd_change, :resale_number,
       :sales_group, :sales_readiness, :sales_rep, :sales_team_list, :salutation, :send_email, :ship_complete, :shipping_item,
-      :stage, :start_date, :subscriptions_list, :subsidiary, :sync_partner_teams, :tax_exempt, :tax_item, :taxable, :terms,
+      :stage, :start_date, :subscriptions_list, :sync_partner_teams, :tax_exempt, :tax_item, :taxable, :terms,
       :territory, :third_party_acct, :third_party_country, :third_party_zipcode, :title, :unbilled_orders, :url,
       :vat_reg_number, :visits, :web_lead
     ].each do |field|
@@ -88,7 +88,7 @@ describe NetSuite::Records::Customer do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :is_person => true }) }
 
       it 'returns a Customer instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Customer, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Customer, {:external_id => 1}], {}).and_return(response)
         customer = NetSuite::Records::Customer.get(:external_id => 1)
         customer.should be_kind_of(NetSuite::Records::Customer)
         customer.is_person.should be_true
@@ -99,7 +99,7 @@ describe NetSuite::Records::Customer do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Customer, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Customer, {:external_id => 1}], {}).and_return(response)
         lambda {
           NetSuite::Records::Customer.get(:external_id => 1)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -116,7 +116,7 @@ describe NetSuite::Records::Customer do
 
       it 'returns true' do
         NetSuite::Actions::Add.should_receive(:call).
-            with(customer).
+            with([customer], {}).
             and_return(response)
         customer.add.should be_true
       end
@@ -127,7 +127,7 @@ describe NetSuite::Records::Customer do
 
       it 'returns false' do
         NetSuite::Actions::Add.should_receive(:call).
-            with(customer).
+            with([customer], {}).
             and_return(response)
         customer.add.should be_false
       end
@@ -140,7 +140,7 @@ describe NetSuite::Records::Customer do
 
       it 'returns true' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(customer).
+            with([customer], {}).
             and_return(response)
         customer.delete.should be_true
       end
@@ -151,7 +151,7 @@ describe NetSuite::Records::Customer do
 
       it 'returns false' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(customer).
+            with([customer], {}).
             and_return(response)
         customer.delete.should be_false
       end

data/spec/netsuite/records/department_spec.rb

@@ -24,7 +24,7 @@ describe NetSuite::Records::Department do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :name => 'Department 1' }) }
 
       it 'returns a Department instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Department, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Department, {:external_id => 1}], {}).and_return(response)
         department = NetSuite::Records::Department.get(:external_id => 1)
         department.should be_kind_of(NetSuite::Records::Department)
         department.name.should eql('Department 1')
@@ -35,7 +35,7 @@ describe NetSuite::Records::Department do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Department, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Department, {:external_id => 1}], {}).and_return(response)
         lambda {
           NetSuite::Records::Department.get(:external_id => 1)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -53,7 +53,7 @@ describe NetSuite::Records::Department do
       it 'returns true' do
         department = NetSuite::Records::Department.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(department).
+            with([department], {}).
             and_return(response)
         department.add.should be_true
       end
@@ -65,7 +65,7 @@ describe NetSuite::Records::Department do
       it 'returns false' do
         department = NetSuite::Records::Department.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(department).
+            with([department], {}).
             and_return(response)
         department.add.should be_false
       end
@@ -81,7 +81,7 @@ describe NetSuite::Records::Department do
       it 'returns true' do
         department = NetSuite::Records::Department.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(department).
+            with([department], {}).
             and_return(response)
         department.delete.should be_true
       end
@@ -93,7 +93,7 @@ describe NetSuite::Records::Department do
       it 'returns false' do
         department = NetSuite::Records::Department.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(department).
+            with([department], {}).
             and_return(response)
         department.delete.should be_false
       end

data/spec/netsuite/records/deposit_spec.rb

@@ -44,13 +44,13 @@ describe NetSuite::Records::Deposit do
 
   describe '.get' do
     context 'when the response is successful' do
-      let(:response) { NetSuite::Response.new(:success => true, :body => { :alt_shipping_cost => 100 }) }
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :memo => 'transfer for subscriptions' }) }
 
       it 'returns a Deposit instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Deposit, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Deposit, {:external_id => 1}], {}).and_return(response)
         deposit = NetSuite::Records::Deposit.get(:external_id => 1)
         deposit.should be_kind_of(NetSuite::Records::Deposit)
-        deposit.alt_shipping_cost.should eql(100)
+        deposit.memo.should eql('transfer for subscriptions')
       end
     end
 
@@ -58,7 +58,7 @@ describe NetSuite::Records::Deposit do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Deposit, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Deposit, {:external_id => 1}], {}).and_return(response)
         lambda {
           NetSuite::Records::Deposit.get(:external_id => 1)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -76,7 +76,7 @@ describe NetSuite::Records::Deposit do
       it 'returns true' do
         deposit = NetSuite::Records::Deposit.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(deposit).
+            with([deposit], {}).
             and_return(response)
         deposit.add.should be_true
       end
@@ -88,7 +88,7 @@ describe NetSuite::Records::Deposit do
       it 'returns false' do
         deposit = NetSuite::Records::Deposit.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(deposit).
+            with([deposit], {}).
             and_return(response)
         deposit.add.should be_false
       end
@@ -104,7 +104,7 @@ describe NetSuite::Records::Deposit do
       it 'returns true' do
         deposit = NetSuite::Records::Deposit.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(deposit).
+            with([deposit], {}).
             and_return(response)
         deposit.delete.should be_true
       end
@@ -116,7 +116,7 @@ describe NetSuite::Records::Deposit do
       it 'returns false' do
         deposit = NetSuite::Records::Deposit.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(deposit).
+            with([deposit], {}).
             and_return(response)
         deposit.delete.should be_false
       end
@@ -125,12 +125,12 @@ describe NetSuite::Records::Deposit do
 
   describe '#to_record' do
     before do
-      deposit.email   = 'something@example.com'
+      deposit.memo   = 'something@example.com'
       deposit.tran_id = '4'
     end
     it 'can represent itself as a SOAP record' do
       record = {
-        'tranBank:email'  => 'something@example.com',
+        'tranBank:memo'  => 'something@example.com',
         'tranBank:tranId' => '4'
       }
       deposit.to_record.should eql(record)

data/spec/netsuite/records/inventory_item_spec.rb

@@ -104,7 +104,7 @@ describe NetSuite::Records::InventoryItem do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :cost => 100 }) }
 
       it 'returns a InventoryItem instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::InventoryItem, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::InventoryItem, {:external_id => 1}], {}).and_return(response)
         item = NetSuite::Records::InventoryItem.get(:external_id => 1)
         item.should be_kind_of(NetSuite::Records::InventoryItem)
         item.cost.should eql(100)
@@ -115,7 +115,7 @@ describe NetSuite::Records::InventoryItem do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::InventoryItem, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::InventoryItem, {:external_id => 1}], {}).and_return(response)
         lambda {
           NetSuite::Records::InventoryItem.get(:external_id => 1)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -132,7 +132,7 @@ describe NetSuite::Records::InventoryItem do
 
       it 'returns true' do
         NetSuite::Actions::Add.should_receive(:call).
-            with(item).
+            with([item], {}).
             and_return(response)
         item.add.should be_true
       end
@@ -143,7 +143,7 @@ describe NetSuite::Records::InventoryItem do
 
       it 'returns false' do
         NetSuite::Actions::Add.should_receive(:call).
-            with(item).
+            with([item], {}).
             and_return(response)
         item.add.should be_false
       end
@@ -156,7 +156,7 @@ describe NetSuite::Records::InventoryItem do
 
       it 'returns true' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(item).
+            with([item], {}).
             and_return(response)
         item.delete.should be_true
       end
@@ -167,7 +167,7 @@ describe NetSuite::Records::InventoryItem do
 
       it 'returns false' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(item).
+            with([item], {}).
             and_return(response)
         item.delete.should be_false
       end

data/spec/netsuite/records/invoice_spec.rb

@@ -137,7 +137,7 @@ describe NetSuite::Records::Invoice do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :is_person => true }) }
 
       it 'returns an Invoice instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Invoice, :external_id => 10).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Invoice, {:external_id => 10}], {}).and_return(response)
         invoice = NetSuite::Records::Invoice.get(:external_id => 10)
         invoice.should be_kind_of(NetSuite::Records::Invoice)
       end
@@ -147,7 +147,7 @@ describe NetSuite::Records::Invoice do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Invoice, :external_id => 10).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Invoice, {:external_id => 10}], {}).and_return(response)
         lambda {
           NetSuite::Records::Invoice.get(:external_id => 10)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -159,14 +159,14 @@ describe NetSuite::Records::Invoice do
   describe '.initialize' do
     context 'when the request is successful' do
       it 'returns an initialized invoice from the customer entity' do
-        NetSuite::Actions::Initialize.should_receive(:call).with(NetSuite::Records::Invoice, customer).and_return(response)
+        NetSuite::Actions::Initialize.should_receive(:call).with([NetSuite::Records::Invoice, customer], {}).and_return(response)
         invoice = NetSuite::Records::Invoice.initialize(customer)
         invoice.should be_kind_of(NetSuite::Records::Invoice)
       end
     end
 
     context 'when the response is unsuccessful' do
-      pending
+      skip
     end
   end
 
@@ -179,7 +179,7 @@ describe NetSuite::Records::Invoice do
       it 'returns true' do
         invoice = NetSuite::Records::Invoice.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(invoice).
+            with([invoice], {}).
             and_return(response)
         invoice.add.should be_true
       end
@@ -191,7 +191,7 @@ describe NetSuite::Records::Invoice do
       it 'returns false' do
         invoice = NetSuite::Records::Invoice.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(invoice).
+            with([invoice], {}).
             and_return(response)
         invoice.add.should be_false
       end
@@ -205,7 +205,7 @@ describe NetSuite::Records::Invoice do
       it 'returns true' do
         invoice = NetSuite::Records::Invoice.new
         NetSuite::Actions::Delete.should_receive(:call).
-            with(invoice).
+            with([invoice], {}).
             and_return(response)
         invoice.delete.should be_true
       end
@@ -217,7 +217,7 @@ describe NetSuite::Records::Invoice do
       it 'returns false' do
         invoice = NetSuite::Records::Invoice.new
         NetSuite::Actions::Delete.should_receive(:call).
-            with(invoice).
+            with([invoice], {}).
             and_return(response)
         invoice.delete.should be_false
       end

data/spec/netsuite/records/item_fulfillment_package_list_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe NetSuite::Records::ItemFulfillmentPackageList do
+  let(:list) { NetSuite::Records::ItemFulfillmentPackageList.new }
+
+  it 'has a packages attribute' do
+    list.packages.should be_kind_of(Array)
+  end
+
+  describe '#to_record' do
+    before do
+      list.packages << NetSuite::Records::ItemFulfillmentPackage.new(
+        :package_tracking_number => '1Z12354645757686786'
+      )
+    end
+
+    it 'can represent itself as a SOAP record' do
+      record = {
+          'tranSales:package' => [
+            'tranSales:packageTrackingNumber' => '1Z12354645757686786'
+          ]
+        }
+      
+      list.to_record.should eql(record)
+    end
+  end
+end

data/spec/netsuite/records/item_fulfillment_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+module NetSuite
+  module Records
+    describe ItemFulfillment do
+      context "unify package list attributes" do
+        let(:package_ups_list) do
+          {
+            :package_weight_ups=>"4.0",
+            :package_tracking_number_ups=>"1ZYA95390396947456",
+            :packaging_ups=>"_yourPackaging",
+            :use_insured_value_ups=>false,
+            :reference1_ups=>"spree@example.com",
+            :reference2_ups=>"spree@example.com",
+            :package_length_ups=>"24",
+            :package_width_ups=>"24",
+            :package_height_ups=>"24",
+            :additional_handling_ups=>false,
+            :use_cod_ups=>false
+          }
+        end
+
+        let(:list) do
+          { package_ups_list: { package_ups: package_ups_list } }
+        end
+
+        subject do
+          described_class.new(list)
+        end
+
+        it "access ups attributes via package_list" do
+          package = subject.package_list.packages.first
+          expect(package.package_tracking_number).to eq package_ups_list[:package_tracking_number_ups]
+        end
+
+        context "array" do
+          let(:list) do
+            { package_ups_list: { package_ups: [package_ups_list, package_ups_list] } }
+          end
+
+          it "access ups attributes via package_list" do
+            package = subject.package_list.packages.first
+            expect(package.package_tracking_number).to eq package_ups_list[:package_tracking_number_ups]
+          end
+        end
+      end
+    end
+  end
+end