castle-rb 6.0.1 → 7.0.0

data/spec/lib/castle/api/risk_spec.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+describe Castle::API::Risk do
+  pending
+end

data/spec/lib/castle/api/start_impersonation_spec.rb

@@ -22,9 +22,9 @@ describe Castle::API::StartImpersonation do
   before do
     Timecop.freeze(time_now)
     stub_const('Castle::VERSION', '2.2.0')
-    stub_request(:any, /api.castle.io/).with(
-      basic_auth: ['', 'secret']
-    ).to_return(status: 200, body: response_body, headers: {})
+    stub_request(:any, /api.castle.io/)
+      .with(basic_auth: ['', 'secret'])
+      .to_return(status: 200, body: response_body, headers: {})
   end
 
   after { Timecop.return }
@@ -32,8 +32,14 @@ describe Castle::API::StartImpersonation do
   describe 'call' do
     let(:impersonator) { 'test@castle.io' }
     let(:request_body) do
-      { user_id: '1234', sent_at: time_auto,
-        properties: { impersonator: impersonator }, context: context }
+      {
+        user_id: '1234',
+        sent_at: time_auto,
+        properties: {
+          impersonator: impersonator
+        },
+        context: context
+      }
     end
     let(:response_body) { { success: true }.to_json }
     let(:options) do

data/spec/lib/castle/api/track_spec.rb

@@ -24,17 +24,16 @@ describe Castle::API::Track do
   before do
     Timecop.freeze(time_now)
     stub_const('Castle::VERSION', '2.2.0')
-    stub_request(:any, /api.castle.io/).with(
-      basic_auth: ['', 'secret']
-    ).to_return(status: 200, body: response_body, headers: {})
+    stub_request(:any, /api.castle.io/)
+      .with(basic_auth: ['', 'secret'])
+      .to_return(status: 200, body: response_body, headers: {})
   end
 
   after { Timecop.return }
 
   describe 'track' do
     let(:request_body) do
-      { event: '$login.succeeded', context: context, user_id: '1234',
-        sent_at: time_auto }
+      { event: '$login.succeeded', context: context, user_id: '1234', sent_at: time_auto }
     end
 
     before { call }
@@ -53,8 +52,13 @@ describe Castle::API::Track do
           { event: '$login.succeeded', user_id: '1234', timestamp: time_user, context: context }
         end
         let(:request_body) do
-          { event: '$login.succeeded', user_id: '1234', context: context,
-            timestamp: time_user, sent_at: time_auto }
+          {
+            event: '$login.succeeded',
+            user_id: '1234',
+            context: context,
+            timestamp: time_user,
+            sent_at: time_auto
+          }
         end
 
         it do

data/spec/lib/castle/api_spec.rb

@@ -8,31 +8,19 @@ describe Castle::API do
   context 'when request timeouts' do
     before { stub_request(:any, /api.castle.io/).to_timeout }
 
-    it do
-      expect do
-        call
-      end.to raise_error(Castle::RequestError)
-    end
+    it { expect { call }.to raise_error(Castle::RequestError) }
   end
 
   context 'when non-OK response code' do
     before { stub_request(:any, /api.castle.io/).to_return(status: 400) }
 
-    it do
-      expect do
-        call
-      end.to raise_error(Castle::BadRequestError)
-    end
+    it { expect { call }.to raise_error(Castle::BadRequestError) }
   end
 
   context 'when no api_secret' do
     before { allow(Castle.config).to receive(:api_secret).and_return('') }
 
-    it do
-      expect do
-        call
-      end.to raise_error(Castle::ConfigurationError)
-    end
+    it { expect { call }.to raise_error(Castle::ConfigurationError) }
   end
 
   context 'when custom config' do
@@ -43,10 +31,6 @@ describe Castle::API do
       stub_request(:any, /api.castle.io/)
     end
 
-    it do
-      expect do
-        call
-      end.not_to raise_error
-    end
+    it { expect { call }.not_to raise_error }
   end
 end

data/spec/lib/castle/client_id/extract_spec.rb

@@ -8,9 +8,7 @@ describe Castle::ClientId::Extract do
   let(:client_id_header) { 'abcde' }
   let(:cookies) { request.cookies }
   let(:request) { Rack::Request.new(env) }
-  let(:env) do
-    Rack::MockRequest.env_for('/', headers)
-  end
+  let(:env) { Rack::MockRequest.env_for('/', headers) }
 
   context 'with client_id' do
     let(:headers) do
@@ -20,17 +18,12 @@ describe Castle::ClientId::Extract do
       }
     end
 
-    it do
-      expect(extractor.call).to eql(client_id_cookie)
-    end
+    it { expect(extractor.call).to eql(client_id_cookie) }
   end
 
   context 'with X-Castle-Client-Id header' do
     let(:headers) do
-      {
-        'HTTP_X_FORWARDED_FOR' => '1.2.3.4',
-        'HTTP_X_CASTLE_CLIENT_ID' => client_id_header
-      }
+      { 'HTTP_X_FORWARDED_FOR' => '1.2.3.4', 'HTTP_X_CASTLE_CLIENT_ID' => client_id_header }
     end
 
     it 'appends the client_id' do
@@ -42,9 +35,7 @@ describe Castle::ClientId::Extract do
     let(:cookies) { nil }
     let(:headers) { {} }
 
-    it do
-      expect(extractor.call).to eql('')
-    end
+    it { expect(extractor.call).to eql('') }
   end
 
   context 'with X-Castle-Client-Id header and cookies client' do

data/spec/lib/castle/client_spec.rb

@@ -21,9 +21,7 @@ describe Castle::Client do
   let(:client_with_no_timestamp) { described_class.new(context: request_to_context) }
 
   let(:headers) do
-    {
-      'Content-Length': '0', 'User-Agent': ua, 'X-Forwarded-For': ip.to_s, 'Cookie': true
-    }
+    { 'Content-Length': '0', 'User-Agent': ua, 'X-Forwarded-For': ip.to_s, 'Cookie': true }
   end
   let(:context) do
     {
@@ -32,7 +30,10 @@ describe Castle::Client do
       user_agent: ua,
       headers: headers,
       ip: ip,
-      library: { name: 'castle-rb', version: '2.2.0' }
+      library: {
+        name: 'castle-rb',
+        version: '2.2.0'
+      }
     }
   end
 
@@ -44,17 +45,15 @@ describe Castle::Client do
   before do
     Timecop.freeze(time_now)
     stub_const('Castle::VERSION', '2.2.0')
-    stub_request(:any, /api.castle.io/).with(
-      basic_auth: ['', 'secret']
-    ).to_return(status: 200, body: response_body, headers: {})
+    stub_request(:any, /api.castle.io/)
+      .with(basic_auth: ['', 'secret'])
+      .to_return(status: 200, body: response_body, headers: {})
   end
 
   after { Timecop.return }
 
   describe 'parses the request' do
-    before do
-      allow(Castle::API).to receive(:send_request).and_call_original
-    end
+    before { allow(Castle::API).to receive(:send_request).and_call_original }
 
     it do
       client.authenticate(event: '$login.succeeded', user_id: '1234')
@@ -65,8 +64,15 @@ describe Castle::Client do
   describe 'end impersonation' do
     let(:impersonator) { 'test@castle.io' }
     let(:request_body) do
-      { user_id: '1234', timestamp: time_auto, sent_at: time_auto,
-        properties: { impersonator: impersonator }, context: context }
+      {
+        user_id: '1234',
+        timestamp: time_auto,
+        sent_at: time_auto,
+        properties: {
+          impersonator: impersonator
+        },
+        context: context
+      }
     end
     let(:response_body) { { success: true }.to_json }
     let(:options) { { user_id: '1234', properties: { impersonator: impersonator } } }
@@ -93,8 +99,15 @@ describe Castle::Client do
   describe 'start impersonation' do
     let(:impersonator) { 'test@castle.io' }
     let(:request_body) do
-      { user_id: '1234', timestamp: time_auto, sent_at: time_auto,
-        properties: { impersonator: impersonator }, context: context }
+      {
+        user_id: '1234',
+        timestamp: time_auto,
+        sent_at: time_auto,
+        properties: {
+          impersonator: impersonator
+        },
+        context: context
+      }
     end
     let(:response_body) { { success: true }.to_json }
     let(:options) { { user_id: '1234', properties: { impersonator: impersonator } } }
@@ -118,70 +131,17 @@ describe Castle::Client do
     end
   end
 
-  describe 'identify' do
-    let(:request_body) do
-      { user_id: '1234', timestamp: time_auto,
-        sent_at: time_auto, context: context, user_traits: { name: 'Jo' } }
-    end
-
-    before { client.identify(options) }
-
-    context 'when used with symbol keys' do
-      let(:options) { { user_id: '1234', user_traits: { name: 'Jo' } } }
-
-      it do
-        assert_requested :post, 'https://api.castle.io/v1/identify', times: 1 do |req|
-          JSON.parse(req.body) == JSON.parse(request_body.to_json)
-        end
-      end
-
-      context 'when passed timestamp in options and no defined timestamp' do
-        let(:client) { client_with_no_timestamp }
-        let(:options) { { user_id: '1234', user_traits: { name: 'Jo' }, timestamp: time_user } }
-        let(:request_body) do
-          { user_id: '1234', user_traits: { name: 'Jo' }, context: context,
-            timestamp: time_user, sent_at: time_auto }
-        end
-
-        it do
-          assert_requested :post, 'https://api.castle.io/v1/identify', times: 1 do |req|
-            JSON.parse(req.body) == JSON.parse(request_body.to_json)
-          end
-        end
-      end
-
-      context 'with client initialized with timestamp' do
-        let(:client) { client_with_user_timestamp }
-        let(:request_body) do
-          { user_id: '1234', timestamp: time_user, sent_at: time_auto,
-            context: context, user_traits: { name: 'Jo' } }
-        end
-
-        it do
-          assert_requested :post, 'https://api.castle.io/v1/identify', times: 1 do |req|
-            JSON.parse(req.body) == JSON.parse(request_body.to_json)
-          end
-        end
-      end
-    end
-
-    context 'when used with string keys' do
-      let(:options) { { 'user_id' => '1234', 'user_traits' => { 'name' => 'Jo' } } }
-
-      it do
-        assert_requested :post, 'https://api.castle.io/v1/identify', times: 1 do |req|
-          JSON.parse(req.body) == JSON.parse(request_body.to_json)
-        end
-      end
-    end
-  end
-
   describe 'authenticate' do
     let(:options) { { event: '$login.succeeded', user_id: '1234' } }
     let(:request_response) { client.authenticate(options) }
     let(:request_body) do
-      { event: '$login.succeeded', user_id: '1234', context: context,
-        timestamp: time_auto, sent_at: time_auto }
+      {
+        event: '$login.succeeded',
+        user_id: '1234',
+        context: context,
+        timestamp: time_auto,
+        sent_at: time_auto
+      }
     end
 
     context 'when used with symbol keys' do
@@ -197,8 +157,13 @@ describe Castle::Client do
         let(:client) { client_with_no_timestamp }
         let(:options) { { event: '$login.succeeded', user_id: '1234', timestamp: time_user } }
         let(:request_body) do
-          { event: '$login.succeeded', user_id: '1234', context: context,
-            timestamp: time_user, sent_at: time_auto }
+          {
+            event: '$login.succeeded',
+            user_id: '1234',
+            context: context,
+            timestamp: time_user,
+            sent_at: time_auto
+          }
         end
 
         it do
@@ -211,8 +176,13 @@ describe Castle::Client do
       context 'with client initialized with timestamp' do
         let(:client) { client_with_user_timestamp }
         let(:request_body) do
-          { event: '$login.succeeded', user_id: '1234', context: context,
-            timestamp: time_user, sent_at: time_auto }
+          {
+            event: '$login.succeeded',
+            user_id: '1234',
+            context: context,
+            timestamp: time_user,
+            sent_at: time_auto
+          }
         end
 
         it do
@@ -306,8 +276,13 @@ describe Castle::Client do
 
   describe 'track' do
     let(:request_body) do
-      { event: '$login.succeeded', context: context, user_id: '1234',
-        timestamp: time_auto, sent_at: time_auto }
+      {
+        event: '$login.succeeded',
+        context: context,
+        user_id: '1234',
+        timestamp: time_auto,
+        sent_at: time_auto
+      }
     end
 
     before { client.track(options) }
@@ -325,8 +300,13 @@ describe Castle::Client do
         let(:client) { client_with_no_timestamp }
         let(:options) { { event: '$login.succeeded', user_id: '1234', timestamp: time_user } }
         let(:request_body) do
-          { event: '$login.succeeded', user_id: '1234', context: context,
-            timestamp: time_user, sent_at: time_auto }
+          {
+            event: '$login.succeeded',
+            user_id: '1234',
+            context: context,
+            timestamp: time_user,
+            sent_at: time_auto
+          }
         end
 
         it do
@@ -339,8 +319,13 @@ describe Castle::Client do
       context 'with client initialized with timestamp' do
         let(:client) { client_with_user_timestamp }
         let(:request_body) do
-          { event: '$login.succeeded', context: context, user_id: '1234',
-            timestamp: time_user, sent_at: time_auto }
+          {
+            event: '$login.succeeded',
+            context: context,
+            user_id: '1234',
+            timestamp: time_user,
+            sent_at: time_auto
+          }
         end
 
         it do
@@ -375,4 +360,16 @@ describe Castle::Client do
       it { expect(client).to be_tracked }
     end
   end
+
+  describe 'filter' do
+    it_behaves_like 'action request', :filter
+  end
+
+  describe 'risk' do
+    it_behaves_like 'action request', :risk
+  end
+
+  describe 'log' do
+    it_behaves_like 'action request', :log
+  end
 end

data/spec/lib/castle/commands/authenticate_spec.rb

@@ -20,9 +20,7 @@ describe Castle::Commands::Authenticate do
 
     context 'with properties' do
       let(:payload) { default_payload.merge(properties: { test: '1' }) }
-      let(:command_data) do
-        default_payload.merge(properties: { test: '1' }, context: context)
-      end
+      let(:command_data) { default_payload.merge(properties: { test: '1' }, context: context) }
 
       it { expect(command.method).to be_eql(:post) }
       it { expect(command.path).to be_eql('authenticate') }
@@ -31,9 +29,7 @@ describe Castle::Commands::Authenticate do
 
     context 'with user_traits' do
       let(:payload) { default_payload.merge(user_traits: { test: '1' }) }
-      let(:command_data) do
-        default_payload.merge(user_traits: { test: '1' }, context: context)
-      end
+      let(:command_data) { default_payload.merge(user_traits: { test: '1' }, context: context) }
 
       it { expect(command.method).to be_eql(:post) }
       it { expect(command.path).to be_eql('authenticate') }
@@ -42,9 +38,7 @@ describe Castle::Commands::Authenticate do
 
     context 'when active true' do
       let(:payload) { default_payload.merge(context: context.merge(active: true)) }
-      let(:command_data) do
-        default_payload.merge(context: context.merge(active: true))
-      end
+      let(:command_data) { default_payload.merge(context: context.merge(active: true)) }
 
       it { expect(command.method).to be_eql(:post) }
       it { expect(command.path).to be_eql('authenticate') }
@@ -53,9 +47,7 @@ describe Castle::Commands::Authenticate do
 
     context 'when active false' do
       let(:payload) { default_payload.merge(context: context.merge(active: false)) }
-      let(:command_data) do
-        default_payload.merge(context: context.merge(active: false))
-      end
+      let(:command_data) { default_payload.merge(context: context.merge(active: false)) }
 
       it { expect(command.method).to be_eql(:post) }
       it { expect(command.path).to be_eql('authenticate') }
@@ -79,9 +71,10 @@ describe Castle::Commands::Authenticate do
       let(:payload) { {} }
 
       it do
-        expect do
-          validate!
-        end.to raise_error(Castle::InvalidParametersError, 'event is missing or empty')
+        expect { validate! }.to raise_error(
+          Castle::InvalidParametersError,
+          'event is missing or empty'
+        )
       end
     end