netsuite 0.8.5 → 0.8.9

data/spec/netsuite/actions/search_spec.rb

@@ -168,9 +168,31 @@ describe NetSuite::Actions::Search do
 
       expect(search.results.size).to eq(2)
       expect(search.current_page).to eq(1)
+      expect(search.results.first.internal_id).to eq('123')
+      expect(search.results.first.external_id).to eq('456')
       expect(search.results.first.alt_name).to eq('A Awesome Name')
+      expect(search.results.first.custom_field_list.custitem_stringfield.value).to eq('sample string value')
+      expect(search.results.first.custom_field_list.custitem_apcategoryforsales.value.internal_id).to eq('4')
       expect(search.results.last.email).to eq('alessawesome@gmail.com')
     end
+
+    it "should handle an ID search with basic search only field result columns" do
+      response = File.read('spec/support/fixtures/search/saved_search_item.xml')
+      savon.expects(:search)
+        .with(message: {
+          "searchRecord"=>{
+            "@xsi:type"      =>"listAcct:ItemSearchAdvanced",
+            "@savedSearchId" =>42,
+            :content!        =>{"listAcct:criteria"=>{}},
+          }
+        }).returns(response)
+
+      search = NetSuite::Records::InventoryItem.search(saved: 42)
+
+      expect(search.results.first.location_quantity_available).to eq('3307.0')
+      expect(search.results.first.location_re_order_point).to eq('2565.0')
+      expect(search.results.first.location_quantity_on_order).to eq('40000.0')
+    end
   end
 
   context "advanced search" do

data/spec/netsuite/configuration_spec.rb

@@ -29,10 +29,12 @@ describe NetSuite::Configuration do
   end
 
   describe '#connection' do
+    EXAMPLE_ENDPOINT = 'https://1023.suitetalk.api.netsuite.com/services/NetSuitePort_2020_2'
     before(:each) do
       # reset clears out the password info
       config.email       'me@example.com'
       config.password    'me@example.com'
+      config.endpoint    EXAMPLE_ENDPOINT
       config.account     1023
       config.wsdl        "my_wsdl"
       config.api_version "2012_2"
@@ -57,6 +59,19 @@ describe NetSuite::Configuration do
 
       expect(config).to have_received(:cached_wsdl)
     end
+
+    it 'sets the endpoint on the Savon client' do
+      # this is ugly/brittle, but it's hard to see how else to test this
+      savon_configs = config.connection.globals.instance_eval {@options}
+      expect(savon_configs.fetch(:endpoint)).to eq(EXAMPLE_ENDPOINT)
+    end
+
+    it 'handles a nil endpoint' do
+      config.endpoint = nil
+      # this is ugly/brittle, but it's hard to see how else to test this
+      savon_configs = config.connection.globals.instance_eval {@options}
+      expect(savon_configs.fetch(:endpoint)).to eq(nil)
+    end
   end
 
   describe '#wsdl' do
@@ -166,6 +181,23 @@ describe NetSuite::Configuration do
     end
   end
 
+  describe '#endpoint' do
+    it 'can be set with endpoint=' do
+      config.endpoint = 42
+      expect(config.endpoint).to eq(42)
+    end
+
+    it 'can be set with just endpoint(value)' do
+      config.endpoint(42)
+      expect(config.endpoint).to eq(42)
+    end
+
+    it 'supports nil endpoints' do
+      config.endpoint = nil
+      expect(config.endpoint).to eq(nil)
+    end
+  end
+
   describe '#auth_header' do
     context 'when doing user authentication' do
       before do
@@ -326,15 +358,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,6 +399,65 @@ 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('tmplog').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)

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,
@@ -49,6 +50,7 @@ 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,
@@ -61,6 +63,7 @@ describe 'basic records' do
       NetSuite::Records::BinTransfer,
       NetSuite::Records::SerializedAssemblyItem,
       NetSuite::Records::CustomerStatus,
+      NetSuite::Records::CustomerPayment,
       NetSuite::Records::TransactionBodyCustomField,
       NetSuite::Records::TransactionColumnCustomField,
       NetSuite::Records::EntityCustomField
@@ -108,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/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/cost_category_spec.rb

@@ -0,0 +1,105 @@
+require 'spec_helper'
+
+describe NetSuite::Records::CostCategory do
+  let(:cost_category) { described_class.new }
+
+  it 'has all the right fields' do
+    [
+      :is_inactive,
+      :item_cost_type,
+      :name,
+    ].each do |field|
+      expect(cost_category).to have_field(field)
+    end
+  end
+
+  it 'has all the right record refs' do
+    [
+      :account,
+    ].each do |record_ref|
+      expect(cost_category).to have_record_ref(record_ref)
+    end
+  end
+
+  describe '.get' do
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :name => 'CostCategory 1' }) }
+
+      it 'returns a CostCategory instance populated with the data from the response object' do
+        expect(NetSuite::Actions::Get).to receive(:call).with([described_class, {:external_id => 1}], {}).and_return(response)
+        cost_category = described_class.get(:external_id => 1)
+        expect(cost_category).to be_kind_of(described_class)
+        expect(cost_category.name).to eql('CostCategory 1')
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'raises a RecordNotFound exception' do
+        expect(NetSuite::Actions::Get).to receive(:call).with([described_class, {:external_id => 1}], {}).and_return(response)
+        expect {
+          described_class.get(:external_id => 1)
+        }.to raise_error(NetSuite::RecordNotFound,
+          /NetSuite::Records::CostCategory with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+  describe '#add' do
+    let(:test_data) { { :name => 'Test CostCategory' } }
+
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        cost_category = described_class.new(test_data)
+        expect(NetSuite::Actions::Add).to receive(:call).
+            with([cost_category], {}).
+            and_return(response)
+        expect(cost_category.add).to be_truthy
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        cost_category = described_class.new(test_data)
+        expect(NetSuite::Actions::Add).to receive(:call).
+            with([cost_category], {}).
+            and_return(response)
+        expect(cost_category.add).to be_falsey
+      end
+    end
+  end
+
+  describe '#delete' do
+    let(:test_data) { { :internal_id => '1' } }
+
+    context 'when the response is successful' do
+      let(:response)  { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        cost_category = described_class.new(test_data)
+        expect(NetSuite::Actions::Delete).to receive(:call).
+            with([cost_category], {}).
+            and_return(response)
+        expect(cost_category.delete).to be_truthy
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        cost_category = described_class.new(test_data)
+        expect(NetSuite::Actions::Delete).to receive(:call).
+            with([cost_category], {}).
+            and_return(response)
+        expect(cost_category.delete).to be_falsey
+      end
+    end
+  end
+
+end

data/spec/netsuite/records/custom_field_list_spec.rb

@@ -3,17 +3,53 @@ require 'spec_helper'
 describe NetSuite::Records::CustomFieldList do
   let(:list) { NetSuite::Records::CustomFieldList.new }
 
+  before(:all) { savon.mock! }
+  after(:all) { savon.unmock! }
+
   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"})
+    field = NetSuite::Records::CustomField.new({
+      :value=>{:internal_id=>"5", :type_id=>"103"},
+      :script_id=>"custitem_item_category",
+      :"@xsi:type"=>"platformCore:SelectCustomFieldRef"
+    })
+
     list = described_class.new(custom_field: [field])
+
     expect(list.custom_fields).to eq([field])
   end
 
+  it 'properly decodes various custom field types' do
+    savon.
+      expects(:get).
+      with(message: {"platformMsgs:baseRef"=>{"@xsi:type"=>"platformCore:RecordRef", "@internalId"=>123, "@type"=>"creditMemo"}}).
+      returns(File.read('spec/support/fixtures/custom_fields/multi_select.xml'))
+
+    credit_memo_with_custom_fields = NetSuite::Records::CreditMemo.get(123)
+
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_standard_select.value.internal_id).to eq("2")
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_standard_select.value.attributes[:name]).to eq("Manual")
+
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_date_field.value).to be_a(DateTime)
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_date_field.value.to_s).to eq("2021-07-13T22:00:00-07:00")
+
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_string_field.value).to eq("a very nice string")
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_boolean_field.value).to eq(false)
+
+    # even if there's a single value, it should return an array
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_multi_select_field.value).to be_a(Array)
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_multi_select_field.value.size).to eq(1)
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_multi_select_field.value.first.attributes[:name]).to eq("selection value")
+
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_multi_select_with_multiple.value).to be_a(Array)
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_multi_select_with_multiple.value.size).to eq(2)
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_multi_select_with_multiple.value.first.attributes[:name]).to eq("selection value 1")
+    expect(credit_memo_with_custom_fields.custom_field_list.custbody_multi_select_with_multiple.value.last.attributes[:name]).to eq("selection value 2")
+  end
+
   context 'initializing with custom field attributes without a type' do
     it 'does not mutate the attributes' do
       field = {:value=>{:internal_id=>"5", :type_id=>"103"},
@@ -28,7 +64,9 @@ describe NetSuite::Records::CustomFieldList do
     it 'does not mutate the attributes' do
       field = {:value=>{:internal_id=>"5", :type_id=>"103"},
                :script_id=>"custitem_item_category", :"@xsi:type"=>"platformCore:SelectCustomFieldRef"}
+
       described_class.new(custom_field: [field])
+
       expect(field).to eq({:value=>{:internal_id=>"5", :type_id=>"103"},
                :script_id=>"custitem_item_category", :"@xsi:type"=>"platformCore:SelectCustomFieldRef"})
     end
@@ -37,9 +75,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 +156,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

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