arpa 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 63b751460fc4d234d09190f15db5ee4d61fa7712
-  data.tar.gz: d2f7431ad04dbb1a81871b4752733ffdd50cb2ea
+  metadata.gz: 1aed68f2437222532064f2a8edbff5caf37bf9c8
+  data.tar.gz: 02ce1a1c1deb60c086bb0bcdfab56389f1927a14
 SHA512:
-  metadata.gz: 20ac34de3dbbef153702470ee29da44c0cd056ed99ce477edd1f1beb5f278ed80a21f0621b01e7783c73c12a08cd882b7dd074df9e0c78c06512729fae852dd1
-  data.tar.gz: aeb00bc408614baeecf5ccc5c8a1aaea7998f5409c8bd2166efc3a8fe2ba58d1aa41be9b0b1d20608ae3d8807f3faeadbaac919c876177c4b25ea17d0daed564
+  metadata.gz: d130875b378e74eeb65141f715e976365f5c790d74531ca87934bd45090a24a70e8086952482ad55d351d7fe3fcb5f4fce3a79281ec531c0ac127c710ad4381b
+  data.tar.gz: fb3c39a4ce49b9238dc7ae6c587aa556982776bae00beec2338a74374d05a1939eec135ca11e38edf6bfd3591db08fa4fa2b575b00afb8d3fee817fb7ceae3c3

data/lib/arpa/additions/resource.rb

@@ -8,8 +8,15 @@ module Arpa
 
       def has_access?(resource, action)
         return unless has_session_or_current_user?
+        verifier.has_access?(resource, action)
+      end
+
+      def reset_permissions
+        verifier.reset_permissions
+      end
+
+      def verifier
         @verifier ||= Arpa::Services::Verifier.new(session, current_user)
-        @verifier.has_access?(resource, action)
       end
 
       private

data/lib/arpa/entities/action.rb

@@ -15,8 +15,12 @@ module Arpa
         @updated_at  = attrs[:updated_at]
       end
 
+      def resource_name
+        resource.name
+      end
+
       def description
-        I18n.t(name.to_sym, scope: "entities.resources.#{resource.name}.actions.description")
+        I18n.t(name.to_sym, scope: "entities.resources.#{resource_name}.actions.description")
       end
 
     end

data/lib/arpa/entities/permissions.rb

@@ -4,8 +4,8 @@ module Arpa
 
       attr_reader :permissions
 
-      def initialize(resources)
-        @resources   = resources
+      def initialize(actions)
+        @actions   = actions
         @permissions = Array.new
         build_permissions
       end
@@ -17,11 +17,9 @@ module Arpa
 
       private
       def build_permissions
-        @resources.each do |resource|
-          resource.actions.collect do |action|
-            hash_permission = build_hash_permission(resource.name, action.name)
-            @permissions << hash_permission unless permissions.include?(hash_permission)
-          end
+        @actions.collect do |action|
+          hash_permission = build_hash_permission(action.resource_name, action.name)
+          @permissions << hash_permission unless permissions.include?(hash_permission)
         end
       end
 

data/lib/arpa/repositories/actions/finder.rb

@@ -9,6 +9,17 @@ module Arpa
           mapper_instance.map_to_entity(record) if record
         end
 
+        def permissions(profile_ids)
+         records = repository_class.joins(roles: :profiles)
+           .where(repository_profiles: {id: profile_ids})
+
+          actions = records.collect do |action|
+            mapper_instance.map_to_entity(action)
+          end
+
+          Arpa::Entities::Permissions.new(actions)
+        end
+
         def mapper_instance
           Arpa::DataMappers::ActionMapper.instance
         end

data/lib/arpa/repositories/resources/finder.rb

@@ -20,17 +20,6 @@ module Arpa
           mapper_instance.map_to_entity(record) if record
         end
 
-        def permissions(profile_ids)
-         records = repository_class.joins(actions: [roles: :profiles])
-           .where(repository_profiles: {id: profile_ids})
-
-          resources = records.collect do |record|
-            mapper_instance.map_to_entity(record)
-          end
-
-          Arpa::Entities::Permissions.new(resources)
-        end
-
         def mapper_instance
           Arpa::DataMappers::ResourceMapper.instance
         end

data/lib/arpa/services/verifier.rb

@@ -10,14 +10,18 @@ module Arpa
       def has_access?(resource, action)
         free_access_action = action.to_s.split('_')[0]
         return true if free_access_action.empty?
-        @session[:entity_permissions] ||= resource_finder.permissions(@current_user.profile_ids)
+        @session[:entity_permissions] ||= action_finder.permissions(@current_user.profile_ids)
         @session[:entity_permissions].has_permission?(resource.to_s, action.to_s)
       end
 
+      def reset_permissions
+        @session[:entity_permissions] = nil
+      end
+
       private
 
-      def resource_finder
-        @resource_finder ||= Arpa::Repositories::Resources::Finder.new
+      def action_finder
+        @action_finder ||= Arpa::Repositories::Actions::Finder.new
       end
     end
   end

data/lib/arpa/version.rb

@@ -1,3 +1,3 @@
 module Arpa
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/spec/lib/ar/additions/resource_spec.rb

@@ -11,13 +11,18 @@ describe Arpa::Additions::Resource, type: :addition, fast: true do
 
   let(:resource_implementation) { ResourceImplementation.new }
 
+  let(:verifier) { double }
+  let(:resource) { double }
+  let(:action)   { double }
+
+  before do
+    allow(Arpa::Services::Verifier).to receive(:new).and_return(verifier)
+    setup
+  end
+
   describe '#has_access?' do
-    let(:verifier) { double }
-    let(:resource) { double }
-    let(:action)   { double }
 
-    before do
-      allow(Arpa::Services::Verifier).to receive(:new).and_return(verifier)
+    let(:setup) do
       allow(verifier).to receive(:has_access?)
       resource_implementation.has_access?(resource, action)
     end
@@ -31,4 +36,20 @@ describe Arpa::Additions::Resource, type: :addition, fast: true do
     end
   end
 
+   describe '#reset_permissions' do
+
+     let(:setup) do
+      allow(verifier).to receive(:reset_permissions)
+      resource_implementation.reset_permissions
+    end
+
+    it 'should call :new from Arpa::Services::Verifier' do
+      expect(Arpa::Services::Verifier).to have_received(:new).once
+    end
+
+    it 'verifier should call :reset_permissions once' do
+      expect(verifier).to have_received(:reset_permissions).once
+    end
+  end
+
 end

data/spec/lib/ar/entities/permissions_spec.rb

@@ -1,12 +1,12 @@
 require 'spec_helper'
 
 describe Arpa::Entities::Permissions, type: :presenter, fast: true do
-  let(:index_action) { double name: 'index_new' }
-  let(:resource_001) { double name: 'users', actions: [index_action] }
-  let(:resource_002) { double name: 'users', actions: [index_action] }
-  let(:resources)    { [resource_001, resource_002] }
+  let(:action_001) { double name: 'index', resource_name: 'users' }
+  let(:action_002) { double name: 'show', resource_name: 'users' }
+  let(:action_003) { double name: 'show', resource_name: 'users' }
+  let(:actions)    { [action_001, action_002, action_003] }
 
-  subject { Arpa::Entities::Permissions.new(resources)}
+  subject { Arpa::Entities::Permissions.new(actions)}
 
   describe 'intializing' do
     let(:result) { subject.permissions }
@@ -24,7 +24,6 @@ describe Arpa::Entities::Permissions, type: :presenter, fast: true do
     end
 
     context 'when try fill permissions with duplicate resource and action' do
-      let(:hash_permission) { {'resource' => 'users', 'action' => 'index_new' } }
 
       it 'should has only one Hash with that duplicated resource and action' do
         duplicated_hash_permission = result.detect{ |e| result.count(e) > 1 }
@@ -39,7 +38,7 @@ describe Arpa::Entities::Permissions, type: :presenter, fast: true do
     let(:result) { subject.has_permission?(resource_name, action_name) }
 
     context 'when has' do
-      let(:action_name) { 'index_new' }
+      let(:action_name) { 'index' }
 
       it 'should return true' do
         expect(result).to be_truthy
@@ -48,7 +47,7 @@ describe Arpa::Entities::Permissions, type: :presenter, fast: true do
     end
 
     context 'when has not' do
-      let(:action_name) { 'index' }
+      let(:action_name) { 'index_old' }
 
       it 'should return false' do
         expect(result).to be_falsey

data/spec/lib/ar/repositories/actions/finder_spec.rb

@@ -37,4 +37,22 @@ describe Arpa::Repositories::Actions::Finder, type: :repository, slow: true do
 
   end
 
+  describe 'getting permissions from profile_ids' do
+    let(:profile_001) { create :repository_profile, :with_complete_association, name: 'prof_001' }
+    let(:profile_002) { create :repository_profile, :with_complete_association, name: 'prof_002' }
+    let(:profile_ids) { [profile_001.id, profile_002.id] }
+
+    let(:result) { subject.permissions(profile_ids) }
+    let(:first)  { result.first }
+
+    it 'should return an Arpa::Entities::Permissions' do
+      expect(result).to be_an Arpa::Entities::Permissions
+    end
+
+    it 'should has just one permission' do
+      expect(result.permissions.size).to be == 1
+    end
+
+  end
+
 end

data/spec/lib/ar/repositories/resources/finder_spec.rb

@@ -71,22 +71,4 @@ describe Arpa::Repositories::Resources::Finder, type: :repository, slow: true do
     end
 
   end
-
-  describe 'getting permissions from profile_ids' do
-    let(:profile_001) { create :repository_profile, :with_complete_association, name: 'prof_001' }
-    let(:profile_002) { create :repository_profile, :with_complete_association, name: 'prof_002' }
-    let(:profile_ids) { [profile_001.id, profile_002.id] }
-
-    let(:result) { subject.permissions(profile_ids) }
-    let(:first)  { result.first }
-
-    it 'should return an Arpa::Entities::Permissions' do
-      expect(result).to be_an Arpa::Entities::Permissions
-    end
-
-    it 'should has just one permission' do
-      expect(result.permissions.size).to be == 1
-    end
-
-  end
 end

data/spec/lib/ar/services/verifier_spec.rb

@@ -6,6 +6,16 @@ describe Arpa::Services::Verifier do
 
   subject(:verifier) { Arpa::Services::Verifier.new(session, current_user) }
 
+  describe 'cleaning session of permissions' do
+    let(:session) { {entity_permissions: [ {'resource': 'users', 'action': 'index'} ] } }
+
+    before { verifier.reset_permissions }
+
+    it 'session[:entity_permissions] should be set to nil' do
+      expect(session[:entity_permissions]).to be_nil
+    end
+  end
+
   describe '#has_access?' do
     context 'when pass a free action which begin with "_"' do
       it 'should has access' do
@@ -14,24 +24,24 @@ describe Arpa::Services::Verifier do
     end
 
     context 'when pass a non free action' do
-      let(:resource_finder_class) { Arpa::Repositories::Resources::Finder }
-      let(:resource_finder)       { instance_double resource_finder_class }
-      let(:entity_permissions)    { double }
+      let(:action_finder_class) { Arpa::Repositories::Actions::Finder }
+      let(:action_finder)       { instance_double action_finder_class }
+      let(:entity_permissions)  { double }
 
       before do
-        allow(resource_finder_class).to receive(:new).and_return(resource_finder)
-        allow(resource_finder).to receive(:permissions).with([1,2,3]).and_return(entity_permissions)
+        allow(action_finder_class).to receive(:new).and_return(action_finder)
+        allow(action_finder).to receive(:permissions).with([1,2,3]).and_return(entity_permissions)
         allow(entity_permissions).to receive(:has_permission?).with('users', 'index')
 
         subject.has_access?('users', 'index')
       end
 
-      it 'should be called :new from Arpa::Repositories::Resources::Finder' do
-        expect(resource_finder_class).to have_received(:new).once
+      it 'should be called :new from Arpa::Repositories::Actions::Finder' do
+        expect(action_finder_class).to have_received(:new).once
       end
 
-      it 'should be called :permissions from Arpa::Repositories::Resources::Finder' do
-        expect(resource_finder).to have_received(:permissions).with([1,2,3]).once
+      it 'should be called :permissions from Arpa::Repositories::Actions::Finder' do
+        expect(action_finder).to have_received(:permissions).with([1,2,3]).once
       end
 
       it 'should be called :has_permission? from session[:entity_permissions]' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: arpa
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Rachid Calazans
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-25 00:00:00.000000000 Z
+date: 2015-11-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -170,6 +170,9 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- arpa-0.0.2.gem
+- arpa-0.0.3.gem
+- arpa-0.0.4.gem
 - arpa.gemspec
 - lib/arpa.rb
 - lib/arpa/additions/resource.rb