castle-rb 5.0.0 → 6.0.0

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

@@ -1,10 +1,12 @@
 # frozen_string_literal: true
 
 describe Castle::Commands::Authenticate do
-  subject(:instance) { described_class.new(context) }
+  subject(:instance) { described_class }
 
   let(:context) { { test: { test1: '1' } } }
-  let(:default_payload) { { event: '$login.authenticate', user_id: '1234', sent_at: time_auto } }
+  let(:default_payload) do
+    { event: '$login.authenticate', user_id: '1234', sent_at: time_auto, context: context }
+  end
 
   let(:time_now) { Time.now }
   let(:time_auto) { time_now.utc.iso8601(3) }
@@ -16,17 +18,6 @@ describe Castle::Commands::Authenticate do
   describe '.build' do
     subject(:command) { instance.build(payload) }
 
-    context 'with simple merger' do
-      let(:payload) { default_payload.merge(context: { test: { test2: '1' } }) }
-      let(:command_data) do
-        default_payload.merge(context: { test: { test1: '1', test2: '1' } })
-      end
-
-      it { expect(command.method).to be_eql(:post) }
-      it { expect(command.path).to be_eql('authenticate') }
-      it { expect(command.data).to be_eql(command_data) }
-    end
-
     context 'with properties' do
       let(:payload) { default_payload.merge(properties: { test: '1' }) }
       let(:command_data) do
@@ -50,7 +41,7 @@ describe Castle::Commands::Authenticate do
     end
 
     context 'when active true' do
-      let(:payload) { default_payload.merge(context: { active: true }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: true)) }
       let(:command_data) do
         default_payload.merge(context: context.merge(active: true))
       end
@@ -61,7 +52,7 @@ describe Castle::Commands::Authenticate do
     end
 
     context 'when active false' do
-      let(:payload) { default_payload.merge(context: { active: false }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: false)) }
       let(:command_data) do
         default_payload.merge(context: context.merge(active: false))
       end
@@ -72,7 +63,7 @@ describe Castle::Commands::Authenticate do
     end
 
     context 'when active string' do
-      let(:payload) { default_payload.merge(context: { active: 'string' }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: 'string')) }
       let(:command_data) { default_payload.merge(context: context) }
 
       it { expect(command.method).to be_eql(:post) }

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

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+describe Castle::Commands::EndImpersonation do
+  subject(:instance) { described_class }
+
+  let(:context) { { user_agent: 'test', ip: '127.0.0.1', client_id: 'test' } }
+  let(:impersonator) { 'test@castle.io' }
+  let(:default_payload) { { user_id: '1234', sent_at: time_auto, context: context } }
+
+  let(:time_now) { Time.now }
+  let(:time_auto) { time_now.utc.iso8601(3) }
+
+  before { Timecop.freeze(time_now) }
+
+  after { Timecop.return }
+
+  describe '.build' do
+    subject(:command) { instance.build(payload) }
+
+    context 'with impersonator' do
+      let(:payload) { default_payload.merge(properties: { impersonator: impersonator }) }
+      let(:command_data) do
+        default_payload.merge(properties: { impersonator: impersonator }, context: context)
+      end
+
+      it { expect(command.method).to be_eql(:delete) }
+      it { expect(command.path).to be_eql('impersonate') }
+      it { expect(command.data).to be_eql(command_data) }
+    end
+
+    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
+
+      it { expect(command.method).to be_eql(:delete) }
+      it { expect(command.path).to be_eql('impersonate') }
+      it { expect(command.data).to be_eql(command_data) }
+    end
+
+    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
+
+      it { expect(command.method).to be_eql(:delete) }
+      it { expect(command.path).to be_eql('impersonate') }
+      it { expect(command.data).to be_eql(command_data) }
+    end
+
+    context 'when active string' do
+      let(:payload) { default_payload.merge(context: context.merge(active: 'string')) }
+      let(:command_data) { default_payload.merge(context: context) }
+
+      it { expect(command.method).to be_eql(:delete) }
+      it { expect(command.path).to be_eql('impersonate') }
+      it { expect(command.data).to be_eql(command_data) }
+    end
+  end
+
+  describe '#validate!' do
+    subject(:validate!) { instance.build(payload) }
+
+    context 'when user_id not present' do
+      let(:payload) { {} }
+
+      it do
+        expect do
+          validate!
+        end.to raise_error(Castle::InvalidParametersError, 'user_id is missing or empty')
+      end
+    end
+
+    context 'when user_id present' do
+      let(:payload) { { user_id: '1234', context: context } }
+
+      it { expect { validate! }.not_to raise_error }
+    end
+  end
+end

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

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+describe Castle::Commands::GetDevice do
+  subject(:instance) { described_class }
+
+  let(:context) { {} }
+  let(:device_token) { '1234' }
+
+  describe '.build' do
+    subject(:command) { instance.build(device_token: device_token) }
+
+    context 'without device_token' do
+      let(:device_token) { '' }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+
+    context 'with device_token' do
+      it { expect(command.method).to be_eql(:get) }
+      it { expect(command.path).to be_eql("devices/#{device_token}") }
+      it { expect(command.data).to be_nil }
+    end
+  end
+end

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

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+describe Castle::Commands::GetDevicesForUser do
+  subject(:instance) { described_class }
+
+  let(:context) { {} }
+  let(:user_id) { '1234' }
+
+  describe '.build' do
+    subject(:command) { instance.build(user_id: user_id) }
+
+    context 'without user_id' do
+      let(:user_id) { '' }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+
+    context 'with user_id' do
+      it { expect(command.method).to be_eql(:get) }
+      it { expect(command.path).to be_eql("users/#{user_id}/devices") }
+      it { expect(command.data).to be_nil }
+    end
+  end
+end

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
 describe Castle::Commands::Identify do
-  subject(:instance) { described_class.new(context) }
+  subject(:instance) { described_class }
 
   let(:context) { { test: { test1: '1' } } }
-  let(:default_payload) { { user_id: '1234', sent_at: time_auto } }
+  let(:default_payload) { { user_id: '1234', sent_at: time_auto, context: context } }
 
   let(:time_now) { Time.now }
   let(:time_auto) { time_now.utc.iso8601(3) }
@@ -16,17 +16,6 @@ describe Castle::Commands::Identify do
   describe '.build' do
     subject(:command) { instance.build(payload) }
 
-    context 'with simple merger' do
-      let(:payload) { default_payload.merge(context: { test: { test2: '1' } }) }
-      let(:command_data) do
-        default_payload.merge(context: { test: { test1: '1', test2: '1' } })
-      end
-
-      it { expect(command.method).to be_eql(:post) }
-      it { expect(command.path).to be_eql('identify') }
-      it { expect(command.data).to be_eql(command_data) }
-    end
-
     context 'with user_traits' do
       let(:payload) { default_payload.merge(user_traits: { test: '1' }) }
       let(:command_data) do
@@ -39,7 +28,7 @@ describe Castle::Commands::Identify do
     end
 
     context 'when active true' do
-      let(:payload) { default_payload.merge(context: { active: true }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: true)) }
       let(:command_data) do
         default_payload.merge(context: context.merge(active: true))
       end
@@ -50,7 +39,7 @@ describe Castle::Commands::Identify do
     end
 
     context 'when active false' do
-      let(:payload) { default_payload.merge(context: { active: false }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: false)) }
       let(:command_data) do
         default_payload.merge(context: context.merge(active: false))
       end
@@ -61,7 +50,7 @@ describe Castle::Commands::Identify do
     end
 
     context 'when active string' do
-      let(:payload) { default_payload.merge(context: { active: 'string' }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: 'string')) }
       let(:command_data) { default_payload.merge(context: context) }
 
       it { expect(command.method).to be_eql(:post) }

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

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+describe Castle::Commands::ReportDevice do
+  subject(:instance) { described_class }
+
+  let(:context) { {} }
+  let(:device_token) { '1234' }
+
+  describe '.build' do
+    subject(:command) { instance.build(device_token: device_token) }
+
+    context 'without device_token' do
+      let(:device_token) { '' }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+
+    context 'with device_token' do
+      it { expect(command.method).to be_eql(:put) }
+      it { expect(command.path).to be_eql("devices/#{device_token}/report") }
+      it { expect(command.data).to be_nil }
+    end
+  end
+end

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

@@ -7,7 +7,7 @@ describe Castle::Commands::Review do
   let(:review_id) { '1234' }
 
   describe '.build' do
-    subject(:command) { instance.build(review_id) }
+    subject(:command) { instance.build(review_id: review_id) }
 
     context 'without review_id' do
       let(:review_id) { '' }

data/spec/lib/castle/commands/{impersonate_spec.rb → start_impersonation_spec.rb}

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-describe Castle::Commands::Impersonate do
-  subject(:instance) { described_class.new(context) }
+describe Castle::Commands::StartImpersonation do
+  subject(:instance) { described_class }
 
   let(:context) { { user_agent: 'test', ip: '127.0.0.1', client_id: 'test' } }
   let(:impersonator) { 'test@castle.io' }
-  let(:default_payload) { { user_id: '1234', sent_at: time_auto } }
+  let(:default_payload) { { user_id: '1234', sent_at: time_auto, context: context } }
 
   let(:time_now) { Time.now }
   let(:time_auto) { time_now.utc.iso8601(3) }
@@ -17,24 +17,6 @@ describe Castle::Commands::Impersonate do
   describe '.build' do
     subject(:command) { instance.build(payload) }
 
-    context 'with simple merger' do
-      let(:payload) { default_payload.merge(context: { test: { test2: '1' } }) }
-      let(:command_data) do
-        default_payload.merge(
-          context: {
-            test: { test2: '1' },
-            user_agent: 'test',
-            ip: '127.0.0.1',
-            client_id: 'test'
-          }
-        )
-      end
-
-      it { expect(command.method).to be_eql(:post) }
-      it { expect(command.path).to be_eql('impersonate') }
-      it { expect(command.data).to be_eql(command_data) }
-    end
-
     context 'with impersonator' do
       let(:payload) { default_payload.merge(properties: { impersonator: impersonator }) }
       let(:command_data) do
@@ -47,7 +29,7 @@ describe Castle::Commands::Impersonate do
     end
 
     context 'when active true' do
-      let(:payload) { default_payload.merge(context: { active: true }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: true)) }
       let(:command_data) do
         default_payload.merge(context: context.merge(active: true))
       end
@@ -58,7 +40,7 @@ describe Castle::Commands::Impersonate do
     end
 
     context 'when active false' do
-      let(:payload) { default_payload.merge(context: { active: false }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: false)) }
       let(:command_data) do
         default_payload.merge(context: context.merge(active: false))
       end
@@ -69,20 +51,13 @@ describe Castle::Commands::Impersonate do
     end
 
     context 'when active string' do
-      let(:payload) { default_payload.merge(context: { active: 'string' }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: 'string')) }
       let(:command_data) { default_payload.merge(context: context) }
 
       it { expect(command.method).to be_eql(:post) }
       it { expect(command.path).to be_eql('impersonate') }
       it { expect(command.data).to be_eql(command_data) }
     end
-
-    context 'when reset' do
-      let(:payload) { default_payload.merge(reset: true) }
-
-      it { expect(command.method).to be_eql(:delete) }
-      it { expect(command.path).to be_eql('impersonate') }
-    end
   end
 
   describe '#validate!' do
@@ -99,7 +74,7 @@ describe Castle::Commands::Impersonate do
     end
 
     context 'when user_id present' do
-      let(:payload) { { user_id: '1234' } }
+      let(:payload) { { user_id: '1234', context: context } }
 
       it { expect { validate! }.not_to raise_error }
     end

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
 describe Castle::Commands::Track do
-  subject(:instance) { described_class.new(context) }
+  subject(:instance) { described_class }
 
   let(:context) { { test: { test1: '1' } } }
-  let(:default_payload) { { event: '$login.track', sent_at: time_auto } }
+  let(:default_payload) { { event: '$login.track', sent_at: time_auto, context: context } }
 
   let(:time_now) { Time.now }
   let(:time_auto) { time_now.utc.iso8601(3) }
@@ -16,17 +16,6 @@ describe Castle::Commands::Track do
   describe '#build' do
     subject(:command) { instance.build(payload) }
 
-    context 'with simple merger' do
-      let(:payload) { default_payload.merge(context: { test: { test2: '1' } }) }
-      let(:command_data) do
-        default_payload.merge(context: { test: { test1: '1', test2: '1' } })
-      end
-
-      it { expect(command.method).to be_eql(:post) }
-      it { expect(command.path).to be_eql('track') }
-      it { expect(command.data).to be_eql(command_data) }
-    end
-
     context 'with user_id' do
       let(:payload) { default_payload.merge(user_id: '1234') }
       let(:command_data) do
@@ -61,7 +50,7 @@ describe Castle::Commands::Track do
     end
 
     context 'when active true' do
-      let(:payload) { default_payload.merge(context: { active: true }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: true)) }
       let(:command_data) do
         default_payload.merge(context: context.merge(active: true))
       end
@@ -72,7 +61,7 @@ describe Castle::Commands::Track do
     end
 
     context 'when active false' do
-      let(:payload) { default_payload.merge(context: { active: false }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: false)) }
       let(:command_data) do
         default_payload.merge(context: context.merge(active: false))
       end
@@ -83,7 +72,7 @@ describe Castle::Commands::Track do
     end
 
     context 'when active string' do
-      let(:payload) { default_payload.merge(context: { active: 'string' }) }
+      let(:payload) { default_payload.merge(context: context.merge(active: 'string')) }
       let(:command_data) { default_payload.merge(context: context) }
 
       it { expect(command.method).to be_eql(:post) }

data/spec/lib/castle/configuration_spec.rb

@@ -2,146 +2,17 @@
 
 describe Castle::Configuration do
   subject(:config) do
-    described_class.instance
+    described_class.new
   end
 
-  describe 'host' do
-    context 'with default' do
-      it { expect(config.url.host).to be_eql('api.castle.io') }
-    end
+  it_behaves_like 'configuration_host'
+  it_behaves_like 'configuration_request_timeout'
+  it_behaves_like 'configuration_allowlisted'
+  it_behaves_like 'configuration_denylisted'
+  it_behaves_like 'configuration_failover_strategy'
+  it_behaves_like 'configuration_api_secret'
 
-    context 'with setter' do
-      before { config.url = 'http://api.castle.dev/v2' }
-
-      it { expect(config.url.host).to be_eql('api.castle.dev') }
-    end
-  end
-
-  describe 'post' do
-    context 'with default' do
-      it { expect(config.url.port).to be_eql(443) }
-    end
-
-    context 'with setter' do
-      before { config.url = 'http://api.castle.dev:3001/v2' }
-
-      it { expect(config.url.port).to be_eql(3001) }
-    end
-  end
-
-  describe 'api_secret' do
-    context 'with env' do
-      let(:secret_key_env) { 'secret_key_env' }
-      let(:secret_key) { 'secret_key' }
-
-      before do
-        allow(ENV).to receive(:fetch).with(
-          'CASTLE_API_SECRET', ''
-        ).and_return(secret_key_env)
-        config.reset
-      end
-
-      it do
-        expect(config.api_secret).to be_eql(secret_key_env)
-      end
-
-      context 'when key is overwritten' do
-        before { config.api_secret = secret_key }
-
-        it do
-          expect(config.api_secret).to be_eql(secret_key)
-        end
-      end
-    end
-
-    context 'with setter' do
-      let(:value) { 'new_secret' }
-
-      before do
-        config.api_secret = value
-      end
-
-      it do
-        expect(config.api_secret).to be_eql(value)
-      end
-    end
-
-    it do
-      expect(config.api_secret).to be_eql('secret')
-    end
-  end
-
-  describe 'request_timeout' do
-    it do
-      expect(config.request_timeout).to be_eql(500)
-    end
-
-    context 'with setter' do
-      let(:value) { 50.0 }
-
-      before do
-        config.request_timeout = value
-      end
-
-      it do
-        expect(config.request_timeout).to be_eql(value)
-      end
-    end
-  end
-
-  describe 'allowlisted' do
-    it do
-      expect(config.allowlisted.size).to be_eql(0)
-    end
-
-    context 'with setter' do
-      before do
-        config.allowlisted = ['header']
-      end
-
-      it do
-        expect(config.allowlisted).to be_eql(['Header'])
-      end
-    end
-  end
-
-  describe 'denylisted' do
-    it do
-      expect(config.denylisted.size).to be_eql(0)
-    end
-
-    context 'with setter' do
-      before do
-        config.denylisted = ['header']
-      end
-
-      it do
-        expect(config.denylisted).to be_eql(['Header'])
-      end
-    end
-  end
-
-  describe 'failover_strategy' do
-    it do
-      expect(config.failover_strategy).to be_eql(:allow)
-    end
-
-    context 'with setter' do
-      before do
-        config.failover_strategy = :deny
-      end
-
-      it do
-        expect(config.failover_strategy).to be_eql(:deny)
-      end
-    end
-
-    context 'when broken' do
-      it do
-        expect do
-          config.failover_strategy = :unicorn
-        end.to raise_error(Castle::ConfigurationError)
-      end
-    end
+  it do
+    expect(config.api_secret).to be_eql('')
   end
 end