netsuite 0.2.6 → 0.3.0

data/lib/netsuite/version.rb

@@ -1,3 +1,3 @@
 module Netsuite
-  VERSION = '0.2.6'
+  VERSION = '0.3.0'
 end

data/spec/netsuite/actions/search_spec.rb

@@ -81,7 +81,7 @@ describe NetSuite::Actions::Search do
 
                   :attributes! => {
                     "platformCore:customField" => {
-                      "internalId" => ["custentity_customerandcontacttypelist", "custentity_relatedthing"],
+                      "scriptId" => ["custentity_customerandcontacttypelist", "custentity_relatedthing"],
                       "operator" => ["anyOf", "anyOf"],
                       "xsi:type" => ["platformCore:SearchMultiSelectCustomField", "platformCore:SearchMultiSelectCustomField"]
                     }

data/spec/netsuite/actions/update_spec.rb

@@ -7,9 +7,8 @@ describe NetSuite::Actions::Update do
   context 'Customer' do
     let(:customer) { NetSuite::Records::Customer.new }
     let(:attributes) { { :entity_id => 'Shutter Fly', :company_name => 'Shutter Fly, Inc.' } }
-
-    before do
-      savon.expects(:update).with(:message => {
+    let(:message) do
+      {
         'platformMsgs:record' => {
           :content! => {
             'listRel:entityId'    => 'Shutter Fly',
@@ -17,44 +16,76 @@ describe NetSuite::Actions::Update do
           },
           '@xsi:type' => 'listRel:Customer'
         },
-      }).returns(File.read('spec/support/fixtures/update/update_customer.xml'))
+      }
     end
 
-    it 'makes a valid request to the NetSuite API' do
-      NetSuite::Actions::Update.call([NetSuite::Records::Customer, attributes])
-    end
 
-    it 'returns a valid Response object' do
-      response = NetSuite::Actions::Update.call([NetSuite::Records::Customer, attributes])
-      response.should be_kind_of(NetSuite::Response)
-      response.should be_success
-    end
-  end
+    context 'when successful' do
 
-  context 'Invoice' do
-    let(:invoice) { NetSuite::Records::Invoice.new }
-    let(:attributes) { { :source => 'Google', :total => 100.0 } }
+      before do
+        savon.expects(:update).with(:message => message).returns(File.read('spec/support/fixtures/update/update_customer.xml'))
+      end
+
+      it 'makes a valid request to the NetSuite API' do
+        NetSuite::Actions::Update.call([NetSuite::Records::Customer, attributes])
+      end
+
+      it 'returns a valid Response object' do
+        response = NetSuite::Actions::Update.call([NetSuite::Records::Customer, attributes])
+        response.should be_kind_of(NetSuite::Response)
+        response.should be_success
+      end
 
-    before do
-      savon.expects(:update).with(:message => {
-        'platformMsgs:record' => {
-          :content! => {
-            'tranSales:source' => 'Google',
-          },
-          '@xsi:type' => 'tranSales:Invoice'
-        },
-      }).returns(File.read('spec/support/fixtures/update/update_invoice.xml'))
     end
 
-    it 'makes a valid request to the NetSuite API' do
-      NetSuite::Actions::Update.call([NetSuite::Records::Invoice, attributes])
+    context 'when not successful' do
+
+      before do
+        savon.expects(:update).with(:message => message).returns(File.read('spec/support/fixtures/update/update_customer_error.xml'))
+      end
+
+      it 'provides an errors method on the object with details about the errors' do
+        customer.update(attributes)
+        error = customer.errors.first
+
+        error.should be_kind_of(NetSuite::Error)
+        error.type.should eq('ERROR')
+        error.code.should eq('INSUFFICIENT_PERMISSION')
+        error.message.should eq('You do not have permissions to set a value for element addrtext due to one of the following reasons: 1) The field is read-only; 2) An associated feature is disabled; 3) The field is available either when a record is created or updated, but not in both cases.')
+      end
+
+      it 'provides an errors method on the response' do
+        response = NetSuite::Actions::Update.call([NetSuite::Records::Customer, attributes])
+        response.errors.first.should be_kind_of(NetSuite::Error)
+      end
+
     end
 
-    it 'returns a valid Response object' do
-      response = NetSuite::Actions::Update.call([NetSuite::Records::Invoice, attributes])
-      response.should be_kind_of(NetSuite::Response)
-      response.should be_success
+    context 'when not successful with multiple errors' do
+
+      before do
+        savon.expects(:update).with(:message => message).returns(File.read('spec/support/fixtures/update/update_customer_multiple_errors.xml'))
+      end
+
+      it 'provides an errors method on the object with details about the errors' do
+        customer.update(attributes)
+        customer.errors.length.should eq(2)
+
+        first_error = customer.errors.first
+        first_error.should be_kind_of(NetSuite::Error)
+        first_error.type.should eq('ERROR')
+        first_error.code.should eq('First error code')
+        first_error.message.should eq('First error message')
+
+        second_error = customer.errors.last
+        second_error.should be_kind_of(NetSuite::Error)
+        second_error.type.should eq('WARNING')
+        second_error.code.should eq('Second error code')
+        second_error.message.should eq('Second error message')
+
+      end
     end
+
   end
 
 end

data/spec/netsuite/actions/upsert_list_spec.rb

@@ -0,0 +1,112 @@
+require 'spec_helper'
+
+describe NetSuite::Actions::UpsertList do
+  before { savon.mock! }
+  after { savon.unmock! }
+
+  context 'Customers' do
+    context 'one customer' do
+      let(:customers) do
+        [
+          NetSuite::Records::Customer.new(external_id: 'ext2', entity_id: 'Target', company_name: 'Target')
+        ]
+      end
+
+      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 'makes a valid request to the NetSuite API' do
+        NetSuite::Actions::UpsertList.call(customers)
+      end
+
+      it 'returns a valid Response object' do
+        response = NetSuite::Actions::UpsertList.call(customers)
+        response.should be_kind_of(NetSuite::Response)
+        response.should be_success
+      end
+    end
+
+    context 'two customers' do
+      let(:customers) do
+        [
+          NetSuite::Records::Customer.new(external_id: 'ext1', entity_id: 'Shutter Fly', company_name: 'Shutter Fly, Inc.'),
+          NetSuite::Records::Customer.new(external_id: 'ext2', entity_id: 'Target', company_name: 'Target')
+        ]
+      end
+
+      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 'makes a valid request to the NetSuite API' do
+        NetSuite::Actions::UpsertList.call(customers)
+      end
+
+      it 'returns a valid Response object' do
+        response = NetSuite::Actions::UpsertList.call(customers)
+        response.should be_kind_of(NetSuite::Response)
+        response.should be_success
+      end
+    end
+  end
+
+  context 'with errors' do
+    let(:customers) do
+      [
+        NetSuite::Records::Customer.new(external_id: 'ext1-bad', entity_id: 'Shutter Fly', company_name: 'Shutter Fly, Inc.'),
+        NetSuite::Records::Customer.new(external_id: 'ext2-bad', entity_id: 'Target', company_name: 'Target')
+      ]
+    end
+
+    before do
+      savon.expects(:upsert_list).with(:message =>
+        {
+          'record' => [{
+            'listRel:entityId'    => 'Shutter Fly',
+            'listRel:companyName' => 'Shutter Fly, Inc.',
+            '@xsi:type' => 'listRel:Customer',
+            '@externalId' => 'ext1-bad'
+          },
+          {
+            'listRel:entityId'    => 'Target',
+            'listRel:companyName' => 'Target',
+            '@xsi:type' => 'listRel:Customer',
+            '@externalId' => 'ext2-bad'
+          }
+          ]
+        }).returns(File.read('spec/support/fixtures/upsert_list/upsert_list_with_errors.xml'))
+    end
+
+    it 'constructs error objects' do
+      response = NetSuite::Actions::UpsertList.call(customers)
+      expect(response.errors.keys).to match_array(['ext1', 'ext2'])
+      expect(response.errors['ext1'].first.code).to eq('USER_ERROR')
+      expect(response.errors['ext1'].first.message).to eq('Please enter value(s) for: Item')
+      expect(response.errors['ext1'].first.type).to eq('ERROR')
+    end
+  end
+end

data/spec/netsuite/records/account_spec.rb

@@ -71,7 +71,7 @@ describe NetSuite::Records::Account do
         NetSuite::Actions::Add.should_receive(:call).
             with([account], {}).
             and_return(response)
-        account.add.should be_true
+        account.add.should be_truthy
       end
     end
 
@@ -83,7 +83,7 @@ describe NetSuite::Records::Account do
         NetSuite::Actions::Add.should_receive(:call).
             with([account], {}).
             and_return(response)
-        account.add.should be_false
+        account.add.should be_falsey
       end
     end
   end
@@ -99,7 +99,7 @@ describe NetSuite::Records::Account do
         NetSuite::Actions::Delete.should_receive(:call).
             with([account], {}).
             and_return(response)
-        account.delete.should be_true
+        account.delete.should be_truthy
       end
     end
 
@@ -111,7 +111,7 @@ describe NetSuite::Records::Account do
         NetSuite::Actions::Delete.should_receive(:call).
             with([account], {}).
             and_return(response)
-        account.delete.should be_false
+        account.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/accounting_period_spec.rb

@@ -91,7 +91,7 @@ describe NetSuite::Records::AccountingPeriod do
         NetSuite::Actions::Add.should_receive(:call).
             with([accounting_period], {}).
             and_return(response)
-        accounting_period.add.should be_true
+        accounting_period.add.should be_truthy
       end
     end
 
@@ -103,7 +103,7 @@ describe NetSuite::Records::AccountingPeriod do
         NetSuite::Actions::Add.should_receive(:call).
             with([accounting_period], {}).
             and_return(response)
-        accounting_period.add.should be_false
+        accounting_period.add.should be_falsey
       end
     end
   end
@@ -119,7 +119,7 @@ describe NetSuite::Records::AccountingPeriod do
         NetSuite::Actions::Delete.should_receive(:call).
             with([accounting_period], {}).
             and_return(response)
-        accounting_period.delete.should be_true
+        accounting_period.delete.should be_truthy
       end
     end
 
@@ -131,7 +131,7 @@ describe NetSuite::Records::AccountingPeriod do
         NetSuite::Actions::Delete.should_receive(:call).
             with([accounting_period], {}).
             and_return(response)
-        accounting_period.delete.should be_false
+        accounting_period.delete.should be_falsey
       end
     end
   end

data/spec/netsuite/records/assembly_item_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+
+describe NetSuite::Records::AssemblyItem do
+  let(:item) { described_class.new }
+
+  describe '#member_list' do
+    it 'can be set from attributes' do
+      attributes = {
+        :item_member => [{
+          :item => { :internal_id => 1 },
+          :quantity => 20
+        }]
+      }
+      item.member_list = attributes
+      item.member_list.should be_kind_of(NetSuite::Records::MemberList)
+      item.member_list.item_member.length.should eql(1)
+
+      expect(item.member_list.item_member.first.item.internal_id).to eq(1)
+      expect(item.member_list.item_member.first.quantity).to eq(20)
+    end
+  end
+
+  describe '#subsidiary_list' do
+    it 'creates record refs from attributes' do
+      item = described_class.new({
+        subsidiary_list: {
+          record_ref: [
+            { internal_id: 1 },
+            { internal_id: 2 },
+          ]
+        }
+      })
+
+      expect(item.subsidiary_list.record_ref[0].internal_id).to eq(1)
+      expect(item.subsidiary_list.record_ref[1].internal_id).to eq(2)
+    end
+  end
+end

data/spec/netsuite/records/billing_schedule_milestone_list_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe NetSuite::Records::BillingScheduleMilestoneList do
+  let(:list) { NetSuite::Records::BillingScheduleMilestoneList.new }
+  let(:milestone) { NetSuite::Records::BillingScheduleMilestone.new }
+
+  it 'can have milestones be added to it' do
+    list.milestones << milestone
+    milestone_list = list.milestones
+    milestone_list.should be_kind_of(Array)
+    milestone_list.length.should eql(1)
+    milestone_list.each { |i| i.should be_kind_of(NetSuite::Records::BillingScheduleMilestone) }
+  end
+
+  describe '#to_record' do
+    it 'can represent itself as a SOAP record' do
+      record = {
+        'listAcct:billingScheduleMilestone' => []
+      }
+      list.to_record.should eql(record)
+    end
+  end
+
+end

data/spec/netsuite/records/billing_schedule_milestone_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe NetSuite::Records::BillingScheduleMilestone do
+  let(:milestone) { NetSuite::Records::BillingScheduleMilestone.new }
+
+  it 'has the right fields' do
+    [
+      :comments, :milestone_actual_completion_date, :milestone_amount, 
+      :milestone_completed, :milestone_date, :milestone_id
+    ].each do |field|
+      milestone.should have_field(field)
+    end
+  end
+
+  it 'has the right record_refs' do
+    [
+      :milestone_term, :project_task
+    ].each do |record_ref|
+      milestone.should have_record_ref(record_ref)
+    end
+  end
+
+  it 'can initialize from a record' do
+    record = NetSuite::Records::BillingScheduleMilestone.new(:comments => "foo")
+    milestone = NetSuite::Records::BillingScheduleMilestone.new(record)
+    milestone.should be_kind_of(NetSuite::Records::BillingScheduleMilestone)
+    milestone.comments.should eql("foo")
+  end
+
+  describe '#to_record' do
+    before do
+      milestone.comments         = 'comment'
+      milestone.milestone_amount = 1
+    end
+
+    it 'can represent itself as a SOAP record' do
+      record = {
+        'listAcct:comments'        => 'comment',
+        'listAcct:milestoneAmount' => 1
+      }
+      milestone.to_record.should eql(record)
+    end
+  end
+
+  describe '#record_type' do
+    it 'returns a string of the SOAP record type' do
+      milestone.record_type.should eql('listAcct:BillingScheduleMilestone')
+    end
+  end
+
+end

data/spec/netsuite/records/billing_schedule_recurrence_list_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe NetSuite::Records::BillingScheduleRecurrenceList do
+  let(:list) { NetSuite::Records::BillingScheduleRecurrenceList.new }
+  let(:recurrence) { NetSuite::Records::BillingScheduleRecurrence.new }
+
+  it 'can have recurrences be added to it' do
+    list.recurrences << recurrence
+    recurrence_list = list.recurrences
+    recurrence_list.should be_kind_of(Array)
+    recurrence_list.length.should eql(1)
+    recurrence_list.each { |i| i.should be_kind_of(NetSuite::Records::BillingScheduleRecurrence) }
+  end
+
+  describe '#to_record' do
+    it 'can represent itself as a SOAP record' do
+      record = {
+        'listAcct:billingScheduleRecurrence' => []
+      }
+      list.to_record.should eql(record)
+    end
+  end
+
+end