netsuite 0.8.3 → 0.8.4

data/lib/netsuite/version.rb

@@ -1,3 +1,3 @@
 module NetSuite
-  VERSION = '0.8.3'
+  VERSION = '0.8.4'
 end

data/spec/netsuite/actions/login_spec.rb

@@ -41,4 +41,27 @@ describe NetSuite::Actions::Login do
       role: 234
     }) }.to raise_error(Savon::SOAPFault)
   end
+
+  it 'handles a login call when token based auth is in place' do
+    NetSuite.configure do
+      consumer_key '123'
+      consumer_secret '123'
+      token_id '123'
+      token_secret '123'
+
+      api_version '2017_2'
+    end
+
+    message = {"platformMsgs:passport"=>{"platformCore:email"=>"email", "platformCore:password"=>"password", "platformCore:account"=>"1234", "platformCore:role"=>234}}
+    savon.expects(:login).with(:message => message).returns(File.read('spec/support/fixtures/login/success.xml'))
+
+    result = NetSuite::Actions::Login.call({
+      email: 'email',
+      password: 'password',
+      role: 234
+    })
+
+    expect(result.success?).to eq(true)
+    expect(result.body[:user_id]).to_not be_nil
+  end
 end

data/spec/netsuite/actions/update_spec.rb

@@ -19,6 +19,48 @@ describe NetSuite::Actions::Update do
       }
     end
 
+    describe 'updating the external ID' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      # https://github.com/NetSweet/netsuite/pull/416
+      #   if the external ID is set, and the external ID field is ommitted on an update, the external ID field does not change
+      #   if the external_id field is set to nil, it should not be passed to netsuite
+      #   passing an empty string to the external ID field does not remove it
+
+      it 'does not pass the external ID to an update call if not modified or included in update options' do
+        expect(NetSuite::Actions::Update).to receive(:call).
+            with([customer.class, {}], {}).
+            and_return(response)
+
+        expect(customer.update).to be_truthy
+      end
+
+      it 'should update the external ID when the attribute on the record is set' do
+        expect(NetSuite::Actions::Update).to receive(:call).
+            with([customer.class, {external_id: 'foo'}], {}).
+            and_return(response)
+
+        customer.external_id = 'foo'
+        expect(customer.update).to be_truthy
+      end
+
+      it 'should update the external ID to nil when the attribute on the record is set' do
+        expect(NetSuite::Actions::Update).to receive(:call).
+            with([customer.class, {external_id: nil}], {}).
+            and_return(response)
+
+        expect(customer.update(external_id: nil)).to be_truthy
+      end
+
+      it 'should update the external ID to the options value not the attribute value' do
+        expect(NetSuite::Actions::Update).to receive(:call).
+            with([customer.class, {external_id: 'bar'}], {}).
+            and_return(response)
+
+        customer.external_id = 'foo'
+        expect(customer.update(external_id: 'bar')).to be_truthy
+      end
+    end
 
     context 'when successful' do
 

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

@@ -57,7 +57,12 @@ describe 'basic records' do
       NetSuite::Records::LotNumberedAssemblyItem,
       NetSuite::Records::InboundShipment,
       NetSuite::Records::InterCompanyJournalEntry,
-      NetSuite::Records::BinTransfer
+      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/custom_field_list_spec.rb

@@ -180,6 +180,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 +214,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/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

@@ -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
@@ -85,6 +85,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

data/spec/netsuite/records/entity_custom_field_spec.rb

@@ -0,0 +1,34 @@
+require "spec_helper"
+
+describe NetSuite::Records::EntityCustomField do
+  describe ".get" do
+    context "success" do
+      let(:internal_id) { 1 }
+      let(:response) do
+        NetSuite::Response.new(
+          success: true,
+          body: { 
+            access_level: "_edit",
+            field_type: "_integerNumber",
+            label: "Salesforce Customer ID",
+            show_in_list: true,
+          }
+        )
+      end
+
+      it "returns a EntityCustomField instance with populated fields" do
+        expect(NetSuite::Actions::Get)
+          .to receive(:call)
+          .with([NetSuite::Records::EntityCustomField, internal_id: internal_id], {})
+          .and_return(response)
+
+        record = NetSuite::Records::EntityCustomField.get(internal_id: internal_id)
+
+        expect(record.access_level).to eql("_edit")
+        expect(record.field_type).to eql("_integerNumber")
+        expect(record.label).to eql("Salesforce Customer ID")
+        expect(record.show_in_list).to eq(true)
+      end
+    end
+  end
+end

data/spec/netsuite/records/pricing_matrix_spec.rb

@@ -5,21 +5,23 @@ describe NetSuite::Records::PricingMatrix do
     it "behaves appropriately if it gets a hash as attributes[:pricing]" do
       # this is what savon returns if there is only one pricing strategy matrix
       # for the item:
-      matrix = {
-        :pricing=> {
-          :currency=>"US Dollar",
-          :priceLevel=>"Base Price",
-          :priceList=>[
-            {:price=>{:value=>12.0, :quantity=>0.0}},
-            {:price=>{:value=>10.0, :quantity=>10.0}},
-            {:price=>{:value=>9.0, :quantity=>100.0}},
-            {:price=>{:value=>7.0, :quantity=>5000.0}}
-          ]
+      matrix = [
+        {
+          currency: { internal_id: 1 },
+          price_level: { internal_id: 1 },
+          price_list: {
+            price: [
+              {
+                value: 25.0,
+                quantity: 0
+              }
+            ]
+          }
         }
-      }
+      ]
 
-      subject = NetSuite::Records::PricingMatrix.new({pricing: matrix})
-      expect(subject.prices[0].pricing).to eq(matrix[:pricing])
+      subject = described_class.new({pricing: matrix})
+      expect(subject.pricing[0]).to be_kind_of(NetSuite::Records::Pricing)
     end
   end
 end

data/spec/netsuite/records/transaction_body_custom_field_spec.rb

@@ -0,0 +1,32 @@
+require "spec_helper"
+
+describe NetSuite::Records::TransactionBodyCustomField do
+  describe ".get" do
+    context "success" do
+      let(:internal_id) { 1 }
+      let(:response) do
+        NetSuite::Response.new(
+          success: true,
+          body: {
+            access_level: "_edit",
+            field_type: "_freeFormText",
+            label: "Billing System Subdomain",
+          }
+        )
+      end
+
+      it "returns a TransactionColumnCustomField instance with populated fields" do
+        expect(NetSuite::Actions::Get)
+          .to receive(:call)
+          .with([NetSuite::Records::TransactionBodyCustomField, internal_id: internal_id], {})
+          .and_return(response)
+
+        record = NetSuite::Records::TransactionBodyCustomField.get(internal_id: internal_id)
+
+        expect(record.access_level).to eql("_edit")
+        expect(record.field_type).to eql("_freeFormText")
+        expect(record.label).to eql("Billing System Subdomain")
+      end
+    end
+  end
+end

data/spec/netsuite/records/transaction_column_custom_field_spec.rb

@@ -0,0 +1,32 @@
+require "spec_helper"
+
+describe NetSuite::Records::TransactionColumnCustomField do
+  describe ".get" do
+    context "success" do
+      let(:internal_id) { 1 }
+      let(:response) do
+        NetSuite::Response.new(
+          success: true,
+          body: { 
+            access_level: "_none",
+            field_type: "_decimalNumber",
+            label: "Billing System Tax",
+          }
+        )
+      end
+
+      it "returns a TransactionColumnCustomField instance with populated fields" do
+        expect(NetSuite::Actions::Get)
+          .to receive(:call)
+          .with([NetSuite::Records::TransactionColumnCustomField, internal_id: internal_id], {})
+          .and_return(response)
+
+        record = NetSuite::Records::TransactionColumnCustomField.get(internal_id: internal_id)
+
+        expect(record.access_level).to eql("_none")
+        expect(record.field_type).to eql("_decimalNumber")
+        expect(record.label).to eql("Billing System Tax")
+      end
+    end
+  end
+end