netsuite 0.2.6 → 0.3.0

data/spec/netsuite/records/inventory_item_spec.rb

@@ -134,7 +134,7 @@ describe NetSuite::Records::InventoryItem do
         NetSuite::Actions::Add.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.add.should be_true
+        item.add.should be_truthy
       end
     end
 
@@ -145,7 +145,7 @@ describe NetSuite::Records::InventoryItem do
         NetSuite::Actions::Add.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.add.should be_false
+        item.add.should be_falsey
       end
     end
   end
@@ -158,7 +158,7 @@ describe NetSuite::Records::InventoryItem do
         NetSuite::Actions::Delete.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.delete.should be_true
+        item.delete.should be_truthy
       end
     end
 
@@ -169,7 +169,7 @@ describe NetSuite::Records::InventoryItem do
         NetSuite::Actions::Delete.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.delete.should be_false
+        item.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/inventory_transfer_spec.rb

@@ -0,0 +1,58 @@
+require 'spec_helper'
+
+describe NetSuite::Records::InventoryTransfer do
+  let(:inventory_transfer) { NetSuite::Records::InventoryTransfer.new }
+
+  it 'has all the right fields' do
+    [
+      :created_date, :last_modified_date, :tran_date, :tran_id, :memo
+    ].each do |field|
+      inventory_transfer.should have_field(field)
+    end
+  end
+
+  it 'has all the right record refs' do
+    [
+      :posting_period, :location, :transfer_location, :department, :subsidiary
+    ].each do |record_ref|
+      inventory_transfer.should have_record_ref(record_ref)
+    end
+  end
+
+
+  describe '#inventory_list' do
+    it 'can be set from attributes' do
+      attributes = {
+        :inventory => [{
+          :inventory_detail => {
+            :custom_form => {
+              internal_id: 1
+            },
+            :inventory_assignment_list => {
+              :inventory_assignment => [{
+                quantity: 3
+              }]
+            }
+          }
+        }]
+      }
+      inventory_transfer.inventory_list = attributes
+      inventory_transfer.inventory_list.should be_kind_of(NetSuite::Records::InventoryTransferInventoryList)
+      inventory_transfer.inventory_list.inventory.length.should eql(1)
+
+      expect(
+        inventory_transfer.inventory_list.inventory.first.inventory_detail.inventory_assignment_list.inventory_assignment.first.quantity
+      ).to eq(3)
+
+      expect(
+        inventory_transfer.inventory_list.inventory.first.inventory_detail.custom_form.internal_id
+      ).to eq(1)
+    end
+
+    it 'can be set from a InventoryTransferInventoryList object' do
+      inventory_list = NetSuite::Records::InventoryTransferInventoryList.new
+      inventory_transfer.inventory_list = inventory_list
+      inventory_transfer.inventory_list.should eql(inventory_list)
+    end
+  end
+end

data/spec/netsuite/records/invoice_spec.rb

@@ -181,7 +181,7 @@ describe NetSuite::Records::Invoice do
         NetSuite::Actions::Add.should_receive(:call).
             with([invoice], {}).
             and_return(response)
-        invoice.add.should be_true
+        invoice.add.should be_truthy
       end
     end
 
@@ -193,7 +193,7 @@ describe NetSuite::Records::Invoice do
         NetSuite::Actions::Add.should_receive(:call).
             with([invoice], {}).
             and_return(response)
-        invoice.add.should be_false
+        invoice.add.should be_falsey
       end
     end
   end
@@ -207,7 +207,7 @@ describe NetSuite::Records::Invoice do
         NetSuite::Actions::Delete.should_receive(:call).
             with([invoice], {}).
             and_return(response)
-        invoice.delete.should be_true
+        invoice.delete.should be_truthy
       end
     end
 
@@ -219,7 +219,7 @@ describe NetSuite::Records::Invoice do
         NetSuite::Actions::Delete.should_receive(:call).
             with([invoice], {}).
             and_return(response)
-        invoice.delete.should be_false
+        invoice.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/job_spec.rb

@@ -111,7 +111,7 @@ describe NetSuite::Records::Job do
         NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Job, {:external_id => 1}], {}).and_return(response)
         job = NetSuite::Records::Job.get(:external_id => 1)
         job.should be_kind_of(NetSuite::Records::Job)
-        job.account_number.should be_true
+        job.account_number.should be_truthy
       end
     end
 
@@ -138,7 +138,7 @@ describe NetSuite::Records::Job do
         NetSuite::Actions::Add.should_receive(:call).
             with([job], {}).
             and_return(response)
-        job.add.should be_true
+        job.add.should be_truthy
       end
     end
 
@@ -149,7 +149,7 @@ describe NetSuite::Records::Job do
         NetSuite::Actions::Add.should_receive(:call).
             with([job], {}).
             and_return(response)
-        job.add.should be_false
+        job.add.should be_falsey
       end
     end
   end
@@ -162,7 +162,7 @@ describe NetSuite::Records::Job do
         NetSuite::Actions::Delete.should_receive(:call).
             with([job], {}).
             and_return(response)
-        job.delete.should be_true
+        job.delete.should be_truthy
       end
     end
 
@@ -173,7 +173,7 @@ describe NetSuite::Records::Job do
         NetSuite::Actions::Delete.should_receive(:call).
             with([job], {}).
             and_return(response)
-        job.delete.should be_false
+        job.delete.should be_falsey
       end
     end
   end
@@ -185,7 +185,7 @@ describe NetSuite::Records::Job do
       it 'returns true' do
         NetSuite::Actions::Update.should_receive(:call).with([NetSuite::Records::Job, {:external_id => 1, :account_number => 7}], {}).and_return(response)
         job = NetSuite::Records::Job.new(:external_id => 1)
-        job.update(:account_number => 7).should be_true
+        job.update(:account_number => 7).should be_truthy
       end
     end
 
@@ -195,7 +195,7 @@ describe NetSuite::Records::Job do
       it 'raises a RecordNotFound exception' do
         NetSuite::Actions::Update.should_receive(:call).with([NetSuite::Records::Job, {:internal_id => 1, :account_number => 7}], {}).and_return(response)
         job = NetSuite::Records::Job.new(:internal_id => 1)
-        job.update(:account_number => 7).should be_false
+        job.update(:account_number => 7).should be_falsey
       end
     end
   end

data/spec/netsuite/records/journal_entry_spec.rb

@@ -69,7 +69,7 @@ describe NetSuite::Records::JournalEntry do
         NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::JournalEntry, {:external_id => 1}], {}).and_return(response)
         customer = NetSuite::Records::JournalEntry.get(:external_id => 1)
         customer.should be_kind_of(NetSuite::Records::JournalEntry)
-        customer.approved.should be_true
+        customer.approved.should be_truthy
       end
     end
 
@@ -96,7 +96,7 @@ describe NetSuite::Records::JournalEntry do
         NetSuite::Actions::Add.should_receive(:call).
             with([entry], {}).
             and_return(response)
-        entry.add.should be_true
+        entry.add.should be_truthy
       end
     end
 
@@ -107,7 +107,7 @@ describe NetSuite::Records::JournalEntry do
         NetSuite::Actions::Add.should_receive(:call).
             with([entry], {}).
             and_return(response)
-        entry.add.should be_false
+        entry.add.should be_falsey
       end
     end
   end
@@ -120,7 +120,7 @@ describe NetSuite::Records::JournalEntry do
         NetSuite::Actions::Delete.should_receive(:call).
             with([entry], {}).
             and_return(response)
-        entry.delete.should be_true
+        entry.delete.should be_truthy
       end
     end
 
@@ -131,7 +131,7 @@ describe NetSuite::Records::JournalEntry do
         NetSuite::Actions::Delete.should_receive(:call).
             with([entry], {}).
             and_return(response)
-        entry.delete.should be_false
+        entry.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/non_inventory_sale_item_spec.rb

@@ -74,7 +74,7 @@ describe NetSuite::Records::NonInventorySaleItem do
         NetSuite::Actions::Add.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.add.should be_true
+        item.add.should be_truthy
       end
     end
 
@@ -85,7 +85,7 @@ describe NetSuite::Records::NonInventorySaleItem do
         NetSuite::Actions::Add.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.add.should be_false
+        item.add.should be_falsey
       end
     end
   end
@@ -98,7 +98,7 @@ describe NetSuite::Records::NonInventorySaleItem do
         NetSuite::Actions::Delete.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.delete.should be_true
+        item.delete.should be_truthy
       end
     end
 
@@ -109,7 +109,7 @@ describe NetSuite::Records::NonInventorySaleItem do
         NetSuite::Actions::Delete.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.delete.should be_false
+        item.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/payment_method_spec.rb

@@ -33,7 +33,7 @@ describe NetSuite::Records::PaymentMethod do
         NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::PaymentMethod, :external_id => 10], {}).and_return(response)
         payment_method = NetSuite::Records::PaymentMethod.get(:external_id => 10)
         payment_method.should be_kind_of(NetSuite::Records::PaymentMethod)
-        payment_method.is_debit_card.should be_true
+        payment_method.is_debit_card.should be_truthy
       end
     end
 

data/spec/netsuite/records/pomo_code_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+
+describe NetSuite::Records::PromotionCode do
+  let(:promo_code) { NetSuite::Records::PromotionCode.new }
+
+  it 'has all the right fields' do
+    [ :code, :code_pattern, :description, :discount_type, :display_line_discounts, :end_date,
+             :exclude_items, :is_inactive, :is_public, :minimum_order_amount, :name, :number_to_generate,
+             :rate, :start_date
+    ].each do |field|
+      promo_code.should have_field(field)
+    end
+  end
+
+  describe '.get' do
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :code => 'SUMMERSALE' }) }
+
+      it 'returns a Campaign instance populated with the data from the response object' do
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::PromotionCode, {:external_id => 1}], {}).and_return(response)
+        promo_code = NetSuite::Records::PromotionCode.get(:external_id => 1)
+        promo_code.should be_kind_of(NetSuite::Records::PromotionCode)
+        promo_code.code.should eql('SUMMERSALE')
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'raises a RecordNotFound exception' do
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::PromotionCode, {:external_id => 1}], {}).and_return(response)
+        lambda {
+          NetSuite::Records::PromotionCode.get(:external_id => 1)
+        }.should raise_error(NetSuite::RecordNotFound,
+          /NetSuite::Records::PromotionCode with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+end

data/spec/netsuite/records/sales_order_spec.rb

@@ -7,17 +7,17 @@ describe NetSuite::Records::SalesOrder do
 
   it 'has all the right fields' do
     [
-      :alt_handling_cost, :alt_shipping_cost, :amount_paid, :amount_remaining, :applied, :auto_apply, 
-      :balance, :bill_address, :contrib_pct, :created_date, :currency_name, :deferred_revenue, 
-      :discount_rate, :discount_total, :email, :end_date, :est_gross_profit, :est_gross_profit_percent, 
-      :exchange_rate, :exclude_commission, :fax, :gift_cert_applied, :gift_cert_available, 
-      :gift_cert_total, :handling_cost, :handling_tax1_rate, :handling_tax2_rate, :is_taxable, 
-      :last_modified_date, :memo, :message, :on_credit_hold, :other_ref_num, :recognized_revenue, 
-      :rev_rec_on_rev_commitment, :sales_effective_date, :shipping_cost, :shipping_tax1_rate, 
-      :shipping_tax2_rate, :source, :start_date, :status, :sub_total, :sync_partner_teams, 
-      :sync_sales_teams, :tax2_total, :tax_rate, :tax_total, :to_be_emailed, :to_be_faxed, :to_be_printed, 
-      :total, :total_cost_estimate, :tran_date, :tran_id, :tran_is_vsoe_bundle, :unapplied, :vat_reg_num, 
-      :vsoe_auto_calc
+      :alt_handling_cost, :alt_shipping_cost, :amount_paid, :amount_remaining, :applied, :auto_apply,
+      :balance, :bill_address, :contrib_pct, :created_date, :currency_name, :deferred_revenue,
+      :discount_rate, :discount_total, :email, :end_date, :est_gross_profit, :est_gross_profit_percent,
+      :exchange_rate, :exclude_commission, :fax, :gift_cert_applied, :gift_cert_available,
+      :gift_cert_total, :handling_cost, :handling_tax1_rate, :handling_tax2_rate, :is_taxable,
+      :last_modified_date, :memo, :message, :on_credit_hold, :other_ref_num, :recognized_revenue,
+      :rev_rec_on_rev_commitment, :sales_effective_date, :shipping_cost, :shipping_tax1_rate,
+      :shipping_tax2_rate, :source, :start_date, :status, :sub_total, :sync_partner_teams,
+      :sync_sales_teams, :tax2_total, :tax_rate, :tax_total, :to_be_emailed, :to_be_faxed, :to_be_printed,
+      :total, :total_cost_estimate, :tran_date, :tran_id, :tran_is_vsoe_bundle, :unapplied, :vat_reg_num,
+      :vsoe_auto_calc, :cc_approved
     ].each do |field|
       salesorder.should have_field(field)
     end
@@ -25,9 +25,9 @@ describe NetSuite::Records::SalesOrder do
 
   it 'has all the right record refs' do
     [
-      :account, :bill_address_list, :created_from, :currency, :custom_form, :department, :discount_item, 
-      :entity, :gift_cert, :handling_tax_code, :job, :klass, :lead_source, :location, :message_sel, 
-      :opportunity, :partner, :posting_period, :promo_code, :sales_group, :sales_rep, 
+      :account, :bill_address_list, :created_from, :currency, :custom_form, :department, :discount_item,
+      :entity, :gift_cert, :handling_tax_code, :job, :klass, :lead_source, :location, :message_sel,
+      :opportunity, :partner, :posting_period, :promo_code, :sales_group, :sales_rep,
       :ship_method, :shipping_tax_code, :subsidiary, :tax_item
     ].each do |record_ref|
       salesorder.should have_record_ref(record_ref)
@@ -138,7 +138,7 @@ describe NetSuite::Records::SalesOrder do
         NetSuite::Actions::Add.should_receive(:call).
             with([salesorder], {}).
             and_return(response)
-        salesorder.add.should be_true
+        salesorder.add.should be_truthy
       end
     end
 
@@ -150,7 +150,7 @@ describe NetSuite::Records::SalesOrder do
         NetSuite::Actions::Add.should_receive(:call).
             with([salesorder], {}).
             and_return(response)
-        salesorder.add.should be_false
+        salesorder.add.should be_falsey
       end
     end
   end
@@ -166,7 +166,7 @@ describe NetSuite::Records::SalesOrder do
         NetSuite::Actions::Delete.should_receive(:call).
             with([salesorder], {}).
             and_return(response)
-        salesorder.delete.should be_true
+        salesorder.delete.should be_truthy
       end
     end
 
@@ -178,7 +178,7 @@ describe NetSuite::Records::SalesOrder do
         NetSuite::Actions::Delete.should_receive(:call).
             with([salesorder], {}).
             and_return(response)
-        salesorder.delete.should be_false
+        salesorder.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/sales_tax_item_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+describe NetSuite::Records::SalesTaxItem do
+  subject(:sales_tax_item) { described_class.new }
+
+  it 'has all the right fields' do
+    [
+      :item_id, :display_name, :description, :rate, :is_inactive, :effective_from,
+      :valid_until, :include_children, :eccode, :reverse_charge, :service,
+      :exempt, :is_default, :exclude_from_tax_reports, :available, :export,
+      :county, :city, :state, :zip
+    ].each do |field|
+      sales_tax_item.should have_field(field)
+    end
+  end
+
+  it 'has all the right record refs' do
+    [
+      :tax_type, :tax_agency, :tax_account, :purchase_account, :sale_account,
+      :parent, :nexus_country
+    ].each do |record_ref|
+      sales_tax_item.should have_record_ref(record_ref)
+    end
+  end
+
+  describe '#subsidiary_list' do
+    it 'can be set from attributes'
+    it 'can be set from a RecordRefList object'
+  end
+end

data/spec/netsuite/records/service_sale_item_spec.rb

@@ -70,7 +70,7 @@ describe NetSuite::Records::ServiceSaleItem do
         NetSuite::Actions::Add.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.add.should be_true
+        item.add.should be_truthy
       end
     end
 
@@ -81,7 +81,7 @@ describe NetSuite::Records::ServiceSaleItem do
         NetSuite::Actions::Add.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.add.should be_false
+        item.add.should be_falsey
       end
     end
   end
@@ -94,7 +94,7 @@ describe NetSuite::Records::ServiceSaleItem do
         NetSuite::Actions::Delete.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.delete.should be_true
+        item.delete.should be_truthy
       end
     end
 
@@ -105,7 +105,7 @@ describe NetSuite::Records::ServiceSaleItem do
         NetSuite::Actions::Delete.should_receive(:call).
             with([item], {}).
             and_return(response)
-        item.delete.should be_false
+        item.delete.should be_falsey
       end
     end
   end