passpartu 1.0.0 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 982e32959b5022a455e42d2c1de190c0851f358cfd34c699ece8394e6af0d951
-  data.tar.gz: 2e81288b0af25837d0d4d97b14562e33f2813ca0b8741adbce66451568a99487
+  metadata.gz: 7ff6ec8f5f35ec4a606df7c651868203f755d0d6f5b6152d23a152fa91d45126
+  data.tar.gz: 4ed57d9ced50f0c8ab11725835be3cc2ecb37a9309433146429312d22417dd13
 SHA512:
-  metadata.gz: 3cebce1276892e0fd925be52bc3ed817b83b6aeb96d9b056b9d999d288b9cbe891189f36a8d6b904b1dac53bbd90eb10b2d6628e42f727d063bc300abbeb697d
-  data.tar.gz: 6caeffe92ffe0f8fe158c0d10c9d4eca18e88ac3acad3c14d7346673ca6740c7335564d53f10a9263f6ae2bcc533077fb18d84cfb25c68cac71717e7b4c01ff7
+  metadata.gz: c55101f249104c06a591cffa4df5126e3bf01a0f81a1bcd96d766b91d657f7997cfaebc81ae22e4da56265951ec47b751565c97a220ec17042945b9ba3e0fbb7
+  data.tar.gz: 5618f1ecf4e1b8ad3b27f802cefd2213e65ee08125e840aa32aab603b1ca85868fc4d79464bcacf4d9be75103174cff9474c05ef8d6d3058d0fd56381cc86f69

data/README.md

@@ -1,8 +1,14 @@
-# Passpartu v1.0.0 - [changelog](https://github.com/coaxsoft/passpartu/blob/master/CHANGELOG.md)
+# Passpartu v1.0.3 - [changelog](https://github.com/coaxsoft/passpartu/blob/master/CHANGELOG.md)
 
 Passpartu makes policies great again (works awesome with [Pundit](https://rubygems.org/gems/pundit)).
 
+### Tested with ruby:
+- 2.7.3
+- 3.0.0
+- 3.1.1
+
 Instead of this:
+
 ```ruby
 class PostPolicy < ApplicationPolicy
   def update?
@@ -12,6 +18,7 @@ end
 ```
 
 just this:
+
 ```ruby
 class PostPolicy < ApplicationPolicy
   def update?
@@ -19,19 +26,24 @@ class PostPolicy < ApplicationPolicy
   end
 end
 ```
+
 ## Usage
+
 Include `Passpartu` into your policy model.
+
 ```ruby
 class User
   include Passpartu
 end
 ```
+
 NOTE: Your `User` model must respond to `role` method that returns a string or a symbol!
 
 Keep all your policies in one place.
 Create `./config/passpartu.yml` and start writing your policies.
 
 #### Example of `passpartu.yml`
+
 ```yml
 # ./config/passpartu.yml
 manager: &manager
@@ -65,18 +77,25 @@ admin:
 ```
 
 ## Features
+
 #### CRUD
+
 It's possible to use `crud` key to set values for `create`, `read`, `update`, `delete` at once.
 `create`, `read`, `update`, `delete` has higher priority than `crud`
+
 In case `crud: true` and `delete: false` - result `false` 
 
+
 #### Only
+
 It's possible to include specific roles to checks
+
 ```ruby
     user_admin.can?(:orders, :edit) # check policy for admin and returns true if policy true
     user_admin.can?(:orders, :edit, only: :admin) # returns true because the user is an admin and we included only admin
     user_manager.can?(:orders, :edit, only: :admin) # returns false because user is manager and we included only admin
 ```
+
 It's possible to give an array as only attribute
 
 ```ruby
@@ -85,18 +104,21 @@ It's possible to give an array as only attribute
 ```
 
 Note: `only` has higher priority than `except/skip`. Do not use both.
+
 ```ruby
   user_admin.can?(:orders, :edit, only: :admin, except: :admin) # returns true
 ```
 
-
 #### Skip (except)
+
 It's possible to exclude roles from checks
+
 ```ruby
     user_admin.can?(:orders, :edit) # check policy for admin and returns true if policy true
     user_admin.can?(:orders, :edit, except: :admin) # returns false because user is admin and we excluded admin
     
 ```
+
 It's possible to give an array as except attribute
 
 ```ruby
@@ -107,6 +129,7 @@ It's possible to give an array as except attribute
 `skip` alias to `except`
 
 Note: `expect` has higher priority than `skip`. Do not use both.
+
 ```ruby
   user_agent.can?(:orders, :edit, except: [:admin, :manager]) { user_agent.orders.include?(order) }
   # equals to
@@ -114,7 +137,9 @@ Note: `expect` has higher priority than `skip`. Do not use both.
 ```
 
 #### Per role methods
+
 Check user roles AND policy rule
+
 ```ruby
     # check if user admin AND returns true if policy true
     user_admin.admin_can?(:orders, :edit) # true
@@ -124,6 +149,7 @@ Check user roles AND policy rule
 ```
 
 #### Code blocks
+
 ```ruby
   # check rules as usual AND code in the block   
   user_agent.can?(:orders, :edit, except: [:admin, :manager]) { user_agent.orders.include?(order) }
@@ -133,7 +159,9 @@ Check user roles AND policy rule
 ```
 
 #### Waterfall check
+
 Allow or restrict absolutely everything for particular role or/and particular domain.
+
 ```ruby
 # ./config/initializers/passpartu.rb
 
@@ -153,6 +181,7 @@ medium_looser:
     delete: false
   products: true
 ```
+
 ```ruby
 user_super_admin.can?(:do, :whatever, :want) # true
 user_super_loser.can?(:do, :whatever, :want) # false
@@ -161,10 +190,10 @@ user_medium_loser.can?(:orders, :delete) # false
 user_medium_loser.can?(:products, :create) # true
 user_medium_loser.can?(:products, :create, :and_delete) # true
 ```
-
-
 ##### Real life example
+
 You need to check custom rule for agent
+
 ```yml
 # ./config/passpartu.yml
 
@@ -202,9 +231,13 @@ You can configure Passpartu by creating `./config/initializers/passpartu.rb`.
 Passpartu.configure do |config|
   config.policy_file = './config/passpartu.yml'
   config.raise_policy_missed_error = true
+  config.check_waterfall = false
 end
+
 ```
+
 ### Raise policy missed errors
+
 By default Passpartu will raise an PolicyMissedError if policy is missed in `passpartu.yml`. In initializer set  `config.raise_policy_missed_error = false` in order to return `false` in case when policy is not defined. This is a good approach to write only "positive" policies (only true) and automatically restricts everything that is not mentioned in `passpartu.yml`
 
 ## Installation
@@ -244,4 +277,5 @@ The gem is available as open source under the terms of the [MIT License](https:/
 Everyone interacting in the Passpartu project’s codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/coaxsoft/passpartu/blob/master/CODE_OF_CONDUCT.md).
 
 ## Idea
+
 Initially designed and created by [Orest Falchuk](https://github.com/OrestF)

data/lib/passpartu/patcher.rb

@@ -3,6 +3,7 @@
 module Passpartu
   class Patcher
     attr_reader :klass
+
     def initialize(klass)
       raise PolicyYmlNotFoundError if Passpartu.policy.nil?
 
@@ -15,11 +16,11 @@ module Passpartu
 
     def call
       klass.class_eval do
-        define_method('can?') do |*keys, only: nil, except: nil, skip: nil, &block|
+        define_method(:can?) do |*keys, only: nil, except: nil, skip: nil, &block|
           Passpartu::BlockVerify.call(role, keys, only: only, except: except, skip: skip, &block)
         end
 
-        Passpartu.policy.keys.each do |policy_role|
+        Passpartu.policy.each_key do |policy_role|
           define_method("#{policy_role}_can?") do |*keys, only: nil, except: nil, skip: nil, &block|
             role.to_s == policy_role && Passpartu::BlockVerify.call(role, keys,
                                                                     only: only,

data/lib/passpartu/validate_result.rb

@@ -5,6 +5,7 @@ module Passpartu
     class PolicyMissedError < StandardError; end
 
     attr_reader :result
+
     def initialize(result)
       @result = result
     end
@@ -23,7 +24,7 @@ module Passpartu
     private
 
     def boolean?
-      [TrueClass, FalseClass].include?(result.class)
+      [true, false].include?(result)
     end
 
     def raise_error?

data/lib/passpartu/verify.rb

@@ -25,14 +25,14 @@ module Passpartu
     def call
       return false if role_ignore?
 
-      check_waterfall_if
       default_check
       check_crud_if
 
       validate_result
     rescue StandardError => e
       if ['TrueClass does not have #dig method', 'FalseClass does not have #dig method'].include?(e.message)
-        raise WaterfallError.new "Looks like you want to use check_waterfall feature, but it's set to 'false'. Otherwise check your #{Passpartu.config.policy_file} for validness"
+        raise WaterfallError,
+              "Looks like you want to use check_waterfall feature, but it's set to 'false'. Otherwise check your #{Passpartu.config.policy_file} for validness"
       else
         raise e
       end
@@ -72,12 +72,6 @@ module Passpartu
       %w[create read update delete].include?(keys[-1])
     end
 
-    def check_waterfall_if
-      return unless Passpartu.config.check_waterfall && policy_missed?
-
-      @result = Passpartu::CheckWaterfall.call(role, keys)
-    end
-
     def blank?(item)
       item.respond_to?(:empty?) ? !!item.empty? : !item
     end

data/lib/passpartu/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Passpartu
-  VERSION = '1.0.0'
+  VERSION = '1.0.3'
 end

data/lib/passpartu.rb

@@ -6,7 +6,6 @@ require_relative 'passpartu/patcher'
 require_relative 'passpartu/verify'
 require_relative 'passpartu/block_verify'
 require_relative 'passpartu/validate_result'
-require_relative 'passpartu/check_waterfall'
 require_relative 'passpartu/user' # for testing only
 
 module Passpartu
@@ -14,32 +13,32 @@ module Passpartu
   class PolicyYmlNotFoundError < StandardError; end
   class WaterfallError < StandardError; end
 
-  def self.included(policy_class)
-    Passpartu::Patcher.call(policy_class)
-  end
-
-  def self.policy
-    config.policy
-  end
-
   class << self
     attr_accessor :config
-  end
 
-  def self.configure
-    self.config ||= Config.new
-    yield(config)
+    def included(policy_class)
+      Passpartu::Patcher.call(policy_class)
+    end
+
+    def policy
+      config.policy
+    end
+
+    def configure
+      self.config ||= Config.new
+      yield(config)
+    end
   end
 
   class Config
-    attr_accessor :policy, :raise_policy_missed_error, :check_waterfall
-    attr_reader :policy_file
+    attr_accessor :raise_policy_missed_error
+    attr_reader :policy_file, :check_waterfall, :policy
 
     DEFAULT_POLICY_FILE = './config/passpartu.yml'
 
     def initialize
       @policy_file = DEFAULT_POLICY_FILE
-      @policy = YAML.load_file(policy_file) if File.exist?(policy_file)
+      self.policy = load_policy_file(policy_file) if File.exist?(policy_file)
       @raise_policy_missed_error = true
       @check_waterfall = false
     end
@@ -49,11 +48,56 @@ module Passpartu
 
       raise PolicyYmlNotFoundError unless File.exist?(policy_file)
 
-      @policy = YAML.load_file(policy_file)
+      self.policy = load_policy_file(policy_file)
+    end
+
+    def check_waterfall=(value)
+      @check_waterfall = value
+
+      @check_waterfall.tap do |check_waterfall|
+        if check_waterfall
+          @raise_policy_missed_error = false
+          self.policy = @policy
+        end
+      end
     end
-  end
 
-  configure {}
+    private
+
+    def load_policy_file(path)
+      RUBY_VERSION.to_f >= 3.1 ? YAML.load_file(path, aliases: true) : YAML.load_file(path)
+    end
+
+    def policy=(value)
+      @policy = patch_policy_booleans_if(value)
+    end
+
+    # patch all booleans in hash to support check_waterfall
+    def patch_policy_booleans_if(hash)
+      return hash unless @check_waterfall
+
+      hash.transform_values! do |value|
+        case value
+        when true
+          value.define_singleton_method(:dig) { |*_keys| true }
+        when false
+          value.define_singleton_method(:dig) { |*_keys| false }
+        else
+          patch_policy_booleans_if(value)
+        end
+
+        value
+      end
+    end
+
+    def blank?(item)
+      item.respond_to?(:empty?) ? !!item.empty? : !item
+    end
+
+    def present?(item)
+      !blank?(item)
+    end
+  end
 end
 
 initializer = './config/initializers/passpartu.rb'

data/passpartu.gemspec

@@ -38,7 +38,10 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
   spec.files = Dir['README.md', 'lib/**/*', 'lib/*', 'passpartu.gemspec']
 
-  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 'bundler', '~> 2.3'
+  spec.add_development_dependency 'rake', '~> 13.0'
+  spec.add_development_dependency 'rspec', '~> 3.11'
+  spec.add_development_dependency 'codecov', '~> 0.6'
+  spec.add_development_dependency 'dotenv', '~> 2.7'
+  spec.add_development_dependency 'simplecov', '~> 0.21'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: passpartu
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.3
 platform: ruby
 authors:
 - OrestF
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-04-08 00:00:00.000000000 Z
+date: 2022-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -16,42 +16,84 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '2.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '2.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '3.11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '3.11'
+- !ruby/object:Gem::Dependency
+  name: codecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+- !ruby/object:Gem::Dependency
+  name: dotenv
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.7'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
 description: |-
   Passpartu is a great tool to manage your policies.
                           Keep all your policy rules in one file - passpartu.yml.
@@ -64,7 +106,6 @@ files:
 - README.md
 - lib/passpartu.rb
 - lib/passpartu/block_verify.rb
-- lib/passpartu/check_waterfall.rb
 - lib/passpartu/patcher.rb
 - lib/passpartu/user.rb
 - lib/passpartu/validate_result.rb
@@ -75,7 +116,7 @@ homepage: https://github.com/coaxsoft/passpartu
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -90,9 +131,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.7
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Passpartu makes policies great again
 test_files: []

data/lib/passpartu/check_waterfall.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-module Passpartu
-  class CheckWaterfall
-    attr_reader :waterfall, :policy_hash
-    def initialize(role, keys)
-      @waterfall = [role] + keys
-      @policy_hash = Passpartu.policy
-    end
-
-    def self.call(role, keys)
-      new(role, keys).call
-    end
-
-    def call
-      patch_boolean_classes
-      @result = policy_hash.dig(*waterfall)
-      reset_boolean_classes
-
-      @result
-    end
-
-    def patch_boolean_classes
-      TrueClass.define_method(:dig) { |*_keys| true }
-      FalseClass.define_method(:dig) { |*_keys| false }
-    end
-
-    def reset_boolean_classes
-      TrueClass.undef_method(:dig)
-      FalseClass.undef_method(:dig)
-    end
-  end
-end