castle-rb 7.2.0 → 8.1.0

data/spec/lib/castle/commands/list_items/unarchive_spec.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+RSpec.describe Castle::Commands::ListItems::Unarchive do
+  describe '.build' do
+    subject(:command) { described_class.build(options) }
+
+    context 'with valid options' do
+      let(:options) { { list_id: 'list_id', list_item_id: 'list_item_id'} }
+
+      it { expect(command.method).to be(:put) }
+      it { expect(command.path).to eql('lists/list_id/items/list_item_id/unarchive') }
+      it { expect(command.data).to be_nil }
+    end
+
+    context 'with invalid options' do
+      let(:options) { { list_id: 'list_id' } }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+  end
+end

data/spec/lib/castle/commands/list_items/update_spec.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+RSpec.describe Castle::Commands::ListItems::Update do
+  describe '.build' do
+    subject(:command) { described_class.build(options) }
+
+    context 'with valid options' do
+      let(:options) { { list_id: 'list_id', list_item_id: 'list_item_id', comment: 'updating item' } }
+
+      it { expect(command.method).to be(:put) }
+      it { expect(command.path).to eql('lists/list_id/items/list_item_id') }
+      it { expect(command.data).to eql(options.except(:list_id, :list_item_id)) }
+    end
+
+    context 'with invalid options' do
+      let(:options) { { list_id: 'list_id', list_item_id: 'list_item_id' } }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+  end
+end

data/spec/lib/castle/commands/lists/create_spec.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+RSpec.describe Castle::Commands::Lists::Create do
+  describe '.build' do
+    subject(:command) { described_class.build(options) }
+
+    let(:options) { { name: 'list_name', color: '$green', primary_field: 'user.email' } }
+
+    context 'with valid options' do
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('lists') }
+      it { expect(command.data).to eql(options) }
+    end
+
+    context 'without name' do
+      let(:options) { { color: '$green', primary_field: 'user.email' } }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+
+    context 'without color' do
+      let(:options) { { name: 'list_name', primary_field: 'user.email' } }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+
+    context 'without primary_field' do
+      let(:options) { { name: 'list_name', color: '$green' } }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+  end
+end

data/spec/lib/castle/commands/lists/delete_spec.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+RSpec.describe Castle::Commands::Lists::Delete do
+  describe '.build' do
+    subject(:command) { described_class.build(options) }
+
+    let(:options) { { list_id: 'list_id' } }
+
+    context 'with valid options' do
+      it { expect(command.method).to be(:delete) }
+      it { expect(command.path).to eql('lists/list_id') }
+      it { expect(command.data).to be_nil }
+    end
+
+    context 'without list_id' do
+      let(:options) { {} }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+  end
+end

data/spec/lib/castle/commands/lists/get_all_spec.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+RSpec.describe Castle::Commands::Lists::GetAll do
+  describe '.build' do
+    subject(:command) { described_class.build }
+
+    it { expect(command.method).to be(:get) }
+    it { expect(command.path).to eql('lists') }
+    it { expect(command.data).to be_nil }
+  end
+end

data/spec/lib/castle/commands/lists/get_spec.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+RSpec.describe Castle::Commands::Lists::Get do
+  describe '.build' do
+    subject(:command) { described_class.build(options) }
+
+    let(:options) { { list_id: 'list_id' } }
+
+    context 'with valid options' do
+      it { expect(command.method).to be(:get) }
+      it { expect(command.path).to eql('lists/list_id') }
+      it { expect(command.data).to be_nil }
+    end
+
+    context 'without list_id' do
+      let(:options) { {} }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+  end
+end

data/spec/lib/castle/commands/lists/query_spec.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+RSpec.describe Castle::Commands::Lists::Query do
+  describe '.build' do
+    subject(:command) { described_class.build(options) }
+
+    context 'with valid options' do
+      let(:options) { { filters: [{ field: 'primary_field', op: '$eq', value: 'test' }]} }
+
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('lists/query') }
+      it { expect(command.data).to eq(options.except(:list_id)) }
+    end
+
+    context 'with invalid filters' do
+      let(:options) { { filters: [{ field: 'primary_field', op: '$eq' }]} }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+
+    context 'with invalid sort' do
+      let(:options) { { sort: { field: 'primary_field' } } }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+  end
+end

data/spec/lib/castle/commands/lists/update_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+RSpec.describe Castle::Commands::Lists::Update do
+  describe '.build' do
+    subject(:command) { described_class.build(options) }
+
+    context 'with valid options' do
+      let(:options) do
+        {
+          list_id: 'list_id',
+          name: 'name',
+          description: 'description',
+          color: '$yellow',
+          default_item_archivation_time: 6000
+        }
+      end
+
+      it { expect(command.method).to be(:put) }
+      it { expect(command.path).to eql('lists/list_id') }
+      it { expect(command.data).to eql(options.except(:list_id)) }
+    end
+
+    context 'without list_id' do
+      let(:options) { { name: 'name' } }
+
+      it { expect { command }.to raise_error(Castle::InvalidParametersError) }
+    end
+  end
+end

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

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Commands::Log do
+RSpec.describe Castle::Commands::Log do
   subject(:instance) { described_class }
 
   let(:context) { { test: { test1: '1' } } }
@@ -29,45 +29,45 @@ describe Castle::Commands::Log do
       let(:payload) { default_payload.merge(properties: { test: '1' }) }
       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('log') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('log') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'with user_traits' do
       let(:payload) { default_payload.merge(user_traits: { test: '1' }) }
       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('log') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('log') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'when active true' do
       let(:payload) { default_payload.merge(context: context.merge(active: true)) }
       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('log') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('log') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'when active false' do
       let(:payload) { default_payload.merge(context: context.merge(active: false)) }
       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('log') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('log') }
+      it { expect(command.data).to 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(:post) }
-      it { expect(command.path).to be_eql('log') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('log') }
+      it { expect(command.data).to eql(command_data) }
     end
   end
 end

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

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Commands::ReportDevice do
+RSpec.describe Castle::Commands::ReportDevice do
   subject(:instance) { described_class }
 
   let(:context) { {} }
@@ -16,8 +16,8 @@ describe Castle::Commands::ReportDevice do
     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.method).to be(:put) }
+      it { expect(command.path).to eql("devices/#{device_token}/report") }
       it { expect(command.data).to be_nil }
     end
   end

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

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Commands::Risk do
+RSpec.describe Castle::Commands::Risk do
   subject(:instance) { described_class }
 
   let(:context) { { test: { test1: '1' } } }
@@ -29,45 +29,45 @@ describe Castle::Commands::Risk do
       let(:payload) { default_payload.merge(properties: { test: '1' }) }
       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('risk') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('risk') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'with user_traits' do
       let(:payload) { default_payload.merge(user_traits: { test: '1' }) }
       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('risk') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('risk') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'when active true' do
       let(:payload) { default_payload.merge(context: context.merge(active: true)) }
       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('risk') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('risk') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'when active false' do
       let(:payload) { default_payload.merge(context: context.merge(active: false)) }
       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('risk') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('risk') }
+      it { expect(command.data).to 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(:post) }
-      it { expect(command.path).to be_eql('risk') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('risk') }
+      it { expect(command.data).to eql(command_data) }
     end
   end
 end

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

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Commands::StartImpersonation do
+RSpec.describe Castle::Commands::StartImpersonation do
   subject(:instance) { described_class }
 
   let(:context) { { user_agent: 'test', ip: '127.0.0.1', client_id: 'test' } }
@@ -19,40 +19,38 @@ describe Castle::Commands::StartImpersonation do
 
     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
+      let(:command_data) { default_payload.merge(properties: { impersonator: impersonator }, 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) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('impersonate') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'when active true' do
       let(:payload) { default_payload.merge(context: context.merge(active: true)) }
       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('impersonate') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('impersonate') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'when active false' do
       let(:payload) { default_payload.merge(context: context.merge(active: false)) }
       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('impersonate') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('impersonate') }
+      it { expect(command.data).to 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(:post) }
-      it { expect(command.path).to be_eql('impersonate') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('impersonate') }
+      it { expect(command.data).to eql(command_data) }
     end
   end
 
@@ -62,12 +60,7 @@ describe Castle::Commands::StartImpersonation do
     context 'when user_id not present' do
       let(:payload) { {} }
 
-      it do
-        expect { validate! }.to raise_error(
-          Castle::InvalidParametersError,
-          'user_id is missing or empty'
-        )
-      end
+      it { expect { validate! }.to raise_error(Castle::InvalidParametersError, 'user_id is missing or empty') }
     end
 
     context 'when user_id present' do

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

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Commands::Track do
+RSpec.describe Castle::Commands::Track do
   subject(:instance) { described_class }
 
   let(:context) { { test: { test1: '1' } } }
@@ -20,54 +20,54 @@ describe Castle::Commands::Track do
       let(:payload) { default_payload.merge(user_id: '1234') }
       let(:command_data) { default_payload.merge(user_id: '1234', context: context) }
 
-      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) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('track') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'with properties' do
       let(:payload) { default_payload.merge(properties: { test: '1' }) }
       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('track') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('track') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'with user_traits' do
       let(:payload) { default_payload.merge(user_traits: { test: '1' }) }
       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('track') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('track') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'when active true' do
       let(:payload) { default_payload.merge(context: context.merge(active: true)) }
       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('track') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('track') }
+      it { expect(command.data).to eql(command_data) }
     end
 
     context 'when active false' do
       let(:payload) { default_payload.merge(context: context.merge(active: false)) }
       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('track') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('track') }
+      it { expect(command.data).to 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(:post) }
-      it { expect(command.path).to be_eql('track') }
-      it { expect(command.data).to be_eql(command_data) }
+      it { expect(command.method).to be(:post) }
+      it { expect(command.path).to eql('track') }
+      it { expect(command.data).to eql(command_data) }
     end
   end
 
@@ -77,12 +77,7 @@ describe Castle::Commands::Track do
     context 'when event not present' do
       let(:payload) { {} }
 
-      it do
-        expect { validate! }.to raise_error(
-          Castle::InvalidParametersError,
-          'event is missing or empty'
-        )
-      end
+      it { expect { validate! }.to raise_error(Castle::InvalidParametersError, 'event is missing or empty') }
     end
 
     context 'when event present' do

data/spec/lib/castle/configuration_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Configuration do
+RSpec.describe Castle::Configuration do
   subject(:config) { described_class.new }
 
   it_behaves_like 'configuration_host'
@@ -10,5 +10,5 @@ describe Castle::Configuration do
   it_behaves_like 'configuration_failover_strategy'
   it_behaves_like 'configuration_api_secret'
 
-  it { expect(config.api_secret).to be_eql('') }
+  it { expect(config.api_secret).to eql('') }
 end

data/spec/lib/castle/context/get_default_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Context::GetDefault do
+RSpec.describe Castle::Context::GetDefault do
   subject { described_class.new(request, nil) }
 
   let(:ip) { '1.2.3.4' }
@@ -12,7 +12,8 @@ describe Castle::Context::GetDefault do
       'HTTP_X_FORWARDED_FOR' => ip,
       'HTTP_ACCEPT_LANGUAGE' => 'en',
       'HTTP_USER_AGENT' => 'test',
-      'HTTP_COOKIE' => "__cid=#{client_id};other=efgh"
+      'HTTP_COOKIE' => "__cid=#{client_id};other=efgh",
+      'HTTP_CONTENT_LENGTH' => '0'
     )
   end
   let(:request) { Rack::Request.new(env) }
@@ -30,11 +31,11 @@ describe Castle::Context::GetDefault do
 
   before { stub_const('Castle::VERSION', version) }
 
-  it { expect(default_context[:active]).to be_eql(true) }
-  it { expect(default_context[:headers]).to be_eql(result_headers) }
-  it { expect(default_context[:ip]).to be_eql(ip) }
-  it { expect(default_context[:client_id]).to be_eql(client_id) }
-  it { expect(default_context[:library][:name]).to be_eql('castle-rb') }
-  it { expect(default_context[:library][:version]).to be_eql(version) }
-  it { expect(default_context[:user_agent]).to be_eql('test') }
+  it { expect(default_context[:active]).to be(true) }
+  it { expect(default_context[:headers]).to eql(result_headers) }
+  it { expect(default_context[:ip]).to eql(ip) }
+  it { expect(default_context[:client_id]).to eql(client_id) }
+  it { expect(default_context[:library][:name]).to eql('castle-rb') }
+  it { expect(default_context[:library][:version]).to eql(version) }
+  it { expect(default_context[:user_agent]).to eql('test') }
 end

data/spec/lib/castle/context/merge_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Context::Merge do
+RSpec.describe Castle::Context::Merge do
   let(:first) { { test: { test1: { c: '4' } } } }
 
   describe '#call' do

data/spec/lib/castle/context/prepare_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Context::Prepare do
+RSpec.describe Castle::Context::Prepare do
   let(:ip) { '1.2.3.4' }
   let(:cookie_id) { 'abcd' }
   let(:ua) { 'Chrome' }
@@ -9,7 +9,8 @@ describe Castle::Context::Prepare do
       '/',
       'HTTP_USER_AGENT' => ua,
       'HTTP_X_FORWARDED_FOR' => ip,
-      'HTTP_COOKIE' => "__cid=#{cookie_id};other=efgh"
+      'HTTP_COOKIE' => "__cid=#{cookie_id};other=efgh",
+      'HTTP_CONTENT_LENGTH' => '0'
     )
   end
   let(:request) { Rack::Request.new(env) }
@@ -27,9 +28,7 @@ describe Castle::Context::Prepare do
     }
   end
 
-  let(:headers) do
-    { 'Content-Length': '0', 'User-Agent': ua, 'X-Forwarded-For': ip.to_s, 'Cookie': true }
-  end
+  let(:headers) { { 'Content-Length': '0', 'User-Agent': ua, 'X-Forwarded-For': ip.to_s, Cookie: true } }
 
   before { stub_const('Castle::VERSION', '6.0.0') }
 

data/spec/lib/castle/context/sanitize_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Context::Sanitize do
+RSpec.describe Castle::Context::Sanitize do
   let(:paylod) { { test: 'test' } }
 
   describe '#call' do

data/spec/lib/castle/core/get_connection_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Castle::Core::GetConnection do
+RSpec.describe Castle::Core::GetConnection do
   describe '.call' do
     subject(:class_call) { described_class.call }