spec/minitel/client_spec.rb in minitel-0.1.0 vs spec/minitel/client_spec.rb in minitel-0.1.1

@@ -1,20 +1,30 @@
 require 'spec_helper'
 
 describe Minitel::Client, '#initialize' do
-  it 'requires an https url' do
-    expect{ Minitel::Client.new('http://user:[e-mail]')  }.to     raise_error(ArgumentError)
-    expect{ Minitel::Client.new('https://user:[e-mail]') }.to_not raise_error
+  before do
+    url = "https://u:[e-mail]"
+    client = Minitel::Client.new(url)
+    @data = client.connection.data
   end
 
   it 'uses the given url and credentials' do
-    url = "https://u:[e-mail]"
-    mock = double(:excon)
-    expect(Excon).to receive(:new).with(url).and_return(mock)
+    expect(@data[:host]).to     eq('foo.com')
+    expect(@data[:user]).to     eq('u')
+    expect(@data[:password]).to eq('p')
+  end
 
-    client = Minitel::Client.new(url)
+  it 'includes the minitel and excon versions in the user agent' do
+    user_agent = @data[:headers]['User-Agent']
+    expect(user_agent).to match('minitel')
+    expect(user_agent).to match(Minitel::VERSION)
+    expect(user_agent).to match('excon')
+    expect(user_agent).to match(Excon::VERSION)
+  end
 
-    expect(client.connection).to eq(mock)
+  it 'requires an https url' do
+    expect{ Minitel::Client.new('http://user:[e-mail]')  }.to     raise_error(ArgumentError)
+    expect{ Minitel::Client.new('https://user:[e-mail]') }.to_not raise_error
   end
 end
 
 describe Minitel::Client, '#notify_app' do
   describe 'arguments' do