consent 0.4 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81dca1628aeb1c52c3d9d3b892e3c148a9556bf2a2c0c5199ddd6c160155897a
-  data.tar.gz: 3e77560cbe9f555e8f8edd42186b3e61b5b2617aa5cc040255bbf9d42c25dbc2
+  metadata.gz: 5f54a2338cba28ba0e887ec44e7e43f9aae2209d4b1017638eaf4334564eae2c
+  data.tar.gz: 2dc433ca07b4615fc56a344d32c3b6adfc5267420419d6d1fd201040ee9bdb31
 SHA512:
-  metadata.gz: 5a4b2b6f8d4b797fbe2b7ce0c272f2bd968fcfb059f0545ff31b230ecd0ea62bf0a198940160dab1c494295f08b149fb0b553903896089a0ff93fe8ab45bdce2
-  data.tar.gz: 639e12a757f9708cb063d20e77ebbf3d89029811f73e1c7b8730437029aab729d2c4edfa296ce208860c8eca96781b24dd1460425a3d77d5831c5d45166e492a
+  metadata.gz: e1ddfc63c31df91ccac74ada8806408353e81f2626d0592dcb954f34c3ffd03aacde585c0f625d9534cd749b2999deebb05860cd2cc3be34ba9b40c2a8645ac4
+  data.tar.gz: 5389891aeef0e65d3752ada1c647c9e0dc1c60debb67fc22019f067dbeb4a4b0c5ac07ab14d0fb760680818c38beab6e4fb4c225f5d07639945b9377cf2dd0bd

data/.rubocop.yml

@@ -0,0 +1 @@
+inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -0,0 +1,21 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2019-11-20 02:06:29 -0300 using RuboCop version 0.65.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 5
+# Configuration parameters: CountComments, ExcludedMethods.
+# ExcludedMethods: refine
+Metrics/BlockLength:
+  Exclude:
+    - 'spec/**/*'
+    - 'lib/consent/rspec.rb'
+
+# Offense count: 9
+Style/Documentation:
+  Exclude:
+    - 'spec/**/*'
+    - 'test/**/*'

data/.travis.yml

@@ -2,7 +2,11 @@ sudo: false
 language: ruby
 rvm:
 - 2.2.2
-before_install: gem install bundler -v 1.12.1
+- 2.5.0
+before_install: gem install bundler -v 1.17.3
+script:
+  - bundle exec rubocop
+  - bundle exec rspec
 deploy:
   provider: rubygems
   api_key:
@@ -11,3 +15,4 @@ deploy:
   on:
     tags: true
     repo: powerhome/consent
+    ruby: 2.5.0

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in consent.gemspec

data/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright Power Home Remodeling Group, LLC
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

data/Rakefile

@@ -1,6 +1,8 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+# frozen_string_literal: true
+
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec

data/bin/console

@@ -1,7 +1,8 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
-require "bundler/setup"
-require "consent"
+require 'bundler/setup'
+require 'consent'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -10,5 +11,5 @@ require "consent"
 # require "pry"
 # Pry.start
 
-require "irb"
+require 'irb'
 IRB.start

data/consent.gemspec

@@ -1,5 +1,6 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+# frozen_string_literal: true
+
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'consent/version'
 
@@ -12,14 +13,17 @@ Gem::Specification.new do |spec|
   spec.summary       = 'Consent'
   spec.description   = 'Consent'
 
+  spec.licenses = ['MIT']
+
   spec.files = `git ls-files`.split.reject do |file|
     file =~ /^(test|spec|features)/
   end
   spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'activesupport', '~> 3.2.22'
+  spec.add_development_dependency 'activesupport', '>= 4.1.11'
+  spec.add_development_dependency 'bundler', '>= 1.17.3'
   spec.add_development_dependency 'cancancan', '~> 1.15.0'
-  spec.add_development_dependency 'bundler', '~> 2.0'
   spec.add_development_dependency 'rake', '~> 10.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'rubocop', '~> 0.65.0'
 end

data/lib/consent.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'consent/version'
 require 'consent/subject'
 require 'consent/view'
@@ -8,23 +10,42 @@ require 'consent/permissions'
 require 'consent/ability' if defined?(CanCan)
 require 'consent/railtie' if defined?(Rails)
 
+# Consent makes defining permissions easier by providing a clean,
+# concise DSL for authorization so that all abilities do not have
+# to be in your `Ability` class.
 module Consent
-  FULL_ACCESS = %w(1 true).freeze
+  FULL_ACCESS = %w[1 true].freeze
 
+  # Default views available to every permission
+  #
+  # i.e.:
+  #  Defining a view with no conditions:
+  #  Consent.default_views[:all] = Consent::View.new(:all, "All")
+  #
+  # @return [Hash<Symbol,Consent::View>]
   def self.default_views
     @default_views ||= {}
   end
 
+  # Subjects defined in Consent
+  #
+  # @return [Array<Consent::Subject>]
   def self.subjects
     @subjects ||= []
   end
 
+  # Finds all subjects defined with the given key
+  #
+  # @return [Array<Consent::Subject>]
   def self.find_subjects(subject_key)
-    @subjects.find do |subject|
+    @subjects.find_all do |subject|
       subject.key.eql?(subject_key)
     end
   end
 
+  # Finds an action within a subject context
+  #
+  # @return [Consent::Action,nil]
   def self.find_action(subject_key, action_key)
     Consent.find_subjects(subject_key)
            .map(&:actions).flatten
@@ -33,18 +54,36 @@ module Consent
            end
   end
 
+  # Finds a view within a subject context
+  #
+  # @return [Consent::View,nil]
   def self.find_view(subject_key, view_key)
     views = Consent.find_subjects(subject_key)
-                   .map{|subject| subject.views}
+                   .map(&:views)
                    .reduce({}, &:merge)
     views[view_key]
   end
 
-  def self.load_subjects!(paths)
-    permission_files = paths.map { |dir| dir.join('*.rb') }
-    Dir[*permission_files].each(&Kernel.method(:load))
+  # Loads all permission (ruby) files from the given directory
+  # and using the given mechanism (default: :require)
+  #
+  # @param paths [Array<String,#to_s>] paths where the ruby files are located
+  # @param mechanism [:require,:load] mechanism to load the files
+  def self.load_subjects!(paths, mechanism = :require)
+    permission_files = paths.map { |dir| File.join(dir, '*.rb') }
+    Dir[*permission_files].each(&Kernel.method(mechanism))
   end
 
+  # Defines a subject with the given key, label and options
+  #
+  # i.e:
+  #   Consent.define :users, "User management" do
+  #     view :department, "Same department only" do |user|
+  #       { department_id: user.department_id }
+  #     end
+  #     action :read, "Can view users"
+  #     action :update, "Can edit existing user", views: :department
+  #   end
   def self.define(key, label, options = {}, &block)
     defaults = options.fetch(:defaults, {})
     subjects << Subject.new(key, label).tap do |subject|
@@ -52,6 +91,9 @@ module Consent
     end
   end
 
+  # Maps a permissions hash to a Consent::Permissions
+  #
+  # @return [Consent::Permissions]
   def self.permissions(permissions)
     Permissions.new(permissions)
   end

data/lib/consent/ability.rb

@@ -1,12 +1,15 @@
+# frozen_string_literal: true
+
 module Consent
+  # Defines a CanCan(Can)::Ability class based on a permissions hash
   class Ability
     include CanCan::Ability
 
     def initialize(permissions, *args)
       Consent.permissions(permissions).each do |permission|
         conditions = permission.conditions(*args)
-        object_conditions = permission.object_conditions(*args)
-        can permission.action_key, permission.subject_key, conditions, &object_conditions
+        ocond = permission.object_conditions(*args)
+        can permission.action_key, permission.subject_key, conditions, &ocond
       end
     end
   end

data/lib/consent/action.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Consent
-  class Action
+  class Action # :nodoc:
     attr_reader :key, :label, :options
 
     def initialize(key, label, options = {})

data/lib/consent/dsl.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Consent
-  class DSL
+  class DSL # :nodoc:
     attr_reader :subject
 
     def initialize(subject, defaults)
@@ -11,13 +13,13 @@ module Consent
       DSL.build(@subject, @defaults.merge(new_defaults), &block)
     end
 
-    # rubocop:disable Link/UnusedBlockArgument, Link/Eval
+    # rubocop:disable Lint/UnusedBlockArgument, Security/Eval
     def eval_view(key, label, collection_conditions)
       view key, label do |user|
         eval(collection_conditions)
       end
     end
-    # rubocop:enable Link/UnusedBlockArgument, Link/Eval
+    # rubocop:enable Lint/UnusedBlockArgument, Security/Eval
 
     def view(key, label, instance = nil, collection = nil, &block)
       collection ||= block

data/lib/consent/permission.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Consent
-  class Permission
+  class Permission # :nodoc:
     def initialize(subject, action, view = nil)
       @subject = subject
       @action = action
@@ -14,6 +16,9 @@ module Consent
       @action.key
     end
 
+    # Disables Sytle/SafeNavigation to keep this code
+    # compatible with ruby < 2.3
+    # rubocop:disable Style/SafeNavigation
     def view_key
       @view && @view.key
     end
@@ -25,5 +30,6 @@ module Consent
     def object_conditions(*args)
       @view && @view.object_conditions(*args)
     end
+    # rubocop:enable Style/SafeNavigation
   end
 end

data/lib/consent/permissions.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Consent
-  class Permissions
+  class Permissions # :nodoc:
     include Enumerable
 
     def initialize(permissions)
@@ -25,12 +27,14 @@ module Consent
 
     def full(subject, action, view_key)
       return unless Consent::FULL_ACCESS.include?(view_key.to_s.strip)
+
       Permission.new(subject, action)
     end
 
     def partial(subject, action, view_key)
       view = subject.view_for(action, view_key.to_s.to_sym)
       return if view.nil?
+
       Permission.new(subject, action, view)
     end
   end

data/lib/consent/railtie.rb

@@ -1,19 +1,26 @@
+# frozen_string_literal: true
+
+require 'consent/reloader'
+
 module Consent
   # Plugs consent permission load to the Rails class loading cycle
   class Railtie < Rails::Railtie
-    config.before_configuration do
-      default_path = Rails.root.join('app', 'permissions')
-      config.consent = Struct.new(:paths).new([default_path])
+    config.before_configuration do |app|
+      default_path = app.root.join('app', 'permissions')
+      config.consent = Consent::Reloader.new(
+        default_path,
+        ActiveSupport::Dependencies.mechanism
+      )
     end
 
-    config.to_prepare do
-      Consent.subjects.clear
-      Consent.load_subjects! Rails.application.config.consent.paths
+    config.after_initialize do |app|
+      app.config.consent.execute
     end
 
-    config.after_initialize do
-      permissions_paths = config.consent.paths.map(&:to_s)
-      ActiveSupport::Dependencies.autoload_paths -= permissions_paths
+    initializer 'initialize consent permissions reloading' do |app|
+      app.reloaders << config.consent
+      ActiveSupport::Dependencies.autoload_paths -= config.consent.paths
+      config.to_prepare { app.config.consent.execute }
     end
   end
 end

data/lib/consent/reloader.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Consent
+  # Rails file reloader to detect permission changes and apply them to consent
+  class Reloader
+    attr_reader :paths
+    delegate :updated?, :execute, :execute_if_updated, to: :updater
+
+    def initialize(default_path, mechanism)
+      @paths = [default_path]
+      @mechanism = mechanism
+    end
+
+    private
+
+    def reload!
+      Consent.subjects.clear
+      Consent.load_subjects! paths, @mechanism
+    end
+
+    def updater
+      @updater ||= ActiveSupport::FileUpdateChecker.new([], globs) { reload! }
+    end
+
+    def globs
+      pairs = paths.map { |path| [path.to_s, %w[rb]] }
+      Hash[pairs]
+    end
+  end
+end

data/lib/consent/rspec.rb

@@ -1,6 +1,33 @@
+# frozen_string_literal: true
+
 require 'consent'
 
 module Consent
+  # RSpec helpers for consent. Given permissions are loaded,
+  # gives you the ability of defining permission specs like
+  #
+  # Given "users" permissions
+  # Consent.define :users, "User management" do
+  #   view :department, "Same department only" do |user|
+  #     { department_id: user.department_id }
+  #   end
+  #   action :read, "Can view users"
+  #   action :update, "Can edit existing user", views: :department
+  # end
+  #
+  # RSpec.describe "User permissions" do
+  #   include Consent::Rspec
+  #   let(:user) { double(department_id: 15) }
+  #
+  #   it do
+  #     is_expected.to consent_view(:department, department_id: 15).to(user)
+  #   end
+  #   it { is_expected.to consent_action(:read) }
+  #   it { is_expected.to consent_action(:update).with_views(:department) }
+  # end
+  #
+  # Find more examples at:
+  # https://github.com/powerhome/consent
   module Rspec
     extend RSpec::Matchers::DSL
 
@@ -11,17 +38,25 @@ module Consent
 
       match do |subject_key|
         action = Consent.find_action(subject_key, action_key)
-        action && @views ? action.view_keys.sort.eql?(@views.sort) : !action.nil?
+        if action && @views
+          values_match?(action.view_keys.sort, @views.sort)
+        else
+          !action.nil?
+        end
       end
 
       failure_message do |subject_key|
         action = Consent.find_action(subject_key, action_key)
-        message = "expected %s (%s) to provide action %s" % [
-          subject_key.to_s, subject.class, action_key
-        ]
+        message = format(
+          'expected %<skey>s (%<sclass>s) to provide action %<action>s',
+          skey: subject_key.to_s, sclass: subject.class, action: action_key
+        )
 
         if action && @views
-          '%s with views %s, but actual views are %p' % [message, @views, action.view_keys]
+          format(
+            '%<message>s with views %<views>s, but actual views are %<keys>p',
+            message: message, views: @views, keys: action.view_keys
+          )
         else
           message
         end
@@ -35,21 +70,36 @@ module Consent
 
       match do |subject_key|
         view = Consent.find_view(subject_key, view_key)
-        conditions ? view.try(:conditions, *@context).eql?(conditions) : !view.nil?
+        if conditions
+          view&.conditions(*@context).eql?(conditions)
+        else
+          !view.nil?
+        end
       end
 
       failure_message do |subject_key|
         view = Consent.find_view(subject_key, view_key)
-        message = "expected %s (%s) to provide view %s with %p, but" % [
-          subject_key.to_s, subject.class, view_key, conditions
-        ]
+        message = format(
+          'expected %<skey>s (%<sclass>s) to provide view %<view>s with` \
+          `%<conditions>p, but',
+          skey: subject_key.to_s, sclass: subject.class,
+          view: view_key, conditions: conditions
+        )
 
         if view && conditions
           actual_conditions = view.conditions(*@context)
-          '%s conditions are %p' % [message, actual_conditions]
+          format(
+            '%<message>s conditions are %<conditions>p',
+            message: message, conditions: actual_conditions
+          )
         else
-          actual_views = Consent.find_subjects(subject_key).map(&:views).map(&:keys).flatten
-          '%s available views are %p' % [message, actual_views]
+          actual_views = Consent.find_subjects(subject_key)
+                                .map(&:views)
+                                .map(&:keys).flatten
+          format(
+            '%<message>s available views are %<views>p',
+            message: message, views: actual_views
+          )
         end
       end
     end

data/lib/consent/subject.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Consent
-  class Subject
+  class Subject # :nodoc:
     attr_reader :key, :label, :actions, :views
 
     def initialize(key, label)

data/lib/consent/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Consent
-  VERSION = '0.4'.freeze
+  VERSION = '0.6.0'
 end

data/lib/consent/view.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Consent
-  class View
+  class View # :nodoc:
     attr_reader :key, :label
 
     def initialize(key, label, instance = nil, collection = nil)
@@ -11,11 +13,13 @@ module Consent
 
     def conditions(*args)
       return @collection unless @collection.respond_to?(:call)
+
       @collection.call(*args)
     end
 
     def object_conditions(*args)
       return @instance unless @instance.respond_to?(:curry)
+
       @instance.curry[*args]
     end
   end

data/lib/generators/consent/permissions_generator.rb

@@ -1,14 +1,21 @@
+# frozen_string_literal: true
+
 module Consent
-  class PermissionsGenerator < Rails::Generators::NamedBase
-    source_root File.expand_path('../templates', __FILE__)
+  class PermissionsGenerator < Rails::Generators::NamedBase # :nodoc:
+    source_root File.expand_path('templates', __dir__)
     argument :description, type: :string, required: false
 
     def create_permissions
-      template "permissions.rb.erb", "app/permissions/#{file_path}.rb", assigns: {
-        description: description
-      }
+      template(
+        'permissions.rb.erb',
+        "app/permissions/#{file_path}.rb",
+        assigns: { description: description }
+      )
 
-      template "permissions_spec.rb.erb", "spec/permissions/#{file_path}_spec.rb"
+      template(
+        'permissions_spec.rb.erb',
+        "spec/permissions/#{file_path}_spec.rb"
+      )
     end
   end
 end

metadata

@@ -1,57 +1,57 @@
 --- !ruby/object:Gem::Specification
 name: consent
 version: !ruby/object:Gem::Version
-  version: '0.4'
+  version: 0.6.0
 platform: ruby
 authors:
 - Carlos Palhares
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-27 00:00:00.000000000 Z
+date: 2020-12-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.2.22
+        version: 4.1.11
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.2.22
+        version: 4.1.11
 - !ruby/object:Gem::Dependency
-  name: cancancan
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.15.0
+        version: 1.17.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.15.0
+        version: 1.17.3
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: cancancan
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: 1.15.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: 1.15.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.65.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.65.0
 description: Consent
 email:
 - chjunior@gmail.com
@@ -89,9 +103,12 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
+- ".rubocop_todo.yml"
 - ".ruby-version"
 - ".travis.yml"
 - Gemfile
+- LICENSE
 - README.md
 - Rakefile
 - TODO.md
@@ -105,6 +122,7 @@ files:
 - lib/consent/permission.rb
 - lib/consent/permissions.rb
 - lib/consent/railtie.rb
+- lib/consent/reloader.rb
 - lib/consent/rspec.rb
 - lib/consent/subject.rb
 - lib/consent/version.rb
@@ -113,7 +131,8 @@ files:
 - lib/generators/consent/templates/permissions.rb.erb
 - lib/generators/consent/templates/permissions_spec.rb.erb
 homepage: 
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []