netsuite 0.8.10 → 0.8.11

data/spec/netsuite/records/return_authorization_spec.rb

@@ -0,0 +1,62 @@
+require 'spec_helper'
+
+describe NetSuite::Records::ReturnAuthorization do
+  let(:return_authorization) { NetSuite::Records::ReturnAuthorization.new }
+
+
+  describe '#shipping_address' do
+    it 'can be set from attributes' do
+      attributes = {
+        :country => "_unitedStates",
+        :attention => "William Sanders",
+        :addressee => "William Sanders",
+        :addr1 => "test1",
+        :addr2 => "test2",
+        :city => "San Francisco",
+        :state => "CA",
+        :zip => "94131",
+        :addr_text => "William Sanders<br>William Sanders<br>test1<br>test2<br>San Francisco CA 94131",
+        :override => false,
+        :"@xmlns:platform_common" => "urn:common_2016_1.platform.webservices.netsuite.com"
+      }
+
+      return_authorization.shipping_address = attributes
+      expect(return_authorization.shipping_address).to be_kind_of(NetSuite::Records::Address)
+      expect(return_authorization.shipping_address.addressee).to eql("William Sanders")
+    end
+
+    it 'can be set from a ItemVendorList object' do
+      shipping_address = NetSuite::Records::Address.new
+      return_authorization.shipping_address = shipping_address
+      expect(return_authorization.shipping_address).to eql(shipping_address)
+    end
+  end
+
+  describe '#billing_address' do
+    it 'can be set from attributes' do
+      attributes = {
+        :country => "_unitedStates",
+        :attention => "William Sanders",
+        :addressee => "William Sanders",
+        :addr1 => "test1",
+        :addr2 => "test2",
+        :city => "San Francisco",
+        :state => "CA",
+        :zip => "94131",
+        :addr_text => "William Sanders<br>William Sanders<br>test1<br>test2<br>San Francisco CA 94131",
+        :override => false,
+        :"@xmlns:platform_common" => "urn:common_2016_1.platform.webservices.netsuite.com"
+      }
+
+      return_authorization.billing_address = attributes
+      expect(return_authorization.billing_address).to be_kind_of(NetSuite::Records::Address)
+      expect(return_authorization.billing_address.addressee).to eql("William Sanders")
+    end
+
+    it 'can be set from a ItemVendorList object' do
+      billing_address = NetSuite::Records::Address.new
+      return_authorization.billing_address = billing_address
+      expect(return_authorization.billing_address).to eql(billing_address)
+    end
+  end
+end

data/spec/netsuite/records/sales_order_spec.rb

@@ -174,6 +174,35 @@ describe NetSuite::Records::SalesOrder do
     end
   end
 
+  describe '#attach_file' do
+    let(:test_data) { { :email => 'test@example.com', :fax => '1234567890' } }
+    let(:file) { double('file') }
+
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        sales_order = NetSuite::Records::SalesOrder.new(test_data)
+        expect(NetSuite::Actions::AttachFile).to receive(:call).
+          with([sales_order, file], {}).
+          and_return(response)
+        expect(sales_order.attach_file(file)).to be_truthy
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        sales_order = NetSuite::Records::SalesOrder.new(test_data)
+        expect(NetSuite::Actions::AttachFile).to receive(:call).
+          with([sales_order, file], {}).
+          and_return(response)
+        expect(sales_order.attach_file(file)).to be_falsey
+      end
+    end
+  end
+
   describe '#delete' do
     let(:test_data) { { :internal_id => '1' } }
 

data/spec/netsuite/records/service_resale_item_spec.rb

@@ -5,31 +5,136 @@ describe NetSuite::Records::ServiceResaleItem do
 
   it 'has the right fields' do
     [
-      :available_to_partners, :cost, :cost_estimate, :cost_estimate_type, :cost_estimate_units, :create_job, :created_date,
-      :display_name, :dont_show_price, :enforce_min_qty_internally, :exclude_from_sitemap, :featured_description,
-      :include_children, :is_donation_item, :is_fulfillable, :is_gco_compliant, :is_inactive, :is_online, :is_taxable,
-      :item_id, :last_modified_date, :matrix_option_list, :matrix_type, :max_donation_amount, :meta_tag_html,
-      :minimum_quantity, :minimum_quantity_units, :no_price_message, :offer_support, :on_special, :out_of_stock_behavior,
-      :out_of_stock_message, :overall_quantity_pricing_type, :page_title, :presentation_item_list, :prices_include_tax,
-      :show_default_donation_amount, :site_category_list, :sitemap_priority, :soft_descriptor, :specials_description,
-      :store_description, :store_detailed_description, :store_display_name, :translations_list, :upc_code, :url_component,
-      :use_marginal_rates, :vsoe_deferral, :vsoe_delivered, :vsoe_permit_discount, :vsoe_price, :vsoe_sop_group
+      :amortization_period,
+      :available_to_partners,
+      :contingent_revenue_handling,
+      :cost,
+      :cost_estimate,
+      :cost_estimate_type,
+      :cost_estimate_units,
+      :cost_units,
+      :created_date,
+      :create_job,
+      :currency,
+      :defer_rev_rec,
+      :direct_revenue_posting,
+      :display_name,
+      :dont_show_price,
+      :enforce_min_qty_internally,
+      :exclude_from_sitemap,
+      :featured_description,
+      :generate_accruals,
+      :include_children,
+      :is_donation_item,
+      :is_fulfillable,
+      :is_gco_compliant,
+      :is_inactive,
+      :is_online,
+      :is_taxable,
+      :item_id,
+      :last_modified_date,
+      :manufacturing_charge_item,
+      :matrix_item_name_template,
+      :matrix_type,
+      :max_donation_amount,
+      :maximum_quantity,
+      :meta_tag_html,
+      :minimum_quantity,
+      :minimum_quantity_units,
+      :no_price_message,
+      :offer_support,
+      :on_special,
+      :out_of_stock_behavior,
+      :out_of_stock_message,
+      :overall_quantity_pricing_type,
+      :page_title,
+      :prices_include_tax,
+      :purchase_description,
+      :purchase_order_amount,
+      :purchase_order_quantity,
+      :purchase_order_quantity_diff,
+      :rate,
+      :receipt_amount,
+      :receipt_quantity,
+      :receipt_quantity_diff,
+      :related_items_description,
+      :residual,
+      :sales_description,
+      :search_keywords,
+      :show_default_donation_amount,
+      :sitemap_priority,
+      :soft_descriptor,
+      :specials_description,
+      :store_description,
+      :store_detailed_description,
+      :store_display_name,
+      :upc_code,
+      :url_component,
+      :use_marginal_rates,
+      :vendor_name,
+      :vsoe_deferral,
+      :vsoe_delivered,
+      :vsoe_permit_discount,
+      :vsoe_price,
+      :vsoe_sop_group,
     ].each do |field|
       expect(item).to have_field(field)
     end
+  end
 
-    # TODO there is a probably a more robust way to test this
-    expect(item.custom_field_list.class).to eq(NetSuite::Records::CustomFieldList)
-    expect(item.pricing_matrix.class).to eq(NetSuite::Records::PricingMatrix)
-    expect(item.subsidiary_list.class).to eq(NetSuite::Records::RecordRefList)
+  it 'has all the right fields with specific classes' do
+    {
+      custom_field_list: NetSuite::Records::CustomFieldList,
+      item_vendor_list: NetSuite::Records::ItemVendorList,
+      matrix_option_list: NetSuite::Records::MatrixOptionList,
+      pricing_matrix: NetSuite::Records::PricingMatrix,
+      subsidiary_list: NetSuite::Records::RecordRefList,
+    }.each do |field, klass|
+      expect(item).to have_field(field, klass)
+    end
   end
 
   it 'has the right record_refs' do
     [
-      :billing_schedule, :cost_category, :custom_form, :deferred_revenue_account, :department, :income_account,
-      :issue_product, :item_options_list, :klass, :location, :parent, :pricing_group, :purchase_tax_code,
-      :quantity_pricing_schedule, :rev_rec_schedule, :sale_unit, :sales_tax_code, :store_display_image,
-      :store_display_thumbnail, :store_item_template, :tax_schedule, :units_type
+      :amortization_template,
+      :bill_exch_rate_variance_acct,
+      :billing_schedule,
+      :bill_price_variance_acct,
+      :bill_qty_variance_acct,
+      :klass,
+      :consumption_unit,
+      :cost_category,
+      :create_revenue_plans_on,
+      :custom_form,
+      :deferral_account,
+      :deferred_revenue_account,
+      :department,
+      :expense_account,
+      :income_account,
+      :interco_def_rev_account,
+      :interco_expense_account,
+      :interco_income_account,
+      :issue_product,
+      :item_revenue_category,
+      :location,
+      :parent,
+      :pricing_group,
+      :purchase_tax_code,
+      :purchase_unit,
+      :quantity_pricing_schedule,
+      :revenue_allocation_group,
+      :revenue_recognition_rule,
+      :rev_rec_forecast_rule,
+      :rev_reclass_f_x_account,
+      :rev_rec_schedule,
+      :sales_tax_code,
+      :sale_unit,
+      :store_display_image,
+      :store_display_thumbnail,
+      :store_item_template,
+      :tax_schedule,
+      :units_type,
+      :vendor,
     ].each do |record_ref|
       expect(item).to have_record_ref(record_ref)
     end

data/spec/netsuite/records/translation_list_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe NetSuite::Records::TranslationList do
+  let(:list) { NetSuite::Records::TranslationList.new }
+
+  it 'has a translations attribute' do
+    expect(list.translations).to be_kind_of(Array)
+  end
+
+  describe '#to_record' do
+    before do
+      list.translations << NetSuite::Records::Translation.new(
+        locale: '_englishUK',
+        language: 'English (UK)',
+        display_name: 'display name',
+        sales_description: 'sales description'
+      )
+    end
+
+    it 'can represent itself as a SOAP record' do
+      record = {
+        "listAcct:translation" => [
+          {
+            "listAcct:locale" => "_englishUK",
+            "listAcct:language" => "English (UK)",
+            "listAcct:displayName" => "display name",
+            "listAcct:salesDescription" => "sales description"
+          }
+        ]
+      }
+      expect(list.to_record).to eql(record)
+    end
+  end
+end

data/spec/netsuite/records/translation_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe NetSuite::Records::Translation do
+  let(:translation) { NetSuite::Records::Translation.new }
+
+  it 'has dynamic fields' do
+    [
+      :description,
+      :display_name,
+      :featured_description,
+      :language,
+      :locale,
+      :locale_description,
+      :name,
+      :no_price_message,
+      :out_of_stock_message,
+      :page_title,
+      :replace_all,
+      :sales_description,
+      :specials_description,
+      :store_description,
+      :store_detailed_description,
+      :store_display_name
+    ].each do |field|
+      expect(translation).to have_field(field)
+    end
+  end
+end

data/spec/netsuite/support/fields_spec.rb

@@ -1,12 +1,13 @@
 require 'spec_helper'
 
 describe NetSuite::Support::Fields do
-  DummyRecord = Class.new.send(:include, NetSuite::Support::Fields)
-  let(:klass) { DummyRecord }
+  let(:klass) do
+    Class.new do
+      include NetSuite::Support::Fields
+    end
+  end
   let(:instance) { klass.new }
 
-  before { klass.fields.clear }
-
   describe '.fields' do
     context 'with arguments' do
       it 'calls .field with each argument passed to it' do
@@ -34,10 +35,21 @@ describe NetSuite::Support::Fields do
     end
 
     it 'errors when already a field' do
+      DummyRecord = klass
+
       klass.field :one
 
       expect { klass.field :one }.to raise_error('one already defined on DummyRecord')
     end
+
+    it 'errors when conflicting with a public method' do
+      DummyRecordWithMethod = Class.new(klass) do
+        def existing_method
+        end
+      end
+
+      expect { DummyRecordWithMethod.field :existing_method }.to raise_error('existing_method conflicts with a method defined on DummyRecordWithMethod')
+    end
   end
 
   describe '.read_only_fields' do

data/spec/netsuite/support/search_result_spec.rb

@@ -1,6 +1,9 @@
 require 'spec_helper'
 
 describe NetSuite::Support::SearchResult do
+  before(:all) { savon.mock! }
+  after(:all) { savon.unmock! }
+
   describe '#results' do
     context 'empty page' do
       it 'returns empty array' do
@@ -20,5 +23,14 @@ describe NetSuite::Support::SearchResult do
         expect(results).to eq []
       end
     end
+
+    it 'handles a recordList with a single element' do
+      response = File.read('spec/support/fixtures/search/single_search_result.xml')
+      savon.expects(:search).with(message: {}).returns(response)
+
+      results = NetSuite::Records::Account.search(basic: [])
+
+      expect(results.results.count).to eq 1
+    end
   end
 end

data/spec/netsuite/utilities_spec.rb

@@ -13,11 +13,19 @@ describe NetSuite::Utilities do
 
         no_dst_stamp = DateTime.parse('Sun, November 6 2017 00:00:00 -0000')
         formatted_date = NetSuite::Utilities.normalize_time_to_netsuite_date(no_dst_stamp.to_time.to_i)
-        expect(formatted_date).to eq('2017-11-06T00:00:00-08:00')
+        if Gem.loaded_specs.has_key?('tzinfo')
+          expect(formatted_date).to eq('2017-11-06T00:00:00-08:00')
+        else
+          expect(formatted_date).to eq('2017-11-06T00:00:00-07:00')
+        end
 
         no_dst_stamp_with_time = DateTime.parse('Sun, November 6 2017 12:11:10 -0000')
         formatted_date = NetSuite::Utilities.normalize_time_to_netsuite_date(no_dst_stamp_with_time.to_time.to_i)
-        expect(formatted_date).to eq('2017-11-06T00:00:00-08:00')
+        if Gem.loaded_specs.has_key?('tzinfo')
+          expect(formatted_date).to eq('2017-11-06T00:00:00-08:00')
+        else
+          expect(formatted_date).to eq('2017-11-06T00:00:00-07:00')
+        end
       end
     end
   end

data/spec/support/fixtures/add/add_file.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+  <soapenv:Header>
+    <platformMsgs:documentInfo xmlns:platformMsgs="urn:messages_2020_2.platform.webservices.netsuite.com">
+      <platformMsgs:nsId>REDACTED</platformMsgs:nsId>
+    </platformMsgs:documentInfo>
+  </soapenv:Header>
+  <soapenv:Body>
+    <addResponse xmlns="urn:messages_2020_2.platform.webservices.netsuite.com">
+      <writeResponse>
+        <platformCore:status xmlns:platformCore="urn:core_2020_2.platform.webservices.netsuite.com" isSuccess="true">
+          <platformCore:statusDetail>
+            <platformCore:afterSubmitFailed>false</platformCore:afterSubmitFailed>
+          </platformCore:statusDetail>
+        </platformCore:status>
+        <baseRef xmlns:platformCore="urn:core_2020_2.platform.webservices.netsuite.com" internalId="23556" type="file" xsi:type="platformCore:RecordRef"/>
+      </writeResponse>
+    </addResponse>
+  </soapenv:Body>
+</soapenv:Envelope>

data/spec/support/fixtures/attach/attach_file_to_sales_order.xml

@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+  <soapenv:Header>
+    <platformMsgs:documentInfo xmlns:platformMsgs="urn:messages_2011_2.platform.webservices.netsuite.com">
+      <platformMsgs:nsId>WEBSERVICES_3392464_1220201115821392011296470399_67055c545d0</platformMsgs:nsId>
+    </platformMsgs:documentInfo>
+  </soapenv:Header>
+  <soapenv:Body>
+    <attachResponse xmlns="urn:messages_2_5.platform.webservices.netsuite.com">
+      <writeResponse xmlns="urn:messages_2_5.platform.webservices.netsuite.com">
+        <ns1:status isSuccess="true" xmlns:ns1="urn:core_2_5.platform.webservices.netsuite.com"/>
+        <baseRef internalId="999" type="salesOrder" xsi:type="ns2:RecordRef" xmlns:ns2="urn:core_2_5.platform.webservices.netsuite.com"/>
+      </writeResponse>
+    </attachResponse>
+  </soapenv:Body>
+</soapenv:Envelope>

data/spec/support/fixtures/attach/attach_file_to_sales_order_error.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+  <soapenv:Header>
+    <platformMsgs:documentInfo xmlns:platformMsgs="urn:messages_2011_2.platform.webservices.netsuite.com">
+      <platformMsgs:nsId>WEBSERVICES_3392464_1220201115821392011296470399_67055c545d0</platformMsgs:nsId>
+    </platformMsgs:documentInfo>
+  </soapenv:Header>
+  <soapenv:Body>
+    <attachResponse xmlns="urn:messages_2_5.platform.webservices.netsuite.com">
+      <writeResponse xmlns="urn:messages_2_5.platform.webservices.netsuite.com">
+        <platformCore:status xmlns:platformCore="urn:core_2011_2.platform.webservices.netsuite.com" isSuccess="false">
+          <platformCore:statusDetail type="ERROR">
+            <platformCore:code>INVALID</platformCore:code>
+            <platformCore:message>Invalid request.</platformCore:message>
+          </platformCore:statusDetail>
+        </platformCore:status>
+      </writeResponse>
+    </attachResponse>
+  </soapenv:Body>
+</soapenv:Envelope>

data/spec/support/fixtures/search/basic_search_contact.xml

@@ -0,0 +1,39 @@
+<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+  <soapenv:Header>
+    <platformMsgs:documentInfo xmlns:platformMsgs="urn:messages_2020_2.platform.webservices.netsuite.com">
+      <platformMsgs:nsId>WEBSERVICES_123456</platformMsgs:nsId>
+    </platformMsgs:documentInfo>
+  </soapenv:Header>
+  <soapenv:Body>
+    <searchResponse xmlns="urn:messages_2020_2.platform.webservices.netsuite.com">
+      <platformCore:searchResult xmlns:platformCore="urn:core_2020_2.platform.webservices.netsuite.com">
+        <platformCore:status isSuccess="true"/>
+        <platformCore:totalRecords>1</platformCore:totalRecords>
+        <platformCore:pageSize>1000</platformCore:pageSize>
+        <platformCore:totalPages>1</platformCore:totalPages>
+        <platformCore:pageIndex>1</platformCore:pageIndex>
+        <platformCore:searchId>WEBSERVICES_123456</platformCore:searchId>
+        <platformCore:recordList>
+          <platformCore:record xmlns:listRel="urn:relationships_2020_2.lists.webservices.netsuite.com" internalId="7272" externalId="customer_contact_3338" xsi:type="listRel:Contact">
+            <listRel:entityId>Mary</listRel:entityId>
+            <listRel:company internalId="7270">
+              <platformCore:name>1357 What a Company</platformCore:name>
+            </listRel:company>
+            <listRel:title>President/CEO</listRel:title>
+            <listRel:phone>(123) 555-1234</listRel:phone>
+            <listRel:email>***FILTERED***</listRel:email>
+            <listRel:defaultAddress>123 Main St</listRel:defaultAddress>
+            <listRel:isPrivate>false</listRel:isPrivate>
+            <listRel:isInactive>false</listRel:isInactive>
+            <listRel:subsidiary internalId="1">
+              <platformCore:name>Awesome Company</platformCore:name>
+            </listRel:subsidiary>
+            <listRel:globalSubscriptionStatus>_softOptOut</listRel:globalSubscriptionStatus>
+            <listRel:dateCreated>2021-03-03T08:26:56.000-08:00</listRel:dateCreated>
+            <listRel:lastModifiedDate>2021-07-07T09:49:51.000-07:00</listRel:lastModifiedDate>
+          </platformCore:record>
+        </platformCore:recordList>
+      </platformCore:searchResult>
+    </searchResponse>
+  </soapenv:Body>
+</soapenv:Envelope>

data/spec/support/fixtures/search/single_search_result.xml

@@ -0,0 +1,46 @@
+<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+  <soapenv:Header>
+    <platformMsgs:documentInfo xmlns:platformMsgs="urn:messages_2021_1.platform.webservices.netsuite.com">
+      <platformMsgs:nsId>WEBSERVICES_3603434_08292013908151921913870884_2f1b68b787b4d</platformMsgs:nsId>
+    </platformMsgs:documentInfo>
+  </soapenv:Header>
+  <soapenv:Body>
+    <searchResponse xmlns="urn:messages_2021_1.platform.webservices.netsuite.com">
+      <platformCore:searchResult xmlns:platformCore="urn:core_2021_1.platform.webservices.netsuite.com">
+        <platformCore:status isSuccess="true"/>
+        <platformCore:totalRecords>1</platformCore:totalRecords>
+        <platformCore:pageSize>1000</platformCore:pageSize>
+        <platformCore:totalPages>1</platformCore:totalPages>
+        <platformCore:pageIndex>1</platformCore:pageIndex>
+        <platformCore:searchId>WEBSERVICES_3603434_08292013908151921913870884_2f1b68b787b4d</platformCore:searchId>
+        <platformCore:recordList>
+          <platformCore:record xmlns:listAcct="urn:accounting_2021_1.lists.webservices.netsuite.com" internalId="355" xsi:type="listAcct:Account">
+            <listAcct:acctType>_costOfGoodsSold</listAcct:acctType>
+            <listAcct:acctNumber>4508</listAcct:acctNumber>
+            <listAcct:acctName>Selling Fees/Commissions - COS</listAcct:acctName>
+            <listAcct:includeChildren>false</listAcct:includeChildren>
+            <listAcct:generalRate>_average</listAcct:generalRate>
+            <listAcct:parent internalId="246">
+              <platformCore:name>4500 Cost of Sales</platformCore:name>
+            </listAcct:parent>
+            <listAcct:cashFlowRate>_average</listAcct:cashFlowRate>
+            <listAcct:isInactive>false</listAcct:isInactive>
+            <listAcct:inventory>false</listAcct:inventory>
+            <listAcct:revalue>false</listAcct:revalue>
+            <listAcct:customFieldList>
+              <platformCore:customField internalId="3602" scriptId="custrecord_cps_bdc_lastupdatedbyimp_acc" xsi:type="platformCore:BooleanCustomFieldRef">
+                <platformCore:value>false</platformCore:value>
+              </platformCore:customField>
+              <platformCore:customField internalId="3609" scriptId="custrecord_cps_bdc_updatedbyimp_acc" xsi:type="platformCore:BooleanCustomFieldRef">
+                <platformCore:value>false</platformCore:value>
+              </platformCore:customField>
+              <platformCore:customField internalId="1145" scriptId="custrecord_has_mx_localization" xsi:type="platformCore:BooleanCustomFieldRef">
+                <platformCore:value>false</platformCore:value>
+              </platformCore:customField>
+            </listAcct:customFieldList>
+          </platformCore:record>
+        </platformCore:recordList>
+      </platformCore:searchResult>
+    </searchResponse>
+  </soapenv:Body>
+</soapenv:Envelope>