forest_liana 6.0.0.pre.beta.2 → 6.0.0.pre.beta.3

data/spec/services/forest_liana/permissions_formatter_spec.rb

@@ -0,0 +1,222 @@
+module ForestLiana
+  describe PermissionsFormatter do
+    describe '#convert_to_new_format' do
+      let(:rendering_id) { 1 }
+      let(:old_format_collection_permissions) {
+        {
+          'list'=>true,
+          'show'=>false,
+          'create'=>true,
+          'update'=>false,
+          'delete'=>true,
+          'export'=>false,
+          'searchToEdit'=>false
+        }
+      }
+      let(:old_format_action_permissions) { { 'allowed' => true, 'users' => nil } }
+      let(:old_format_scope_permissions) { nil }
+      let(:old_format_permissions) {
+        {
+          'collection_1' => {
+            'collection' => old_format_collection_permissions,
+            'actions' => {
+              'action_1' => old_format_action_permissions
+            },
+            'scope' => old_format_scope_permissions
+          }
+        }
+      }
+
+      let(:converted_permission) { described_class.convert_to_new_format(old_format_permissions, rendering_id) }
+
+      describe 'collection permissions' do
+        subject { converted_permission['collections']['collection_1']['collection'] }
+
+        let(:expected_new_collection_permissions_format) {
+          {
+            'browseEnabled'=>true,
+            'readEnabled'=>false,
+            'addEnabled'=>true,
+            'editEnabled'=>false,
+            'deleteEnabled'=>true,
+            'exportEnabled'=>false
+          }
+        }
+
+        it 'should convert the old format to the new one' do
+          expect(subject).to eq expected_new_collection_permissions_format
+        end
+
+        describe 'with searchToEdit true and list false' do
+          let(:old_format_collection_permissions) {
+            {
+              'list'=>false,
+              'show'=>false,
+              'create'=>false,
+              'update'=>false,
+              'delete'=>false,
+              'export'=>false,
+              'searchToEdit'=>true
+            }
+          }
+
+          let(:expected_new_collection_permissions_format) {
+            {
+              'browseEnabled'=>true,
+              'readEnabled'=>false,
+              'addEnabled'=>false,
+              'editEnabled'=>false,
+              'deleteEnabled'=>false,
+              'exportEnabled'=>false
+            }
+          }
+
+          it 'should convert the old format to the new one with browseEnabled at true' do
+            expect(subject).to eq expected_new_collection_permissions_format
+          end
+        end
+
+        describe 'with searchToEdit false and list true' do
+          let(:old_format_collection_permissions) {
+            {
+              'list'=>true,
+              'show'=>false,
+              'create'=>false,
+              'update'=>false,
+              'delete'=>false,
+              'export'=>false,
+              'searchToEdit'=>false
+            }
+          }
+
+          let(:expected_new_collection_permissions_format) {
+            {
+              'browseEnabled'=>true,
+              'readEnabled'=>false,
+              'addEnabled'=>false,
+              'editEnabled'=>false,
+              'deleteEnabled'=>false,
+              'exportEnabled'=>false
+            }
+          }
+
+          it 'should convert the old format to the new one with browseEnabled at true' do
+            expect(subject).to eq expected_new_collection_permissions_format
+          end
+        end
+
+        describe 'with searchToEdit false and list false' do
+          let(:old_format_collection_permissions) {
+            {
+              'list'=>false,
+              'show'=>false,
+              'create'=>false,
+              'update'=>false,
+              'delete'=>false,
+              'export'=>false,
+              'searchToEdit'=>false
+            }
+          }
+
+          let(:expected_new_collection_permissions_format) {
+            {
+              'browseEnabled'=>false,
+              'readEnabled'=>false,
+              'addEnabled'=>false,
+              'editEnabled'=>false,
+              'deleteEnabled'=>false,
+              'exportEnabled'=>false
+            }
+          }
+
+          it 'should convert the old format to the new one with browseEnabled at false' do
+            expect(subject).to eq expected_new_collection_permissions_format
+          end
+        end
+      end
+
+      describe 'action permissions' do
+        subject { converted_permission['collections']['collection_1']['actions']['action_1'] }
+
+        context 'when allowed is true' do
+          context 'when users is nil' do
+            let(:old_format_action_permissions) { { 'allowed' => true, 'users' => nil } }
+            let(:expected_new_action_permissions_format) { { 'triggerEnabled' => true } }
+
+            it 'expected action permission triggerEnabled field should be true' do
+              expect(subject).to eq expected_new_action_permissions_format
+            end
+          end
+
+          context 'when users is an empty array' do
+            let(:old_format_action_permissions) { { 'allowed' => true, 'users' => [] } }
+            let(:expected_new_action_permissions_format) { { 'triggerEnabled' => [] } }
+
+            it 'expected action permission triggerEnabled field should be an empty array' do
+              expect(subject).to eq expected_new_action_permissions_format
+            end
+          end
+
+          context 'when users is NOT an empty array' do
+            let(:old_format_action_permissions) { { 'allowed' => true, 'users' => [2, 3] } }
+            let(:expected_new_action_permissions_format) { { 'triggerEnabled' => [2, 3] } }
+
+            it 'expected action permission triggerEnabled field should be equal to the users array' do
+              expect(subject).to eq expected_new_action_permissions_format
+            end
+          end
+        end
+
+        context 'when allowed is false' do
+          context 'when users is nil' do
+            let(:old_format_action_permissions) { { 'allowed' => false, 'users' => nil } }
+            let(:expected_new_action_permissions_format) { { 'triggerEnabled' => false } }
+
+            it 'expected action permission triggerEnabled field should be false' do
+              expect(subject).to eq expected_new_action_permissions_format
+            end
+          end
+
+          context 'when users is an empty array' do
+            let(:old_format_action_permissions) { { 'allowed' => false, 'users' => [] } }
+            let(:expected_new_action_permissions_format) { { 'triggerEnabled' => false } }
+
+            it 'expected action permission triggerEnabled field should be false' do
+              expect(subject).to eq expected_new_action_permissions_format
+            end
+          end
+
+          context 'when users is NOT an empty array' do
+            let(:old_format_action_permissions) { { 'allowed' => false, 'users' => [2, 3] } }
+            let(:expected_new_action_permissions_format) { { 'triggerEnabled' => false } }
+
+            it 'expected action permission triggerEnabled field should be false' do
+              expect(subject).to eq expected_new_action_permissions_format
+            end
+          end
+        end
+      end
+
+      describe 'scope permissions' do
+        subject { converted_permission['renderings'][rendering_id]['collection_1']['scope'] }
+        let(:expected_new_format_permissions) { old_format_scope_permissions }
+
+        context 'when scope permissions are set' do
+          let(:old_format_scope_permissions) { { 'dynamicScopesValues' => {}, 'filter' => { 'aggregator' => 'and', 'conditions' => [{ 'field' => 'field_1', 'operator' => 'equal', 'value' => true }] } } }
+
+          it 'expected scope permissions should be set' do
+            expect(subject).to eq expected_new_format_permissions
+          end
+        end
+
+        context 'when scope permissions are nil' do
+          let(:old_format_scope_permissions) { nil }
+
+          it 'expected scope permissions should be nil' do
+            expect(subject).to eq expected_new_format_permissions
+          end
+        end
+      end
+    end
+  end
+end

data/spec/services/forest_liana/permissions_getter_spec.rb

@@ -0,0 +1,83 @@
+module ForestLiana
+  describe PermissionsGetter do
+    describe '#get_permissions_api_route' do
+      it 'should respond with the v3 permissions route' do
+        expect(described_class.get_permissions_api_route).to eq '/liana/v3/permissions'
+      end
+    end
+
+    describe '#get_permissions_for_rendering' do
+      let(:rendering_id) { 34 }
+      let(:liana_permissions_url) { 'https://api.forestadmin.com/liana/v3/permissions' }
+      let(:liana_permissions_api_call_response) { instance_double(HTTParty::Response) }
+      let(:expected_request_parameters) {
+        {
+          :headers => {
+            "Content-Type" => "application/json",
+            "forest-secret-key" => "env_secret_test"
+          },
+          :query => expected_query_parameters,
+          :verify => false
+        }
+      }
+
+      before do
+        allow(HTTParty).to receive(:get).and_return(liana_permissions_api_call_response)
+        allow(liana_permissions_api_call_response).to receive(:response).and_return(liana_permissions_api_call_response_content)
+        allow(liana_permissions_api_call_response_content).to receive(:body).and_return(liana_permissions_api_call_response_content_body)
+      end
+
+      describe 'when the API returns a success' do
+        let(:liana_permissions_api_call_response_content) { Net::HTTPOK.new({}, 200, liana_permissions_api_call_response_content_body) }
+        let(:liana_permissions_api_call_response_content_body) { '{"test": true}' }
+        let(:expected_parsed_result) { { "test" => true } }
+
+        describe 'when NOT calling for rendering specific only' do
+          let(:expected_query_parameters) { { "renderingId" => rendering_id } }
+
+          it 'should call the API with correct URL' do
+            described_class.get_permissions_for_rendering(rendering_id)
+            expect(HTTParty).to have_received(:get).with(liana_permissions_url, expected_request_parameters)
+          end
+
+          it 'should return the expected JSON body' do
+            expect(described_class.get_permissions_for_rendering(rendering_id)).to eq expected_parsed_result
+          end
+        end
+
+        describe 'when calling for rendering specific only' do
+          let(:expected_query_parameters) { { "renderingId" => rendering_id, 'renderingSpecificOnly' => true } }
+
+          it 'should call the API with correct URL and parameters' do
+            described_class.get_permissions_for_rendering(rendering_id, rendering_specific_only: true)
+            expect(HTTParty).to have_received(:get).with(liana_permissions_url, expected_request_parameters)
+          end
+
+          it 'should return the expected JSON body' do
+            expect(described_class.get_permissions_for_rendering(rendering_id, rendering_specific_only: true)).to eq expected_parsed_result
+          end
+        end
+      end
+
+      describe 'when the API returns a not found error' do
+        let(:liana_permissions_api_call_response_content) { Net::HTTPNotFound.new({}, 404, liana_permissions_api_call_response_content_body) }
+        let(:liana_permissions_api_call_response_content_body) { 'Not Found' }
+
+        before do
+          allow(FOREST_LOGGER).to receive(:error)
+        end
+
+        it 'should return nil' do
+          expect(described_class.get_permissions_for_rendering(rendering_id)).to eq nil
+        end
+
+        it 'should log the not found error' do
+          described_class.get_permissions_for_rendering(rendering_id)
+          expect(FOREST_LOGGER).to have_received(:error).with('Cannot retrieve the permissions from the Forest server.')
+          expect(FOREST_LOGGER).to have_received(:error).with('Which was caused by:')
+          expect(FOREST_LOGGER).to have_received(:error).with(' Forest API returned an HTTP error 404')
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,3 +1,6 @@
+require 'simplecov'
+SimpleCov.start 'rails'
+
 # This file was generated by the `rails generate rspec:install` command. Conventionally, all
 # specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
 # The generated `.rspec` file contains `--require spec_helper` which will cause

data/test/dummy/app/assets/config/manifest.js

@@ -0,0 +1 @@
+{}

data/test/dummy/config/application.rb

@@ -20,7 +20,7 @@ module Dummy
     # config.i18n.default_locale = :de
 
     # Do not swallow errors in after_commit/after_rollback callbacks.
-    config.active_record.raise_in_transactional_callbacks = true
+    # config.active_record.raise_in_transactional_callbacks = true
   end
 end
 

data/test/dummy/db/migrate/20150608130516_create_date_field.rb

@@ -1,4 +1,4 @@
-class CreateDateField < ActiveRecord::Migration
+class CreateDateField < ActiveRecord::Migration[4.2]
   def change
     create_table :date_fields do |t|
       t.date :field

data/test/dummy/db/migrate/20150608131430_create_integer_field.rb

@@ -1,4 +1,4 @@
-class CreateIntegerField < ActiveRecord::Migration
+class CreateIntegerField < ActiveRecord::Migration[4.2]
   def change
     create_table :integer_fields do |t|
       t.integer :field

data/test/dummy/db/migrate/20150608131603_create_decimal_field.rb

@@ -1,4 +1,4 @@
-class CreateDecimalField < ActiveRecord::Migration
+class CreateDecimalField < ActiveRecord::Migration[4.2]
   def change
     create_table :decimal_fields do |t|
       t.decimal :field

data/test/dummy/db/migrate/20150608131610_create_float_field.rb

@@ -1,4 +1,4 @@
-class CreateFloatField < ActiveRecord::Migration
+class CreateFloatField < ActiveRecord::Migration[4.2]
   def change
     create_table :float_fields do |t|
       t.float :field

data/test/dummy/db/migrate/20150608132159_create_boolean_field.rb

@@ -1,4 +1,4 @@
-class CreateBooleanField < ActiveRecord::Migration
+class CreateBooleanField < ActiveRecord::Migration[4.2]
   def change
     create_table :boolean_fields do |t|
       t.boolean :field

data/test/dummy/db/migrate/20150608132621_create_string_field.rb

@@ -1,4 +1,4 @@
-class CreateStringField < ActiveRecord::Migration
+class CreateStringField < ActiveRecord::Migration[4.2]
   def change
     create_table :string_fields do |t|
       t.string :field

data/test/dummy/db/migrate/20150608133038_create_belongs_to_field.rb

@@ -1,4 +1,4 @@
-class CreateBelongsToField < ActiveRecord::Migration
+class CreateBelongsToField < ActiveRecord::Migration[4.2]
   def change
     create_table :belongs_to_fields do |t|
       t.references :has_one_field, index: true

data/test/dummy/db/migrate/20150608133044_create_has_one_field.rb

@@ -1,4 +1,4 @@
-class CreateHasOneField < ActiveRecord::Migration
+class CreateHasOneField < ActiveRecord::Migration[4.2]
   def change
     create_table :has_one_fields do |t|
       t.boolean :checked

data/test/dummy/db/migrate/20150608150016_create_has_many_field.rb

@@ -1,4 +1,4 @@
-class CreateHasManyField < ActiveRecord::Migration
+class CreateHasManyField < ActiveRecord::Migration[4.2]
   def change
     create_table :has_many_fields do |t|
       t.references :has_many_through_field, index: true

data/test/dummy/db/migrate/20150609114636_create_belongs_to_class_name_field.rb

@@ -1,4 +1,4 @@
-class CreateBelongsToClassNameField < ActiveRecord::Migration
+class CreateBelongsToClassNameField < ActiveRecord::Migration[4.2]
   def change
     create_table :belongs_to_class_name_fields do |t|
       t.references :foo, index: true

data/test/dummy/db/migrate/20150612112520_create_has_and_belongs_to_many_field.rb

@@ -1,4 +1,4 @@
-class CreateHasAndBelongsToManyField < ActiveRecord::Migration
+class CreateHasAndBelongsToManyField < ActiveRecord::Migration[4.2]
   def change
     create_table :has_and_belongs_to_many_fields do |t|
     end

data/test/dummy/db/migrate/20150616150629_create_polymorphic_field.rb

@@ -1,4 +1,4 @@
-class CreatePolymorphicField < ActiveRecord::Migration
+class CreatePolymorphicField < ActiveRecord::Migration[4.2]
   def change
     create_table :polymorphic_fields do |t|
       t.references :has_one_field, index: true

data/test/dummy/db/migrate/20150623115554_create_has_many_class_name_field.rb

@@ -1,4 +1,4 @@
-class CreateHasManyClassNameField < ActiveRecord::Migration
+class CreateHasManyClassNameField < ActiveRecord::Migration[4.2]
   def change
     create_table :has_many_class_name_fields do |t|
     end

data/test/dummy/db/migrate/20150814081918_create_has_many_through_field.rb

@@ -1,4 +1,4 @@
-class CreateHasManyThroughField < ActiveRecord::Migration
+class CreateHasManyThroughField < ActiveRecord::Migration[4.2]
   def change
     create_table :has_many_through_fields do |t|
     end

data/test/dummy/db/migrate/20160627172810_create_owner.rb

@@ -1,4 +1,4 @@
-class CreateOwner < ActiveRecord::Migration
+class CreateOwner < ActiveRecord::Migration[4.2]
   def change
     create_table :owners do |t|
       t.string :name

data/test/dummy/db/migrate/20160627172951_create_tree.rb

@@ -1,4 +1,4 @@
-class CreateTree < ActiveRecord::Migration
+class CreateTree < ActiveRecord::Migration[4.2]
   def change
     create_table :trees do |t|
       t.string :name

data/test/dummy/db/migrate/20160628173505_add_timestamps.rb

@@ -1,4 +1,4 @@
-class AddTimestamps < ActiveRecord::Migration
+class AddTimestamps < ActiveRecord::Migration[4.2]
   def change
     add_column :owners, :created_at, :datetime
     add_column :owners, :updated_at, :datetime

data/test/dummy/db/migrate/20170614141921_create_serialize_field.rb

@@ -1,4 +1,4 @@
-class CreateSerializeField < ActiveRecord::Migration
+class CreateSerializeField < ActiveRecord::Migration[4.2]
   def change
     create_table :serialize_fields do |t|
       t.string :field