netsuite 0.8.12 → 0.9.1

data/spec/netsuite/actions/get_select_value_spec.rb

@@ -0,0 +1,43 @@
+require 'spec_helper'
+
+describe NetSuite::Actions::GetSelectValue do
+  before(:all) { savon.mock!  }
+  after(:all) { savon.unmock! }
+
+  describe 'fulfillment ship method' do
+    subject do
+      NetSuite::Records::BaseRefList.get_select_value(recordType: 'itemFulfillment', field: 'shipMethod')
+    end
+
+    let(:response) { File.read('spec/support/fixtures/get_select_value/item_fulfillment_ship_method.xml') }
+
+    before do
+      savon.expects(:get_select_value).with(:message => {
+        pageIndex: 1,
+        fieldDescription: {
+          'platformCore:recordType' => 'itemFulfillment',
+          'platformCore:field' => 'shipMethod'
+        }
+      }).returns(response)
+    end
+
+    it 'makes a valid request to the NetSuite API' do
+      subject
+    end
+
+    it 'returns a BaseRefList object' do
+      expect(subject).to be_kind_of(NetSuite::Records::BaseRefList)
+      expect(subject.base_refs[0].internal_id).to eq('94')
+      expect(subject.base_refs[0].name).to eq('Ground (Custom)')
+    end
+
+    context 'with empty result' do
+      let(:response) { File.read('spec/support/fixtures/get_select_value/empty_result.xml') }
+
+      it 'returns an empty list' do
+        expect(subject).to be_kind_of(NetSuite::Records::BaseRefList)
+        expect(subject.base_refs).to be_empty
+      end
+    end
+  end
+end

data/spec/netsuite/configuration_spec.rb

@@ -14,6 +14,44 @@ describe NetSuite::Configuration do
       config.reset!
       expect(config.attributes).to be_empty
     end
+
+    it 'treats attributes as shared/global in default single tenant mode' do
+      config.attributes[:blah] = 'something'
+      expect(config.attributes[:blah]).to eq('something')
+
+      thread = Thread.new {
+        expect(config.attributes[:blah]).to eq('something')
+
+        config.attributes[:blah] = 'something_else'
+        expect(config.attributes[:blah]).to eq('something_else')
+      }
+
+      thread.join
+
+      expect(config.attributes[:blah]).to eq('something_else')
+    end
+
+    it 'treats attributes as thread-local in multi-tenant mode, which each thread starting with empty attributes' do
+      begin
+        config.multi_tenant!
+
+        config.attributes[:blah] = 'something'
+        expect(config.attributes[:blah]).to eq('something')
+
+        thread = Thread.new {
+          expect(config.attributes[:blah]).to be_nil
+
+          config.attributes[:blah] = 'something_else'
+          expect(config.attributes[:blah]).to eq('something_else')
+        }
+
+        thread.join
+
+        expect(config.attributes[:blah]).to eq('something')
+      ensure
+        config.instance_variable_set(:@multi_tenant, false)
+      end
+    end
   end
 
   describe '#filters' do
@@ -87,7 +125,7 @@ describe NetSuite::Configuration do
 
     context 'when the wsdl has not been set' do
       it 'returns a path to the WSDL to use for the API' do
-        expect(config.wsdl).to eq("https://webservices.netsuite.com/wsdl/v2015_1_0/netsuite.wsdl")
+        expect(config.wsdl).to eq("https://webservices.netsuite.com/wsdl/v2016_2_0/netsuite.wsdl")
       end
     end
 
@@ -337,8 +375,8 @@ describe NetSuite::Configuration do
 
   describe '#api_version' do
     context 'when no api_version is defined' do
-      it 'defaults to 2015_1' do
-        expect(config.api_version).to eq('2015_1')
+      it 'defaults to 2016_2' do
+        expect(config.api_version).to eq('2016_2')
       end
     end
   end
@@ -476,4 +514,25 @@ describe NetSuite::Configuration do
     end
   end
 
+  describe '#proxy' do
+    it 'defaults to nil' do
+      expect(config.proxy).to be_nil
+    end
+
+    it 'can be set with proxy=' do
+      config.proxy = "https://my-proxy"
+
+      expect(config.proxy).to eql("https://my-proxy")
+
+      # ensure no exception is raised
+      config.connection
+    end
+
+    it 'can be set with proxy(value)' do
+      config.proxy("https://my-proxy")
+
+      expect(config.proxy).to eql("https://my-proxy")
+    end
+  end
+
 end

data/spec/netsuite/records/customer_spec.rb

@@ -5,30 +5,268 @@ describe NetSuite::Records::Customer do
 
   it 'has all the right fields' 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_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,
-      :group_pricing_list, :home_phone, :image, :is_budget_approved, :is_inactive, :is_person, :item_pricing_list, :keywords,
-      :language, :last_modified_date, :last_name, :last_page_visited, :last_visit, :middle_name, :mobile_phone,
-      :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, :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
+      :account_number,
+      :aging,
+      :aging1,
+      :aging2,
+      :aging3,
+      :aging4,
+      :alcohol_recipient_type,
+      :alt_email,
+      :alt_name,
+      :alt_phone,
+      :bill_pay,
+      :click_stream,
+      :comments,
+      :company_name,
+      :consol_aging,
+      :consol_aging1,
+      :consol_aging2,
+      :consol_aging3,
+      :consol_aging4,
+      :consol_days_overdue,
+      :contrib_pct,
+      :credit_hold_override,
+      :credit_limit,
+      :date_created,
+      :days_overdue,
+      :default_address,
+      :default_order_priority,
+      :display_symbol,
+      :email,
+      :email_preference,
+      :email_transactions,
+      :end_date,
+      :entity_id,
+      :estimated_budget,
+      :fax,
+      :fax_transactions,
+      :first_name,
+      :first_visit,
+      :give_access,
+      :global_subscription_status,
+      :home_phone,
+      :is_budget_approved,
+      :is_inactive,
+      :is_person,
+      :keywords,
+      :language,
+      :last_modified_date,
+      :last_name,
+      :last_page_visited,
+      :last_visit,
+      :middle_name,
+      :mobile_phone,
+      :monthly_closing,
+      :negative_number_format,
+      :number_format,
+      :opening_balance,
+      :opening_balance_date,
+      :override_currency_format,
+      :password,
+      :password2,
+      :phone,
+      :phonetic_name,
+      :print_on_check_as,
+      :print_transactions,
+      :referrer,
+      :reminder_days,
+      :require_pwd_change,
+      :resale_number,
+      :salutation,
+      :send_email,
+      :ship_complete,
+      :stage,
+      :start_date,
+      :symbol_placement,
+      :sync_partner_teams,
+      :taxable,
+      :tax_exempt,
+      :third_party_acct,
+      :third_party_country,
+      :third_party_zipcode,
+      :title,
+      :url,
+      :vat_reg_number,
+      :visits,
+      :web_lead,
     ].each do |field|
       expect(customer).to have_field(field)
     end
   end
 
+  it 'has all the right fields with specific classes' do
+    {
+      addressbook_list: NetSuite::Records::CustomerAddressbookList,
+      contact_roles_list: NetSuite::Records::ContactAccessRolesList,
+      credit_cards_list: NetSuite::Records::CustomerCreditCardsList,
+      currency_list: NetSuite::Records::CustomerCurrencyList,
+      custom_field_list: NetSuite::Records::CustomFieldList,
+      # download_list: NetSuite::Records::CustomerDownloadList, # TODO Implement me
+      # group_pricing_list: NetSuite::Records::CustomerGroupPricingList, # TODO: Implement me
+      # item_pricing_list: NetSuite::Records::CustomerItemPricingList, # TODO: Implement me
+      partners_list: NetSuite::Records::CustomerPartnersList,
+      sales_team_list: NetSuite::Records::CustomerSalesTeamList,
+      subscriptions_list: NetSuite::Records::SubscriptionsList,
+      # tax_registration_list: NetSuite::Records::CustomerTaxRegistrationList, # TODO: Implement me
+    }.each do |field, klass|
+      expect(customer).to have_field(field, klass)
+    end
+  end
+
+  it 'has all the right read_only_fields' do
+    [
+      :balance,
+      :consol_balance,
+      :deposit_balance,
+      :consol_deposit_balance,
+      :overdue_balance,
+      :consol_overdue_balance,
+      :unbilled_orders,
+      :consol_unbilled_orders,
+    ].each do |field|
+      expect(NetSuite::Records::Customer).to have_read_only_field(field)
+    end
+  end
+
+  it 'has all the right search_only_fields' do
+    [
+      :address,
+      :address1,
+      :address2,
+      :address3,
+      :addressee,
+      :address_internal_id,
+      :address_label,
+      :address_phone,
+      :alt_contact,
+      :assigned_site,
+      :assigned_site_id,
+      :attention,
+      :available_offline,
+      :bill_address,
+      :bill_address1,
+      :bill_address2,
+      :bill_address3,
+      :bill_addressee,
+      :bill_attention,
+      :bill_city,
+      :bill_country,
+      :bill_country_code,
+      :bill_phone,
+      :bill_state,
+      :bill_zip_code,
+      :cc_customer_code,
+      :cc_default,
+      :cc_exp_date,
+      :cc_holder_name,
+      :cc_internal_id,
+      :cc_number,
+      :cc_state,
+      :cc_state_from,
+      :cc_type,
+      :city,
+      :contact,
+      :contribution,
+      :contribution_primary,
+      :conversion_date,
+      :country,
+      :country_code,
+      :credit_hold,
+      :date_closed,
+      :entity_number,
+      :explicit_conversion,
+      :first_order_date,
+      :first_sale_date,
+      :fx_balance,
+      :fx_consol_balance,
+      :fx_consol_unbilled_orders,
+      :fx_unbilled_orders,
+      :group_pricing_level,
+      :has_duplicates,
+      :is_default_billing,
+      :is_default_shipping,
+      :is_ship_address,
+      :item_pricing_level,
+      :item_pricing_unit_price,
+      :job_end_date,
+      :job_projected_end,
+      :job_start_date,
+      :job_type,
+      :last_order_date,
+      :last_sale_date,
+      :lead_date,
+      :level,
+      :manual_credit_hold,
+      :on_credit_hold,
+      :partner_contribution,
+      :partner_role,
+      :partner_team_member,
+      :pec,
+      :permission,
+      :pricing_group,
+      :pricing_item,
+      :prospect_date,
+      :role,
+      :sales_team_member,
+      :sales_team_role,
+      :ship_address,
+      :ship_address1,
+      :ship_address2,
+      :ship_address3,
+      :ship_addressee,
+      :ship_attention,
+      :ship_city,
+      :ship_country,
+      :ship_country_code,
+      :ship_phone,
+      :ship_state,
+      :ship_zip,
+      :source_site,
+      :source_site_id,
+      :state,
+      :subscription,
+      :subscription_date,
+      :subscription_status,
+      :zip_code,
+    ].each do |field|
+      expect(NetSuite::Records::Customer).to have_search_only_field(field)
+    end
+  end
+
   it 'has the right record_refs' do
     [
-      :access_role, :currency, :custom_form, :entity_status, :partner, :sales_rep, :terms, :parent, :territory, :shipping_item, :tax_item
+      :access_role,
+      :assigned_web_site,
+      :buying_reason,
+      :buying_time_frame,
+      :campaign_category,
+      :category,
+      :currency,
+      :custom_form,
+      :default_allocation_strategy,
+      :default_tax_reg,
+      :dr_account,
+      :entity_status,
+      :fx_account,
+      :image,
+      :lead_source,
+      :opening_balance_account,
+      :parent,
+      :partner,
+      :pref_cc_processor,
+      :price_level,
+      :receivables_account,
+      :representing_subsidiary,
+      :sales_group,
+      :sales_readiness,
+      :sales_rep,
+      :shipping_item,
+      :source_web_site,
+      :subsidiary,
+      :tax_item,
+      :terms,
+      :territory,
     ].each do |record_ref|
       expect(customer).to have_record_ref(record_ref)
     end
@@ -115,12 +353,12 @@ describe NetSuite::Records::Customer do
           }
         ]
       }
-      expect(customer.subscriptions_list).to be_kind_of(NetSuite::Records::CustomerSubscriptionsList)
+      expect(customer.subscriptions_list).to be_kind_of(NetSuite::Records::SubscriptionsList)
       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
+    it 'can be set from a SubscriptionsList object' do
+      customer_subscriptions_list = NetSuite::Records::SubscriptionsList.new
       customer.subscriptions_list = customer_subscriptions_list
       expect(customer.subscriptions_list).to eql(customer_subscriptions_list)
     end
@@ -202,6 +440,35 @@ describe NetSuite::Records::Customer 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
+        customer = NetSuite::Records::Customer.new(test_data)
+        expect(NetSuite::Actions::AttachFile).to receive(:call).
+          with([customer, file], {}).
+          and_return(response)
+        expect(customer.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
+        customer = NetSuite::Records::Customer.new(test_data)
+        expect(NetSuite::Actions::AttachFile).to receive(:call).
+          with([customer, file], {}).
+          and_return(response)
+        expect(customer.attach_file(file)).to be_falsey
+      end
+    end
+  end
+
   describe '#to_record' do
     let(:customer) { NetSuite::Records::Customer.new(:entity_id => 'TEST CUSTOMER', :is_person => true) }
 

data/spec/netsuite/records/item_availability_spec.rb

@@ -0,0 +1,59 @@
+require 'spec_helper'
+
+describe NetSuite::Records::ItemAvailability do
+  before(:all) { savon.mock!  }
+  after(:all) { savon.unmock! }
+
+  let(:item_availability) { NetSuite::Records::ItemAvailability.new }
+
+  it 'has all the right fields' do
+    [
+      :quantity_on_hand,
+      :on_hand_value_mli,
+      :reorder_point,
+      :quantity_on_order,
+      :quantity_committed,
+      :quantity_available,
+    ].each do |field|
+      expect(item_availability).to have_field(field)
+    end
+  end
+
+  it { expect(item_availability).to have_field(:item, NetSuite::Records::InventoryItem) }
+  it { expect(item_availability).to have_field(:location_id, NetSuite::Records::Location) }
+
+  describe 'get_item_availability' do
+    let(:inventory_item_ref_list) { 
+      NetSuite::Records::RecordRefList.new(
+        record_ref: [
+          NetSuite::Records::RecordRef.new(internal_id: 57)
+        ]
+      )
+    }
+    let(:result) { NetSuite::Records::ItemAvailability.get_item_availability(inventory_item_ref_list) }
+
+    before do
+      savon.expects(:get_item_availability).with(:message => {
+        "platformMsgs:itemAvailabilityFilter" => {
+          "platformCore:item"=>{"platformCore:recordRef"=>[{:@internalId=>57}]}
+        }
+      }).returns(File.read('spec/support/fixtures/get_item_availability/get_item_availability.xml'))
+    end
+
+    it 'returns ItemAvailability records' do
+      expect(result).to be_kind_of(Array)
+      expect(result).not_to be_empty
+      expect(result[0]).to be_kind_of(NetSuite::Records::ItemAvailability)
+      expect(result[0]).to have_attributes(
+        item: be_kind_of(NetSuite::Records::InventoryItem),
+        location_id: NetSuite::Records::Location,
+        quantity_on_hand: '264.0',
+        on_hand_value_mli: '129.36',
+        reorder_point: '50.0',
+        quantity_on_order: '0.0',
+        quantity_committed: '0.0',
+        quantity_available: '264.0',
+      )
+    end
+  end
+end

data/spec/netsuite/records/item_fulfillment_package_fed_ex_list_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe NetSuite::Records::ItemFulfillmentPackageFedExList do
+  let(:list) { NetSuite::Records::ItemFulfillmentPackageFedExList.new }
+
+  it 'has a packages attribute' do
+    expect(list.packages).to be_kind_of(Array)
+  end
+
+  describe '#to_record' do
+    before do
+      list.packages << NetSuite::Records::ItemFulfillmentPackageFedEx.new(
+        :package_tracking_number_fed_ex => '1Z12354645757686786'
+      )
+    end
+
+    it 'can represent itself as a SOAP record' do
+      record = {
+          'tranSales:packageFedEx' => [
+            'tranSales:packageTrackingNumberFedEx' => '1Z12354645757686786'
+          ]
+        }
+      
+      expect(list.to_record).to eql(record)
+    end
+  end
+end

data/spec/netsuite/records/item_fulfillment_package_ups_list_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe NetSuite::Records::ItemFulfillmentPackageUpsList do
+  let(:list) { NetSuite::Records::ItemFulfillmentPackageUpsList.new }
+
+  it 'has a packages attribute' do
+    expect(list.packages).to be_kind_of(Array)
+  end
+
+  describe '#to_record' do
+    before do
+      list.packages << NetSuite::Records::ItemFulfillmentPackageUps.new(
+        :package_tracking_number_ups => '1Z12354645757686786'
+      )
+    end
+
+    it 'can represent itself as a SOAP record' do
+      record = {
+          'tranSales:packageUps' => [
+            'tranSales:packageTrackingNumberUps' => '1Z12354645757686786'
+          ]
+        }
+      
+      expect(list.to_record).to eql(record)
+    end
+  end
+end

data/spec/netsuite/records/item_fulfillment_package_usps_list_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe NetSuite::Records::ItemFulfillmentPackageUspsList do
+  let(:list) { NetSuite::Records::ItemFulfillmentPackageUspsList.new }
+
+  it 'has a packages attribute' do
+    expect(list.packages).to be_kind_of(Array)
+  end
+
+  describe '#to_record' do
+    before do
+      list.packages << NetSuite::Records::ItemFulfillmentPackageUsps.new(
+        :package_tracking_number_usps => '1Z12354645757686786'
+      )
+    end
+
+    it 'can represent itself as a SOAP record' do
+      record = {
+          'tranSales:packageUsps' => [
+            'tranSales:packageTrackingNumberUsps' => '1Z12354645757686786'
+          ]
+        }
+      
+      expect(list.to_record).to eql(record)
+    end
+  end
+end

data/spec/netsuite/records/sales_order_item_spec.rb

@@ -13,9 +13,10 @@ describe NetSuite::Records::SalesOrderItem do
      :quantity, :quantity_back_ordered, :quantity_billed,
      :quantity_committed, :quantity_fulfilled,
      :rate, :rev_rec_end_date, :rev_rec_start_date,
-     :rev_rec_term_in_months, :serial_numbers, :tax1_amt, :tax_rate1,
-     :tax_rate2, :vsoe_allocation, :vsoe_amount, :vsoe_deferral,
-     :vsoe_delivered, :vsoe_permit_discount, :vsoe_price
+     :rev_rec_term_in_months, :serial_numbers, :ship_group,
+     :tax1_amt, :tax_rate1, :tax_rate2,
+     :vsoe_allocation, :vsoe_amount, :vsoe_deferral,
+     :vsoe_delivered, :vsoe_permit_discount, :vsoe_price,
     ].each do |field|
       expect(item).to have_field(field)
     end

data/spec/netsuite/records/sales_order_spec.rb

@@ -23,6 +23,23 @@ describe NetSuite::Records::SalesOrder do
     end
   end
 
+  it 'has all the right fields with specific classes' do
+    {
+      billing_address: NetSuite::Records::Address,
+      custom_field_list: NetSuite::Records::CustomFieldList,
+      gift_cert_redemption_list: NetSuite::Records::GiftCertRedemptionList,
+      item_list: NetSuite::Records::SalesOrderItemList,
+      null_field_list: NetSuite::Records::NullFieldList,
+      promotions_list: NetSuite::Records::PromotionsList,
+      ship_group_list: NetSuite::Records::SalesOrderShipGroupList,
+      shipping_address: NetSuite::Records::Address,
+      transaction_bill_address: NetSuite::Records::BillAddress,
+      transaction_ship_address: NetSuite::Records::ShipAddress,
+    }.each do |field, klass|
+      expect(salesorder).to have_field(field, klass)
+    end
+  end
+
   it 'has all the right record refs' do
     [
       :account, :bill_address_list, :created_from, :currency, :custom_form, :department, :discount_item,

data/spec/netsuite/records/{customer_subscription_spec.rb → subscription_spec.rb}

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe NetSuite::Records::CustomerSubscription do
+describe NetSuite::Records::Subscription do
 
   before do
     NetSuite::Configuration.api_version = '2014_2'
@@ -16,7 +16,7 @@ describe NetSuite::Records::CustomerSubscription do
     }
   end
 
-  let(:list) { NetSuite::Records::CustomerSubscription.new(attributes) }
+  let(:list) { NetSuite::Records::Subscription.new(attributes) }
 
   it 'has all the right fields' do
     [

data/spec/netsuite/records/{customer_subscriptions_list_spec.rb → subscriptions_list_spec.rb}

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
-describe NetSuite::Records::CustomerSubscriptionsList do
-  let(:list) { NetSuite::Records::CustomerSubscriptionsList.new }
+describe NetSuite::Records::SubscriptionsList do
+  let(:list) { NetSuite::Records::SubscriptionsList.new }
 
   it 'has a subscriptions attribute' do
     expect(list.subscriptions).to be_kind_of(Array)