netsuite 0.8.2 → 0.8.7

data/spec/netsuite/records/address_spec.rb

@@ -134,6 +134,16 @@ describe NetSuite::Records::Address do
       expect(addressbook.country.to_record).to eql ""
       expect(addressbook.to_record["platformCommon:country"]).to eql ""
     end
+
+    it 'changes the netsuite identifier based on the current API version' do
+      NetSuite::Configuration.api_version = '2015_1'
+      addressbook = NetSuite::Records::Address.new country: "GB"
+      expect(addressbook.to_record["platformCommon:country"]).to eql "_unitedKingdomGB"
+      NetSuite::Configuration.api_version = '2018_1'
+
+      addressbook = NetSuite::Records::Address.new country: "GB"
+      expect(addressbook.to_record["platformCommon:country"]).to eql "_unitedKingdom"
+    end
   end
 
 end

data/spec/netsuite/records/basic_record_spec.rb

@@ -1,6 +1,7 @@
 require 'spec_helper'
 
 describe 'basic records' do
+  # all records with internal IDs should be added to this list
   let(:basic_record_list) {
     [
       NetSuite::Records::Currency,
@@ -25,6 +26,7 @@ describe 'basic records' do
       NetSuite::Records::CustomerDeposit,
       NetSuite::Records::NonInventoryPurchaseItem,
       NetSuite::Records::NonInventoryResaleItem,
+      NetSuite::Records::LotNumberedInventoryItem,
       NetSuite::Records::TaxGroup,
       NetSuite::Records::Folder,
       NetSuite::Records::CustomerCategory,
@@ -48,12 +50,23 @@ describe 'basic records' do
       NetSuite::Records::SerializedInventoryItem,
       NetSuite::Records::DepositApplication,
       NetSuite::Records::InventoryAdjustment,
+      NetSuite::Records::Vendor,
       NetSuite::Records::VendorReturnAuthorization,
       NetSuite::Records::AssemblyBuild,
       NetSuite::Records::AssemblyUnbuild,
       NetSuite::Records::AssemblyComponent,
       NetSuite::Records::InventoryNumber,
-      NetSuite::Records::LotNumberedAssemblyItem
+      NetSuite::Records::PriceLevel,
+      NetSuite::Records::LotNumberedAssemblyItem,
+      NetSuite::Records::InboundShipment,
+      NetSuite::Records::InterCompanyJournalEntry,
+      NetSuite::Records::BinTransfer,
+      NetSuite::Records::SerializedAssemblyItem,
+      NetSuite::Records::CustomerStatus,
+      NetSuite::Records::CustomerPayment,
+      NetSuite::Records::TransactionBodyCustomField,
+      NetSuite::Records::TransactionColumnCustomField,
+      NetSuite::Records::EntityCustomField
     ]
   }
 
@@ -98,8 +111,12 @@ describe 'basic records' do
 
       if !sublist_fields.empty?
         sublist_fields.each do |sublist_field|
+          sublist = record_instance.send(sublist_field)
+
           # TODO make a sublist entry with some fields valid for that sublist item
-          record_instance.send(sublist_field) << {}
+          sublist << {}
+
+          expect(sublist.send(sublist.sublist_key).count).to be(1)
         end
       end
 

data/spec/netsuite/records/bin_number_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe NetSuite::Records::BinNumber do
+  it "has all the right fields" do
+    bin_number = described_class.new
+
+    [:preferred_bin, :location].each do |field|
+      expect(bin_number).to have_field(field)
+    end
+  end
+
+  it "has all the right record refs" do
+    bin_number = described_class.new
+
+    expect(bin_number).to have_record_ref(:bin_number)
+  end
+
+  it "has the right namespace" do
+    bin_number = described_class.new
+
+    expect(bin_number.record_namespace).to eq('listAcct')
+  end
+end

data/spec/netsuite/records/classification_spec.rb

@@ -5,12 +5,21 @@ describe NetSuite::Records::Classification do
 
   it 'has all the right fields' do
     [
-      :name, :include_children, :is_inactive, :class_translation_list, :custom_field_list
+      :name, :include_children, :is_inactive, :class_translation_list
     ].each do |field|
       expect(classification).to have_field(field)
     end
 
     expect(classification.subsidiary_list.class).to eq(NetSuite::Records::RecordRefList)
+    expect(classification.custom_field_list.class).to eq(NetSuite::Records::CustomFieldList)
+  end
+
+  it 'has all the right record refs' do
+    [
+      :parent
+    ].each do |record_ref|
+      expect(classification).to have_record_ref(record_ref)
+    end
   end
 
   describe '.get' do

data/spec/netsuite/records/custom_field_list_spec.rb

@@ -37,9 +37,11 @@ describe NetSuite::Records::CustomFieldList do
   context 'writing convience methods' do
     it "should create a custom field entry when none exists" do
       list.custrecord_somefield = 'a value'
-      list.custom_fields.size.should == 1
-      list.custom_fields.first.value.should == 'a value'
-      list.custom_fields.first.type.should == 'platformCore:StringCustomFieldRef'
+      custom_fields = list.custom_fields
+
+      expect(custom_fields.size).to eq(1)
+      expect(custom_fields.first.value).to eq('a value')
+      expect(custom_fields.first.type).to eq('platformCore:StringCustomFieldRef')
     end
 
     # https://github.com/NetSweet/netsuite/issues/325
@@ -116,7 +118,7 @@ describe NetSuite::Records::CustomFieldList do
           end
 
           it "should raise an error if custom field entry does not exist" do
-            expect{ list.nonexisting_custom_field }.to raise_error
+            expect{ list.nonexisting_custom_field }.to raise_error(NoMethodError)
           end
         end
       end
@@ -180,6 +182,20 @@ describe NetSuite::Records::CustomFieldList do
               script_id: "custbody_salesclassification",
               type: "platformCore:StringCustomFieldRef",
               value: "foobar"
+            },
+            {
+              script_id: 'custbody_multipleselectfield',
+              type: "platformCore:MultiSelectCustomFieldRef",
+              value: [
+                {
+                  internal_id: 405,
+                  type_id: 157
+                },
+                {
+                  internal_id: 419,
+                  type_id: 157
+                }
+              ]
             }
           ]
         }
@@ -200,9 +216,28 @@ describe NetSuite::Records::CustomFieldList do
               "@xsi:type" => "platformCore:StringCustomFieldRef",
               "platformCore:value" => "foobar",
             },
+            {
+              '@scriptId' => 'custbody_multipleselectfield',
+              '@xsi:type' => 'platformCore:MultiSelectCustomFieldRef',
+              'platformCore:value' => [
+                {
+                  :@internalId => 405,
+                  :@typeId => "157"
+                },
+                {
+                  :@internalId => 419,
+                  :@typeId => "157"
+                }
+              ]
+            }
           ]
         }
 
+        # field accessors are tested elsewhere, but let's run tests here to check various field types
+        expect(list).to respond_to(:custbody_multipleselectfield)
+        expect(list).to respond_to(:custbody_salesclassification)
+        expect(list).to respond_to(:custentity_registeredonline)
+
         expect(list.to_record).to eql(record)
         expect(list.to_record.length).to eq(1)
       end

data/spec/netsuite/records/custom_record_spec.rb

@@ -18,7 +18,7 @@ describe NetSuite::Records::CustomRecord do
 
   it 'has all the right record_refs' do
     [
-      :custom_form, :owner
+      :custom_form, :owner, :rec_type, :parent
     ].each do |record_ref|
       expect(record).to have_record_ref(record_ref)
     end

data/spec/netsuite/records/customer_credit_cards_list_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe NetSuite::Records::CustomerCreditCardsList do
+  let(:list) { NetSuite::Records::CustomerCreditCardsList.new }
+
+  it 'has a credit_cards attribute' do
+    expect(list.credit_cards).to be_kind_of(Array)
+  end
+
+  describe '#to_record' do
+    it 'can represent itself as a SOAP record' do
+      list.replace_all = true
+
+      record = {
+        'listRel:creditCards' => [],
+        'listRel:replaceAll' => true
+      }
+
+      expect(list.to_record).to eql(record)
+    end
+  end
+
+end

data/spec/netsuite/records/customer_payment_credit_list_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe NetSuite::Records::CustomerPaymentCreditList do
+  let(:list) { NetSuite::Records::CustomerPaymentCreditList.new }
+  let(:apply) { NetSuite::Records::CustomerPaymentCredit.new }
+
+  it 'can have credits be added to it' do
+    list.credits << apply
+    credit_list = list.credits
+    expect(credit_list).to be_kind_of(Array)
+    expect(credit_list.length).to eql(1)
+    credit_list.each { |i| expect(i).to be_kind_of(NetSuite::Records::CustomerPaymentCredit) }
+  end
+
+  describe '#to_record' do
+    it 'can represent itself as a SOAP record' do
+      record = {
+          'tranCust:credit' => [{},{}]
+      }
+
+      list.credits.concat([apply, apply])
+      expect(list.to_record).to eql(record)
+    end
+  end
+
+end

data/spec/netsuite/records/customer_payment_spec.rb

@@ -18,7 +18,7 @@ describe NetSuite::Records::CustomerPayment do
 
   it 'has all the right record refs' do
     [
-      :account, :ar_acct, :credit_card, :credit_card_processor, :custom_form, :customer, :department, :klass, :location, :payment_method, :posting_period, :subsidiary
+      :account, :ar_acct, :credit_card, :credit_card_processor, :custom_form, :customer, :department, :klass, :location, :payment_method, :payment_option, :posting_period, :subsidiary
     ].each do |record_ref|
       expect(payment).to have_record_ref(record_ref)
     end
@@ -49,11 +49,6 @@ describe NetSuite::Records::CustomerPayment do
     it 'can be set from a CustomerPaymentApplyList object'
   end
 
-  describe '#credit_list' do
-    it 'can be set from attributes'
-    it 'can be set from a CustomerPaymentCreditList object'
-  end
-
   describe '#deposit_list' do
     it 'can be set from attributes'
     it 'can be set from a CustomerPaymentDepositList object'

data/spec/netsuite/records/customer_sales_team_list_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe NetSuite::Records::CustomerSalesTeamList do
+  let(:list) { NetSuite::Records::CustomerSalesTeamList.new }
+
+  it 'has an customer_sales_team attribute' do
+    expect(list.sales_team).to be_kind_of(Array)
+  end
+
+  describe '#to_record' do
+    it 'can represent itself as a SOAP record' do
+      list.replace_all = true
+
+      record = {
+        'listRel:salesTeam' => [],
+        'listRel:replaceAll' => true
+      }
+      expect(list.to_record).to eql(record)
+    end
+  end
+
+  describe "#replace_all" do
+    it "can be changed via accessor" do
+      list.replace_all = false
+
+      expect(list.replace_all).to eql(false)
+    end
+
+    it "coerces to a boolean" do
+      list.replace_all = "goober"
+
+      record = {
+        'listRel:salesTeam' => [],
+        'listRel:replaceAll' => true
+      }
+
+      expect(list.to_record).to eql(record)
+    end
+  end
+
+end

data/spec/netsuite/records/customer_spec.rb

@@ -8,7 +8,7 @@ describe NetSuite::Records::Customer do
       :account_number, :aging, :alt_email, :alt_name, :alt_phone, :balance, :bill_pay,
       :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, :contrib_pct, :credit_cards_list, :credit_hold_override, :credit_limit,
+      :consol_unbilled_orders, :contrib_pct, :credit_hold_override, :credit_limit,
       :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,
@@ -17,7 +17,7 @@ describe NetSuite::Records::Customer do
       :opening_balance, :opening_balance_account, :opening_balance_date, :overdue_balance,
       :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_team_list, :salutation, :send_email, :ship_complete,
+      :sales_group, :sales_readiness, :salutation, :send_email, :ship_complete,
       :stage, :start_date, :sync_partner_teams, :tax_exempt, :taxable,
       :third_party_acct, :third_party_country, :third_party_zipcode, :title, :unbilled_orders, :url,
       :vat_reg_number, :visits, :web_lead
@@ -65,6 +65,27 @@ describe NetSuite::Records::Customer do
     end
   end
 
+  describe '#credit_cards_list' do
+    it 'can be set from attributes' do
+      customer.credit_cards_list = {
+        :credit_cards => {
+          :internal_id       => '1234567',
+          :cc_default        => true,
+          :cc_expire_date    => '2099-12-01T00:00:00.000-08:00'
+        }
+      }
+
+      expect(customer.credit_cards_list).to be_kind_of(NetSuite::Records::CustomerCreditCardsList)
+      expect(customer.credit_cards_list.credit_cards.length).to eql(1)
+    end
+
+    it 'can be set from a CustomerCreditCardsList object' do
+      customer_credit_cards_list = NetSuite::Records::CustomerCreditCardsList.new
+      customer.credit_cards_list = customer_credit_cards_list
+      expect(customer.credit_cards_list).to eql(customer_credit_cards_list)
+    end
+  end
+
   describe '#custom_field_list' do
     it 'can be set from attributes' do
       attributes = {
@@ -85,6 +106,27 @@ describe NetSuite::Records::Customer do
     end
   end
 
+  describe '#subscriptions_list' do
+    it 'can be set from attributes' do
+      customer.subscriptions_list = {
+        subscriptions: [
+          {
+            :subscribed         => true
+          }
+        ]
+      }
+      expect(customer.subscriptions_list).to be_kind_of(NetSuite::Records::CustomerSubscriptionsList)
+      expect(customer.subscriptions_list.subscriptions.length).to eql(1)
+    end
+
+    it 'can be set from a CustomerSubscriptionsList object' do
+      customer_subscriptions_list = NetSuite::Records::CustomerSubscriptionsList.new
+      customer.subscriptions_list = customer_subscriptions_list
+      expect(customer.subscriptions_list).to eql(customer_subscriptions_list)
+    end
+  end
+
+
   describe '.get' do
     context 'when the response is successful' do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :is_person => true }) }

data/spec/netsuite/records/customer_subscription_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe NetSuite::Records::CustomerSubscription do
+
+  before do
+    NetSuite::Configuration.api_version = '2014_2'
+  end
+
+  let(:attributes) do
+    {
+      :subscription => {
+        :subscribed          => true,
+        :subscription        => NetSuite::Records::RecordRef.new(internal_id: '123123', name: 'My Sub'),
+        :last_modified_date  => DateTime.new(2018, 1, 1, 0, 0, 0)
+      }
+    }
+  end
+
+  let(:list) { NetSuite::Records::CustomerSubscription.new(attributes) }
+
+  it 'has all the right fields' do
+    [
+      :subscribed, :last_modified_date
+    ].each do |field|
+      expect(list).to have_field(field)
+    end
+
+    expect(list.subscription).to_not be_nil
+  end
+
+  describe '#initialize' do
+    context 'when taking in a hash of attributes' do
+      it 'sets the attributes for the object given the attributes hash' do
+        expect(list.subscription.subscribed).to eql(true)
+        expect(list.subscription.subscription.internal_id).to eql('123123')
+        expect(list.subscription.subscription.name).to eql('My Sub')
+        expect(list.subscription.last_modified_date).to_not be_nil
+      end
+    end
+  end
+end

data/spec/netsuite/records/customer_subscriptions_list_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe NetSuite::Records::CustomerSubscriptionsList do
+  let(:list) { NetSuite::Records::CustomerSubscriptionsList.new }
+
+  it 'has a subscriptions attribute' do
+    expect(list.subscriptions).to be_kind_of(Array)
+  end
+
+  describe '#to_record' do
+    it 'can represent itself as a SOAP record' do
+
+      record = {
+        'listRel:subscriptions' => []
+      }
+      expect(list.to_record).to eql(record)
+    end
+  end
+end