netsuite 0.2.4 → 0.2.5

data/spec/netsuite/actions/search_spec.rb

@@ -42,8 +42,8 @@ describe NetSuite::Actions::Search do
       }.not_to raise_error
     end
 
-    pending "should handle a ID search with basic params"
-    pending "should handle a search with joined params"
+    skip "should handle a ID search with basic params"
+    skip "should handle a search with joined params"
 
     it "should handle a search with joined params containing custom field search" do
       savon.expects(:search).with(:message => {
@@ -146,12 +146,12 @@ describe NetSuite::Actions::Search do
   end
 
   context "advanced search" do
-    pending "should handle search column definitions"
-    pending "should handle joined search results"
+    skip "should handle search column definitions"
+    skip "should handle joined search results"
   end
 
   context "basic search" do
-    pending "should handle searching basic fields"
-    pending "should handle searching with joined fields"
+    skip "should handle searching basic fields"
+    skip "should handle searching with joined fields"
   end
 end

data/spec/netsuite/actions/update_spec.rb

@@ -21,11 +21,11 @@ describe NetSuite::Actions::Update do
     end
 
     it 'makes a valid request to the NetSuite API' do
-      NetSuite::Actions::Update.call(NetSuite::Records::Customer, attributes)
+      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 = NetSuite::Actions::Update.call([NetSuite::Records::Customer, attributes])
       response.should be_kind_of(NetSuite::Response)
       response.should be_success
     end
@@ -47,11 +47,11 @@ describe NetSuite::Actions::Update do
     end
 
     it 'makes a valid request to the NetSuite API' do
-      NetSuite::Actions::Update.call(NetSuite::Records::Invoice, attributes)
+      NetSuite::Actions::Update.call([NetSuite::Records::Invoice, attributes])
     end
 
     it 'returns a valid Response object' do
-      response = NetSuite::Actions::Update.call(NetSuite::Records::Invoice, attributes)
+      response = NetSuite::Actions::Update.call([NetSuite::Records::Invoice, attributes])
       response.should be_kind_of(NetSuite::Response)
       response.should be_success
     end

data/spec/netsuite/actions/upsert_spec.rb

@@ -22,11 +22,11 @@ describe NetSuite::Actions::Upsert do
     end
 
     it 'makes a valid request to the NetSuite API' do
-      NetSuite::Actions::Upsert.call(customer)
+      NetSuite::Actions::Upsert.call([customer])
     end
 
     it 'returns a valid Response object' do
-      response = NetSuite::Actions::Upsert.call(customer)
+      response = NetSuite::Actions::Upsert.call([customer])
       response.should be_kind_of(NetSuite::Response)
       response.should be_success
     end
@@ -50,11 +50,11 @@ describe NetSuite::Actions::Upsert do
       end
 
       it 'makes a valid request to the NetSuite API' do
-        NetSuite::Actions::Upsert.call(invoice)
+        NetSuite::Actions::Upsert.call([invoice])
       end
 
       it 'returns a valid Response object' do
-        response = NetSuite::Actions::Upsert.call(invoice)
+        response = NetSuite::Actions::Upsert.call([invoice])
         response.should be_kind_of(NetSuite::Response)
         response.should be_success
       end
@@ -83,7 +83,7 @@ describe NetSuite::Actions::Upsert do
       end
 
       it 'provides an error method on the response' do
-        response = NetSuite::Actions::Upsert.call(invoice)
+        response = NetSuite::Actions::Upsert.call([invoice])
         response.errors.first.should be_kind_of(NetSuite::Error)
       end
     end

data/spec/netsuite/records/account_spec.rb

@@ -40,7 +40,7 @@ describe NetSuite::Records::Account do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :acct_name => 'Account 1' }) }
 
       it 'returns a Account instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Account, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Account, {:external_id => 1}], {}).and_return(response)
         account = NetSuite::Records::Account.get(:external_id => 1)
         account.should be_kind_of(NetSuite::Records::Account)
         account.acct_name.should eql('Account 1')
@@ -51,7 +51,7 @@ describe NetSuite::Records::Account do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Account, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Account, {:external_id => 1}], {}).and_return(response)
         lambda {
           NetSuite::Records::Account.get(:external_id => 1)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -69,7 +69,7 @@ describe NetSuite::Records::Account do
       it 'returns true' do
         account = NetSuite::Records::Account.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(account).
+            with([account], {}).
             and_return(response)
         account.add.should be_true
       end
@@ -81,7 +81,7 @@ describe NetSuite::Records::Account do
       it 'returns false' do
         account = NetSuite::Records::Account.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(account).
+            with([account], {}).
             and_return(response)
         account.add.should be_false
       end
@@ -97,7 +97,7 @@ describe NetSuite::Records::Account do
       it 'returns true' do
         account = NetSuite::Records::Account.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(account).
+            with([account], {}).
             and_return(response)
         account.delete.should be_true
       end
@@ -109,7 +109,7 @@ describe NetSuite::Records::Account do
       it 'returns false' do
         account = NetSuite::Records::Account.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(account).
+            with([account], {}).
             and_return(response)
         account.delete.should be_false
       end

data/spec/netsuite/records/accounting_period_spec.rb

@@ -24,7 +24,7 @@ describe NetSuite::Records::AccountingPeriod do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :period_name => 'Accounting Period 1' }) }
 
       it 'returns a Account instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::AccountingPeriod, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::AccountingPeriod, {:external_id => 1}], {}).and_return(response)
         accounting_period = NetSuite::Records::AccountingPeriod.get(:external_id => 1)
         accounting_period.should be_kind_of(NetSuite::Records::AccountingPeriod)
         accounting_period.period_name.should eql('Accounting Period 1')
@@ -35,7 +35,7 @@ describe NetSuite::Records::AccountingPeriod do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::AccountingPeriod, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::AccountingPeriod, {:external_id => 1}], {}).and_return(response)
         lambda {
           NetSuite::Records::AccountingPeriod.get(:external_id => 1)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -53,7 +53,7 @@ describe NetSuite::Records::AccountingPeriod do
       it 'returns true' do
         accounting_period = NetSuite::Records::AccountingPeriod.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(accounting_period).
+            with([accounting_period], {}).
             and_return(response)
         accounting_period.add.should be_true
       end
@@ -65,7 +65,7 @@ describe NetSuite::Records::AccountingPeriod do
       it 'returns false' do
         accounting_period = NetSuite::Records::AccountingPeriod.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(accounting_period).
+            with([accounting_period], {}).
             and_return(response)
         accounting_period.add.should be_false
       end
@@ -81,7 +81,7 @@ describe NetSuite::Records::AccountingPeriod do
       it 'returns true' do
         accounting_period = NetSuite::Records::AccountingPeriod.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(accounting_period).
+            with([accounting_period], {}).
             and_return(response)
         accounting_period.delete.should be_true
       end
@@ -93,7 +93,7 @@ describe NetSuite::Records::AccountingPeriod do
       it 'returns false' do
         accounting_period = NetSuite::Records::AccountingPeriod.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(accounting_period).
+            with([accounting_period], {}).
             and_return(response)
         accounting_period.delete.should be_false
       end

data/spec/netsuite/records/classification_spec.rb

@@ -16,7 +16,7 @@ describe NetSuite::Records::Classification do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :name => 'Retail' }) }
 
       it 'returns an Invoice instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Classification, :external_id => 10).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Classification, {:external_id => 10}], {}).and_return(response)
         invoice = NetSuite::Records::Classification.get(:external_id => 10)
         invoice.should be_kind_of(NetSuite::Records::Classification)
       end
@@ -26,7 +26,7 @@ describe NetSuite::Records::Classification do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::Classification, :external_id => 10).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::Classification, {:external_id => 10}], {}).and_return(response)
         lambda {
           NetSuite::Records::Classification.get(:external_id => 10)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -44,7 +44,7 @@ describe NetSuite::Records::Classification do
       it 'returns true' do
         classification = NetSuite::Records::Classification.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(classification).
+            with([classification], {}).
             and_return(response)
         classification.delete.should be_true
       end
@@ -56,7 +56,7 @@ describe NetSuite::Records::Classification do
       it 'returns false' do
         classification = NetSuite::Records::Classification.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(classification).
+            with([classification], {}).
             and_return(response)
         classification.delete.should be_false
       end

data/spec/netsuite/records/credit_memo_spec.rb

@@ -98,7 +98,7 @@ describe NetSuite::Records::CreditMemo do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :alt_shipping_cost => 100 }) }
 
       it 'returns a CreditMemo instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::CreditMemo, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::CreditMemo, {:external_id => 1}], {}).and_return(response)
         memo = NetSuite::Records::CreditMemo.get(:external_id => 1)
         memo.should be_kind_of(NetSuite::Records::CreditMemo)
         memo.alt_shipping_cost.should eql(100)
@@ -109,7 +109,7 @@ describe NetSuite::Records::CreditMemo do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::CreditMemo, :external_id => 1).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::CreditMemo, {:external_id => 1}], {}).and_return(response)
         lambda {
           NetSuite::Records::CreditMemo.get(:external_id => 1)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -121,14 +121,14 @@ describe NetSuite::Records::CreditMemo do
   describe '.initialize' do
     context 'when the request is successful' do
       it 'returns an initialized credit memo from the customer entity' do
-        NetSuite::Actions::Initialize.should_receive(:call).with(NetSuite::Records::CreditMemo, customer).and_return(response)
+        NetSuite::Actions::Initialize.should_receive(:call).with([NetSuite::Records::CreditMemo, customer], {}).and_return(response)
         invoice = NetSuite::Records::CreditMemo.initialize(customer)
         invoice.should be_kind_of(NetSuite::Records::CreditMemo)
       end
     end
 
     context 'when the response is unsuccessful' do
-      pending
+      skip
     end
   end
 
@@ -141,7 +141,7 @@ describe NetSuite::Records::CreditMemo do
       it 'returns true' do
         memo = NetSuite::Records::CreditMemo.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(memo).
+            with([memo], {}).
             and_return(response)
         memo.add.should be_true
       end
@@ -153,7 +153,7 @@ describe NetSuite::Records::CreditMemo do
       it 'returns false' do
         memo = NetSuite::Records::CreditMemo.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(memo).
+            with([memo], {}).
             and_return(response)
         memo.add.should be_false
       end
@@ -169,7 +169,7 @@ describe NetSuite::Records::CreditMemo do
       it 'returns true' do
         memo = NetSuite::Records::CreditMemo.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(memo).
+            with([memo], {}).
             and_return(response)
         memo.delete.should be_true
       end
@@ -181,7 +181,7 @@ describe NetSuite::Records::CreditMemo do
       it 'returns false' do
         memo = NetSuite::Records::CreditMemo.new(test_data)
         NetSuite::Actions::Delete.should_receive(:call).
-            with(memo).
+            with([memo], {}).
             and_return(response)
         memo.delete.should be_false
       end

data/spec/netsuite/records/custom_field_list_spec.rb

@@ -15,6 +15,16 @@ describe NetSuite::Records::CustomFieldList do
       list.custom_fields.first.type.should == 'platformCore:StringCustomFieldRef'
     end
 
+    it "should handle date custom field creation" do
+      list.custrecord_somefield = Date.parse("12/12/2012")
+      list.custom_fields.first.value.should == '2012-12-12T00:00:00+00:00'
+    end
+
+    it "should handle datetime custom field creation" do
+      list.custrecord_somefield = DateTime.parse("12/12/2012 10:05am")
+      list.custom_fields.first.value.should == '2012-12-12T10:05:00+00:00'
+    end
+
     it "should convert a list of numbers into a list of custom field refs" do
       list.custrecord_somefield = [1,2]
       list.custom_fields.first.type.should == 'platformCore:MultiSelectCustomFieldRef'

data/spec/netsuite/records/custom_field_spec.rb

@@ -1,5 +1,5 @@
 require 'spec_helper'
 
 describe NetSuite::Records::CustomField do
-  pending
+  skip
 end

data/spec/netsuite/records/custom_record_ref_spec.rb

@@ -4,5 +4,5 @@ describe NetSuite::Records::CustomRecordRef do
   #<attribute name="internalId" type="xsd:string"/>
   #<attribute name="externalId" type="xsd:string"/>
   #<attribute name="typeId" type="xsd:string" use="required"/>
-  pending
+  skip
 end

data/spec/netsuite/records/custom_record_spec.rb

@@ -51,7 +51,7 @@ describe NetSuite::Records::CustomRecord do
 
       it 'returns a Customer instance populated with the data from the response object' do
         NetSuite::Actions::Get.should_receive(:call).
-          with(NetSuite::Records::CustomRecord, :external_id => 1, :type_id => nil, :custom => true).
+          with([NetSuite::Records::CustomRecord, {:external_id => 1, :type_id => nil, :custom => true}], {}).
           and_return(response)
         customer = NetSuite::Records::CustomRecord.get(:external_id => 1)
         customer.should be_kind_of(NetSuite::Records::CustomRecord)
@@ -64,7 +64,7 @@ describe NetSuite::Records::CustomRecord do
 
       it 'raises a RecordNotFound exception' do
         NetSuite::Actions::Get.should_receive(:call).
-          with(NetSuite::Records::CustomRecord, :external_id => 1, :type_id => nil, :custom => true).
+          with([NetSuite::Records::CustomRecord, {:external_id => 1, :type_id => nil, :custom => true}], {}).
           and_return(response)
         lambda {
           NetSuite::Records::CustomRecord.get(:external_id => 1)
@@ -80,7 +80,7 @@ describe NetSuite::Records::CustomRecord do
 
       it 'returns true' do
         NetSuite::Actions::Add.should_receive(:call).
-            with(record).
+            with([record], {}).
             and_return(response)
         record.add.should be_true
       end
@@ -91,7 +91,7 @@ describe NetSuite::Records::CustomRecord do
 
       it 'returns false' do
         NetSuite::Actions::Add.should_receive(:call).
-            with(record).
+            with([record], {}).
             and_return(response)
         record.add.should be_false
       end
@@ -104,7 +104,7 @@ describe NetSuite::Records::CustomRecord do
 
       it 'returns true' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(record, { :custom => true }).
+            with([record, { :custom => true }], {}).
             and_return(response)
         record.delete.should be_true
       end
@@ -115,7 +115,7 @@ describe NetSuite::Records::CustomRecord do
 
       it 'returns false' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(record, { :custom => true }).
+            with([record, { :custom => true }], {}).
             and_return(response)
         record.delete.should be_false
       end

data/spec/netsuite/records/custom_record_type_spec.rb

@@ -84,7 +84,7 @@ describe NetSuite::Records::CustomRecordType do
 
       it 'returns a Customer instance populated with the data from the response object' do
         NetSuite::Actions::Get.should_receive(:call).
-          with(NetSuite::Records::CustomRecordType, :external_id => 1).
+          with([NetSuite::Records::CustomRecordType, :external_id => 1], {}).
           and_return(response)
         record_type = NetSuite::Records::CustomRecordType.get(:external_id => 1)
         record_type.should be_kind_of(NetSuite::Records::CustomRecordType)
@@ -97,7 +97,7 @@ describe NetSuite::Records::CustomRecordType do
 
       it 'raises a RecordNotFound exception' do
         NetSuite::Actions::Get.should_receive(:call).
-          with(NetSuite::Records::CustomRecordType, :external_id => 1).
+          with([NetSuite::Records::CustomRecordType, :external_id => 1], {}).
           and_return(response)
         lambda {
           NetSuite::Records::CustomRecordType.get(:external_id => 1)
@@ -113,7 +113,7 @@ describe NetSuite::Records::CustomRecordType do
 
       it 'returns true' do
         NetSuite::Actions::Add.should_receive(:call).
-            with(record_type).
+            with([record_type], {}).
             and_return(response)
         record_type.add.should be_true
       end
@@ -124,7 +124,7 @@ describe NetSuite::Records::CustomRecordType do
 
       it 'returns false' do
         NetSuite::Actions::Add.should_receive(:call).
-            with(record_type).
+            with([record_type], {}).
             and_return(response)
         record_type.add.should be_false
       end
@@ -137,7 +137,7 @@ describe NetSuite::Records::CustomRecordType do
 
       it 'returns true' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(record_type).
+            with([record_type], {}).
             and_return(response)
         record_type.delete.should be_true
       end
@@ -148,7 +148,7 @@ describe NetSuite::Records::CustomRecordType do
 
       it 'returns false' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(record_type).
+            with([record_type], {}).
             and_return(response)
         record_type.delete.should be_false
       end

data/spec/netsuite/records/customer_payment_spec.rb

@@ -10,7 +10,7 @@ describe NetSuite::Records::CustomerPayment do
       :applied, :auth_code, :auto_apply, :balance, :cc_approved, :cc_avs_street_match, :cc_avs_zip_match, :cc_expire_date,
       :cc_name, :cc_number, :cc_security_code, :cc_security_code_match, :cc_street, :cc_zip_code, :charge_it, :check_num,
       :created_date, :currency_name, :debit_card_issue_no, :exchange_rate, :ignore_avs, :last_modified_date, :memo, :payment,
-      :pending, :pn_ref_num, :status, :three_d_status_code, :total, :tran_date, :unapplied, :undep_funds, :valid_from
+      :skip, :pn_ref_num, :status, :three_d_status_code, :total, :tran_date, :unapplied, :undep_funds, :valid_from
     ].each do |field|
       payment.should have_field(field)
     end
@@ -64,7 +64,7 @@ describe NetSuite::Records::CustomerPayment do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :memo => 'This is a memo' }) }
 
       it 'returns an CustomerPayment instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::CustomerPayment, :external_id => 7).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::CustomerPayment, {:external_id => 7}], {}).and_return(response)
         payment = NetSuite::Records::CustomerPayment.get(:external_id => 7)
         payment.should be_kind_of(NetSuite::Records::CustomerPayment)
         payment.memo.should eql('This is a memo')
@@ -75,7 +75,7 @@ describe NetSuite::Records::CustomerPayment do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with(NetSuite::Records::CustomerPayment, :external_id => 8).and_return(response)
+        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::CustomerPayment, {:external_id => 8}], {}).and_return(response)
         lambda {
           NetSuite::Records::CustomerPayment.get(:external_id => 8)
         }.should raise_error(NetSuite::RecordNotFound,
@@ -87,14 +87,14 @@ describe NetSuite::Records::CustomerPayment do
   describe '.initialize' do
     context 'when the request is successful' do
       it 'returns an initialized invoice from the customer entity' do
-        NetSuite::Actions::Initialize.should_receive(:call).with(NetSuite::Records::CustomerPayment, invoice).and_return(response)
+        NetSuite::Actions::Initialize.should_receive(:call).with([NetSuite::Records::CustomerPayment, invoice], {}).and_return(response)
         payment = NetSuite::Records::CustomerPayment.initialize(invoice)
         payment.should be_kind_of(NetSuite::Records::CustomerPayment)
       end
     end
 
     context 'when the response is unsuccessful' do
-      pending
+      skip
     end
   end
 
@@ -107,7 +107,7 @@ describe NetSuite::Records::CustomerPayment do
       it 'returns true' do
         payment = NetSuite::Records::CustomerPayment.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(payment).
+            with([payment], {}).
             and_return(response)
         payment.add.should be_true
       end
@@ -119,7 +119,7 @@ describe NetSuite::Records::CustomerPayment do
       it 'returns false' do
         payment = NetSuite::Records::CustomerPayment.new(test_data)
         NetSuite::Actions::Add.should_receive(:call).
-            with(payment).
+            with([payment], {}).
             and_return(response)
         payment.add.should be_false
       end
@@ -132,7 +132,7 @@ describe NetSuite::Records::CustomerPayment do
 
       it 'returns true' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(payment).
+            with([payment], {}).
             and_return(response)
         payment.delete.should be_true
       end
@@ -143,7 +143,7 @@ describe NetSuite::Records::CustomerPayment do
 
       it 'returns false' do
         NetSuite::Actions::Delete.should_receive(:call).
-            with(payment).
+            with([payment], {}).
             and_return(response)
         payment.delete.should be_false
       end