activenetsuite 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: be30c904151989986a967d8a1a3a7c7d2ab5b578
-  data.tar.gz: c86034831bf144333cd794c566edd9b471eec078
+  metadata.gz: a3ae2182bb4b0ed5f80d1b87cfba3619ea67a377
+  data.tar.gz: 7bf7b4fb3bc103ffa0dc5232d65c1171e1083fd4
 SHA512:
-  metadata.gz: 3f50d7ad4fdcc9e4dcb77fd2bffc11b90db0025836156e92228a08f77ef49b3747a401bef3c3fa497f8bfa648fee46d94a784625aa63af53097143470c6ee1ea
-  data.tar.gz: ca254c174566c6947a992c51744f0ce2b95b29dd45c6897f9e5db679294a4087e7adb644da9262e08f5473c25fa700c7eb22639506dea28c9c3c4b66c00a772a
+  metadata.gz: f7795d2cfb911860ef8c60b9b402f64312caac098697792eaf299374d8162387a6e6584f651981799bd16a47ed56a11b2de895d1e3c1fa0cad03a6bdf0347d4b
+  data.tar.gz: b510bf5324e604bb2d0f3276ffb4145fe1c5246001cafd299b46317283503a67c4ed8b54d8e5922e8625a5501b719fb226680c94e5edc2a35b71840246dd4c1d

data/lib/activenetsuite.rb

@@ -12,6 +12,7 @@ module NetSuite
 
   Dir.chdir(File.dirname(__FILE__)) do
     Dir[*%w(
+      activenetsuite/types/*.rb
       activenetsuite/helpers/*.rb
       activenetsuite/accounting/*.rb
       activenetsuite/core/*.rb

data/lib/activenetsuite/core/record.rb

@@ -3,17 +3,9 @@ module ActiveNetsuite
 class Record
   include MethodInflector
 
-  def to_s
-    instance_variables.
-      map do |var|
-        [var, instance_variable_get(var)]
-      end.
-      select do |var, val|
-        val
-      end
-  end
-
   class << self
+    extend Forwardable
+
     def client
       @@client ||= nil
     end
@@ -48,30 +40,17 @@ class Record
       "::#{to_s}SearchBasic".constantize
     end
 
-    def search_next(search_result, page_index)
-      client.search_next(search, page_index)
-    end
+    def_delegators :search, :where, :find_by, :inactive, :active
 
-    def where(*args)
-      search.where(*args)
-    end
-
-    def find_by(*args)
-      search.find_by(*args)
+    def list(objects)
+      objects = [objects] unless objects.respond_to?(:map)
+      client.get_list(refs(objects))
     end
 
     def all
       search.response
     end
 
-    def inactive
-      search.inactive
-    end
-
-    def active
-      search.active
-    end
-
     def delete(objects)
       objects = [objects] unless objects.respond_to?(:map)
       client.delete_list(refs(objects))
@@ -97,8 +76,13 @@ class Record
       client.get_deleted(get_deleted_filter)
     end
 
-    # Convert Hash or Id to RecordRef
-    # @param arg Hash or String
+    # Convert arg to RecordRef
+    # @overload ref(internal_id)
+    #   @param [String] internal_id
+    # @overload ref(hash)
+    #   @param [Hash] hash with internal_id or external_id
+    #   @option opts [String] :internal_id
+    #   @option opts [String] :external_id
     # @return RecordRef
     # @example
     #   ref = Record.arg(12)
@@ -156,10 +140,38 @@ class Record
     res
   end
 
+  def add!
+    raise_on_fail(:add)
+  end
+
   def update
     client.update(self)
   end
 
+  def update!
+    raise_on_fail(:update)
+  end
+
+  def save
+    if new?
+      add
+    else
+      update
+    end
+  end
+
+  def save!
+    if new?
+      add!
+    else
+      update!
+    end
+  end
+
+  def new?
+    !internal_id
+  end
+
   def delete
     res = client.delete(ref)
     if res.success?
@@ -168,6 +180,10 @@ class Record
     res
   end
 
+  def delete!
+    raise_on_error(:delete)
+  end
+
   def client
     self.class.client
   end
@@ -175,9 +191,9 @@ class Record
   def load
     return self if loaded?
 
-    record = find_by_id
-    record.getters.each do |getter|
-      send :"#{getter}=", record.send(getter)
+    result = find_by_id
+    result.getters.each do |getter|
+      send :"#{getter}=", result.send(getter)
     end
     @loaded = true
     self
@@ -188,7 +204,11 @@ class Record
   end
 
   def active?
-    !inactive?
+    !is_inactive
+  end
+
+  def inactive?
+    !active?
   end
 
   def active=(value)
@@ -199,16 +219,6 @@ class Record
     self.is_inactive = !!value
   end
 
-  def activate
-    self.active = true
-    self
-  end
-
-  def inactivate
-    self.active = false
-    self
-  end
-
   def ref
     self.class.ref(internal_id)
   end
@@ -223,6 +233,12 @@ class Record
 
   private
 
+  def raise_on_fail(method)
+    res = send(method)
+    return res if res.success?
+    raise NetsuiteError, res
+  end
+
   def raise_not_found_error
     self.class.raise_not_found_error(ref)
   end

data/lib/activenetsuite/core/search_record.rb

@@ -7,13 +7,7 @@ class SearchRecord
   attr_accessor :record_class
 
   def_delegators :response, :page_index, :page_size, :search_id, :total_pages,
-    :total_records, :has_more?, :next
-
-  def each
-    response.each do |result|
-      yield result if block_given?
-    end
-  end
+    :total_records, :more?, :next, :each, :to_a
 
   def where(*args)
     dup.send(:add, *args)
@@ -23,16 +17,25 @@ class SearchRecord
     where(*args).first
   end
 
-  def to_a
-    response.to_a
-  end
-
   def inactive
-    dup.send(:add, isInactive: true)
+    where(isInactive: true)
   end
 
   def active
-    dup.send(:add, isInactive: false)
+    where(isInactive: false)
+  end
+
+  def deleted(op, val)
+    search_value = SearchDateField.new
+    search_value.xmlattr_operator = SearchDateFieldOperator.make(op)
+    search_value.predefinedSearchValue = SearchDate.make(val)
+
+    search_type = SearchEnumMultiSelectField.new
+    search_type.xmlattr_operator = SearchEnumMultiSelectFieldOperator::AnyOf
+    search_type.searchValue = type
+
+    get_deleted_filter = GetDeletedFilter.new(search_value, search_type)
+    client.get_deleted(get_deleted_filter)
   end
 
   def response
@@ -50,7 +53,7 @@ class SearchRecord
   def add(*args)
     if args.first.is_a?(Hash)
       @method, @value = args.first.flatten
-      @op = :is
+      @op = default_operator
     end
     if args.size == 3
       @method, @op, @value = args
@@ -66,6 +69,19 @@ class SearchRecord
     self
   end
 
+  def default_operator
+    case value
+    when String
+      :is
+    when Fixnum
+      :equal_to
+    when Date, DateTime
+      :on
+    else
+      raise ArgumentError, "Can't find search class for #{value.inspect}"
+    end
+  end
+
   def search_class
     case value
     when String
@@ -89,7 +105,6 @@ class SearchRecord
 
   def record_class
     @record_class
-    #"::#{self.class.to_s.gsub(/SearchBasic$/, '')}".constantize
   end
 end
 

data/lib/activenetsuite/core/search_result.rb

@@ -3,7 +3,7 @@ module ActiveNetsuite
 class SearchResult
   include MethodInflector
 
-  def has_more?
+  def more?
     page_index < total_pages
   end
 end

data/lib/activenetsuite/core/status.rb

@@ -1,20 +1,27 @@
 module ActiveNetsuite
 
 class Status
-  DUP_ITEM = 'DUP_ITEM'
-  DUP_RCRD = 'DUP_RCRD'
-  DUP_ENTITY = 'DUP_ENTITY'
-  DUP_VENDOR_NAME = 'DUP_VENDOR_NAME'
-  RCRD_TYPE_REQD = 'RCRD_TYPE_REQD'
-  USER_ERROR = 'USER_ERROR'
-  MAX_RCRDS_EXCEEDED = 'MAX_RCRDS_EXCEEDED'
-  INVALID_INTERNALID = 'INVALID_INTERNALID'
-  INVALID_FLD_VALUE = 'INVALID_FLD_VALUE'
+  # Make it so Status responds to error_code? style methods
+  # @param [Symbol] method name
+  # @return [Boolean]
+  # @example
+  #  status.code # => 'DUP_ITEM'
+  #  status.dup_item? # => true
+  #  status.cogs_error? # => false
+  def method_missing(method)
+    if method.to_s.end_with?('?')
+      code == method.to_s.chop.upcase
+    else
+      super
+    end
+  end
 
+  # True if status is success
   def success?
-    xmlattr_isSuccess
+    !!xmlattr_isSuccess
   end
 
+  # True if status is failure
   def failure?
     !success?
   end
@@ -27,15 +34,6 @@ class Status
     status_detail.message if status_detail
   end
 
-  constants.each do |constant|
-    name = (constant.to_s.underscore + '?').to_sym
-    value = const_get(constant)
-    next unless value.is_a?(String)
-    define_method name do
-      code == value
-    end
-  end
-
   def duplicate?
     dup_item? || dup_rcrd? || dup_entity? || dup_vendor_name?
   end

data/lib/activenetsuite/core/status_detail.rb

@@ -0,0 +1,6 @@
+module ActiveNetsuite
+
+class StatusDetail
+end
+
+end

data/lib/activenetsuite/helpers/client.rb

@@ -65,7 +65,7 @@ class Client
     @driver.get(GetRequest.new(ref))
   end
 
-  def get_all(refs)
+  def get_list(refs)
     @driver.getList(GetListRequest.new(refs))
   end
 

data/lib/activenetsuite/helpers/error.rb

@@ -1,7 +1,6 @@
 module ActiveNetsuite
 
-class NotFoundError < StandardError
-end
+class NetsuiteError < StandardError; end
+class NotFoundError < NetsuiteError; end
 
 end
-

data/lib/activenetsuite/helpers/search_response_methods.rb

@@ -7,7 +7,7 @@ module SearchResponseMethods
   def_delegators :records, :[]
 
   def_delegators :result, :page_index, :page_size, :search_id, :total_pages,
-    :total_records, :has_more?
+    :total_records, :more?
 
   def each
     records.each do |record|

data/lib/activenetsuite/messages/get_list_response.rb

@@ -0,0 +1,18 @@
+module ActiveNetsuite
+
+class GetListResponse
+  include Enumerable
+  extend Forwardable
+
+  def_delegators :records, :[]
+
+  alias_method :records, :readResponseList
+
+  def each
+    records.each do |record|
+      yield record if block_given?
+    end
+  end
+end
+
+end

data/lib/activenetsuite/types/search_date.rb

@@ -0,0 +1,9 @@
+module ActiveNetsuite
+
+class SearchDate < ::String
+  def self.make(operator)
+    (name + '::' + operator.to_s.camelize).constantize
+  end
+end
+
+end

data/lib/activenetsuite/types/search_date_field_operator.rb

@@ -0,0 +1,9 @@
+module ActiveNetsuite
+
+class SearchDateFieldOperator < ::String
+  def self.make(operator)
+    (name + '::' + operator.to_s.camelize).constantize
+  end
+end
+
+end

data/lib/activenetsuite/version.rb

@@ -1,3 +1,3 @@
 module ActiveNetsuite
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

data/spec/lib/activenetsuite/core/record_spec.rb

@@ -26,21 +26,22 @@ class ActiveNetsuite::Record
 
   attr_accessor :foo
   attr_accessor :bar
+  attr_accessor :is_inactive
 end
 
 class ActiveNetsuite::RecordSearchBasic; end
 
 describe Record do
-  let(:model) { Record.new args }
+  let(:record) { Record.new args }
   let(:args) { nil }
   let(:client) { double }
   before { Record.client = client }
 
   describe '#add' do
-    subject { model.add }
+    subject { record.add }
 
     before do
-      model.client.stub(:add) { result }
+      record.client.stub(:add) { result }
     end
     let(:result) { double success?: success, internal_id: :internal_id }
     let(:success) { true }
@@ -49,31 +50,56 @@ describe Record do
 
     context 'when success' do
       let(:success) { true }
-      it { expect { subject }.to change { model.internal_id }.to(:internal_id) }
+      it { expect { subject }.to change { record.internal_id }.to(:internal_id) }
     end
 
     context 'when not success' do
       let(:success) { false }
       it { expect { subject }.to_not \
-           change { model.internal_id }.to(:internal_id) }
+           change { record.internal_id }.to(:internal_id) }
     end
   end
 
+  describe '#add!' do
+    subject { record.add! }
+
+    before do
+      record.client.stub(:add) { result }
+    end
+    let(:result) { double success?: success, internal_id: :internal_id }
+    let(:success) { true }
+
+    it { should eq result }
+
+    context 'when success' do
+      let(:success) { true }
+      it { expect { subject }.to change { record.internal_id }.to(:internal_id) }
+    end
+
+    context 'when not success' do
+      let(:success) { false }
+      it { expect { subject rescue nil }.to_not \
+           change { record.internal_id }.to(:internal_id) }
+      it { expect { subject }.to raise_error NetsuiteError }
+    end
+
+  end
+
   describe '#update' do
-    subject { model.update }
+    subject { record.update }
 
-    before { model.client.stub(:update).with(model) { result } }
+    before { record.client.stub(:update).with(record) { result } }
     let(:result) { double }
 
     it { should be result }
   end
 
   describe '#delete' do
-    subject { model.delete }
+    subject { record.delete }
 
     before do
       client.stub(:delete) { result }
-      model.internal_id = 100
+      record.internal_id = 100
     end
     let(:result) { double success?: success }
     let(:success) { true }
@@ -81,34 +107,34 @@ describe Record do
 
     context 'when it is success' do
       let(:success) { true }
-      it { expect { subject }.to change { model.internal_id }.to(nil) }
+      it { expect { subject }.to change { record.internal_id }.to(nil) }
     end
 
     context 'when it is failure' do
       let(:success) { false }
-      it { expect { subject }.to_not change { model.internal_id }.from(100) }
+      it { expect { subject }.to_not change { record.internal_id }.from(100) }
     end
   end
 
   describe '#client' do
-    subject { model.client }
-    before { model.class.client = client }
+    subject { record.client }
+    before { record.class.client = client }
     let(:client) { double }
 
     it { should be client }
   end
 
   describe '#load' do
-    subject { model.load }
-    before { model.instance_variable_set(:@loaded, loaded) }
+    subject { record.load }
+    before { record.instance_variable_set(:@loaded, loaded) }
     let(:loaded) { double }
 
-    it { should be model }
+    it { should be record }
 
     context 'when already loaded' do
       let(:loaded) { true }
 
-      it { should be model }
+      it { should be record }
       it do
         client.should_not_receive(:get)
         subject
@@ -119,38 +145,38 @@ describe Record do
       let(:loaded) { false }
 
       before do
-        model.internal_id = 123
-        client.stub(:get) { double record: record }
+        record.internal_id = 123
+        client.stub(:get) { double record: result }
       end
 
-      let(:record) do
+      let(:result) do
         double getters: [:foo, :bar],
                foo: :foo,
                bar: :bar
       end
 
-      it { expect { subject }.to change { model.loaded? }.to(true) }
-      it { expect { subject }.to change { model.foo }.to(:foo) }
-      it { expect { subject }.to change { model.bar }.to(:bar) }
+      it { expect { subject }.to change { record.loaded? }.to(true) }
+      it { expect { subject }.to change { record.foo }.to(:foo) }
+      it { expect { subject }.to change { record.bar }.to(:bar) }
     end
   end
 
   describe '#loaded?' do
-    subject { model.loaded? }
-    before { model.instance_variable_set(:@loaded, true) }
+    subject { record.loaded? }
+    before { record.instance_variable_set(:@loaded, true) }
     it { should be_true }
   end
 
   describe '#ref' do
-    subject { model.ref }
+    subject { record.ref }
 
     it { should be_a RecordRef }
     its(:type) { should eq "record" }
-    its(:internal_id) { should eq model.internal_id }
+    its(:internal_id) { should eq record.internal_id }
   end
 
   describe '#setters' do
-    subject { model.setters }
+    subject { record.setters }
     it do
       should include :foo=, :bar=, :nullFieldList=,
                    :xmlattr_externalId=, :xmlattr_internalId=
@@ -158,12 +184,68 @@ describe Record do
   end
 
   describe '#getters' do
-    subject { model.getters }
+    subject { record.getters }
     it do
       should include :foo, :bar, :xmlattr_externalId, :xmlattr_internalId
     end
   end
 
+  describe '.list' do
+    subject { described_class.list(objects) }
+
+    let(:ids) { [10, 20] }
+
+    let(:recs) do
+      ids.map do |id|
+        rec = Record.new
+        rec.internal_id = id
+        rec
+      end
+    end
+
+    let(:refs) do
+      ids.map do |id|
+        ref = RecordRef.new
+        ref.internal_id = id
+        ref.type = 'record'
+        ref
+      end
+    end
+
+    context 'when internal ids given' do
+      let(:objects) { ids }
+
+      it do
+        described_class.client.should_receive(:get_list) do |args|
+          args == refs
+        end
+        subject
+      end
+    end
+
+    context 'when Records given' do
+      let(:objects) { recs }
+
+      it do
+        described_class.client.should_receive(:get_list) do |args|
+          args == refs
+        end
+        subject
+      end
+    end
+
+    context 'when Refs given' do
+      let(:objects) { refs }
+
+      it do
+        described_class.client.should_receive(:get_list) do |args|
+          args == refs
+        end
+        subject
+      end
+    end
+  end
+
   describe '.delete' do
     subject { described_class.delete(objects) }
 
@@ -295,4 +377,61 @@ describe Record do
     subject { Record.basic_search_class }
     it { should be RecordSearchBasic }
   end
+
+  describe '#active?' do
+    subject { record.active? }
+
+    context 'when is_inactive=true' do
+      before { record.is_inactive = true }
+      it { should be_false }
+    end
+
+    context 'when is_inactive=false' do
+      before { record.is_inactive = false }
+      it { should be_true }
+    end
+  end
+
+  describe '#save' do
+    subject { record.save }
+    let :result do
+      double success?: true, internal_id: 123
+    end
+    before do
+      client.stub(:add).with(record) { result }
+    end
+
+    context 'when new record' do
+      before { record.internal_id = nil }
+      specify do
+        client.should_receive(:add).with(record) { result }
+        subject
+      end
+
+      it { expect { subject }.to change { record.internal_id }.to(123) }
+    end
+
+    context 'when existing record' do
+      before { record.internal_id = 123 }
+      specify do
+        client.should_receive(:update).with(record) { result }
+        subject
+      end
+    end
+  end
+
+  describe '#new?' do
+    subject { record.new? }
+    before { record.internal_id = internal_id }
+
+    context 'when there is no internal_id' do
+      let(:internal_id) { nil }
+      it { should be_true }
+    end
+
+    context 'when there is internal_id' do
+      let(:internal_id) { 123 }
+      it { should be_false }
+    end
+  end
 end

data/spec/lib/activenetsuite/core/search_record_spec.rb

@@ -10,41 +10,89 @@ describe SearchRecord do
     search.record_class = Record
     search
   end
+  let(:client) { double search: response }
+  let(:response) { double }
 
   before do
     Record.stub(:client) { client }
     SearchRecord.stub(:inflected_method) { :foo }
   end
 
-  describe '#each' do
-    subject { search.each }
-    let(:client) { double search: response }
-    let(:response) { [:a, :b, :c] }
-    it do
-      search.response.should_receive(:each).and_yield(response)
-      subject
-    end
-  end
-
   describe '#where' do
     subject { search.where *args }
     let(:args) { double }
 
     context 'when foo: "bar" given' do
       let(:args) { [{foo: 'bar'}] }
-      let :where do
-        where = SearchRecord.new
-        where.foo = SearchStringField.new
-        where.foo.xmlattr_operator = SearchStringFieldOperator::Is
-        where.foo.searchValue = 'bar'
-        where
-      end
 
       it { should be_a SearchRecord }
+      it { should_not be search }
       its(:foo) { should be_a SearchStringField }
       its('foo.xmlattr_operator') { should eq SearchStringFieldOperator::Is }
       its('foo.searchValue') { 'bar' }
     end
+
+    context 'when foo: 10 given' do
+      let(:args) { [{foo: 10} ] }
+
+      it { should be_a SearchRecord }
+      it { should_not be search }
+      its(:foo) { should be_a SearchLongField }
+      its('foo.xmlattr_operator') { should eq SearchLongFieldOperator::EqualTo }
+      its('foo.searchValue') { 10 }
+    end
+
+    context 'when foo: Date.new(2000, 1, 2) given' do
+      let(:args) { [{foo: Date.new(2000, 1, 2)} ] }
+
+      it { should be_a SearchRecord }
+      it { should_not be search }
+      its(:foo) { should be_a SearchDateField }
+      its('foo.xmlattr_operator') { should eq SearchDateFieldOperator::On }
+      its('foo.searchValue') { Date.new(2000, 1, 2) }
+    end
+
+    context 'when :foo, :has_keywords, "key" given' do
+      let(:args) { [:foo, :has_keywords, 'key'] }
+
+      it { should be_a SearchRecord }
+      it { should_not be search }
+      its(:foo) { should be_a SearchStringField }
+      its('foo.xmlattr_operator') { should eq SearchStringFieldOperator::HasKeywords }
+      its('foo.searchValue') { 'key' }
+    end
+  end
+
+  describe '#find_by' do
+    subject { search.find_by *args }
+  end
+
+  describe 'response delegators' do
+    [:page_index, :page_size, :search_id, :total_pages,
+      :total_records, :more?, :next, :each].each do |method|
+
+      describe "#{method}" do
+        subject { search.send method }
+        let(:response) { double method => value }
+        let(:value) { double }
+        specify do
+          client.should_receive(:search).with(search) { response }
+          should eq value
+        end
+      end
+    end
+  end
+
+  describe '#find_by' do
+    subject { search.find_by *args }
+    before do
+      search.stub(:where).with(args) { response }
+    end
+    let(:args) { double }
+    let(:response) { double first: record }
+    let(:record) { double }
+
+    it { should eq record }
   end
 end
 

data/spec/lib/activenetsuite/core/search_result_spec.rb

@@ -3,8 +3,8 @@ require 'spec_helper'
 describe SearchResult do
   let(:result) { SearchResult.new }
 
-  describe '#has_more?' do
-    subject { result.has_more? }
+  describe '#more?' do
+    subject { result.more? }
     before do
       result.page_index = page_index
       result.total_pages = total_pages

data/spec/lib/activenetsuite/core/status_spec.rb

@@ -0,0 +1,95 @@
+require 'spec_helper'
+
+describe Status do
+  let(:status) { described_class.new }
+
+  describe '#success?' do
+    subject { status.success? }
+    before { status.stub(:xmlattr_isSuccess) { false } }
+    it { should be_false }
+  end
+
+  describe '#failure?' do
+    subject { status.failure? }
+    before { status.stub(:xmlattr_isSuccess) { false } }
+    it { should be_true }
+  end
+
+  describe '#code' do
+    subject { status.code }
+    before do
+      status.stub(:statusDetail) { status_detail }
+    end
+
+    context 'when there is statusDetail' do
+      let(:status_detail) { [double(code: :code)] }
+      it { should be :code }
+    end
+
+    context 'when there is no statusDetail' do
+      let(:status_detail) { nil }
+      it { should be_nil }
+    end
+  end
+
+  describe '#message' do
+    subject { status.message }
+    before do
+      status.stub(:statusDetail) { status_detail }
+    end
+
+    context 'when there is statusDetail' do
+      let(:status_detail) { [double(message: :message)] }
+      it { should be :message }
+    end
+
+    context 'when there is no statusDetail' do
+      let(:status_detail) { nil }
+      it { should be_nil }
+    end
+  end
+
+  describe '#duplicate?' do
+    subject { status.duplicate? }
+    before do
+      status.stub(:statusDetail) { [double(code: code)] }
+    end
+
+    context 'when code==DUP_ITEM for example' do
+      let(:code) { 'DUP_ITEM' }
+      it { should be_true }
+    end
+
+    context 'when code==COGS_ERROR for example' do
+      let(:code) { 'COGS_ERROR' }
+      it { should be_false }
+    end
+  end
+
+  describe '#missing_method' do
+    subject { status.send method }
+    before do
+      status.stub(:statusDetail) { [double(code: code)] }
+    end
+
+    context 'when method ends with ? and matches code' do
+      let(:method) { :dup_item? }
+      let(:code) { 'DUP_ITEM' }
+      it { should be_true }
+    end
+
+    context 'when method ends with ? and does not match code' do
+      let(:method) { :dup_item? }
+      let(:code) { 'COGS_ERROR' }
+      it { should be_false }
+    end
+
+    context 'when method does not end with ?' do
+      let(:method) { :missing }
+      it 'does not try to evaluate code' do
+        status.should_not_receive(:code)
+        subject rescue nil
+      end
+    end
+  end
+end

data/spec/lib/activenetsuite/helpers/client_spec.rb

@@ -105,8 +105,39 @@ describe Client do
     it do
       client.driver.should_receive(:addList) do |req|
         req.should be_a AddListRequest
-        req.should have(1).record
-        req.first.should eq recs.first
+        req.should eq recs
+        res
+      end
+      should eq res
+    end
+  end
+
+  describe '#update' do
+    subject { client.update(rec) }
+
+    let(:rec) { double }
+    let(:res) { double }
+
+    it do
+      client.driver.should_receive(:update) do |req|
+        req.should be_a UpdateRequest
+        req.record.should be rec
+        res
+      end
+      should eq res
+    end
+  end
+
+  describe '#update_list' do
+    subject { client.update_list(recs) }
+
+    let(:recs) { [double] }
+    let(:res) { double }
+
+    it do
+      client.driver.should_receive(:updateList) do |req|
+        req.should be_a UpdateListRequest
+        req.should eq recs
         res
       end
       should eq res

data/spec/lib/activenetsuite/types/search_date_field_operator_spec.rb

@@ -0,0 +1,8 @@
+require 'spec_helper'
+
+describe SearchDateFieldOperator do
+  describe '.make' do
+    subject { described_class.make :not_empty }
+    it { should be described_class::NotEmpty }
+  end
+end

data/spec/lib/activenetsuite/types/search_date_spec.rb

@@ -0,0 +1,8 @@
+require 'spec_helper'
+
+describe SearchDate do
+  describe '.make' do
+    subject { described_class.make :next_week }
+    it { should be described_class::NextWeek }
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activenetsuite
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Stan Mazhara
@@ -149,6 +149,7 @@ files:
 - lib/activenetsuite/core/search_record.rb
 - lib/activenetsuite/core/search_result.rb
 - lib/activenetsuite/core/status.rb
+- lib/activenetsuite/core/status_detail.rb
 - lib/activenetsuite/helpers/client.rb
 - lib/activenetsuite/helpers/error.rb
 - lib/activenetsuite/helpers/method_inflector.rb
@@ -159,6 +160,7 @@ files:
 - lib/activenetsuite/messages/delete_response.rb
 - lib/activenetsuite/messages/get_deleted_response.rb
 - lib/activenetsuite/messages/get_deleted_result.rb
+- lib/activenetsuite/messages/get_list_response.rb
 - lib/activenetsuite/messages/get_response.rb
 - lib/activenetsuite/messages/read_response.rb
 - lib/activenetsuite/messages/search_more_with_id_response.rb
@@ -173,6 +175,8 @@ files:
 - lib/activenetsuite/soap/default.rb
 - lib/activenetsuite/soap/defaultDriver.rb
 - lib/activenetsuite/soap/defaultMappingRegistry.rb
+- lib/activenetsuite/types/search_date.rb
+- lib/activenetsuite/types/search_date_field_operator.rb
 - lib/activenetsuite/version.rb
 - spec/lib/activenetsuite/accounting/account_spec.rb
 - spec/lib/activenetsuite/accounting/non_inventory_sale_item_spec.rb
@@ -180,6 +184,7 @@ files:
 - spec/lib/activenetsuite/core/record_spec.rb
 - spec/lib/activenetsuite/core/search_record_spec.rb
 - spec/lib/activenetsuite/core/search_result_spec.rb
+- spec/lib/activenetsuite/core/status_spec.rb
 - spec/lib/activenetsuite/helpers/client_spec.rb
 - spec/lib/activenetsuite/helpers/method_inflector_spec.rb
 - spec/lib/activenetsuite/helpers/search_response_methods_spec.rb
@@ -189,6 +194,8 @@ files:
 - spec/lib/activenetsuite/relationships/vendor_spec.rb
 - spec/lib/activenetsuite/sales/cash_refund_spec.rb
 - spec/lib/activenetsuite/sales/cash_sale_spec.rb
+- spec/lib/activenetsuite/types/search_date_field_operator_spec.rb
+- spec/lib/activenetsuite/types/search_date_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/smazhara/activenetsuite
 licenses: