spec/netsuite/configuration_spec.rb in netsuite-0.3.0 vs spec/netsuite/configuration_spec.rb in netsuite-0.3.1

@@ -8,49 +8,49 @@
   end
 
   describe '#reset!' do
     it 'clears the attributes hash' do
       config.attributes[:blah] = 'something'
-      config.attributes.should_not be_empty
+      expect(config.attributes).not_to be_empty
       config.reset!
-      config.attributes.should be_empty
+      expect(config.attributes).to be_empty
     end
   end
 
   describe '#connection' do
     it 'returns a Savon::Client object that allows requests to the service' do
       # reset clears out the password info
       config.email '[e-mail]'
       config.password '[e-mail]'
       config.account 1023
 
-      config.connection.should be_kind_of(Savon::Client)
+      expect(config.connection).to be_kind_of(Savon::Client)
     end
   end
 
   describe '#wsdl' do
     context 'when the wsdl has been set' do
       before do
         config.wsdl = 'https://system.sandbox.netsuite.com/wsdl/v2011_2_0/netsuite.wsdl'
       end
 
       it 'returns a path to the WSDL to use for the API' do
-        config.wsdl.should eql('https://system.sandbox.netsuite.com/wsdl/v2011_2_0/netsuite.wsdl')
+        expect(config.wsdl).to eql('https://system.sandbox.netsuite.com/wsdl/v2011_2_0/netsuite.wsdl')
       end
     end
 
     context 'when the wsdl has not been set' do
       it 'returns a path to the WSDL to use for the API' do
-        config.wsdl.should match(/.*\/netsuite\/wsdl\/2011_2\.wsdl/)
+        expect(config.wsdl).to match(/.*\/netsuite\/wsdl\/2011_2\.wsdl/)
       end
     end
 
     context 'when the wsdl has not been set, but the API has been set' do
       it 'should correctly return the full HTTP sandbox URL' do
         config.api_version '2013_1'
         config.sandbox false
-        config.wsdl.should eql('https://webservices.netsuite.com/wsdl/v2013_1_0/netsuite.wsdl')
+        expect(config.wsdl).to eql('https://webservices.netsuite.com/wsdl/v2013_1_0/netsuite.wsdl')
       end
     end
   end
 
   describe '#auth_header' do
@@ -59,11 +59,11 @@
       config.password = 'myPassword'
       config.account  = 1234
     end
 
     it 'returns a hash representation of the authentication header' do
-      config.auth_header.should eql({
+      expect(config.auth_header).to eql({
         'platformMsgs:passport' => {
           'platformCore:email'    => '[e-mail]',
           'platformCore:password' => 'myPassword',
           'platformCore:account'  => '1234',
           'platformCore:role'     => { :@internalId => '3' },
@@ -83,11 +83,11 @@
       config.soap_header = {
         'platformMsgs:preferences' => {
           'platformMsgs:ignoreReadOnlyFields' => true,
         }
       }
-      config.soap_header.should eql({
+      expect(config.soap_header).to eql({
         'platformMsgs:preferences' => {
           'platformMsgs:ignoreReadOnlyFields' => true,
         }
       })
     end
@@ -98,19 +98,19 @@
       before do
         config.email = '[e-mail]'
       end
 
       it 'returns the email' do
-        config.email.should eql('[e-mail]')
+        expect(config.email).to eql('[e-mail]')
       end
     end
 
     context 'when the email has not been set' do
       it 'raises a ConfigurationError' do
-        lambda {
+        expect {
           config.email
-        }.should raise_error(NetSuite::ConfigurationError,
+        }.to raise_error(NetSuite::ConfigurationError,
           '#email is a required configuration value. Please set it by calling NetSuite::Configuration.email = "[e-mail]"')
       end
     end
   end
 
@@ -119,19 +119,19 @@
       before do
         config.password = 'password'
       end
 
       it 'returns the password' do
-        config.password.should eql('password')
+        expect(config.password).to eql('password')
       end
     end
 
     context 'when the password has not been set' do
       it 'raises a ConfigurationError' do
-        lambda {
+        expect {
           config.password
-        }.should raise_error(NetSuite::ConfigurationError,
+        }.to raise_error(NetSuite::ConfigurationError,
           '#password is a required configuration value. Please set it by calling NetSuite::Configuration.password = "my_pass"')
       end
     end
   end
 
@@ -140,52 +140,52 @@
       before do
         config.account = 4321
       end
 
       it 'returns the account' do
-        config.account.should eql(4321)
+        expect(config.account).to eql(4321)
       end
     end
 
     context 'when the account has not been set' do
       it 'raises a ConfigurationError' do
-        lambda {
+        expect {
           config.account
-        }.should raise_error(NetSuite::ConfigurationError,
+        }.to raise_error(NetSuite::ConfigurationError,
           '#account is a required configuration value. Please set it by calling NetSuite::Configuration.account = 1234')
       end
     end
   end
 
 
   describe '#role' do
     context 'when no role is defined' do
       it 'defaults to "3"' do
-        config.role.should == "3"
+        expect(config.role).to eq("3")
       end
     end
   end
 
   describe '#role=' do
     it 'sets the role according to the input value' do
       config.role = "6"
-      config.role.should == "6"
+      expect(config.role).to eq("6")
     end
   end
 
   describe '#api_version' do
     context 'when no api_version is defined' do
       it 'defaults to 2011_2' do
-        config.api_version.should == '2011_2'
+        expect(config.api_version).to eq('2011_2')
       end
     end
   end
 
   describe '#api_version=' do
     context 'when api version is defined' do
       it 'sets the api_version of the application' do
         config.api_version = '2012_1'
-        config.api_version.should == '2012_1'
+        expect(config.api_version).to eq('2012_1')
       end
     end
   end
 
 end