netsuite 0.7.9 → 0.8.6

data/spec/netsuite/records/inventory_item_spec.rb

@@ -231,4 +231,61 @@ describe NetSuite::Records::InventoryItem do
     end
   end
 
+  describe '.update_list' do
+    before { savon.mock! }
+    after { savon.unmock! }
+
+    context 'with one item' do
+      before do
+        savon.expects(:update_list).with(:message =>
+          {
+            'record' => [{
+              'listAcct:itemId' => 'Target',
+              '@xsi:type' => 'listAcct:InventoryItem',
+              '@internalId' => '624113'
+            }]
+        }).returns(File.read('spec/support/fixtures/update_list/update_list_one_item.xml'))
+      end
+
+      it 'returns collection with one InventoryItem instances populated with the data from the response object' do
+        items = NetSuite::Records::InventoryItem.update_list([
+                  NetSuite::Records::InventoryItem.new(internal_id: '624113', item_id: 'Target', upccode: 'Target')
+                ])
+        shutter_fly = items[0]
+        expect(shutter_fly).to be_kind_of(NetSuite::Records::InventoryItem)
+        expect(shutter_fly.item_id).to eq('Target')
+        expect(shutter_fly.internal_id).to eq('624113')
+      end
+    end
+
+    context 'with two items' do
+      before do
+        savon.expects(:update_list).with(:message =>
+          {
+            'record' => [{
+                'listAcct:itemId' => 'Shutter Fly',
+                '@xsi:type' => 'listAcct:InventoryItem',
+                '@internalId' => '624172'
+              },
+              {
+                'listAcct:itemId' => 'Target',
+                '@xsi:type' => 'listAcct:InventoryItem',
+                '@internalId' => '624113'
+              }
+            ]
+        }).returns(File.read('spec/support/fixtures/update_list/update_list_items.xml'))
+      end
+
+      it 'returns collection of InventoryItem instances populated with the data from the response object' do
+        items = NetSuite::Records::InventoryItem.update_list( [
+                  NetSuite::Records::InventoryItem.new(internal_id: '624172', item_id: 'Shutter Fly', upccode: 'Shutter Fly, Inc.'),
+                  NetSuite::Records::InventoryItem.new(internal_id: '624113', item_id: 'Target', upccode: 'Target')
+                ])
+        shutter_fly = items[0]
+        expect(shutter_fly).to be_kind_of(NetSuite::Records::InventoryItem)
+        expect(shutter_fly.item_id).to eq('Shutter Fly')
+        expect(shutter_fly.internal_id).to eq('624172')
+      end
+    end
+  end
 end

data/spec/netsuite/records/inventory_transfer_spec.rb

@@ -19,6 +19,27 @@ describe NetSuite::Records::InventoryTransfer do
     end
   end
 
+  describe '#to_record' do
+    it 'includes the inventory_list' do
+      attributes = {
+        :inventory => [{
+          :inventory_detail => {
+            :custom_form => {
+              internal_id: 1
+            },
+            :inventory_assignment_list => {
+              :inventory_assignment => [{
+                quantity: 3
+              }]
+            }
+          }
+        }]
+      }
+      inventory_transfer.inventory_list = attributes
+      list = inventory_transfer.to_record.fetch("tranInvt:inventoryList").fetch("tranInvt:inventory")
+      expect(list.count).to eq(1)
+    end
+  end
 
   describe '#inventory_list' do
     it 'can be set from attributes' do

data/spec/netsuite/records/invoice_spec.rb

@@ -9,14 +9,14 @@ describe NetSuite::Records::Invoice do
     [
       :balance, :bill_address,
       :billing_schedule, :contrib_pct, :created_date, :currency_name,
-      :deferred_revenue, :discount_amount, :discount_date, :discount_item, :discount_rate,
+      :deferred_revenue, :discount_amount, :discount_date, :discount_rate,
       :due_date, :email, :end_date, :est_gross_profit, :est_gross_profit_percent, :exchange_rate,
       :exclude_commission, :exp_cost_disc_amount, :exp_cost_disc_print, :exp_cost_disc_rate, :exp_cost_disc_tax_1_amt,
       :exp_cost_disc_taxable, :exp_cost_discount, :exp_cost_list, :exp_cost_tax_code, :exp_cost_tax_rate_1,
       :exp_cost_tax_rate_2, :fax, :fob, :gift_cert_redemption_list, :handling_tax_1_rate,
       :handling_tax_2_rate, :handling_tax_code, :is_taxable, :item_cost_disc_amount, :item_cost_disc_print,
       :item_cost_disc_rate, :item_cost_disc_tax_1_amt, :item_cost_disc_taxable, :item_cost_discount, :item_cost_list,
-      :item_cost_tax_code, :item_cost_tax_rate_1, :item_cost_tax_rate_2, :job, :last_modified_date,
+      :item_cost_tax_code, :item_cost_tax_rate_1, :item_cost_tax_rate_2, :last_modified_date,
       :linked_tracking_numbers, :memo, :message, :message_sel, :on_credit_hold, :opportunity,
       :other_ref_num, :partners_list, :rev_rec_end_date,
       :rev_rec_on_rev_commitment, :rev_rec_schedule, :rev_rec_start_date, :revenue_status, :sales_effective_date,
@@ -43,8 +43,8 @@ describe NetSuite::Records::Invoice do
 
   it 'has the right record_refs' do
     [
-      :account, :bill_address_list, :custom_form, :department, :entity, :klass, :posting_period, :ship_address_list, :terms,
-      :created_from, :location, :sales_rep, :ship_method, :tax_item, :partner, :lead_source, :promo_code, :subsidiary
+      :account, :bill_address_list, :job, :custom_form, :department, :entity, :klass, :posting_period, :ship_address_list, :terms,
+      :created_from, :location, :sales_rep, :ship_method, :tax_item, :partner, :lead_source, :promo_code, :subsidiary, :discount_item
     ].each do |record_ref|
       expect(invoice).to have_record_ref(record_ref)
     end

data/spec/netsuite/records/matrix_option_list_spec.rb

@@ -5,20 +5,30 @@ module NetSuite
   module Records
     describe MatrixOptionList do
       it "deals with hash properly" do
-        hash = {:value=>{:@internal_id=>"1", :@type_id=>"36"}}
+        hash = {:value=>{:@internal_id=>"1", :@type_id=>"36", :name=>"some value"}, :@script_id=>'cust_field_1'}
 
         list = described_class.new({ matrix_option: hash })
-        expect(list.options.first.value_id).to eq "1"
+        option = list.options.first
+        expect(option.value_id).to eq "1"
+        expect(option.type_id).to eq "36"
+        expect(option.name).to eq "some value"
+        expect(option.script_id).to eq "cust_field_1"
       end
 
       it "deals with arrays properly" do
         array = [
-          {:value=>{:@internal_id=>"2", :@type_id=>"28"}},
-          {:value=>{:@internal_id=>"1", :@type_id=>"29"}}
+          {:value=>{:@internal_id=>"2", :@type_id=>"28", :name=>"some value 28"}, :@script_id=>'cust_field_28'},
+          {:value=>{:@internal_id=>"1", :@type_id=>"29", :name=>"some value 29"}, :@script_id=>'cust_field_29'}
         ]
 
         list = described_class.new({ matrix_option: array })
-        expect(list.options.first.value_id).to eq "2"
+        expect(list.options.count).to eq 2
+
+        option = list.options.first
+        expect(option.value_id).to eq "2"
+        expect(option.type_id).to eq "28"
+        expect(option.name).to eq "some value 28"
+        expect(option.script_id).to eq "cust_field_28"
       end
     end
   end

data/spec/netsuite/records/message_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe NetSuite::Records::Message do
+  let(:message) { NetSuite::Records::Message.new }
+
+  it 'has all the right fields' do
+    [
+      :bcc, :cc, :compress_attachments, :date_time, :emailed,
+      :incoming, :message, :record_name, :record_type_name, :subject,
+      :last_modified_date, :message_date
+    ].each do |field|
+      expect(message).to have_field(field)
+    end
+  end
+
+  it 'has the right record_refs' do
+    [
+      :activity, :author, :recipient, :transaction
+    ].each do |record_ref|
+      expect(message).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 => {message: 'message text'})}
+
+      it 'returns a Message instance populated with data from the response object' do
+        expect(NetSuite::Actions::Get).to receive(:call).with([NetSuite::Records::Message, :external_id => 1], {}).and_return(response)
+
+        message = NetSuite::Records::Message.get(:external_id => 1)
+        expect(message).to be_kind_of(NetSuite::Records::Message)
+        expect(message.message).to eq('message text')
+      end
+    end
+
+    context 'when 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([NetSuite::Records::Message, :external_id => 1], {}).and_return(response)
+        expect {
+          NetSuite::Records::Message.get(:external_id => 1)
+        }.to raise_error(NetSuite::RecordNotFound,
+                         /NetSuite::Records::Message with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+end

data/spec/netsuite/records/non_inventory_resale_item_spec.rb

@@ -0,0 +1,165 @@
+require 'spec_helper'
+
+describe NetSuite::Records::NonInventoryResaleItem do
+  let(:item) { NetSuite::Records::NonInventoryResaleItem.new }
+
+  it 'has the right fields' do
+    [
+      :available_to_partners, :cost_estimate, :cost_estimate_type, :cost_estimate_units, :country_of_manufacture, :created_date,
+      :display_name, :dont_show_price, :enforce_min_qty_internally, :exclude_from_sitemap,
+      :featured_description, :handling_cost, :handling_cost_units, :include_children, :is_donation_item, :is_fulfillable,
+      :is_gco_compliant, :is_inactive, :is_online, :is_taxable, :item_id, :last_modified_date, :manufacturer, :manufacturer_addr1,
+      :manufacturer_city, :manufacturer_state, :manufacturer_tariff, :manufacturer_tax_id, :manufacturer_zip, :matrix_option_list,
+      :matrix_type, :max_donation_amount, :meta_tag_html, :minimum_quantity, :minimum_quantity_units, :mpn,
+      :mult_manufacture_addr, :nex_tag_category, :no_price_message, :offer_support, :on_special, :out_of_stock_behavior,
+      :out_of_stock_message, :overall_quantity_pricing_type, :page_title, :preference_criterion, :presentation_item_list,
+      :prices_include_tax, :producer, :product_feed_list, :rate, :related_items_description, :sales_description,
+      :schedule_b_code, :schedule_b_number, :schedule_b_quantity, :search_keywords, :ship_individually, :shipping_cost,
+      :shipping_cost_units, :shopping_dot_com_category, :shopzilla_category_id, :show_default_donation_amount,
+      :site_category_list, :sitemap_priority, :soft_descriptor, :specials_description, :stock_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, :weight, :weight_unit, :weight_units
+    ].each do |field|
+      expect(item).to have_field(field)
+    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)
+    expect(item.item_vendor_list.class).to eq(NetSuite::Records::ItemVendorList)
+
+  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, :ship_package, :store_display_image,
+        :store_display_thumbnail, :store_item_template, :tax_schedule, :units_type, :expense_account
+    ].each do |record_ref|
+      expect(item).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 => { :manufacturer_zip => '90401' }) }
+
+      it 'returns a NonInventoryResaleItem instance populated with the data from the response object' do
+        expect(NetSuite::Actions::Get).to receive(:call).with([NetSuite::Records::NonInventoryResaleItem, {:external_id => 20}], {}).and_return(response)
+        customer = NetSuite::Records::NonInventoryResaleItem.get(:external_id => 20)
+        expect(customer).to be_kind_of(NetSuite::Records::NonInventoryResaleItem)
+        expect(customer.manufacturer_zip).to eql('90401')
+      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([NetSuite::Records::NonInventoryResaleItem, {:external_id => 20}], {}).and_return(response)
+        expect {
+          NetSuite::Records::NonInventoryResaleItem.get(:external_id => 20)
+        }.to raise_error(NetSuite::RecordNotFound,
+          /NetSuite::Records::NonInventoryResaleItem with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+  describe '#add' do
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        expect(NetSuite::Actions::Add).to receive(:call).
+            with([item], {}).
+            and_return(response)
+        expect(item.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
+        expect(NetSuite::Actions::Add).to receive(:call).
+            with([item], {}).
+            and_return(response)
+        expect(item.add).to be_falsey
+      end
+    end
+  end
+
+  describe '#update' do
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        expect(NetSuite::Actions::Update).to receive(:call).
+            with([item.class, {external_id: 'foo'}], {}).
+            and_return(response)
+        item.external_id = 'foo'
+        expect(item.update).to be_truthy
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        expect(NetSuite::Actions::Update).to receive(:call).
+            with([item.class, {external_id: 'foo'}], {}).
+            and_return(response)
+        item.external_id = 'foo'
+        expect(item.update).to be_falsey
+      end
+    end
+  end
+
+  describe '#delete' do
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        expect(NetSuite::Actions::Delete).to receive(:call).
+            with([item], {}).
+            and_return(response)
+        expect(item.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
+        expect(NetSuite::Actions::Delete).to receive(:call).
+            with([item], {}).
+            and_return(response)
+        expect(item.delete).to be_falsey
+      end
+    end
+  end
+
+  describe '#to_record' do
+    before do
+      item.handling_cost = 100.0
+      item.is_online     = true
+    end
+
+    it 'can represent itself as a SOAP record' do
+      record = {
+        'listAcct:handlingCost' => 100.0,
+        'listAcct:isOnline'     => true
+      }
+      expect(item.to_record).to eql(record)
+    end
+  end
+
+  describe '#record_type' do
+    it 'returns a string of the SOAP type' do
+      expect(item.record_type).to eql('listAcct:NonInventoryResaleItem')
+    end
+  end
+
+end

data/spec/netsuite/records/non_inventory_sale_item_spec.rb

@@ -6,7 +6,7 @@ describe NetSuite::Records::NonInventorySaleItem do
   it 'has the right fields' do
     [
       :available_to_partners, :cost_estimate, :cost_estimate_type, :cost_estimate_units, :country_of_manufacture, :created_date,
-      :display_name, :dont_show_price, :enforce_min_qty_internally, :exclude_from_sitemap,
+      :direct_revenue_posting, :display_name, :dont_show_price, :enforce_min_qty_internally, :exclude_from_sitemap,
       :featured_description, :handling_cost, :handling_cost_units, :include_children, :is_donation_item, :is_fulfillable,
       :is_gco_compliant, :is_inactive, :is_online, :is_taxable, :item_id, :last_modified_date, :manufacturer, :manufacturer_addr1,
       :manufacturer_city, :manufacturer_state, :manufacturer_tariff, :manufacturer_tax_id, :manufacturer_zip, :matrix_option_list,

data/spec/netsuite/records/partner_spec.rb

@@ -0,0 +1,141 @@
+require 'spec_helper'
+
+describe NetSuite::Records::Partner do
+  let(:partner) { NetSuite::Records::Partner.new }
+
+  it 'has all the right fields' do
+    [
+        :phone, :home_phone, :first_name, :last_name, :alt_name, :is_inactive, :email, :give_access,
+        :partner_code, :is_person, :company_name, :eligible_for_commission, :entity_id, :last_modified_date,
+        :date_created, :title, :mobile_phone, :comments, :middle_name, :send_email, :password, :password2
+    ].each do |field|
+      expect(partner).to have_field(field)
+    end
+  end
+
+  it 'has all the right record refs' do
+    [
+        :klass, :access_role, :department, :subsidiary
+    ].each do |record_ref|
+      expect(partner).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 => { :account_number => 7 }) }
+
+      it 'returns an Partner instance populated with the data from the response object' do
+        expect(NetSuite::Actions::Get).to receive(:call).with([NetSuite::Records::Partner, { :external_id => 1 }], {}).and_return(response)
+        Partner = NetSuite::Records::Partner.get(:external_id => 1)
+        expect(Partner).to be_kind_of(NetSuite::Records::Partner)
+      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([NetSuite::Records::Partner, { :external_id => 1 }], {}).and_return(response)
+        expect {
+          NetSuite::Records::Partner.get(:external_id => 1)
+        }.to raise_error(NetSuite::RecordNotFound,
+                         /NetSuite::Records::Partner with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+  describe '#add' do
+    let(:partner) { NetSuite::Records::Partner.new(:email => 'dale.cooper@example.com') }
+
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        expect(NetSuite::Actions::Add).to receive(:call).
+                                            with([partner], {}).
+                                            and_return(response)
+        expect(partner.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
+        expect(NetSuite::Actions::Add).to receive(:call).
+                                            with([partner], {}).
+                                            and_return(response)
+        expect(partner.add).to be_falsey
+      end
+    end
+  end
+
+  describe '#delete' do
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        expect(NetSuite::Actions::Delete).to receive(:call).
+                                               with([partner], {}).
+                                               and_return(response)
+        expect(partner.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
+        expect(NetSuite::Actions::Delete).to receive(:call).
+                                               with([partner], {}).
+                                               and_return(response)
+        expect(partner.delete).to be_falsey
+      end
+    end
+  end
+
+  describe '.update' do
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :email => 'leland.palmer@example.com' }) }
+
+      it 'returns true' do
+        expect(NetSuite::Actions::Update).to receive(:call).with([NetSuite::Records::Partner, { :internal_id => 1, :email => 'leland.palmer@example.com' }], {}).and_return(response)
+        partner = NetSuite::Records::Partner.new(:internal_id => 1)
+        expect(partner.update(:email => 'leland.palmer@example.com')).to be_truthy
+      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::Update).to receive(:call).with([NetSuite::Records::Partner, { :internal_id => 1, :account_number => 7 }], {}).and_return(response)
+        partner = NetSuite::Records::Partner.new(:internal_id => 1)
+        expect(partner.update(:account_number => 7)).to be_falsey
+      end
+    end
+  end
+
+  describe '#to_record' do
+    let(:partner) { NetSuite::Records::Partner.new(:email => 'bob@example.com') }
+
+    it 'returns a hash of attributes that can be used in a SOAP request' do
+      expect(partner.to_record).to eql({ 'listRel:email' => 'bob@example.com' })
+    end
+  end
+
+  describe '#record_type' do
+    it 'returns a string type for the record to be used in a SOAP request' do
+      expect(partner.record_type).to eql('listRel:Partner')
+    end
+  end
+
+  it 'has the right record_refs' do
+    [
+        :klass, :access_role, :department, :subsidiary
+    ].each do |record_ref|
+      expect(partner).to have_record_ref(record_ref)
+    end
+  end
+end