vero 0.7.0 → 0.10.0

data/spec/lib/context_spec.rb

@@ -1,57 +1,59 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
-describe Vero::Context do 
+describe Vero::Context do
   let(:context) { Vero::Context.new }
 
-  it "accepts multiple parameter types in contructor" do
-    context1 = Vero::Context.new({ :api_key => 'blah', :secret => 'didah' })
-    context1.should be_a(Vero::Context)
-    context1.config.api_key.should == 'blah'
-    context1.config.secret.should == 'didah'
+  it 'accepts multiple parameter types in contructor' do
+    context1 = Vero::Context.new({ api_key: 'blah', secret: 'didah' })
+    expect(context1).to be_a(Vero::Context)
+    expect(context1.config.api_key).to eq('blah')
+    expect(context1.config.secret).to eq('didah')
 
     context2 = Vero::Context.new(context1)
-    context2.should be_a(Vero::Context)
-    context2.should_not be(context1)
-    context2.config.api_key.should == 'blah'
-    context2.config.secret.should == 'didah'
+    expect(context2).to be_a(Vero::Context)
+    expect(context2).not_to be(context1)
+    expect(context2.config.api_key).to eq('blah')
+    expect(context2.config.secret).to eq('didah')
 
     context3 = Vero::Context.new VeroUser.new('api_key', 'secret')
-    context3.should be_a(Vero::Context)
-    context3.config.api_key.should == 'api_key'
-    context3.config.secret.should == 'secret'
+    expect(context3).to be_a(Vero::Context)
+    expect(context3.config.api_key).to eq('api_key')
+    expect(context3.config.secret).to eq('secret')
   end
 
   describe :configure do
-    it "should ignore configuring the config if no block is provided" do
+    it 'should ignore configuring the config if no block is provided' do
       context.configure
-      context.configured?.should be_false
+      expect(context.configured?).to be(false)
     end
 
-    it "should pass configuration defined in the block to the config file" do
+    it 'should pass configuration defined in the block to the config file' do
       context.configure do |c|
-        c.api_key = "abcd1234"
+        c.api_key = 'abcd1234'
       end
-      context.config.api_key.should == "abcd1234"
+      expect(context.config.api_key).to eq('abcd1234')
     end
 
-    it "should init should be able to set async" do
+    it 'should init should be able to set async' do
       context.configure do |c|
         c.async = false
       end
-      context.config.async.should be_false
+      expect(context.config.async).to be(false)
 
       context.configure do |c|
         c.async = true
       end
-      context.config.async.should be_true
+      expect(context.config.async).to be(true)
     end
   end
 
   describe :disable_requests! do
-    it "should change config.disabled" do
-      context.config.disabled.should be_false
+    it 'should change config.disabled' do
+      expect(context.config.disabled).to be(false)
       context.disable_requests!
-      context.config.disabled.should be_true            
+      expect(context.config.disabled).to be(true)
     end
   end
-end
+end

data/spec/lib/dsl_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Vero::DSL do
@@ -18,8 +20,8 @@ describe Vero::DSL::Proxy do
       expect(proxy.users).to eql(Vero::Api::Users)
     end
 
-    it "should respond to reidentify!" do
-      expect(proxy.users.respond_to?(:reidentify!)).to be_true
+    it 'should respond to reidentify!' do
+      expect(proxy.users.respond_to?(:reidentify!)).to be(true)
     end
   end
 

data/spec/lib/sender_spec.rb

@@ -1,33 +1,22 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Vero::Sender do
   subject { Vero::Sender }
 
-  describe ".senders" do
-    it "should be a Hash" do
-      subject.senders.should be_a(Hash)
-    end
-
-    context 'when using Ruby with verion greater than 1.8.7' do
-      before do
-        stub_const('RUBY_VERSION', '1.9.3')
-      end
-
-      it "should have a default set of senders (true, false, none, thread)" do
-        subject.senders.should == {
-          true          => Vero::Senders::Thread,
-          false         => Vero::Senders::Base,
-          :none         => Vero::Senders::Base,
-          :thread       => Vero::Senders::Thread,
-        }
-      end
+  describe '.senders' do
+    it 'should automatically find senders that are not defined' do
+      expect(subject.senders[:delayed_job]).to eq(Vero::Senders::DelayedJob)
+      expect(subject.senders[:sucker_punch]).to eq(Vero::Senders::SuckerPunch)
+      expect(subject.senders[:resque]).to eq(Vero::Senders::Resque)
+      expect(subject.senders[:sidekiq]).to eq(Vero::Senders::Sidekiq)
+      expect(subject.senders[:invalid]).to eq(Vero::Senders::Invalid)
+      expect(subject.senders[:none]).to eq(Vero::Senders::Base)
     end
 
-    it "should automatically find senders that are not defined" do
-      subject.senders[:delayed_job].should  == Vero::Senders::DelayedJob
-      subject.senders[:resque].should       == Vero::Senders::Resque
-      subject.senders[:invalid].should      == Vero::Senders::Invalid
-      subject.senders[:none].should         == Vero::Senders::Base
+    it 'should fallback to Vero::Senders::Base' do
+      expect(subject.senders[:unsupported_sender]).to eq(Vero::Senders::Base)
     end
   end
 end

data/spec/lib/senders/sidekiq_spec.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Vero::Senders::Sidekiq do
+  subject { Vero::Senders::Sidekiq.new }
+  describe :call do
+    it 'should perform_async a Vero::SidekiqWorker' do
+      expect(Vero::SidekiqWorker).to(
+        receive(:perform_async)
+        .with('Vero::Api::Workers::Events::TrackAPI', 'abc', { test: 'abc' })
+        .once
+      )
+      subject.call(Vero::Api::Workers::Events::TrackAPI, 'abc', { test: 'abc' })
+    end
+  end
+end
+
+describe Vero::SidekiqWorker do
+  subject { Vero::SidekiqWorker.new }
+  describe :perform do
+    it 'should call the api method' do
+      mock_api = double(Vero::Api::Workers::Events::TrackAPI)
+      expect(mock_api).to receive(:perform).once
+
+      allow(Vero::Api::Workers::Events::TrackAPI).to receive(:new).and_return(mock_api)
+      expect(Vero::Api::Workers::Events::TrackAPI).to receive(:new).with('abc', { test: 'abc' }).once
+
+      subject.perform('Vero::Api::Workers::Events::TrackAPI', 'abc', { test: 'abc' })
+    end
+  end
+end

data/spec/lib/trackable_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 def vero_context(user, logging = true, async = false, disabled = true)
@@ -13,29 +15,29 @@ end
 describe Vero::Trackable do
   before :each do
     @request_params = {
-      :event_name => 'test_event',
-      :auth_token => 'YWJjZDEyMzQ6ZWZnaDU2Nzg=',
-      :identity => {:email => 'durkster@gmail.com', :age => 20, :_user_type => "User"},
-      :data => { :test => 1 },
-      :development_mode => true
+      event_name: 'test_event',
+      auth_token: 'YWJjZDEyMzQ6ZWZnaDU2Nzg=',
+      identity: { email: 'durkster@gmail.com', age: 20, _user_type: 'User' },
+      data: { test: 1 },
+      development_mode: true
     }
-    @url = "https://api.getvero.com/api/v1/track.json"
+    @url = 'https://api.getvero.com/api/v1/track.json'
     @user = User.new
 
-    @content_type_params = {:content_type => :json, :accept => :json}
+    @content_type_params = { content_type: :json, accept: :json }
   end
 
-  context "the gem has not been configured" do
+  context 'the gem has not been configured' do
     before { Vero::App.reset! }
-    it "should raise an error when API requests are made" do
-      expect { @user.track(@request_params[:event_name], @request_params[:data]) }.to raise_error
+    it 'should raise an error when API requests are made' do
+      expect { @user.track(@request_params[:event_name], @request_params[:data]) }.to raise_error(RuntimeError)
 
-      @user.stub(:post_later).and_return('success')
-      expect { @user.identity! }.to raise_error
+      allow(@user).to receive(:post_later).and_return('success')
+      expect { @user.identity! }.to raise_error(NoMethodError)
     end
   end
 
-  context "the gem has been configured" do
+  context 'the gem has been configured' do
     before do
       Vero::App.init do |c|
         c.api_key = 'abcd1234'
@@ -47,48 +49,44 @@ describe Vero::Trackable do
     describe :track! do
       before do
         @request_params = {
-          :event_name => 'test_event',
-          :identity => {:email => 'durkster@gmail.com', :age => 20, :_user_type => "User"},
-          :data => { :test => 1 },
-          :extras => {}
+          event_name: 'test_event',
+          identity: { email: 'durkster@gmail.com', age: 20, _user_type: 'User' },
+          data: { test: 1 },
+          extras: {}
         }
-        @url = "https://api.getvero.com/api/v1/track.json"
+        @url = 'https://api.getvero.com/api/v1/track.json'
       end
 
-      it "should not send a track request when the required parameters are invalid" do
-        expect { @user.track!(nil) }.to raise_error
-        expect { @user.track!('') }.to raise_error
-        expect { @user.track!('test', '') }.to raise_error
+      it 'should not send a track request when the required parameters are invalid' do
+        expect { @user.track!(nil) }.to raise_error(ArgumentError)
+        expect { @user.track!('') }.to raise_error(ArgumentError)
+        expect { @user.track!('test', '') }.to raise_error(ArgumentError)
       end
 
-      it "should send a `track!` request when async is set to false" do
+      it 'should send a `track!` request when async is set to false' do
         context = vero_context(@user)
-        @user.stub(:with_vero_context).and_return(context)
+        allow(@user).to receive(:with_vero_context).and_return(context)
 
-        RestClient.stub(:post).and_return(200)
+        allow(RestClient).to receive(:post).and_return(200)
 
-        Vero::Api::Events.stub(:track!).and_return(200)
-        Vero::Api::Events.should_receive(:track!).with(@request_params, context)
-        @user.track!(@request_params[:event_name], @request_params[:data]).should == 200
+        allow(Vero::Api::Events).to receive(:track!).and_return(200)
+        expect(Vero::Api::Events).to receive(:track!).with(@request_params, context)
+        expect(@user.track!(@request_params[:event_name], @request_params[:data])).to eq(200)
 
-        Vero::Api::Events.stub(:track!).and_return(200)
-        Vero::Api::Events.should_receive(:track!).with(@request_params.merge(:data => {}), context)
-        @user.track!(@request_params[:event_name]).should == 200
+        allow(Vero::Api::Events).to receive(:track!).and_return(200)
+        expect(Vero::Api::Events).to receive(:track!).with(@request_params.merge(data: {}), context)
+        expect(@user.track!(@request_params[:event_name])).to eq(200)
       end
 
       context 'when set to be async' do
         before do
           @context = vero_context(@user, true, true)
-          @user.stub(:with_vero_context).and_return(@context)
+          allow(@user).to receive(:with_vero_context).and_return(@context)
         end
 
-        context 'not using Ruby 1.8.7' do
-          before { stub_const('RUBY_VERSION', '1.9.3') }
-
-          it 'sends' do
-            @user.track!(@request_params[:event_name], @request_params[:data]).should be_nil
-            @user.track!(@request_params[:event_name]).should be_nil
-          end
+        it 'sends' do
+          expect(@user.track!(@request_params[:event_name], @request_params[:data])).to be_nil
+          expect(@user.track!(@request_params[:event_name])).to be_nil
         end
       end
     end
@@ -96,35 +94,31 @@ describe Vero::Trackable do
     describe :identify! do
       before do
         @request_params = {
-          :id => nil,
-          :email => 'durkster@gmail.com',
-          :data => {:email => 'durkster@gmail.com', :age => 20, :_user_type => "User"}
+          id: nil,
+          email: 'durkster@gmail.com',
+          data: { email: 'durkster@gmail.com', age: 20, _user_type: 'User' }
         }
-        @url = "https://api.getvero.com/api/v2/users/track.json"
+        @url = 'https://api.getvero.com/api/v2/users/track.json'
       end
 
-      it "should send an `identify` request when async is set to false" do
+      it 'should send an `identify` request when async is set to false' do
         context = vero_context(@user)
-        @user.stub(:with_vero_context).and_return(context)
+        allow(@user).to receive(:with_vero_context).and_return(context)
 
-        Vero::Api::Users.stub(:track!).and_return(200)
-        Vero::Api::Users.should_receive(:track!).with(@request_params, context)
+        allow(Vero::Api::Users).to receive(:track!).and_return(200)
+        expect(Vero::Api::Users).to receive(:track!).with(@request_params, context)
 
-        @user.identify!.should == 200
+        expect(@user.identify!).to eq(200)
       end
 
       context 'when set to use async' do
         before do
           @context = vero_context(@user, false, true)
-          @user.stub(:with_vero_context).and_return(@context)
+          allow(@user).to receive(:with_vero_context).and_return(@context)
         end
 
-        context 'and not using Ruby 1.8.7' do
-          before { stub_const('RUBY_VERSION', '1.9.3') }
-
-          it 'sends' do
-            @user.identify!.should be_nil
-          end
+        it 'sends' do
+          expect(@user.identify!).to be_nil
         end
       end
     end
@@ -132,46 +126,33 @@ describe Vero::Trackable do
     describe :update_user! do
       before do
         @request_params = {
-          :id => nil,
-          :email => 'durkster@gmail.com',
-          :changes => {:email => 'durkster@gmail.com', :age => 20, :_user_type => "User"},
+          id: nil,
+          email: 'durkster@gmail.com',
+          changes: { email: 'durkster@gmail.com', age: 20, _user_type: 'User' }
         }
-        @url = "https://api.getvero.com/api/v2/users/edit.json"
+        @url = 'https://api.getvero.com/api/v2/users/edit.json'
       end
 
-      it "should send an `update_user` request when async is set to false" do
+      it 'should send an `update_user` request when async is set to false' do
         context = Vero::Context.new(Vero::App.default_context)
         context.subject = @user
 
-        @user.stub(:with_vero_context).and_return(context)
+        allow(@user).to receive(:with_vero_context).and_return(context)
 
-        Vero::Api::Users.stub(:edit_user!).and_return(200)
-        Vero::Api::Users.should_receive(:edit_user!).with(@request_params, context)
+        allow(Vero::Api::Users).to receive(:edit_user!).and_return(200)
+        expect(Vero::Api::Users).to receive(:edit_user!).with(@request_params, context)
 
-        @user.with_vero_context.update_user!.should == 200
+        expect(@user.with_vero_context.update_user!).to eq(200)
       end
 
       context 'when set to use async' do
         before do
           @context = vero_context(@user, false, true)
-          @user.stub(:with_vero_context).and_return(@context)
+          allow(@user).to receive(:with_vero_context).and_return(@context)
         end
 
-        context 'and using Ruby 1.8.7' do
-          before { stub_const('RUBY_VERSION', '1.8.7') }
-
-          it 'raises an error' do
-            @context.config.disabled = false
-            expect { @user.with_vero_context.update_user! }.to raise_error
-          end
-        end
-
-        context 'and not using Ruby 1.8.7' do
-          before { stub_const('RUBY_VERSION', '1.9.3') }
-
-          it 'sends' do
-            @user.with_vero_context.update_user!.should be_nil
-          end
+        it 'sends' do
+          expect(@user.with_vero_context.update_user!).to be_nil
         end
       end
     end
@@ -179,74 +160,68 @@ describe Vero::Trackable do
     describe :update_user_tags! do
       before do
         @request_params = {
-          :id => nil,
-          :email => 'durkster@gmail.com',
-          :add => [],
-          :remove => []
+          id: nil,
+          email: 'durkster@gmail.com',
+          add: [],
+          remove: []
         }
-        @url = "https://api.getvero.com/api/v2/users/tags/edit.json"
+        @url = 'https://api.getvero.com/api/v2/users/tags/edit.json'
       end
 
-      it "should send an `update_user_tags` request when async is set to false" do
+      it 'should send an `update_user_tags` request when async is set to false' do
         context = Vero::Context.new(Vero::App.default_context)
         context.subject = @user
         context.config.async = false
 
-        @user.stub(:with_vero_context).and_return(context)
+        allow(@user).to receive(:with_vero_context).and_return(context)
 
-        Vero::Api::Users.stub(:edit_user_tags!).and_return(200)
-        Vero::Api::Users.should_receive(:edit_user_tags!).with(@request_params, context)
+        allow(Vero::Api::Users).to receive(:edit_user_tags!).and_return(200)
+        expect(Vero::Api::Users).to receive(:edit_user_tags!).with(@request_params, context)
 
-        @user.with_vero_context.update_user_tags!.should == 200
+        expect(@user.with_vero_context.update_user_tags!).to eq(200)
       end
 
-      if RUBY_VERSION =~ /1\.9\./
-        it "should send using another thread when async is set to true" do
-          context = Vero::Context.new(Vero::App.default_context)
-          context.subject = @user
-          context.config.async = true
+      it 'should send using another thread when async is set to true' do
+        context = Vero::Context.new(Vero::App.default_context)
+        context.subject = @user
+        context.config.async = true
 
-          @user.stub(:with_vero_context).and_return(context)
+        allow(@user).to receive(:with_vero_context).and_return(context)
 
-          @user.with_vero_context.update_user_tags!.should be_nil
-        end
+        expect(@user.with_vero_context.update_user_tags!).to be_nil
       end
     end
 
     describe :unsubscribe! do
       before do
         @request_params = {
-          :id => nil,
-          :email => 'durkster@gmail.com'
+          id: nil,
+          email: 'durkster@gmail.com'
         }
-        @url = "https://api.getvero.com/api/v2/users/unsubscribe.json"
+        @url = 'https://api.getvero.com/api/v2/users/unsubscribe.json'
       end
 
-      it "should send an `update_user` request when async is set to false" do
+      it 'should send an `update_user` request when async is set to false' do
         context = Vero::Context.new(Vero::App.default_context)
         context.subject = @user
         context.config.async = false
 
-        @user.stub(:with_vero_context).and_return(context)
+        allow(@user).to receive(:with_vero_context).and_return(context)
 
-        Vero::Api::Users.stub(:unsubscribe!).and_return(200)
-        Vero::Api::Users.should_receive(:unsubscribe!).with(@request_params, context)
+        allow(Vero::Api::Users).to receive(:unsubscribe!).and_return(200)
+        expect(Vero::Api::Users).to receive(:unsubscribe!).with(@request_params, context)
 
-        @user.with_vero_context.unsubscribe!.should == 200
+        expect(@user.with_vero_context.unsubscribe!).to eq(200)
       end
 
       context 'when using async' do
         before do
           @context = vero_context(@user, false, true)
-          @user.stub(:with_vero_context).and_return(@context)
+          allow(@user).to receive(:with_vero_context).and_return(@context)
         end
 
-        context 'and using Ruby 1.9.3' do
-          before { stub_const('RUBY_VERSION', '1.9.3') }
-
-          it 'sends' do
-            @user.with_vero_context.unsubscribe!.should be_nil
-          end
+        it 'sends' do
+          expect(@user.with_vero_context.unsubscribe!).to be_nil
         end
       end
     end
@@ -254,20 +229,20 @@ describe Vero::Trackable do
     describe :trackable do
       before { User.reset_trackable_map! }
 
-      it "should build an array of trackable params" do
+      it 'should build an array of trackable params' do
         User.trackable :email, :age
-        User.trackable_map.should == [:email, :age]
+        expect(User.trackable_map).to eq(%i[email age])
       end
 
-      it "should append new trackable items to an existing trackable map" do
+      it 'should append new trackable items to an existing trackable map' do
         User.trackable :email, :age
         User.trackable :hair_colour
-        User.trackable_map.should == [:email, :age, :hair_colour]
+        expect(User.trackable_map).to eq(%i[email age hair_colour])
       end
 
       it "should append an extra's hash to the trackable map" do
-        User.trackable :email, {:extras => :properties}
-        User.trackable_map.should == [:email, {:extras => :properties}]
+        User.trackable :email, { extras: :properties }
+        expect(User.trackable_map).to eq([:email, { extras: :properties }])
       end
     end
 
@@ -277,79 +252,78 @@ describe Vero::Trackable do
         User.trackable :email, :age
       end
 
-      it "should return a hash of all values mapped by trackable" do
+      it 'should return a hash of all values mapped by trackable' do
         user = User.new
-        user.to_vero.should == {:email => 'durkster@gmail.com', :age => 20, :_user_type => "User"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', age: 20, _user_type: 'User' })
 
         user = UserWithoutEmail.new
-        user.to_vero.should == {:email => 'durkster@gmail.com', :age => 20, :_user_type => "UserWithoutEmail"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', age: 20, _user_type: 'UserWithoutEmail' })
 
         user = UserWithEmailAddress.new
-        user.to_vero.should == {:email => 'durkster@gmail.com', :age => 20, :_user_type => "UserWithEmailAddress"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', age: 20, _user_type: 'UserWithEmailAddress' })
 
         user = UserWithoutInterface.new
-        user.to_vero.should == {:email => 'durkster@gmail.com', :age => 20, :_user_type => "UserWithoutInterface"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', age: 20, _user_type: 'UserWithoutInterface' })
 
         user = UserWithNilAttributes.new
-        user.to_vero.should == {:email => 'durkster@gmail.com', :_user_type => "UserWithNilAttributes"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', _user_type: 'UserWithNilAttributes' })
       end
 
-      it "should take into account any defined extras" do
+      it 'should take into account any defined extras' do
         user = UserWithExtras.new
         user.properties = nil
-        user.to_vero.should == {:email => 'durkster@gmail.com', :_user_type => "UserWithExtras"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', _user_type: 'UserWithExtras' })
 
-        user.properties = "test"
-        user.to_vero.should == {:email => 'durkster@gmail.com', :_user_type => "UserWithExtras"}
+        user.properties = 'test'
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', _user_type: 'UserWithExtras' })
 
         user.properties = {}
-        user.to_vero.should == {:email => 'durkster@gmail.com', :_user_type => "UserWithExtras"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', _user_type: 'UserWithExtras' })
 
         user.properties = {
-          :age => 20,
-          :gender => "female"
+          age: 20,
+          gender: 'female'
         }
-        user.to_vero.should == {:email => 'durkster@gmail.com', :age => 20, :gender => "female", :_user_type => "UserWithExtras"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', age: 20, gender: 'female', _user_type: 'UserWithExtras' })
 
         user = UserWithPrivateExtras.new
-        user.to_vero.should == {:email => 'durkster@gmail.com', :age => 26, :_user_type => "UserWithPrivateExtras"}
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', age: 26, _user_type: 'UserWithPrivateExtras' })
       end
 
-      it "should allow extras to be provided instead :id or :email" do
+      it 'should allow extras to be provided instead :id or :email' do
         user = UserWithOnlyExtras.new
-        user.properties = {:email => user.email}
-        user.to_vero.should == {:email => 'durkster@gmail.com', :_user_type => "UserWithOnlyExtras"}
+        user.properties = { email: user.email }
+        expect(user.to_vero).to eq({ email: 'durkster@gmail.com', _user_type: 'UserWithOnlyExtras' })
       end
     end
 
     describe :with_vero_context do
-      it "should be able to change contexts" do
+      it 'should be able to change contexts' do
         user = User.new
-        user.with_default_vero_context.config.config_params.should == {:api_key=>"abcd1234", :secret=>"efgh5678"}
-        user.with_vero_context({:api_key => "boom", :secret => "tish"}).config.config_params.should == {:api_key=>"boom", :secret=>"tish"}
+        expect(user.with_default_vero_context.config.config_params).to eq({ api_key: 'abcd1234', secret: 'efgh5678' })
+        expect(user.with_vero_context({ api_key: 'boom', secret: 'tish' }).config.config_params).to eq({ api_key: 'boom', secret: 'tish' })
       end
     end
 
-    it "should work when Vero::Trackable::Interface is not included" do
+    it 'should work when Vero::Trackable::Interface is not included' do
       user = UserWithoutInterface.new
 
       request_params = {
-        :event_name => 'test_event',
-        :auth_token => 'YWJjZDEyMzQ6ZWZnaDU2Nzg=',
-        :identity => {:email => 'durkster@gmail.com', :age => 20, :_user_type => "UserWithoutInterface"},
-        :data => { :test => 1 },
-        :development_mode => true
+        event_name: 'test_event',
+        auth_token: 'YWJjZDEyMzQ6ZWZnaDU2Nzg=',
+        identity: { email: 'durkster@gmail.com', age: 20, _user_type: 'UserWithoutInterface' },
+        data: { test: 1 },
+        development_mode: true
       }
-      url = "https://api.getvero.com/api/v1/track.json"
 
       context = Vero::Context.new(Vero::App.default_context)
       context.subject = user
-      context.stub(:post_now).and_return(200)
+      allow(context).to receive(:post_now).and_return(200)
 
-      user.stub(:with_vero_context).and_return(context)
+      allow(user).to receive(:with_vero_context).and_return(context)
 
-      RestClient.stub(:post).and_return(200)
-      user.vero_track(request_params[:event_name], request_params[:data]).should == 200
+      allow(RestClient).to receive(:post).and_return(200)
+      expect(user.vero_track(request_params[:event_name], request_params[:data])).to eq(200)
     end
   end
 end