netsuite 0.2.6 → 0.3.0

data/spec/netsuite/records/customer_addressbook_list_spec.rb

@@ -10,10 +10,28 @@ describe NetSuite::Records::CustomerAddressbookList do
   describe '#to_record' do
     it 'can represent itself as a SOAP record' do
       record = {
-        'listRel:addressbook' => []
+        'listRel:addressbook' => [],
+        'listRel:replaceAll' => true
       }
       list.to_record.should eql(record)
     end
   end
 
+  describe "#replace_all" do
+    it "returns true by default" do
+      list.replace_all.should eql(true)
+    end
+
+    it "can be changed via accessor" do
+      list.replace_all = false
+
+      list.replace_all.should eql(false)
+    end
+
+    it "coerces to a boolean" do
+      list.replace_all = "goober"
+      list.replace_all.should eql(true)
+    end
+  end
+
 end

data/spec/netsuite/records/customer_addressbook_spec.rb

@@ -24,24 +24,32 @@ describe NetSuite::Records::CustomerAddressbook do
   it 'has all the right fields' do
     [
       :default_shipping, :default_billing, :is_residential, :label, :attention, :addressee,
-      :phone, :addr1, :addr2, :addr3, :city, :zip, :country, :addr_text, :override, :state
+      :phone, :addr1, :addr2, :addr3, :city, :zip, :addr_text, :override, :state
     ].each do |field|
       list.should have_field(field)
     end
   end
 
+  it 'has all the right read_only_fields' do
+    [
+      :addr_text
+    ].each do |field|
+      NetSuite::Records::CustomerAddressbook.should have_read_only_field(field)
+    end
+  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
         list.addr1.should eql('123 Happy Lane')
         list.addr_text.should eql("123 Happy Lane\nLos Angeles CA 90007")
         list.city.should eql('Los Angeles')
-        list.country.should eql('_unitedStates')
-        list.default_billing.should be_true
-        list.default_shipping.should be_true
-        list.is_residential.should be_false
+        list.country.to_record.should eql('_unitedStates')
+        list.default_billing.should be_truthy
+        list.default_shipping.should be_truthy
+        list.is_residential.should be_falsey
         list.label.should eql('123 Happy Lane')
-        list.override.should be_false
+        list.override.should be_falsey
         list.state.should eql('CA')
         list.zip.should eql('90007')
         list.internal_id.should eql('567')
@@ -55,12 +63,12 @@ describe NetSuite::Records::CustomerAddressbook do
         list.addr1.should eql('123 Happy Lane')
         list.addr_text.should eql("123 Happy Lane\nLos Angeles CA 90007")
         list.city.should eql('Los Angeles')
-        list.country.should eql('_unitedStates')
-        list.default_billing.should be_true
-        list.default_shipping.should be_true
-        list.is_residential.should be_false
+        list.country.to_record.should eql('_unitedStates')
+        list.default_billing.should be_truthy
+        list.default_shipping.should be_truthy
+        list.is_residential.should be_falsey
         list.label.should eql('123 Happy Lane')
-        list.override.should be_false
+        list.override.should be_falsey
         list.state.should eql('CA')
         list.zip.should eql('90007')
         list.internal_id.should eql('567')
@@ -72,7 +80,6 @@ describe NetSuite::Records::CustomerAddressbook do
     it 'can represent itself as a SOAP record' do
       record = {
         'listRel:addr1'           => '123 Happy Lane',
-        'listRel:addrText'        => "123 Happy Lane\nLos Angeles CA 90007",
         'listRel:city'            => 'Los Angeles',
         'listRel:country'         => '_unitedStates',
         'listRel:defaultBilling'  => true,
@@ -81,12 +88,34 @@ describe NetSuite::Records::CustomerAddressbook do
         'listRel:label'           => '123 Happy Lane',
         'listRel:override'        => false,
         'listRel:state'           => 'CA',
-        'listRel:zip'             => '90007'
+        'listRel:zip'             => '90007',
+        "listRel:internalId"      => "567"
       }
       list.to_record.should eql(record)
     end
   end
 
+  describe "#country" do
+    context "when it's specified as a 2 character ISO code" do
+      it "is converted to the appropriate NetSuite enum value" do
+        addressbook = NetSuite::Records::CustomerAddressbook.new country: "US"
+        addressbook.to_record["listRel:country"].should eql "_unitedStates"
+      end
+    end
+
+    context "when the country code is a YAML reserved word (NO)" do
+      it "is converted to the appropriate NetSuite enum value" do
+        addressbook = NetSuite::Records::CustomerAddressbook.new country: "NO"
+        addressbook.to_record["listRel:country"].should eql "_norway"
+      end
+    end
+
+    it "can be specified as the NetSuite enum value" do
+      addressbook = NetSuite::Records::CustomerAddressbook.new country: "_unitedStates"
+      addressbook.to_record["listRel:country"].should eql "_unitedStates"
+    end
+  end
+
   describe '#record_type' do
     it 'returns a string of the record SOAP type' do
       list.record_type.should eql('listRel:CustomerAddressbook')

data/spec/netsuite/records/customer_payment_spec.rb

@@ -109,7 +109,7 @@ describe NetSuite::Records::CustomerPayment do
         NetSuite::Actions::Add.should_receive(:call).
             with([payment], {}).
             and_return(response)
-        payment.add.should be_true
+        payment.add.should be_truthy
       end
     end
 
@@ -121,7 +121,7 @@ describe NetSuite::Records::CustomerPayment do
         NetSuite::Actions::Add.should_receive(:call).
             with([payment], {}).
             and_return(response)
-        payment.add.should be_false
+        payment.add.should be_falsey
       end
     end
   end
@@ -134,7 +134,7 @@ describe NetSuite::Records::CustomerPayment do
         NetSuite::Actions::Delete.should_receive(:call).
             with([payment], {}).
             and_return(response)
-        payment.delete.should be_true
+        payment.delete.should be_truthy
       end
     end
 
@@ -145,7 +145,7 @@ describe NetSuite::Records::CustomerPayment do
         NetSuite::Actions::Delete.should_receive(:call).
             with([payment], {}).
             and_return(response)
-        payment.delete.should be_false
+        payment.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/customer_refund_spec.rb

@@ -131,7 +131,7 @@ describe NetSuite::Records::CustomerRefund do
         NetSuite::Actions::Add.should_receive(:call).
             with([refund], {}).
             and_return(response)
-        refund.add.should be_true
+        refund.add.should be_truthy
       end
     end
 
@@ -143,7 +143,7 @@ describe NetSuite::Records::CustomerRefund do
         NetSuite::Actions::Add.should_receive(:call).
             with([refund], {}).
             and_return(response)
-        refund.add.should be_false
+        refund.add.should be_falsey
       end
     end
   end
@@ -157,7 +157,7 @@ describe NetSuite::Records::CustomerRefund do
         NetSuite::Actions::Delete.should_receive(:call).
             with([refund], {}).
             and_return(response)
-        refund.delete.should be_true
+        refund.delete.should be_truthy
       end
     end
 
@@ -169,7 +169,7 @@ describe NetSuite::Records::CustomerRefund do
         NetSuite::Actions::Delete.should_receive(:call).
             with([refund], {}).
             and_return(response)
-        refund.delete.should be_false
+        refund.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/customer_spec.rb

@@ -8,7 +8,7 @@ describe NetSuite::Records::Customer do
       :access_role, :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, :contact_roles_list, :contrib_pct, :credit_cards_list, :credit_hold_override, :credit_limit,
+      :consol_unbilled_orders, :contrib_pct, :credit_cards_list, :credit_hold_override, :credit_limit,
       :currency, :currency_list, :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,
@@ -34,6 +34,8 @@ describe NetSuite::Records::Customer do
     end
   end
 
+  # TODO test contact_roles_list
+
   describe '#addressbook_list' do
     it 'can be set from attributes' do
       customer.addressbook_list = {
@@ -91,7 +93,7 @@ describe NetSuite::Records::Customer do
         NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Customer, {:external_id => 1}], {}).and_return(response)
         customer = NetSuite::Records::Customer.get(:external_id => 1)
         customer.should be_kind_of(NetSuite::Records::Customer)
-        customer.is_person.should be_true
+        customer.is_person.should be_truthy
       end
     end
 
@@ -118,7 +120,7 @@ describe NetSuite::Records::Customer do
         NetSuite::Actions::Add.should_receive(:call).
             with([customer], {}).
             and_return(response)
-        customer.add.should be_true
+        customer.add.should be_truthy
       end
     end
 
@@ -129,7 +131,7 @@ describe NetSuite::Records::Customer do
         NetSuite::Actions::Add.should_receive(:call).
             with([customer], {}).
             and_return(response)
-        customer.add.should be_false
+        customer.add.should be_falsey
       end
     end
   end
@@ -142,7 +144,7 @@ describe NetSuite::Records::Customer do
         NetSuite::Actions::Delete.should_receive(:call).
             with([customer], {}).
             and_return(response)
-        customer.delete.should be_true
+        customer.delete.should be_truthy
       end
     end
 
@@ -153,7 +155,7 @@ describe NetSuite::Records::Customer do
         NetSuite::Actions::Delete.should_receive(:call).
             with([customer], {}).
             and_return(response)
-        customer.delete.should be_false
+        customer.delete.should be_falsey
       end
     end
   end
@@ -175,4 +177,76 @@ describe NetSuite::Records::Customer do
     end
   end
 
+  describe '.upsert_list' do
+    before { savon.mock! }
+    after { savon.unmock! }
+
+    context 'with one customer' do
+      before do
+        savon.expects(:upsert_list).with(:message =>
+          {
+            'record' => [{
+              'listRel:entityId'    => 'Target',
+              'listRel:companyName' => 'Target',
+              '@xsi:type' => 'listRel:Customer',
+              '@externalId' => 'ext2'
+            }]
+        }).returns(File.read('spec/support/fixtures/upsert_list/upsert_list_one_customer.xml'))
+      end
+
+      it 'returns collection with one Customer instances populated with the data from the response object' do
+        customers = NetSuite::Records::Customer.upsert_list([
+          {
+            external_id: 'ext2',
+            entity_id: 'Target',
+            company_name: 'Target'
+          }
+        ])
+        shutter_fly = customers[0]
+        expect(shutter_fly).to be_kind_of(NetSuite::Records::Customer)
+        expect(shutter_fly.entity_id).to eq('Target')
+        expect(shutter_fly.internal_id).to eq('974')
+      end
+    end
+
+    context 'with two customers' do
+      before do
+        savon.expects(:upsert_list).with(:message =>
+          {
+            'record' => [{
+                'listRel:entityId'    => 'Shutter Fly',
+                'listRel:companyName' => 'Shutter Fly, Inc.',
+                '@xsi:type' => 'listRel:Customer',
+                '@externalId' => 'ext1'
+              },
+              {
+                'listRel:entityId'    => 'Target',
+                'listRel:companyName' => 'Target',
+                '@xsi:type' => 'listRel:Customer',
+                '@externalId' => 'ext2'
+              }
+            ]
+        }).returns(File.read('spec/support/fixtures/upsert_list/upsert_list_customers.xml'))
+      end
+
+      it 'returns collection of Customer instances populated with the data from the response object' do
+        customers = NetSuite::Records::Customer.upsert_list([
+          {
+            external_id: 'ext1',
+            entity_id: 'Shutter Fly',
+            company_name: 'Shutter Fly, Inc.'
+          },
+          {
+            external_id: 'ext2',
+            entity_id: 'Target',
+            company_name: 'Target'
+          }
+        ])
+        shutter_fly = customers[0]
+        expect(shutter_fly).to be_kind_of(NetSuite::Records::Customer)
+        expect(shutter_fly.entity_id).to eq('Shutter Fly')
+        expect(shutter_fly.internal_id).to eq('970')
+      end
+    end
+  end
 end

data/spec/netsuite/records/department_spec.rb

@@ -55,7 +55,7 @@ describe NetSuite::Records::Department do
         NetSuite::Actions::Add.should_receive(:call).
             with([department], {}).
             and_return(response)
-        department.add.should be_true
+        department.add.should be_truthy
       end
     end
 
@@ -67,7 +67,7 @@ describe NetSuite::Records::Department do
         NetSuite::Actions::Add.should_receive(:call).
             with([department], {}).
             and_return(response)
-        department.add.should be_false
+        department.add.should be_falsey
       end
     end
   end
@@ -83,7 +83,7 @@ describe NetSuite::Records::Department do
         NetSuite::Actions::Delete.should_receive(:call).
             with([department], {}).
             and_return(response)
-        department.delete.should be_true
+        department.delete.should be_truthy
       end
     end
 
@@ -95,7 +95,7 @@ describe NetSuite::Records::Department do
         NetSuite::Actions::Delete.should_receive(:call).
             with([department], {}).
             and_return(response)
-        department.delete.should be_false
+        department.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/deposit_spec.rb

@@ -42,6 +42,24 @@ describe NetSuite::Records::Deposit do
     end
   end
 
+  describe '#cash_back_list' do
+    it 'can be set from attributes' do
+      attributes = {
+        amount: 100,
+        memo: "test"
+      }
+      deposit.cash_back_list.cashback = attributes
+      deposit.cash_back_list.should be_kind_of(NetSuite::Records::DepositCashBackList)
+      deposit.cash_back_list.cashbacks.length.should eql(1)
+    end
+
+    it 'can be set from a DepositCashBackList object' do
+      item_list = NetSuite::Records::DepositCashBackList.new
+      deposit.cash_back_list = item_list
+      deposit.cash_back_list.should eql(item_list)
+    end
+  end
+
   describe '.get' do
     context 'when the response is successful' do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :memo => 'transfer for subscriptions' }) }
@@ -78,7 +96,7 @@ describe NetSuite::Records::Deposit do
         NetSuite::Actions::Add.should_receive(:call).
             with([deposit], {}).
             and_return(response)
-        deposit.add.should be_true
+        deposit.add.should be_truthy
       end
     end
 
@@ -90,7 +108,7 @@ describe NetSuite::Records::Deposit do
         NetSuite::Actions::Add.should_receive(:call).
             with([deposit], {}).
             and_return(response)
-        deposit.add.should be_false
+        deposit.add.should be_falsey
       end
     end
   end
@@ -106,7 +124,7 @@ describe NetSuite::Records::Deposit do
         NetSuite::Actions::Delete.should_receive(:call).
             with([deposit], {}).
             and_return(response)
-        deposit.delete.should be_true
+        deposit.delete.should be_truthy
       end
     end
 
@@ -118,7 +136,7 @@ describe NetSuite::Records::Deposit do
         NetSuite::Actions::Delete.should_receive(:call).
             with([deposit], {}).
             and_return(response)
-        deposit.delete.should be_false
+        deposit.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/discount_item_spec.rb

@@ -0,0 +1,124 @@
+require 'spec_helper'
+
+describe NetSuite::Records::DiscountItem do
+  let(:item) { NetSuite::Records::DiscountItem.new }
+
+  it 'has the right fields' do
+    [
+      :available_to_partners, :created_date, :description, :display_name, :include_children, :is_inactive, :is_pretax,
+      :item_id, :last_modified_date, :non_posting, :rate, :upc_code, :vendor_name
+    ].each do |field|
+      item.should have_field(field)
+    end
+
+    # TODO there is a probably a more robust way to test this
+    item.custom_field_list.class.should == NetSuite::Records::CustomFieldList
+  end
+
+  it 'has the right record_refs' do
+    [
+      :account, :custom_form, :deferred_revenue_account, :department, :expense_account,
+      :income_account, :issue_product, :klass, :location, :parent, :rev_rec_schedule, :sales_tax_code,
+      :subsidiary_list, :tax_schedule
+    ].each do |record_ref|
+      item.should 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 => { :item_id => 'penguins' }) }
+
+      it 'returns a DiscountItem instance populated with the data from the response object' do
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::DiscountItem, :external_id => 20], {}).and_return(response)
+        customer = NetSuite::Records::DiscountItem.get(:external_id => 20)
+        customer.should be_kind_of(NetSuite::Records::DiscountItem)
+        customer.item_id.should eql('penguins')
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'raises a RecordNotFound exception' do
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::DiscountItem, :external_id => 20], {}).and_return(response)
+        lambda {
+          NetSuite::Records::DiscountItem.get(:external_id => 20)
+        }.should raise_error(NetSuite::RecordNotFound,
+          /NetSuite::Records::DiscountItem with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+  describe '#add' do
+    # let(:item) { NetSuite::Records::DiscountItem.new(:cost => 100, :is_inactive => false) }
+
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        NetSuite::Actions::Add.should_receive(:call).
+            with([item], {}).
+            and_return(response)
+        item.add.should be_truthy
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        NetSuite::Actions::Add.should_receive(:call).
+            with([item], {}).
+            and_return(response)
+        item.add.should 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
+        NetSuite::Actions::Delete.should_receive(:call).
+            with([item], {}).
+            and_return(response)
+        item.delete.should be_truthy
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        NetSuite::Actions::Delete.should_receive(:call).
+            with([item], {}).
+            and_return(response)
+        item.delete.should be_falsey
+      end
+    end
+  end
+
+  describe '#to_record' do
+    before do
+      item.item_id = 'penguins'
+      item.is_inactive = true
+    end
+
+    it 'can represent itself as a SOAP record' do
+      record = {
+        'listAcct:itemId' => 'penguins',
+        'listAcct:isInactive'     => true
+      }
+      item.to_record.should eql(record)
+    end
+  end
+
+  describe '#record_type' do
+    it 'returns a string of the SOAP type' do
+      item.record_type.should eql('listAcct:DiscountItem')
+    end
+  end
+
+end