netsuite 0.7.9 → 0.8.6

data/spec/netsuite/configuration_spec.rb

@@ -326,15 +326,11 @@ describe NetSuite::Configuration do
 
   describe "#credentials" do
     context "when none are defined" do
-      skip "should properly create the auth credentials" do
-
-      end
+      skip "should properly create the auth credentials"
     end
 
     context "when they are defined" do
-      it "should properly replace the default auth credentials" do
-
-      end
+      skip "should properly replace the default auth credentials"
     end
   end
 
@@ -371,4 +367,81 @@ describe NetSuite::Configuration do
     end
   end
 
+  describe "#log" do
+    it 'allows a file path to be set as the log destination' do
+      file_path = Tempfile.new.path
+      config.log = file_path
+      config.logger.info "foo"
+
+      log_contents = open(file_path).read
+      expect(log_contents).to include("foo")
+    end
+
+    it 'allows an IO device to bet set as the log destination' do
+      stream = StringIO.new
+      config.log = stream
+      config.logger.info "foo"
+
+      expect(stream.string).to include("foo")
+    end
+  end
+
+  describe '#log_level' do
+    it 'defaults to :debug' do
+      expect(config.log_level).to eq(:debug)
+    end
+
+    it 'can be initially set to any log level' do
+      config.log_level(:info)
+
+      expect(config.log_level).to eq(:info)
+    end
+
+    it 'can override itself' do
+      config.log_level = :info
+
+      expect(config.log_level).to eq(:info)
+
+      config.log_level(:debug)
+
+      expect(config.log_level).to eq(:debug)
+    end
+  end
+
+  describe '#log_level=' do
+    it 'can set the initial log_level' do
+      config.log_level = :info
+
+      expect(config.log_level).to eq(:info)
+    end
+
+    it 'can override a previously set log level' do
+      config.log_level = :info
+
+      expect(config.log_level).to eq(:info)
+
+      config.log_level = :debug
+
+      expect(config.log_level).to eq(:debug)
+    end
+  end
+
+  describe 'timeouts' do
+    it 'has defaults' do
+      expect(config.read_timeout).to eql(60)
+      expect(config.open_timeout).to be_nil
+    end
+
+    it 'sets timeouts' do
+      config.read_timeout = 100
+      config.open_timeout = 60
+
+      expect(config.read_timeout).to eql(100)
+      expect(config.open_timeout).to eql(60)
+
+      # ensure no exception is raised
+      config.connection
+    end
+  end
+
 end

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

@@ -4,6 +4,8 @@ describe 'basic records' do
   let(:basic_record_list) {
     [
       NetSuite::Records::Currency,
+      NetSuite::Records::CashSale,
+      NetSuite::Records::CashRefund,
       NetSuite::Records::Location,
       NetSuite::Records::JobStatus,
       NetSuite::Records::TimeBill,
@@ -23,6 +25,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,
@@ -46,7 +49,21 @@ describe 'basic records' do
       NetSuite::Records::SerializedInventoryItem,
       NetSuite::Records::DepositApplication,
       NetSuite::Records::InventoryAdjustment,
-      NetSuite::Records::VendorReturnAuthorization
+      NetSuite::Records::VendorReturnAuthorization,
+      NetSuite::Records::AssemblyBuild,
+      NetSuite::Records::AssemblyUnbuild,
+      NetSuite::Records::AssemblyComponent,
+      NetSuite::Records::InventoryNumber,
+      NetSuite::Records::PriceLevel,
+      NetSuite::Records::LotNumberedAssemblyItem,
+      NetSuite::Records::InboundShipment,
+      NetSuite::Records::InterCompanyJournalEntry,
+      NetSuite::Records::BinTransfer,
+      NetSuite::Records::SerializedAssemblyItem,
+      NetSuite::Records::CustomerStatus,
+      NetSuite::Records::TransactionBodyCustomField,
+      NetSuite::Records::TransactionColumnCustomField,
+      NetSuite::Records::EntityCustomField
     ]
   }
 

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

@@ -6,7 +6,7 @@ describe NetSuite::Records::CustomFieldList do
   it 'has a custom_fields attribute' do
     expect(list.custom_fields).to be_kind_of(Array)
   end
-  
+
   it 'accepts a collection of CustomField records' do
     field = NetSuite::Records::CustomField.new({:value=>{:internal_id=>"5", :type_id=>"103"},
              :script_id=>"custitem_item_category", :"@xsi:type"=>"platformCore:SelectCustomFieldRef"})
@@ -37,9 +37,17 @@ 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
+    it 'should create a custom field entry when some fields exist without scriptIds' do
+      list.custom_fields << NetSuite::Records::CustomField.new
+      list.custrecord_somefield = 123
     end
   end
 
@@ -110,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
@@ -174,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
+                }
+              ]
             }
           ]
         }
@@ -194,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_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