Merge pull request #123 from PChambino/ntlm

Don't warn about missing rubyntlm gem if it is optional

lib/httpi/adapter/net_http.rb

@@ -11,8 +11,7 @@
   unless Net::NTLM::VERSION::STRING >= '0.3.2'
     raise ArgumentError, 'Invalid version of rubyntlm. Please use v0.3.2+.'
   end
-rescue LoadError => e
-  HTTPI.logger.debug('Net::NTLM is not available. Install via gem install rubyntlm.')
+rescue LoadError
 end
 
 module HTTPI
@@ -86,7 +85,7 @@ def setup
 
       def negotiate_ntlm_auth(http, &requester)
         unless Net.const_defined?(:NTLM)
-          HTTPI.logger.fatal('Cannot negotiate ntlm auth if net/ntlm is not present. Perhaps the net/ntlm gem is not installed?')
+          raise NotSupportedError, 'Net::NTLM is not available. Install via gem install rubyntlm.'
         end
 
         # first figure out if we should use NTLM or Negotiate

spec/httpi/adapter/net_http_spec.rb

@@ -76,15 +76,14 @@
       expect(response.body).to eq("ntlm-auth")
     end
 
-    it 'fatal logs when net/ntlm is not available, but ntlm authentication was requested' do
+    it 'does not support ntlm authentication when Net::NTLM is not available' do
       Net.expects(:const_defined?).with(:NTLM).returns false
 
       request = HTTPI::Request.new(@server.url + 'ntlm-auth')
       request.auth.ntlm("testing", "failures")
-      HTTPI.logger.expects(:fatal)
 
-      response = HTTPI.get(request, adapter)
-      expect(response.body).to eq("ntlm-auth")
+      expect { HTTPI.get(request, adapter) }.
+        to raise_error(HTTPI::NotSupportedError, /Net::NTLM is not available/)
     end
   end