netsuite 0.6.7 → 0.6.8

data/lib/netsuite.rb

@@ -167,6 +167,7 @@ module NetSuite
     autoload :OpportunityItem,                  'netsuite/records/opportunity_item'
     autoload :OpportunityItemList,              'netsuite/records/opportunity_item_list'
     autoload :Partner,                          'netsuite/records/partner'
+    autoload :PaymentItem,                      'netsuite/records/payment_item'
     autoload :PaymentMethod,                    'netsuite/records/payment_method'
     autoload :PayrollItem,                      'netsuite/records/payroll_item'
     autoload :PhoneCall,                        'netsuite/records/phone_call'

data/lib/netsuite/records/job.rb

@@ -22,6 +22,7 @@ module NetSuite
       field :estimated_time_override, Duration
       field :actual_time,             Duration
       field :time_remaining,          Duration
+      field :custom_field_list,       CustomFieldList
 
       record_refs :billing_schedule, :category, :currency, :custom_form, :entity_status, :estimate_rev_rec_template, :job_item,
         :job_type, :language, :parent, :subsidiary, :workplace

data/lib/netsuite/records/payment_item.rb

@@ -0,0 +1,36 @@
+module NetSuite
+  module Records
+    class PaymentItem
+      include Support::Fields
+      include Support::RecordRefs
+      include Support::Records
+      include Support::Actions
+      include Namespaces::ListAcct
+
+      actions :get, :get_list, :add, :delete, :search, :update, :upsert
+
+      fields :available_to_partners, :created_date, :description, :display_name, :include_children, :is_inactive, :item_id, :last_modified_date, :undep_funds
+
+      record_refs :account, :custom_form, :department, :issue_product, :klass, :location, :payment_method
+
+      field :custom_field_list, CustomFieldList
+      field :subsidiary_list, RecordRefList
+      
+      # TODO custom records need to be implemented
+      # field :translations_list, TranslationList
+
+      attr_reader :internal_id
+      attr_accessor :external_id
+
+      def initialize(attributes = {})
+        @internal_id = attributes.delete(:internal_id) || attributes.delete(:@internal_id)
+        @external_id = attributes.delete(:external_id) || attributes.delete(:@external_id)
+        initialize_from_attributes_hash(attributes)
+      end
+
+      def self.search_class_name
+        "Item"
+      end
+    end
+  end
+end

data/lib/netsuite/records/pricing_matrix.rb

@@ -4,6 +4,8 @@ module NetSuite
       include Namespaces::PlatformCore
 
       def initialize(attributes = {})
+        attributes[:pricing] = [attributes[:pricing]] unless
+          attributes[:pricing].is_a? Array
         attributes[:pricing].each do |pricing|
           prices << RecordRef.new(pricing)
         end if attributes[:pricing]

data/lib/netsuite/records/vendor_bill_expense_list.rb

@@ -1,31 +1,12 @@
 module NetSuite
   module Records
-    class VendorBillExpenseList
-      include Support::Fields
+    class VendorBillExpenseList < Support::Sublist
       include Namespaces::TranPurch
 
-      fields :expense
+      sublist :expense, VendorBillExpense
 
-      def initialize(attributes = {})
-        initialize_from_attributes_hash(attributes)
-      end
-
-      def expense=(expenses)
-        case expenses
-          when Hash
-            self.expenses << VendorBillExpense.new(expenses)
-          when Array
-            expenses.each { |expense| self.expenses << VendorBillExpense.new(expense) }
-        end
-      end
-
-      def expenses
-        @expenses ||= []
-      end
-
-      def to_record
-        { "#{record_namespace}:expense" => expenses.map(&:to_record) }
-      end
+      # legacy support
+      alias :expenses :expense
 
     end
   end

data/lib/netsuite/version.rb

@@ -1,3 +1,3 @@
 module Netsuite
-  VERSION = '0.6.7'
+  VERSION = '0.6.8'
 end

data/spec/netsuite/records/basic_record_spec.rb

@@ -12,6 +12,7 @@ describe 'basic records' do
       NetSuite::Records::PayrollItem,
       NetSuite::Records::Opportunity,
       NetSuite::Records::VendorCategory,
+      NetSuite::Records::VendorBill,
       NetSuite::Records::Deposit,
       NetSuite::Records::RevRecTemplate,
       NetSuite::Records::RevRecSchedule,
@@ -31,6 +32,7 @@ describe 'basic records' do
       NetSuite::Records::ItemReceipt,
       NetSuite::Records::GiftCertificate,
       NetSuite::Records::ContactRole,
+      NetSuite::Records::PaymentItem,
     ]
   }
 

data/spec/netsuite/records/payment_item_spec.rb

@@ -0,0 +1,117 @@
+require 'spec_helper'
+
+describe NetSuite::Records::PaymentItem do
+  let(:item) { NetSuite::Records::PaymentItem.new }
+
+  it 'has all the right fields' do
+    [
+      :available_to_partners, :created_date, :description, :display_name, :include_children, :is_inactive, :item_id, :last_modified_date, :undep_funds
+    ].each do |field|
+      expect(item).to have_field(field)
+    end
+  end
+
+  it 'has all the right record refs' do
+    [
+      :account, :custom_form, :department, :issue_product, :klass, :location, :payment_method
+    ].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 => { :item_id => "Item100" }) }
+
+      it 'returns a PaymentItem instance populated with the data from the response object' do
+        expect(NetSuite::Actions::Get).to receive(:call).with([NetSuite::Records::PaymentItem, {:external_id => 1}], {}).and_return(response)
+        item = NetSuite::Records::PaymentItem.get(:external_id => 1)
+        expect(item).to be_kind_of(NetSuite::Records::PaymentItem)
+        expect(item.item_id).to eql("Item100")
+      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::PaymentItem, {:external_id => 1}], {}).and_return(response)
+        expect {
+          NetSuite::Records::PaymentItem.get(:external_id => 1)
+        }.to raise_error(NetSuite::RecordNotFound,
+          /NetSuite::Records::PaymentItem with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+  describe '#add' do
+    let(:item) { NetSuite::Records::PaymentItem.new(:item_id => "Item100", :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
+        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 '#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.item_id = "Item100"
+      item.is_inactive = false
+    end
+    it 'can represent itself as a SOAP record' do
+      record = {
+        'listAcct:itemId'       => "Item100",
+        'listAcct:isInactive' => false
+      }
+      expect(item.to_record).to eql(record)
+    end
+  end
+
+  describe '#record_type' do
+    it 'returns a string representation of the SOAP type' do
+      expect(item.record_type).to eql('listAcct:PaymentItem')
+    end
+  end
+
+end

data/spec/netsuite/records/pricing_matrix_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe NetSuite::Records::PricingMatrix do
+  describe "#initialize" do
+    it "behaves appropriately if it gets a hash as attributes[:pricing]" do
+      # this is what savon returns if there is only one pricing strategy matrix
+      # for the item:
+      matrix = {
+        :pricing=> {
+          :currency=>"US Dollar",
+          :priceLevel=>"Base Price",
+          :priceList=>[
+            {:price=>{:value=>12.0, :quantity=>0.0}},
+            {:price=>{:value=>10.0, :quantity=>10.0}},
+            {:price=>{:value=>9.0, :quantity=>100.0}},
+            {:price=>{:value=>7.0, :quantity=>5000.0}}
+          ]
+        }
+      }
+
+      subject = NetSuite::Records::PricingMatrix.new({pricing: matrix})
+      expect(subject.prices[0].pricing).to eq(matrix[:pricing])
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: netsuite
 version: !ruby/object:Gem::Version
-  version: 0.6.7
+  version: 0.6.8
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-25 00:00:00.000000000 Z
+date: 2016-07-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: savon
@@ -205,6 +205,7 @@ files:
 - lib/netsuite/records/opportunity_item.rb
 - lib/netsuite/records/opportunity_item_list.rb
 - lib/netsuite/records/partner.rb
+- lib/netsuite/records/payment_item.rb
 - lib/netsuite/records/payment_method.rb
 - lib/netsuite/records/payroll_item.rb
 - lib/netsuite/records/phone_call.rb
@@ -345,9 +346,11 @@ files:
 - spec/netsuite/records/location_spec.rb
 - spec/netsuite/records/matrix_option_list_spec.rb
 - spec/netsuite/records/non_inventory_sale_item_spec.rb
+- spec/netsuite/records/payment_item_spec.rb
 - spec/netsuite/records/payment_method_spec.rb
 - spec/netsuite/records/phone_call_spec.rb
 - spec/netsuite/records/pomo_code_spec.rb
+- spec/netsuite/records/pricing_matrix_spec.rb
 - spec/netsuite/records/record_ref_spec.rb
 - spec/netsuite/records/rev_rec_template_spec.rb
 - spec/netsuite/records/sales_order_item_list_spec.rb
@@ -523,9 +526,11 @@ test_files:
 - spec/netsuite/records/location_spec.rb
 - spec/netsuite/records/matrix_option_list_spec.rb
 - spec/netsuite/records/non_inventory_sale_item_spec.rb
+- spec/netsuite/records/payment_item_spec.rb
 - spec/netsuite/records/payment_method_spec.rb
 - spec/netsuite/records/phone_call_spec.rb
 - spec/netsuite/records/pomo_code_spec.rb
+- spec/netsuite/records/pricing_matrix_spec.rb
 - spec/netsuite/records/record_ref_spec.rb
 - spec/netsuite/records/rev_rec_template_spec.rb
 - spec/netsuite/records/sales_order_item_list_spec.rb