netsuite 0.0.47 → 0.0.48

data/README.md

@@ -18,13 +18,32 @@ Or install it yourself as:
 
     $ gem install netsuite
 
+This gem is built for ruby 1.9.x, checkout the [1-8-stable](https://github.com/RevolutionPrep/netsuite/tree/1-8-stable) branch for ruby 1.8.x support.
 ## Usage
 
+### Configuration
+```ruby
+NetSuite.configure do
+  reset!
+  
+  api_version	'2012_1'
+  
+  # specify full wsdl URL for sandbox / production switching
+  wsdl          "https://webservices.sandbox.netsuite.com/wsdl/v#{api_version}_0/netsuite.wsdl"
+  
+  # login information
+  email    		'email@domain.com'
+  password 		'password'
+  account   	'12345'
+  role      	1111
+end
+```
+
 ### Customer
 
 * Initializing a customer can be done using a hash of attributes.
 
-#### Get
+### Get
 
 * Retrieves the customer by internalId.
 
@@ -33,7 +52,6 @@ Or install it yourself as:
     customer.is_person                            # => true
     ```
 
-<a name='extending'>
 ## Additions
 
 * Please submit a pull request for any models or actions that you would like to be included. The API is quite large and so we will necessarily not cover all of it.
@@ -99,6 +117,18 @@ Or install it yourself as:
 
 1. Fork it
 2. Create your feature branch (`git checkout -b my-new-feature`)
-3. Commit your changes (`git commit -am 'Added some feature'`)
-4. Push to the branch (`git push origin my-new-feature`)
-5. Create new Pull Request
+3. Write new test and ensure that `bundle exec rspec` doesn't fail
+4. Commit your changes (`git commit -am 'Added some feature'`)
+5. Push to the branch (`git push origin my-new-feature`)
+6. Create new Pull Request
+
+## Fields and RecordRefs
+
+Note that some record attributes are specified as Fields while others are specified as RecordRefs. In some cases
+attributes are actually RecordRefs in the schema, but we indicate them as Fields. Our experience has shown
+this works as long as the attribute is only read from and is not written to. Writing a value for an attribute
+that has been wrongly specified will result in an error. Be careful when initializing objects from other objects --
+they may carry attributes that write to the new object.
+
+As we build up this gem we will replace these inconsistent Fields with RecordRefs. Feel free to contribute new Record 
+definitions to help us along.

data/lib/netsuite.rb

@@ -1,8 +1,10 @@
 require 'set'
 
+require 'savon'
 require 'netsuite/version'
 require 'netsuite/errors'
 require 'netsuite/xml_logger'
+require 'netsuite/core_ext/string/lower_camelcase'
 
 module NetSuite
   autoload :Configuration, 'netsuite/configuration'
@@ -37,6 +39,7 @@ module NetSuite
   end
 
   module Records
+    autoload :AssemblyItem,               'netsuite/records/assembly_item'
     autoload :Account,                    'netsuite/records/account'
     autoload :AccountingPeriod,           'netsuite/records/accounting_period'
     autoload :BillAddress,                'netsuite/records/bill_address'
@@ -60,6 +63,7 @@ module NetSuite
     autoload :CustomerRefundApplyList,    'netsuite/records/customer_refund_apply_list'
     autoload :CustomerRefundDeposit,      'netsuite/records/customer_refund_deposit'
     autoload :CustomerRefundDepositList,  'netsuite/records/customer_refund_deposit_list'
+    autoload :Department,                 'netsuite/records/department'
     autoload :Duration,                   'netsuite/records/duration'
     autoload :InventoryItem,              'netsuite/records/inventory_item'
     autoload :Invoice,                    'netsuite/records/invoice'
@@ -72,9 +76,11 @@ module NetSuite
     autoload :Location,                   'netsuite/records/location'
     autoload :NonInventorySaleItem,       'netsuite/records/non_inventory_sale_item'
     autoload :PaymentMethod,              'netsuite/records/payment_method'
+    autoload :PricingMatrix,              'netsuite/records/pricing_matrix'
     autoload :RecordRef,                  'netsuite/records/record_ref'
     autoload :RevRecTemplate,             'netsuite/records/rev_rec_template'
     autoload :ShipAddress,                'netsuite/records/ship_address'
+    autoload :Term,                       'netsuite/records/term'
   end
 
   def self.configure(&block)

data/lib/netsuite/core_ext/string/lower_camelcase.rb

@@ -0,0 +1,9 @@
+class String
+  def lower_camelcase
+    str = dup
+    str.gsub!(/\/(.?)/) { "::#{$1.upcase}" }
+    str.gsub!(/(?:_+|-+)([a-z])/) { $1.upcase }
+    str.gsub!(/(\A|\s)([A-Z])/) { $1 + $2.downcase }
+    str
+  end
+end

data/lib/netsuite/records/assembly_item.rb

@@ -0,0 +1,55 @@
+module NetSuite
+  module Records
+    class AssemblyItem
+      include Support::Fields
+      include Support::RecordRefs
+      include Support::Records
+      include Support::Actions
+      include Namespaces::ListAcct
+
+      actions :get, :add, :delete
+
+      fields :auto_lead_time, :auto_preferred_stock_level, :auto_reorder_point, :available_to_partners, :average_cost,
+        :copy_description, :cost, :cost_estimate, :cost_estimate_type, :cost_estimate_units, :cost_units, :costing_method,
+        :costing_method_display, :country_of_manufacture, :created_date, :currency, :date_converted_to_inv,
+        :default_return_cost, :demand_modifier, :display_name, :dont_show_price, :enforce_min_qty_internally,
+        :exclude_from_sitemap, :featured_description, :fixed_lot_size, :handling_cost, :handling_cost_units, :include_children,
+        :is_donation_item, :is_drop_ship_item, :is_gco_compliant, :is_inactive, :is_online, :is_special_order_item, :is_taxable,
+        :item_id, :last_modified_date, :last_purchase_price, :lead_time, :manufacturer, :manufacturer_addr1, :manufacturer_city,
+        :manufacturer_state, :manufacturer_tariff, :manufacturer_tax_id, :manufacturer_zip, :match_bill_to_receipt,
+        :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_hand_value_mli, :on_special,
+        :original_item_subtype, :original_item_type, :out_of_stock_behavior, :out_of_stock_message,
+        :overall_quantity_pricing_type, :page_title, :preference_criterion, :preferred_stock_level, :preferred_stock_level_days,
+        :preferred_stock_level_units, :prices_include_tax, :producer, :purchase_description, :quantity_available,
+        :quantity_available_units, :quantity_back_ordered, :quantity_committed, :quantity_committed_units, :quantity_on_hand,
+        :quantity_on_hand_units, :quantity_on_order, :quantity_on_order_units, :quantity_reorder_units, :rate,
+        :related_items_description, :reorder_multiple, :reorder_point, :reorder_point_units, :safety_stock_level,
+        :safety_stock_level_days, :safety_stock_level_units, :sales_description, :schedule_b_code, :schedule_b_number,
+        :schedule_b_quantity, :search_keywords, :seasonal_demand, :ship_individually, :shipping_cost, :shipping_cost_units,
+        :shopping_dot_com_category, :shopzilla_category_id, :show_default_donation_amount, :sitemap_priority,
+        :specials_description, :stock_description, :store_description, :store_detailed_description, :store_display_name,
+        :total_value, :track_landed_cost, :transfer_price, :upc_code, :url_component, :use_bins, :use_marginal_rates,
+        :vendor_name, :vsoe_deferral, :vsoe_delivered, :vsoe_permit_discount, :vsoe_price, :weight, :weight_unit, :weight_units
+
+      record_refs :alternate_demand_source_item, :asset_account, :bill_exch_rate_variance_acct, :bill_price_variance_acct,
+        :bill_qty_variance_acct, :billing_schedule, :cogs_account, :cost_category, :custom_form, :deferred_revenue_account,
+        :demand_source, :department, :expense_account, :gain_loss_account, :income_account, :issue_product, :klass, :location,
+        :parent, :preferred_location, :pricing_group, :purchase_price_variance_acct, :purchase_tax_code, :purchase_unit,
+        :quantity_pricing_schedule, :rev_rec_schedule, :sale_unit, :sales_tax_code, :ship_package, :soft_descriptor,
+        :stock_unit, :store_display_image, :store_display_thumbnail, :store_item_template, :supply_lot_sizing_method,
+        :supply_replenishment_method, :supply_type, :tax_schedule, :units_type, :vendor
+
+      field :custom_field_list, CustomFieldList
+
+      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
+    end
+  end
+end

data/lib/netsuite/records/custom_field_list.rb

@@ -10,11 +10,24 @@ module NetSuite
         when Array
           attributes[:custom_field].each { |custom_field| custom_fields << CustomField.new(custom_field) }
         end
+        
+        @custom_fields_assoc = Hash.new
+        custom_fields.each { |custom_field| @custom_fields_assoc[custom_field.internal_id.to_sym] = custom_field }
       end
 
       def custom_fields
         @custom_fields ||= []
       end
+      
+      def method_missing(sym, *args, &block)
+        return @custom_fields_assoc[sym] if @custom_fields_assoc.include? sym
+        super(sym, *args, &block)
+      end
+
+      def respond_to?(sym, include_private = false)
+        return true if @custom_fields_assoc.include? sym
+        super
+      end
 
       def to_record
         custom_fields.map { |custom_field|

data/lib/netsuite/records/department.rb

@@ -0,0 +1,25 @@
+module NetSuite
+  module Records
+    class Department
+      include Support::Fields
+      include Support::RecordRefs
+      include Support::Actions
+
+      actions :get, :add, :delete
+
+      fields :name, :is_inactive
+      
+      record_refs :parent
+
+      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
+
+    end
+  end
+end

data/lib/netsuite/records/inventory_item.rb

@@ -40,6 +40,9 @@ module NetSuite
         :stock_unit, :store_display_image, :store_display_thumbnail, :store_item_template, :supply_lot_sizing_method,
         :supply_replenishment_method, :supply_type, :tax_schedule, :units_type, :vendor
 
+      field :pricing_matrix, PricingMatrix
+      field :custom_field_list, CustomFieldList
+
       attr_reader :internal_id
       attr_accessor :external_id
 

data/lib/netsuite/records/invoice.rb

@@ -11,7 +11,7 @@ module NetSuite
 
       fields :alt_handling_cost, :alt_shipping_cost, :balance, :bill_address,
         :billing_schedule, :contrib_pct, :created_date, :created_from, :currency_name, :custom_field_list,
-        :deferred_revenue, :department, :discount_amount, :discount_date, :discount_item, :discount_rate,
+        :deferred_revenue, :discount_amount, :discount_date, :discount_item, :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,
@@ -25,7 +25,7 @@ module NetSuite
         :sales_group, :sales_rep, :sales_team_list, :ship_address, :ship_date, :ship_group_list,
         :ship_method, :shipping_cost, :shipping_tax_1_rate, :shipping_tax_2_rate, :shipping_tax_code, :source, :start_date,
         :status, :subsidiary, :sync_partner_teams, :sync_sales_teams, :tax_2_total, :tax_item, :tax_rate,
-        :tax_total, :terms, :time_disc_amount, :time_disc_print, :time_disc_rate, :time_disc_tax_1_amt, :time_disc_taxable,
+        :tax_total, :time_disc_amount, :time_disc_print, :time_disc_rate, :time_disc_tax_1_amt, :time_disc_taxable,
         :time_discount, :time_list, :time_tax_code, :time_tax_rate_1, :time_tax_rate_2, :to_be_emailed, :to_be_faxed,
         :to_be_printed, :total_cost_estimate, :tracking_numbers, :tran_date, :tran_id, :tran_is_vsoe_bundle,
         :transaction_bill_address, :transaction_ship_address, :vat_reg_num, :vsoe_auto_calc
@@ -37,7 +37,7 @@ module NetSuite
 
       read_only_fields :sub_total, :discount_total, :total, :recognized_revenue, :amount_remaining, :amount_paid
 
-      record_refs :account, :bill_address_list, :custom_form, :entity, :klass, :posting_period, :ship_address_list
+      record_refs :account, :bill_address_list, :custom_form, :department, :entity, :klass, :posting_period, :ship_address_list, :terms
 
       attr_reader   :internal_id
       attr_accessor :external_id

data/lib/netsuite/records/non_inventory_sale_item.rb

@@ -10,7 +10,7 @@ module NetSuite
       actions :get, :add, :delete
 
       fields :available_to_partners, :cost_estimate, :cost_estimate_type, :cost_estimate_units, :country_of_manufacture,
-        :created_date, :custom_field_list, :display_name, :dont_show_price, :enforce_min_qty_internally, :exclude_from_sitemap,
+        :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,
@@ -30,6 +30,9 @@ module NetSuite
         :quantity_pricing_schedule, :rev_rec_schedule, :sale_unit, :sales_tax_code, :ship_package, :store_display_image,
         :store_display_thumbnail, :store_item_template, :subsidiary_list, :tax_schedule, :units_type
 
+      field :pricing_matrix, PricingMatrix
+      field :custom_field_list, CustomFieldList
+      
       attr_reader   :internal_id
       attr_accessor :external_id
 

data/lib/netsuite/records/pricing_matrix.rb

@@ -0,0 +1,21 @@
+module NetSuite
+  module Records
+    class PricingMatrix
+      include Namespaces::PlatformCore
+
+      def initialize(attributes = {})
+        attributes[:pricing].each do |pricing|
+          prices << RecordRef.new(pricing)
+        end if attributes[:pricing]
+      end
+
+      def prices
+        @prices ||= []
+      end
+
+      def to_record
+        { "#{record_namespace}:item" => prices.map(&:to_record) }
+      end
+    end
+  end
+end

data/lib/netsuite/records/term.rb

@@ -0,0 +1,25 @@
+module NetSuite
+  module Records
+    class Term
+      include Support::Fields
+      include Support::RecordRefs
+      include Support::Actions
+
+      actions :get, :add, :delete
+
+      fields :due_next_month_if_within_days, :name, :date_driven, :days_until_expiry, :days_until_next_due,
+        :day_discount_expires, :day_of_month_net_due, :discount_percent, :discount_percent_date_driven, :is_inactive,
+        :preferred
+      
+      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
+
+    end
+  end
+end

data/lib/netsuite/support/records.rb

@@ -6,11 +6,9 @@ module NetSuite
 
       def to_record
         attributes.reject { |k,v| self.class.read_only_fields.include?(k) }.inject({}) do |hash, (k,v)|
-          kname = if k == :klass
-                    "#{record_namespace}:class"
-                  else
-                    "#{record_namespace}:#{k.to_s.lower_camelcase}"
-                  end
+          kname = "#{record_namespace}:"
+          kname += k == :klass ? 'class' : k.to_s.lower_camelcase
+
           to_attributes!(hash, kname, v)
           if Array === v
             v = v.map { |i| i.respond_to?(:to_record) ? i.to_record : i }

data/lib/netsuite/version.rb

@@ -1,3 +1,3 @@
 module Netsuite
-  VERSION = '0.0.47'
+  VERSION = '0.0.48'
 end

data/netsuite.gemspec

@@ -1,10 +1,9 @@
 # -*- encoding: utf-8 -*-
-$:.unshift File.expand_path("../lib", __FILE__)
-require 'netsuite/version'
+require File.expand_path('../lib/netsuite/version', __FILE__)
 
 Gem::Specification.new do |gem|
-  gem.authors       = ['Ryan Moran']
-  gem.email         = ['ryan.moran@gmail.com']
+  gem.authors       = ['Ryan Moran', 'Michael Bianco']
+  gem.email         = ['ryan.moran@gmail.com', 'info@cliffsidedev.com']
   gem.description   = %q{NetSuite SuiteTalk API Wrapper}
   gem.summary       = %q{NetSuite SuiteTalk API Wrapper}
   gem.homepage      = 'https://github.com/RevolutionPrep/netsuite'
@@ -16,10 +15,10 @@ Gem::Specification.new do |gem|
   gem.require_paths = ['lib']
   gem.version       = Netsuite::VERSION
 
-  gem.add_dependency 'savon', '0.9.9'
-  gem.add_dependency 'nokogiri'
+  gem.add_dependency 'savon', '~> 1.2.0'
+  gem.add_dependency 'nokogiri', '~> 1.5.0'
 
-  gem.add_development_dependency 'rspec',               '2.8.0'
-  gem.add_development_dependency 'autotest-standalone', '4.5.9'
-  gem.add_development_dependency 'savon_spec',          '0.1.6'
+  gem.add_development_dependency 'rspec',                 '~> 2.10'
+  gem.add_development_dependency 'savon_spec',            '~> 1.3.0'
+  gem.add_development_dependency 'autotest-standalone',   '~> 4.5'
 end

data/spec/netsuite/actions/add_spec.rb

@@ -14,7 +14,7 @@ describe NetSuite::Actions::Add do
           'listRel:companyName' => 'Shutter Fly, Inc.'
         },
         :attributes! => {
-          'platformMsgs:baseRef' => {
+          'platformMsgs:record' => {
             'xsi:type' => 'listRel:Customer'
           }
         }
@@ -40,12 +40,11 @@ describe NetSuite::Actions::Add do
     before do
       savon.expects(:add).with({
         'platformMsgs:record' => {
-          'listRel:source' => 'Google',
-          'listRel:total'  => 100.0
+          'tranSales:source' => 'Google'
         },
         :attributes! => {
-          'platformMsgs:baseRef' => {
-            'xsi:type' => 'listRel:Invoice'
+          'platformMsgs:record' => {
+            'xsi:type' => 'tranSales:Invoice'
           }
         }
       }).returns(:add_invoice)

data/spec/netsuite/actions/get_spec.rb

@@ -8,8 +8,8 @@ describe NetSuite::Actions::Get do
         'platformMsgs:baseRef' => {},
         :attributes! => {
           'platformMsgs:baseRef' => {
-            :externalId => 1,
-            :type       => 'customer',
+            'externalId' => 1,
+            'type'       => 'customer',
             'xsi:type'  => 'platformCore:RecordRef'
           }
         }
@@ -32,8 +32,8 @@ describe NetSuite::Actions::Get do
         'platformMsgs:baseRef' => {},
         :attributes! => {
           'platformMsgs:baseRef' => {
-            :externalId => 10,
-            :type       => 'invoice',
+            'externalId' => 1,
+            'type'       => 'invoice',
             'xsi:type'  => 'platformCore:RecordRef'
           }
         }

data/spec/netsuite/actions/initialize_spec.rb

@@ -6,13 +6,11 @@ describe NetSuite::Actions::Initialize do
   before do
     savon.expects(:initialize).with({
       'platformMsgs:initializeRecord' => {
-        'platformCore:type' => 'invoice',
-        'platformCore:reference' => {
-          'platformCore:name' => 'Ryan Moran'
-        },
+        'platformCore:type' => 'customer',
+        'platformCore:reference' => {},
         :attributes! => {
           'platformCore:reference' => {
-            'internalId' => '1',
+            'internalId' => 1,
             :type        => 'customer'
           }
         }

data/spec/netsuite/actions/update_spec.rb

@@ -13,7 +13,7 @@ describe NetSuite::Actions::Update do
           'listRel:companyName' => 'Shutter Fly, Inc.'
         },
         :attributes! => {
-          'platformMsgs:baseRef' => {
+          'platformMsgs:record' => {
             'xsi:type' => 'listRel:Customer'
           }
         }
@@ -38,12 +38,11 @@ describe NetSuite::Actions::Update do
     before do
       savon.expects(:update).with({
         'platformMsgs:record' => {
-          'listRel:source' => 'Google',
-          'listRel:total'  => 100.0
+          'tranSales:source' => 'Google',
         },
         :attributes! => {
-          'platformMsgs:baseRef' => {
-            'xsi:type' => 'listRel:Invoice'
+          'platformMsgs:record' => {
+            'xsi:type' => 'tranSales:Invoice'
           }
         }
       }).returns(:update_invoice)

data/spec/netsuite/records/custom_record_spec.rb

@@ -28,12 +28,14 @@ describe NetSuite::Records::CustomRecord do
     it 'can be set from attributes' do
       attributes = {
         :custom_field => {
-          :amount => 10
+          :amount => 10,
+          :internal_id => 'custfield_amount'
         }
       }
       record.custom_field_list = attributes
       record.custom_field_list.should be_kind_of(NetSuite::Records::CustomFieldList)
       record.custom_field_list.custom_fields.length.should eql(1)
+      record.custom_field_list.custfield_amount.attributes[:amount].should eq(10)
     end
 
     it 'can be set from a CustomFieldList object' do

data/spec/netsuite/records/customer_payment_spec.rb

@@ -28,7 +28,8 @@ describe NetSuite::Records::CustomerPayment do
     it 'can be set from attributes' do
       attributes = {
         :custom_field => {
-          :amount => 10
+          :amount => 10,
+          :internal_id => 'custfield_amount'
         }
       }
       payment.custom_field_list = attributes

data/spec/netsuite/records/customer_refund_spec.rb

@@ -28,7 +28,8 @@ describe NetSuite::Records::CustomerRefund do
     it 'can be set from attributes' do
       attributes = {
         :custom_field => {
-          :amount => 10
+          :amount => 10,
+          :internal_id => 'custfield_amount'
         }
       }
       refund.custom_field_list = attributes

data/spec/netsuite/records/customer_spec.rb

@@ -67,7 +67,8 @@ describe NetSuite::Records::Customer do
     it 'can be set from attributes' do
       attributes = {
         :custom_field => {
-          :value => 10
+          :value => 10,
+          :internal_id => 'custfield_something'
         }
       }
       customer.custom_field_list = attributes

data/spec/netsuite/records/department_spec.rb

@@ -0,0 +1,103 @@
+require 'spec_helper'
+
+describe NetSuite::Records::Department do
+  let(:department) { NetSuite::Records::Department.new }
+
+  it 'has all the right fields' do
+    [
+      :name, :is_inactive
+    ].each do |field|
+      department.should have_field(field)
+    end
+  end
+
+  it 'has all the right record refs' do
+    [
+      :parent
+    ].each do |record_ref|
+      department.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 => { :name => 'Department 1' }) }
+
+      it 'returns a Department instance populated with the data from the response object' do
+        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Department, :external_id => 1).and_return(response)
+        department = NetSuite::Records::Department.get(:external_id => 1)
+        department.should be_kind_of(NetSuite::Records::Department)
+        department.name.should eql('Department 1')
+      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::Department, :external_id => 1).and_return(response)
+        lambda {
+          NetSuite::Records::Department.get(:external_id => 1)
+        }.should raise_error(NetSuite::RecordNotFound,
+          /NetSuite::Records::Department with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+  describe '#add' do
+    let(:test_data) { { :acct_name => 'Test Department', :description => 'An example Department' } }
+
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        department = NetSuite::Records::Department.new(test_data)
+        NetSuite::Actions::Add.should_receive(:call).
+            with(department).
+            and_return(response)
+        department.add.should be_true
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        department = NetSuite::Records::Department.new(test_data)
+        NetSuite::Actions::Add.should_receive(:call).
+            with(department).
+            and_return(response)
+        department.add.should be_false
+      end
+    end
+  end
+
+  describe '#delete' do
+    let(:test_data) { { :internal_id => '1' } }
+
+    context 'when the response is successful' do
+      let(:response)  { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        department = NetSuite::Records::Department.new(test_data)
+        NetSuite::Actions::Delete.should_receive(:call).
+            with(department).
+            and_return(response)
+        department.delete.should be_true
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        department = NetSuite::Records::Department.new(test_data)
+        NetSuite::Actions::Delete.should_receive(:call).
+            with(department).
+            and_return(response)
+        department.delete.should be_false
+      end
+    end
+  end
+
+end

data/spec/netsuite/records/invoice_item_spec.rb

@@ -37,12 +37,14 @@ describe NetSuite::Records::InvoiceItem do
     it 'can be set from attributes' do
       attributes = {
         :custom_field => {
-          :value => 10
+          :value => 10,
+          :internal_id => 'custfield_value'
         }
       }
       item.custom_field_list = attributes
       item.custom_field_list.should be_kind_of(NetSuite::Records::CustomFieldList)
       item.custom_field_list.custom_fields.length.should eql(1)
+      item.custom_field_list.custfield_value.attributes[:value].should eq(10)
     end
 
     it 'can be set from a CustomFieldList object' do

data/spec/netsuite/records/invoice_spec.rb

@@ -9,7 +9,7 @@ describe NetSuite::Records::Invoice do
     [
       :alt_handling_cost, :alt_shipping_cost, :amount_paid, :amount_remaining, :balance, :bill_address,
       :billing_schedule, :contrib_pct, :created_date, :created_from, :currency_name,
-      :deferred_revenue, :department, :discount_amount, :discount_date, :discount_item, :discount_rate,
+      :deferred_revenue, :discount_amount, :discount_date, :discount_item, :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,
@@ -23,7 +23,7 @@ describe NetSuite::Records::Invoice do
       :sales_group, :sales_rep, :sales_team_list, :ship_address, :ship_date, :ship_group_list,
       :ship_method, :shipping_cost, :shipping_tax_1_rate, :shipping_tax_2_rate, :shipping_tax_code, :source, :start_date,
       :status, :subsidiary, :sync_partner_teams, :sync_sales_teams, :tax_2_total, :tax_item, :tax_rate,
-      :tax_total, :terms, :time_disc_amount, :time_disc_print, :time_disc_rate, :time_disc_tax_1_amt, :time_disc_taxable,
+      :tax_total, :time_disc_amount, :time_disc_print, :time_disc_rate, :time_disc_tax_1_amt, :time_disc_taxable,
       :time_discount, :time_list, :time_tax_code, :time_tax_rate_1, :time_tax_rate_2, :to_be_emailed, :to_be_faxed,
       :to_be_printed, :total_cost_estimate, :tracking_numbers, :tran_date, :tran_id, :tran_is_vsoe_bundle,
       :vat_reg_num, :vsoe_auto_calc
@@ -42,7 +42,7 @@ describe NetSuite::Records::Invoice do
 
   it 'has the right record_refs' do
     [
-      :account, :bill_address_list, :custom_form, :entity, :klass, :posting_period, :ship_address_list
+      :account, :bill_address_list, :custom_form, :department, :entity, :klass, :posting_period, :ship_address_list, :terms
     ].each do |record_ref|
       invoice.should have_record_ref(record_ref)
     end
@@ -52,7 +52,8 @@ describe NetSuite::Records::Invoice do
     it 'can be set from attributes' do
       attributes = {
         :custom_field => {
-          :amount => 10
+          :amount => 10,
+          :internal_id => 'custfield_amount'
         }
       }
       invoice.custom_field_list = attributes

data/spec/netsuite/records/journal_entry_line_spec.rb

@@ -23,7 +23,8 @@ describe NetSuite::Records::JournalEntryLine do
     it 'can be set from attributes' do
       attributes = {
         :custom_field => {
-          :amount => 10
+          :amount => 10,
+          :internal_id => 'custfield_amount'
         }
       }
       line.custom_field_list = attributes

data/spec/netsuite/records/journal_entry_spec.rb

@@ -25,12 +25,14 @@ describe NetSuite::Records::JournalEntry do
     it 'can be set from attributes' do
       attributes = {
         :custom_field => {
-          :amount => 10
+          :amount => 10,
+          :internal_id => 'custfield_amount'
         }
       }
       entry.custom_field_list = attributes
       entry.custom_field_list.should be_kind_of(NetSuite::Records::CustomFieldList)
       entry.custom_field_list.custom_fields.length.should eql(1)
+      entry.custom_field_list.custfield_amount.attributes[:amount].should eq(10)
     end
 
     it 'can be set from a CustomFieldList object' do

data/spec/netsuite/records/non_inventory_sale_item_spec.rb

@@ -6,14 +6,14 @@ 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,
-      :custom_field_list, :display_name, :dont_show_price, :enforce_min_qty_internally, :exclude_from_sitemap,
+      :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, :pricing_matrix, :producer, :product_feed_list, :rate, :related_items_description, :sales_description,
+      :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,
@@ -22,6 +22,9 @@ describe NetSuite::Records::NonInventorySaleItem do
     ].each do |field|
       item.should have_field(field)
     end
+
+    item.pricing_matrix.class.should == NetSuite::Records::PricingMatrix
+    item.custom_field_list.class.should == NetSuite::Records::CustomFieldList
   end
 
   it 'has the right record_refs' do

data/spec/netsuite/records/term_spec.rb

@@ -0,0 +1,96 @@
+require 'spec_helper'
+
+describe NetSuite::Records::Term do
+  let(:term) { NetSuite::Records::Term.new }
+
+  it 'has all the right fields' do
+    [
+      :name, :date_driven, :days_until_next_due, :discount_percent, :days_until_expiry, :day_of_month_net_due,
+      :due_next_month_if_within_days, :discount_percent_date_driven, :day_discount_expires, :preferred, :is_inactive
+    ].each do |field|
+      term.should have_field(field)
+    end
+  end
+
+  describe '.get' do
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :name => 'Term 1' }) }
+
+      it 'returns a Term instance populated with the data from the response object' do
+        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Term, :external_id => 1).and_return(response)
+        term = NetSuite::Records::Term.get(:external_id => 1)
+        term.should be_kind_of(NetSuite::Records::Term)
+        term.name.should eql('Term 1')
+      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::Term, :external_id => 1).and_return(response)
+        lambda {
+          NetSuite::Records::Term.get(:external_id => 1)
+        }.should raise_error(NetSuite::RecordNotFound,
+          /NetSuite::Records::Term with OPTIONS=(.*) could not be found/)
+      end
+    end
+  end
+
+  describe '#add' do
+    let(:test_data) { { :name => 'Test Term' } }
+
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        term = NetSuite::Records::Term.new(test_data)
+        NetSuite::Actions::Add.should_receive(:call).
+            with(term).
+            and_return(response)
+        term.add.should be_true
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        term = NetSuite::Records::Term.new(test_data)
+        NetSuite::Actions::Add.should_receive(:call).
+            with(term).
+            and_return(response)
+        term.add.should be_false
+      end
+    end
+  end
+
+  describe '#delete' do
+    let(:test_data) { { :internal_id => '1' } }
+
+    context 'when the response is successful' do
+      let(:response)  { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        term = NetSuite::Records::Term.new(test_data)
+        NetSuite::Actions::Delete.should_receive(:call).
+            with(term).
+            and_return(response)
+        term.delete.should be_true
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        term = NetSuite::Records::Term.new(test_data)
+        NetSuite::Actions::Delete.should_receive(:call).
+            with(term).
+            and_return(response)
+        term.delete.should be_false
+      end
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -1,8 +1,8 @@
-$LOAD_PATH.unshift(File.dirname(__FILE__))
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
-require 'netsuite'
+$LOAD_PATH.unshift(File.expand_path(File.join(File.dirname(__FILE__), '..') ))
+
 require 'rspec'
 require 'rspec/autorun'
+require 'netsuite'
 require 'savon_spec'
 
 # Requires supporting ruby files with custom matchers and macros, etc,

metadata

@@ -1,115 +1,108 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: netsuite
-version: !ruby/object:Gem::Version 
-  hash: 65
+version: !ruby/object:Gem::Version
+  version: 0.0.48
   prerelease: 
-  segments: 
-  - 0
-  - 0
-  - 47
-  version: 0.0.47
 platform: ruby
-authors: 
+authors:
 - Ryan Moran
+- Michael Bianco
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2012-03-16 00:00:00 -07:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2012-10-05 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: savon
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - "="
-      - !ruby/object:Gem::Version 
-        hash: 41
-        segments: 
-        - 0
-        - 9
-        - 9
-        version: 0.9.9
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.2.0
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: nokogiri
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.2.0
+- !ruby/object:Gem::Dependency
+  name: nokogiri
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.5.0
   type: :runtime
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: rspec
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.5.0
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - "="
-      - !ruby/object:Gem::Version 
-        hash: 47
-        segments: 
-        - 2
-        - 8
-        - 0
-        version: 2.8.0
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.10'
   type: :development
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: autotest-standalone
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - "="
-      - !ruby/object:Gem::Version 
-        hash: 57
-        segments: 
-        - 4
-        - 5
-        - 9
-        version: 4.5.9
-  type: :development
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.10'
+- !ruby/object:Gem::Dependency
   name: savon_spec
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.3.0
+  type: :development
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - "="
-      - !ruby/object:Gem::Version 
-        hash: 23
-        segments: 
-        - 0
-        - 1
-        - 6
-        version: 0.1.6
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.3.0
+- !ruby/object:Gem::Dependency
+  name: autotest-standalone
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '4.5'
   type: :development
-  version_requirements: *id005
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '4.5'
 description: NetSuite SuiteTalk API Wrapper
-email: 
+email:
 - ryan.moran@gmail.com
+- info@cliffsidedev.com
 executables: []
-
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .autotest
 - .gitignore
 - .rspec
-- .rvmrc
 - Gemfile
 - LICENSE
 - README.md
@@ -121,6 +114,7 @@ files:
 - lib/netsuite/actions/initialize.rb
 - lib/netsuite/actions/update.rb
 - lib/netsuite/configuration.rb
+- lib/netsuite/core_ext/string/lower_camelcase.rb
 - lib/netsuite/errors.rb
 - lib/netsuite/namespaces/list_acct.rb
 - lib/netsuite/namespaces/list_rel.rb
@@ -132,6 +126,7 @@ files:
 - lib/netsuite/namespaces/tran_sales.rb
 - lib/netsuite/records/account.rb
 - lib/netsuite/records/accounting_period.rb
+- lib/netsuite/records/assembly_item.rb
 - lib/netsuite/records/bill_address.rb
 - lib/netsuite/records/classification.rb
 - lib/netsuite/records/credit_memo.rb
@@ -153,6 +148,7 @@ files:
 - lib/netsuite/records/customer_refund_apply_list.rb
 - lib/netsuite/records/customer_refund_deposit.rb
 - lib/netsuite/records/customer_refund_deposit_list.rb
+- lib/netsuite/records/department.rb
 - lib/netsuite/records/duration.rb
 - lib/netsuite/records/inventory_item.rb
 - lib/netsuite/records/invoice.rb
@@ -165,9 +161,11 @@ files:
 - lib/netsuite/records/location.rb
 - lib/netsuite/records/non_inventory_sale_item.rb
 - lib/netsuite/records/payment_method.rb
+- lib/netsuite/records/pricing_matrix.rb
 - lib/netsuite/records/record_ref.rb
 - lib/netsuite/records/rev_rec_template.rb
 - lib/netsuite/records/ship_address.rb
+- lib/netsuite/records/term.rb
 - lib/netsuite/response.rb
 - lib/netsuite/support/actions.rb
 - lib/netsuite/support/attributes.rb
@@ -207,6 +205,7 @@ files:
 - spec/netsuite/records/customer_refund_deposit_spec.rb
 - spec/netsuite/records/customer_refund_spec.rb
 - spec/netsuite/records/customer_spec.rb
+- spec/netsuite/records/department_spec.rb
 - spec/netsuite/records/duration_spec.rb
 - spec/netsuite/records/inventory_item_spec.rb
 - spec/netsuite/records/invoice_item_list_spec.rb
@@ -222,6 +221,7 @@ files:
 - spec/netsuite/records/record_ref_spec.rb
 - spec/netsuite/records/rev_rec_template_spec.rb
 - spec/netsuite/records/ship_address_spec.rb
+- spec/netsuite/records/term_spec.rb
 - spec/netsuite/response_spec.rb
 - spec/netsuite/support/actions_spec.rb
 - spec/netsuite/support/attributes_spec.rb
@@ -246,41 +246,31 @@ files:
 - spec/support/savon.rb
 - wsdl/2011_2.wsdl
 - wsdl/2012_1.wsdl
-has_rdoc: true
 homepage: https://github.com/RevolutionPrep/netsuite
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.4.2
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: NetSuite SuiteTalk API Wrapper
-test_files: 
+test_files:
 - spec/netsuite/actions/add_spec.rb
 - spec/netsuite/actions/delete_spec.rb
 - spec/netsuite/actions/get_spec.rb
@@ -310,6 +300,7 @@ test_files:
 - spec/netsuite/records/customer_refund_deposit_spec.rb
 - spec/netsuite/records/customer_refund_spec.rb
 - spec/netsuite/records/customer_spec.rb
+- spec/netsuite/records/department_spec.rb
 - spec/netsuite/records/duration_spec.rb
 - spec/netsuite/records/inventory_item_spec.rb
 - spec/netsuite/records/invoice_item_list_spec.rb
@@ -325,6 +316,7 @@ test_files:
 - spec/netsuite/records/record_ref_spec.rb
 - spec/netsuite/records/rev_rec_template_spec.rb
 - spec/netsuite/records/ship_address_spec.rb
+- spec/netsuite/records/term_spec.rb
 - spec/netsuite/response_spec.rb
 - spec/netsuite/support/actions_spec.rb
 - spec/netsuite/support/attributes_spec.rb

data/.rvmrc

@@ -1 +0,0 @@
-rvm use ree-1.8.7-2010.02@netsuite --create