netsuite 0.3.0 → 0.3.1

data/spec/netsuite/records/custom_record_type_spec.rb

@@ -11,7 +11,7 @@ describe NetSuite::Records::CustomRecordType do
       :record_name, :script_id, :show_creation_date, :show_creation_date_on_list, :show_id, :show_last_modified,
       :show_last_modified_on_list, :show_notes, :show_owner, :show_owner_allow_change, :show_owner_on_list, :use_permissions
     ].each do |field|
-      record_type.should have_field(field)
+      expect(record_type).to have_field(field)
     end
   end
 
@@ -19,7 +19,7 @@ describe NetSuite::Records::CustomRecordType do
     [
       :owner
     ].each do |record_ref|
-      record_type.should have_record_ref(record_ref)
+      expect(record_type).to have_record_ref(record_ref)
     end
   end
 
@@ -83,12 +83,12 @@ describe NetSuite::Records::CustomRecordType do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :allow_attachments => true }) }
 
       it 'returns a Customer instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).
+        expect(NetSuite::Actions::Get).to receive(:call).
           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)
-        record_type.allow_attachments.should be_truthy
+        expect(record_type).to be_kind_of(NetSuite::Records::CustomRecordType)
+        expect(record_type.allow_attachments).to be_truthy
       end
     end
 
@@ -96,12 +96,12 @@ describe NetSuite::Records::CustomRecordType do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).
+        expect(NetSuite::Actions::Get).to receive(:call).
           with([NetSuite::Records::CustomRecordType, :external_id => 1], {}).
           and_return(response)
-        lambda {
+        expect {
           NetSuite::Records::CustomRecordType.get(:external_id => 1)
-        }.should raise_error(NetSuite::RecordNotFound,
+        }.to raise_error(NetSuite::RecordNotFound,
           /NetSuite::Records::CustomRecordType with OPTIONS=(.*) could not be found/)
       end
     end
@@ -112,10 +112,10 @@ describe NetSuite::Records::CustomRecordType do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
 
       it 'returns true' do
-        NetSuite::Actions::Add.should_receive(:call).
+        expect(NetSuite::Actions::Add).to receive(:call).
             with([record_type], {}).
             and_return(response)
-        record_type.add.should be_truthy
+        expect(record_type.add).to be_truthy
       end
     end
 
@@ -123,10 +123,10 @@ describe NetSuite::Records::CustomRecordType do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'returns false' do
-        NetSuite::Actions::Add.should_receive(:call).
+        expect(NetSuite::Actions::Add).to receive(:call).
             with([record_type], {}).
             and_return(response)
-        record_type.add.should be_falsey
+        expect(record_type.add).to be_falsey
       end
     end
   end
@@ -136,10 +136,10 @@ describe NetSuite::Records::CustomRecordType do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
 
       it 'returns true' do
-        NetSuite::Actions::Delete.should_receive(:call).
+        expect(NetSuite::Actions::Delete).to receive(:call).
             with([record_type], {}).
             and_return(response)
-        record_type.delete.should be_truthy
+        expect(record_type.delete).to be_truthy
       end
     end
 
@@ -147,10 +147,10 @@ describe NetSuite::Records::CustomRecordType do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'returns false' do
-        NetSuite::Actions::Delete.should_receive(:call).
+        expect(NetSuite::Actions::Delete).to receive(:call).
             with([record_type], {}).
             and_return(response)
-        record_type.delete.should be_falsey
+        expect(record_type.delete).to be_falsey
       end
     end
   end

data/spec/netsuite/records/customer_addressbook_list_spec.rb

@@ -4,7 +4,7 @@ describe NetSuite::Records::CustomerAddressbookList do
   let(:list) { NetSuite::Records::CustomerAddressbookList.new }
 
   it 'has an addressbooks attribute' do
-    list.addressbooks.should be_kind_of(Array)
+    expect(list.addressbooks).to be_kind_of(Array)
   end
 
   describe '#to_record' do
@@ -13,7 +13,7 @@ describe NetSuite::Records::CustomerAddressbookList do
         'listRel:addressbook' => [],
         'listRel:replaceAll' => true
       }
-      list.to_record.should eql(record)
+      expect(list.to_record).to eql(record)
     end
   end
 

data/spec/netsuite/records/customer_addressbook_spec.rb

@@ -26,7 +26,7 @@ describe NetSuite::Records::CustomerAddressbook do
       :default_shipping, :default_billing, :is_residential, :label, :attention, :addressee,
       :phone, :addr1, :addr2, :addr3, :city, :zip, :addr_text, :override, :state
     ].each do |field|
-      list.should have_field(field)
+      expect(list).to have_field(field)
     end
   end
 
@@ -34,25 +34,25 @@ describe NetSuite::Records::CustomerAddressbook do
     [
       :addr_text
     ].each do |field|
-      NetSuite::Records::CustomerAddressbook.should have_read_only_field(field)
+      expect(NetSuite::Records::CustomerAddressbook).to have_read_only_field(field)
     end
   end
 
   describe '#initialize' do
     context 'when taking in a hash of attributes' do
       it 'sets the attributes for the object given the attributes hash' do
-        list.addr1.should eql('123 Happy Lane')
-        list.addr_text.should eql("123 Happy Lane\nLos Angeles CA 90007")
-        list.city.should eql('Los Angeles')
-        list.country.to_record.should eql('_unitedStates')
-        list.default_billing.should be_truthy
-        list.default_shipping.should be_truthy
-        list.is_residential.should be_falsey
-        list.label.should eql('123 Happy Lane')
-        list.override.should be_falsey
-        list.state.should eql('CA')
-        list.zip.should eql('90007')
-        list.internal_id.should eql('567')
+        expect(list.addr1).to eql('123 Happy Lane')
+        expect(list.addr_text).to eql("123 Happy Lane\nLos Angeles CA 90007")
+        expect(list.city).to eql('Los Angeles')
+        expect(list.country.to_record).to eql('_unitedStates')
+        expect(list.default_billing).to be_truthy
+        expect(list.default_shipping).to be_truthy
+        expect(list.is_residential).to be_falsey
+        expect(list.label).to eql('123 Happy Lane')
+        expect(list.override).to be_falsey
+        expect(list.state).to eql('CA')
+        expect(list.zip).to eql('90007')
+        expect(list.internal_id).to eql('567')
       end
     end
 
@@ -60,18 +60,18 @@ describe NetSuite::Records::CustomerAddressbook do
       it 'sets the attributes for the object given the record attributes' do
         old_list = NetSuite::Records::CustomerAddressbook.new(attributes)
         list     = NetSuite::Records::CustomerAddressbook.new(old_list)
-        list.addr1.should eql('123 Happy Lane')
-        list.addr_text.should eql("123 Happy Lane\nLos Angeles CA 90007")
-        list.city.should eql('Los Angeles')
-        list.country.to_record.should eql('_unitedStates')
-        list.default_billing.should be_truthy
-        list.default_shipping.should be_truthy
-        list.is_residential.should be_falsey
-        list.label.should eql('123 Happy Lane')
-        list.override.should be_falsey
-        list.state.should eql('CA')
-        list.zip.should eql('90007')
-        list.internal_id.should eql('567')
+        expect(list.addr1).to eql('123 Happy Lane')
+        expect(list.addr_text).to eql("123 Happy Lane\nLos Angeles CA 90007")
+        expect(list.city).to eql('Los Angeles')
+        expect(list.country.to_record).to eql('_unitedStates')
+        expect(list.default_billing).to be_truthy
+        expect(list.default_shipping).to be_truthy
+        expect(list.is_residential).to be_falsey
+        expect(list.label).to eql('123 Happy Lane')
+        expect(list.override).to be_falsey
+        expect(list.state).to eql('CA')
+        expect(list.zip).to eql('90007')
+        expect(list.internal_id).to eql('567')
       end
     end
   end
@@ -91,7 +91,7 @@ describe NetSuite::Records::CustomerAddressbook do
         'listRel:zip'             => '90007',
         "listRel:internalId"      => "567"
       }
-      list.to_record.should eql(record)
+      expect(list.to_record).to eql(record)
     end
   end
 
@@ -113,12 +113,22 @@ describe NetSuite::Records::CustomerAddressbook do
     it "can be specified as the NetSuite enum value" do
       addressbook = NetSuite::Records::CustomerAddressbook.new country: "_unitedStates"
       addressbook.to_record["listRel:country"].should eql "_unitedStates"
+
+      # country with two uppercase letters (looks like ISO2)
+      addressbook = NetSuite::Records::CustomerAddressbook.new country: "_unitedKingdomGB"
+      addressbook.to_record["listRel:country"].should eql "_unitedKingdomGB"
+    end
+
+    it "can be unspecified" do
+      addressbook = NetSuite::Records::CustomerAddressbook.new country: ''
+      addressbook.country.to_record.should eql ""
+      addressbook.to_record["listRel:country"].should eql ""
     end
   end
 
   describe '#record_type' do
     it 'returns a string of the record SOAP type' do
-      list.record_type.should eql('listRel:CustomerAddressbook')
+      expect(list.record_type).to eql('listRel:CustomerAddressbook')
     end
   end
 

data/spec/netsuite/records/customer_payment_spec.rb

@@ -12,7 +12,7 @@ describe NetSuite::Records::CustomerPayment do
       :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
     ].each do |field|
-      payment.should have_field(field)
+      expect(payment).to have_field(field)
     end
   end
 
@@ -20,7 +20,7 @@ describe NetSuite::Records::CustomerPayment do
     [
       :account, :ar_acct, :credit_card, :credit_card_processor, :custom_form, :customer, :department, :klass, :location, :payment_method, :posting_period, :subsidiary
     ].each do |record_ref|
-      payment.should have_record_ref(record_ref)
+      expect(payment).to have_record_ref(record_ref)
     end
   end
 
@@ -33,14 +33,14 @@ describe NetSuite::Records::CustomerPayment do
         }
       }
       payment.custom_field_list = attributes
-      payment.custom_field_list.should be_kind_of(NetSuite::Records::CustomFieldList)
-      payment.custom_field_list.custom_fields.length.should eql(1)
+      expect(payment.custom_field_list).to be_kind_of(NetSuite::Records::CustomFieldList)
+      expect(payment.custom_field_list.custom_fields.length).to eql(1)
     end
 
     it 'can be set from a CustomFieldList object' do
       custom_field_list = NetSuite::Records::CustomFieldList.new
       payment.custom_field_list = custom_field_list
-      payment.custom_field_list.should eql(custom_field_list)
+      expect(payment.custom_field_list).to eql(custom_field_list)
     end
   end
 
@@ -64,10 +64,10 @@ 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)
+        expect(NetSuite::Actions::Get).to 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')
+        expect(payment).to be_kind_of(NetSuite::Records::CustomerPayment)
+        expect(payment.memo).to eql('This is a memo')
       end
     end
 
@@ -75,10 +75,10 @@ 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)
-        lambda {
+        expect(NetSuite::Actions::Get).to receive(:call).with([NetSuite::Records::CustomerPayment, {:external_id => 8}], {}).and_return(response)
+        expect {
           NetSuite::Records::CustomerPayment.get(:external_id => 8)
-        }.should raise_error(NetSuite::RecordNotFound,
+        }.to raise_error(NetSuite::RecordNotFound,
           /NetSuite::Records::CustomerPayment with OPTIONS=(.*) could not be found/)
       end
     end
@@ -87,9 +87,9 @@ 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)
+        expect(NetSuite::Actions::Initialize).to receive(:call).with([NetSuite::Records::CustomerPayment, invoice], {}).and_return(response)
         payment = NetSuite::Records::CustomerPayment.initialize(invoice)
-        payment.should be_kind_of(NetSuite::Records::CustomerPayment)
+        expect(payment).to be_kind_of(NetSuite::Records::CustomerPayment)
       end
     end
 
@@ -106,10 +106,10 @@ describe NetSuite::Records::CustomerPayment do
 
       it 'returns true' do
         payment = NetSuite::Records::CustomerPayment.new(test_data)
-        NetSuite::Actions::Add.should_receive(:call).
+        expect(NetSuite::Actions::Add).to receive(:call).
             with([payment], {}).
             and_return(response)
-        payment.add.should be_truthy
+        expect(payment.add).to be_truthy
       end
     end
 
@@ -118,10 +118,10 @@ describe NetSuite::Records::CustomerPayment do
 
       it 'returns false' do
         payment = NetSuite::Records::CustomerPayment.new(test_data)
-        NetSuite::Actions::Add.should_receive(:call).
+        expect(NetSuite::Actions::Add).to receive(:call).
             with([payment], {}).
             and_return(response)
-        payment.add.should be_falsey
+        expect(payment.add).to be_falsey
       end
     end
   end
@@ -131,10 +131,10 @@ describe NetSuite::Records::CustomerPayment do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
 
       it 'returns true' do
-        NetSuite::Actions::Delete.should_receive(:call).
+        expect(NetSuite::Actions::Delete).to receive(:call).
             with([payment], {}).
             and_return(response)
-        payment.delete.should be_truthy
+        expect(payment.delete).to be_truthy
       end
     end
 
@@ -142,10 +142,10 @@ describe NetSuite::Records::CustomerPayment do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'returns false' do
-        NetSuite::Actions::Delete.should_receive(:call).
+        expect(NetSuite::Actions::Delete).to receive(:call).
             with([payment], {}).
             and_return(response)
-        payment.delete.should be_falsey
+        expect(payment.delete).to be_falsey
       end
     end
   end
@@ -160,13 +160,13 @@ describe NetSuite::Records::CustomerPayment do
         'tranCust:ccName'   => 'Ryan Moran',
         'tranCust:ccNumber' => '1234567890123456'
       }
-      payment.to_record.should eql(record)
+      expect(payment.to_record).to eql(record)
     end
   end
 
   describe '#record_type' do
     it 'returns a string representation of the SOAP type' do
-      payment.record_type.should eql('tranCust:CustomerPayment')
+      expect(payment.record_type).to eql('tranCust:CustomerPayment')
     end
   end
 

data/spec/netsuite/records/customer_refund_apply_list_spec.rb

@@ -4,7 +4,7 @@ describe NetSuite::Records::CustomerRefundApplyList do
   let(:list) { NetSuite::Records::CustomerRefundApplyList.new }
 
   it 'has a applies attribute' do
-    list.applies.should be_kind_of(Array)
+    expect(list.applies).to be_kind_of(Array)
   end
 
   describe '#to_record' do
@@ -20,7 +20,7 @@ describe NetSuite::Records::CustomerRefundApplyList do
           }
         }
       ]
-      list.to_record.should eql(record)
+      expect(list.to_record).to eql(record)
     end
   end
 

data/spec/netsuite/records/customer_refund_apply_spec.rb

@@ -7,7 +7,7 @@ describe NetSuite::Records::CustomerRefundApply do
     [
       :amount, :apply, :apply_date, :currency, :doc, :due, :line, :ref_num, :total, :type
     ].each do |field|
-      apply.should have_field(field)
+      expect(apply).to have_field(field)
     end
   end
 

data/spec/netsuite/records/customer_refund_deposit_list_spec.rb

@@ -4,7 +4,7 @@ describe NetSuite::Records::CustomerRefundDepositList do
   let(:list) { NetSuite::Records::CustomerRefundDepositList.new }
 
   it 'has a deposits attribute' do
-    list.deposits.should be_kind_of(Array)
+    expect(list.deposits).to be_kind_of(Array)
   end
 
   describe '#to_record' do
@@ -18,7 +18,7 @@ describe NetSuite::Records::CustomerRefundDepositList do
           'tranCust:apply' => false
         }]
       }
-      list.to_record.should eql(record)
+      expect(list.to_record).to eql(record)
     end
   end
 

data/spec/netsuite/records/customer_refund_deposit_spec.rb

@@ -7,7 +7,7 @@ describe NetSuite::Records::CustomerRefundDeposit do
     [
       :amount, :apply, :currency, :deposit_date, :doc, :line, :ref_num, :remaining, :total
     ].each do |field|
-      deposit.should have_field(field)
+      expect(deposit).to have_field(field)
     end
   end
 

data/spec/netsuite/records/customer_refund_spec.rb

@@ -11,7 +11,7 @@ describe NetSuite::Records::CustomerRefund do
       :created_date, :currency_name, :debit_card_issue_no, :exchange_rate, :last_modified_date, :memo, :pn_ref_num, :status,
       :to_be_printed, :total, :tran_date, :tran_id, :valid_from
     ].each do |field|
-      refund.should have_field(field)
+      expect(refund).to have_field(field)
     end
   end
 
@@ -20,7 +20,7 @@ describe NetSuite::Records::CustomerRefund do
       :account, :ar_acct, :credit_card, :credit_card_processor, :custom_form, :customer, :department, :klass, :location,
       :payment_method, :posting_period, :subsidiary, :void_journal
     ].each do |record_ref|
-      refund.should have_record_ref(record_ref)
+      expect(refund).to have_record_ref(record_ref)
     end
   end
 
@@ -33,14 +33,14 @@ describe NetSuite::Records::CustomerRefund do
         }
       }
       refund.custom_field_list = attributes
-      refund.custom_field_list.should be_kind_of(NetSuite::Records::CustomFieldList)
-      refund.custom_field_list.custom_fields.length.should eql(1)
+      expect(refund.custom_field_list).to be_kind_of(NetSuite::Records::CustomFieldList)
+      expect(refund.custom_field_list.custom_fields.length).to eql(1)
     end
 
     it 'can be set from a CustomFieldList object' do
       custom_field_list = NetSuite::Records::CustomFieldList.new
       refund.custom_field_list = custom_field_list
-      refund.custom_field_list.should eql(custom_field_list)
+      expect(refund.custom_field_list).to eql(custom_field_list)
     end
   end
 
@@ -52,14 +52,14 @@ describe NetSuite::Records::CustomerRefund do
         }
       }
       refund.apply_list = attributes
-      refund.apply_list.should be_kind_of(NetSuite::Records::CustomerRefundApplyList)
-      refund.apply_list.applies.length.should eql(1)
+      expect(refund.apply_list).to be_kind_of(NetSuite::Records::CustomerRefundApplyList)
+      expect(refund.apply_list.applies.length).to eql(1)
     end
 
     it 'can be set from a CustomerRefundApplyList object' do
       apply_list = NetSuite::Records::CustomerRefundApplyList.new
       refund.apply_list = apply_list
-      refund.apply_list.should eql(apply_list)
+      expect(refund.apply_list).to eql(apply_list)
     end
   end
 
@@ -71,14 +71,14 @@ describe NetSuite::Records::CustomerRefund do
         }
       }
       refund.deposit_list = attributes
-      refund.deposit_list.should be_kind_of(NetSuite::Records::CustomerRefundDepositList)
-      refund.deposit_list.deposits.length.should eql(1)
+      expect(refund.deposit_list).to be_kind_of(NetSuite::Records::CustomerRefundDepositList)
+      expect(refund.deposit_list.deposits.length).to eql(1)
     end
 
     it 'can be set from a CustomerRefundDepositList object' do
       deposit_list = NetSuite::Records::CustomerRefundDepositList.new
       refund.deposit_list = deposit_list
-      refund.deposit_list.should eql(deposit_list)
+      expect(refund.deposit_list).to eql(deposit_list)
     end
   end
 
@@ -87,9 +87,9 @@ describe NetSuite::Records::CustomerRefund do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :is_person => true }) }
 
       it 'returns an CustomerRefund instance populated with the data from the response object' do
-        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::CustomerRefund, {:external_id => 10}], {}).and_return(response)
+        expect(NetSuite::Actions::Get).to receive(:call).with([NetSuite::Records::CustomerRefund, {:external_id => 10}], {}).and_return(response)
         refund = NetSuite::Records::CustomerRefund.get(:external_id => 10)
-        refund.should be_kind_of(NetSuite::Records::CustomerRefund)
+        expect(refund).to be_kind_of(NetSuite::Records::CustomerRefund)
       end
     end
 
@@ -97,10 +97,10 @@ describe NetSuite::Records::CustomerRefund do
       let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
 
       it 'raises a RecordNotFound exception' do
-        NetSuite::Actions::Get.should_receive(:call).with([NetSuite::Records::CustomerRefund, {:external_id => 10}], {}).and_return(response)
-        lambda {
+        expect(NetSuite::Actions::Get).to receive(:call).with([NetSuite::Records::CustomerRefund, {:external_id => 10}], {}).and_return(response)
+        expect {
           NetSuite::Records::CustomerRefund.get(:external_id => 10)
-        }.should raise_error(NetSuite::RecordNotFound,
+        }.to raise_error(NetSuite::RecordNotFound,
           /NetSuite::Records::CustomerRefund with OPTIONS=(.*) could not be found/)
       end
     end
@@ -109,9 +109,9 @@ describe NetSuite::Records::CustomerRefund 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::CustomerRefund, memo], {}).and_return(response)
+        expect(NetSuite::Actions::Initialize).to receive(:call).with([NetSuite::Records::CustomerRefund, memo], {}).and_return(response)
         refund = NetSuite::Records::CustomerRefund.initialize(memo)
-        refund.should be_kind_of(NetSuite::Records::CustomerRefund)
+        expect(refund).to be_kind_of(NetSuite::Records::CustomerRefund)
       end
     end
 
@@ -128,10 +128,10 @@ describe NetSuite::Records::CustomerRefund do
 
       it 'returns true' do
         refund = NetSuite::Records::CustomerRefund.new(test_data)
-        NetSuite::Actions::Add.should_receive(:call).
+        expect(NetSuite::Actions::Add).to receive(:call).
             with([refund], {}).
             and_return(response)
-        refund.add.should be_truthy
+        expect(refund.add).to be_truthy
       end
     end
 
@@ -140,10 +140,10 @@ describe NetSuite::Records::CustomerRefund do
 
       it 'returns false' do
         refund = NetSuite::Records::CustomerRefund.new(test_data)
-        NetSuite::Actions::Add.should_receive(:call).
+        expect(NetSuite::Actions::Add).to receive(:call).
             with([refund], {}).
             and_return(response)
-        refund.add.should be_falsey
+        expect(refund.add).to be_falsey
       end
     end
   end
@@ -154,10 +154,10 @@ describe NetSuite::Records::CustomerRefund do
 
       it 'returns true' do
         refund = NetSuite::Records::CustomerRefund.new
-        NetSuite::Actions::Delete.should_receive(:call).
+        expect(NetSuite::Actions::Delete).to receive(:call).
             with([refund], {}).
             and_return(response)
-        refund.delete.should be_truthy
+        expect(refund.delete).to be_truthy
       end
     end
 
@@ -166,10 +166,10 @@ describe NetSuite::Records::CustomerRefund do
 
       it 'returns false' do
         refund = NetSuite::Records::CustomerRefund.new
-        NetSuite::Actions::Delete.should_receive(:call).
+        expect(NetSuite::Actions::Delete).to receive(:call).
             with([refund], {}).
             and_return(response)
-        refund.delete.should be_falsey
+        expect(refund.delete).to be_falsey
       end
     end
   end
@@ -184,13 +184,13 @@ describe NetSuite::Records::CustomerRefund do
         'tranCust:memo'      => 'This is a memo',
         'tranCust:ccZipCode' => '10101'
       }
-      refund.to_record.should eql(record)
+      expect(refund.to_record).to eql(record)
     end
   end
 
   describe '#record_type' do
     it 'returns a string representation of the SOAP type' do
-      refund.record_type.should eql('tranCust:CustomerRefund')
+      expect(refund.record_type).to eql('tranCust:CustomerRefund')
     end
   end