ae_declarative_authorization 0.6.0.pre2 → 0.6.0.pre3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7c781d5e787dca8a3622047bf89ac2fc18bf6482
-  data.tar.gz: 851f84598cb1b17040034c529e7f0733422d28ad
+  metadata.gz: 9b32026106200839d7c0b87b1b00ce72bc8656bf
+  data.tar.gz: 002a81249ed4976c82902fe1c9c74996d70a1f11
 SHA512:
-  metadata.gz: d9e7a70c4b8b7e278f5fb6ed28d76aa82751dd47387d626622f70cc28caf7db7289c13ae7cd8a0764718b627ad3b2b51e9bddab4418bdf54d35efb8e6b934528
-  data.tar.gz: 1a027f140d59cc974ce34ee61a5a75f718715f1bf5c09a1655be0c047d73f58939b6076bcdad549afd29195234af5ab75cd523715541636fda1a00223127d08b
+  metadata.gz: db6eeaadd8ed3967feb98022e667bc38210d5438277d7f2c032651b579dc661d44a0478df130983ec8865643d971024b20280e3c22709966c8c50d7298efa855
+  data.tar.gz: a25e17d7c478f5df03a9f8f2bcabe8f1cec181fe87ebd6eaa9629fbe0ad48e723d5df19f03029607a108d083787c107f140787f7b3a4b2b1650242c6902b0ce8

data/lib/declarative_authorization/test/helpers.rb

@@ -60,7 +60,7 @@ module DeclarativeAuthorization
           privileges   = @privileges
 
           actions     = options[:to]
-          raise ":to is a required option!" unless actions
+          raise ':to is a required option!' unless actions
 
           actions     = [actions] unless actions.is_a?(Array)
           params_name      = options[:with]
@@ -127,10 +127,14 @@ module DeclarativeAuthorization
       module ClassMethods
         attr_reader :access_tests_defined
 
+        def skip_access_tests_for_actions(*actions)
+          @skipped_access_test_actions ||= []
+          @skipped_access_test_actions += actions.map(&:to_sym)
+        end
+
         def access_tests(&block)
           @access_tests_defined = true
-          @access_tests_defined = true
-          file_output ||= [ 'test/profiles/access_checking', ENV['TEST_ENV_NUMBER'] ].compact.join(".")
+          file_output ||= [ 'test/profiles/access_checking', ENV['TEST_ENV_NUMBER'] ].compact.join('.')
           unless File.exists?(file_output)
             FileUtils.mkdir_p(File.dirname(file_output))
           end
@@ -151,18 +155,22 @@ module DeclarativeAuthorization
         alias :access_tests_not_required :this_is_an_abstract_controller_so_it_needs_no_access_tests
 
         def all_public_actions
-          actions = controller_class.public_instance_methods(false).map(&:to_s)
-          actions += controller_class.superclass.public_instance_methods(false).map(&:to_s)
-          actions.reject! { |method| method =~ /^_/ || method.starts_with?('rescue_action') }
+          actions = controller_class.public_instance_methods(false)
+          actions += controller_class.superclass.public_instance_methods(false)
+          actions.reject! do |method|
+            method =~ /^_/ ||
+              method =~ /^rescue_action/ ||
+              (@skipped_access_test_actions.is_a?(Array) && @skipped_access_test_actions.include?(method))
+          end
 
-          actions.uniq.map(&:to_s) - %w(permitted_to? has_role? has_role_with_hierarchy? has_any_role? has_any_role_with_hierarchy?)
+          actions.uniq
         end
 
         def inherited(child)
           super
 
           child.send(:define_method, :test_access_tests_defined) do
-            assert self.class.access_tests_defined, "Access tests needed but not defined."
+            assert self.class.access_tests_defined, 'Access tests needed but not defined.'
           end
 
           child.send(:define_method, :test_all_public_actions_covered_by_role_tests) do
@@ -183,7 +191,7 @@ module DeclarativeAuthorization
       protected
 
       def response_forbidden?
-        flash[:error] == "You do not have the correct permissions to access that page. Click the back button to return to your previous page." ||
+        flash[:error] == 'You do not have the correct permissions to access that page. Click the back button to return to your previous page.' ||
         flash[:error] =~ /You do not have the correct permissions to view this/ ||
         flash[:error] =~ /You do not have access to/ ||
         flash[:alert] =~ /You need to sign in/ ||
@@ -211,7 +219,7 @@ module DeclarativeAuthorization
       end
 
       def access_test_user(role, privilege)
-        raise "MUST IMPLEMENT!!"
+        raise 'MUST IMPLEMENT!!!'
       end
 
       def forbidden_with_role_and_privilege?(action, role, privilege, params_name = nil, options = {})

data/lib/declarative_authorization/version.rb

@@ -1,3 +1,3 @@
 module DeclarativeAuthorization
-  VERSION = '0.6.0.pre2'.freeze
+  VERSION = '0.6.0.pre3'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ae_declarative_authorization
 version: !ruby/object:Gem::Version
-  version: 0.6.0.pre2
+  version: 0.6.0.pre3
 platform: ruby
 authors:
 - AppFolio
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-06-14 00:00:00.000000000 Z
+date: 2018-06-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: blockenspiel
@@ -60,6 +60,7 @@ files:
 - README.md
 - README.rdoc
 - Rakefile
+- ae_declarative_authorization-0.6.0.pre2.gem
 - authorization_rules.dist.rb
 - declarative_authorization.gemspec
 - gemfiles/4.1.gemfile.lock