Fix ancestor lookup when using prepended modules

.circleci/config.yml

@@ -0,0 +1,37 @@
+version: 2.1
+commands:
+  bundle_install_and_test:
+    steps:
+      - checkout
+      - run: bundle install
+      - run: bundle exec appraisal install
+      - run: bundle exec appraisal rake test
+
+jobs:
+  test-ruby-233:
+    docker:
+      - image: circleci/ruby:2.3.3
+    steps:
+      - bundle_install_and_test
+
+  test-ruby-253:
+    docker:
+      - image: circleci/ruby:2.5.3
+    steps:
+      - bundle_install_and_test
+
+  test-ruby-262:
+    docker:
+      - image: circleci/ruby:2.6.2
+    steps:
+      - bundle_install_and_test
+
+workflows:
+  rc:
+    jobs:
+      - test-ruby-233:
+          context: appfolio_test_context
+      - test-ruby-253:
+          context: appfolio_test_context
+      - test-ruby-262:
+          context: appfolio_test_context

Appraisals

@@ -1,34 +1,26 @@
 case RUBY_VERSION
 
-when '2.3.3' then
+when '2.3.3', '2.5.3', '2.6.2' then
 
-  appraise 'rails4252' do
-    gem 'rails', '4.2.5.2'
-  end
-
-  appraise 'rails4271' do
-    gem 'rails', '4.2.7.1'
-  end
-
-  appraise 'rails507' do
+  appraise "ruby-#{RUBY_VERSION}-rails507" do
     gem 'rails', '5.0.7'
     gem 'grape', '1.1.0'
     gem 'rails-controller-testing'
   end
 
-  appraise 'rails516' do
+  appraise "ruby-#{RUBY_VERSION}-rails516" do
     gem 'rails', '5.1.6'
     gem 'grape', '1.2.3'
     gem 'rails-controller-testing'
   end
 
-  appraise 'rails521' do
+  appraise "ruby-#{RUBY_VERSION}-rails521" do
     gem 'rails', '5.2.1'
     gem 'grape', '1.2.3'
     gem 'rails-controller-testing'
   end
 
-  appraise 'rails522' do
+  appraise "ruby-#{RUBY_VERSION}-rails522" do
     gem 'rails', '5.2.2'
     gem 'grape', '1.2.3'
     gem 'rails-controller-testing'

gemfiles/rails4252.gemfile → gemfiles/ruby_2.5.3_rails507.gemfile

@@ -5,6 +5,8 @@ source "http://rubygems.org"
 gem "appraisal", "~> 2.1"
 gem "mocha", "~> 1.0", require: false
 gem "sqlite3", "~> 1.3.0"
-gem "rails", "4.2.5.2"
+gem "rails", "5.0.7"
+gem "grape", "1.1.0"
+gem "rails-controller-testing"
 
 gemspec path: "../"

gemfiles/rails4271.gemfile → gemfiles/ruby_2.5.3_rails516.gemfile

@@ -5,6 +5,8 @@ source "http://rubygems.org"
 gem "appraisal", "~> 2.1"
 gem "mocha", "~> 1.0", require: false
 gem "sqlite3", "~> 1.3.0"
-gem "rails", "4.2.7.1"
+gem "rails", "5.1.6"
+gem "grape", "1.2.3"
+gem "rails-controller-testing"
 
 gemspec path: "../"

gemfiles/ruby_2.5.3_rails521.gemfile

@@ -0,0 +1,12 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "appraisal", "~> 2.1"
+gem "mocha", "~> 1.0", require: false
+gem "sqlite3", "~> 1.3.0"
+gem "rails", "5.2.1"
+gem "grape", "1.2.3"
+gem "rails-controller-testing"
+
+gemspec path: "../"

gemfiles/ruby_2.5.3_rails522.gemfile

@@ -0,0 +1,12 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "appraisal", "~> 2.1"
+gem "mocha", "~> 1.0", require: false
+gem "sqlite3", "~> 1.3.0"
+gem "rails", "5.2.2"
+gem "grape", "1.2.3"
+gem "rails-controller-testing"
+
+gemspec path: "../"

gemfiles/ruby_2.6.2_rails507.gemfile

@@ -0,0 +1,12 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "appraisal", "~> 2.1"
+gem "mocha", "~> 1.0", require: false
+gem "sqlite3", "~> 1.3.0"
+gem "rails", "5.0.7"
+gem "grape", "1.1.0"
+gem "rails-controller-testing"
+
+gemspec path: "../"

gemfiles/ruby_2.6.2_rails516.gemfile

@@ -0,0 +1,12 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "appraisal", "~> 2.1"
+gem "mocha", "~> 1.0", require: false
+gem "sqlite3", "~> 1.3.0"
+gem "rails", "5.1.6"
+gem "grape", "1.2.3"
+gem "rails-controller-testing"
+
+gemspec path: "../"

gemfiles/ruby_2.6.2_rails521.gemfile

@@ -0,0 +1,12 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "appraisal", "~> 2.1"
+gem "mocha", "~> 1.0", require: false
+gem "sqlite3", "~> 1.3.0"
+gem "rails", "5.2.1"
+gem "grape", "1.2.3"
+gem "rails-controller-testing"
+
+gemspec path: "../"

gemfiles/ruby_2.6.2_rails522.gemfile

@@ -0,0 +1,12 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "appraisal", "~> 2.1"
+gem "mocha", "~> 1.0", require: false
+gem "sqlite3", "~> 1.3.0"
+gem "rails", "5.2.2"
+gem "grape", "1.2.3"
+gem "rails-controller-testing"
+
+gemspec path: "../"

lib/declarative_authorization/controller/dsl.rb

@@ -172,9 +172,9 @@ def decl_auth_context
 
       def filter_access_permissions # :nodoc:
         unless filter_access_permissions?
-          ancestors[1..-1].reverse.each do |mod|
-            mod.filter_access_permissions if mod.respond_to?(:filter_access_permissions, true)
-          end
+          ancestors
+            .select { |mod| mod != self && mod.respond_to?(:filter_access_permissions, true) }
+            .reverse_each { |mod| mod.filter_access_permissions }
         end
         class_variable_set(:@@declarative_authorization_permissions, {}) unless filter_access_permissions?
         class_variable_get(:@@declarative_authorization_permissions)[self.name] ||= []