iknow_view_models 3.6.0 → 3.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 573266179096880a34febada583570484e420b91a242920198cfa119192c3fbc
-  data.tar.gz: c024eb9398a4b0d49103d1806379d0f20312f7c6965e9fe2751b09bb720f2fc3
+  metadata.gz: 0f71d69093b0d35214edcfe799aa3fc4493e0640a349e93c828e0158260d4b6c
+  data.tar.gz: 2c2368cd3b0bb27f37ef8e609eae6a396d4de12c4555b90c153386e8bd2a9998
 SHA512:
-  metadata.gz: 353f42c901c4d52cba6bbe459cc6266c2e59f3a56ca432bac616cd4534c95116d0a0e27b8329799717259640e098c704c4f6b63c83713cd42cb3096c6ca6c5b3
-  data.tar.gz: 4e74aa987e3ff26dd9b14bb3fbebc291ce86087dca8f8bcc93ef9b320d7d7ed22f150ce4f3153e27c51f6a2f8d6561fe53cdcb7ac45734caf156f50b10a543d1
+  metadata.gz: c27c10d2940db8b9ed7f0ec512873053f4a2d94283d11004d68b4b70153f696fe3fd45aa103f7ee02ed75fbf3c5d0d8ac899b9df769196308cf67c32dbb3af35
+  data.tar.gz: 882931e849905b9caaf8af154725339b7cda761b1f3ad2deb4d93c4f489cb11c8699143df1e907e51d572ec0a93c07d2f960c86f3e6c35b844d2f86c2839eb66

data/.circleci/config.yml

@@ -8,7 +8,7 @@ executors:
         default: "2.7"
       pg-version:
         type: string
-        default: "11"
+        default: "12.10"
       gemfile:
         type: string
         default: "Gemfile"
@@ -16,14 +16,14 @@ executors:
       PGHOST: 127.0.0.1
       PGUSER: eikaiwa
     docker:
-      - image: circleci/ruby:<< parameters.ruby-version >>
+      - image: cimg/ruby:<< parameters.ruby-version >>
         environment:
           BUNDLE_JOBS: 3
           BUNDLE_RETRY: 3
           BUNDLE_PATH: vendor/bundle
           RAILS_ENV: test
           BUNDLE_GEMFILE: << parameters.gemfile >>
-      - image: circleci/postgres:<< parameters.pg-version >>-alpine
+      - image: cimg/postgres:<< parameters.pg-version >>
         environment:
           POSTGRES_USER: eikaiwa
           POSTGRES_DB: iknow_view_models
@@ -104,23 +104,28 @@ workflows:
       - test:
           name: 'ruby 2.7 rails 5.2 pg 12'
           ruby-version: "2.7"
-          pg-version: "12"
+          pg-version: "12.10"
           gemfile: gemfiles/rails_5_2.gemfile
       - test:
           name: 'ruby 2.7 rails 6.0 pg 12'
           ruby-version: "2.7"
-          pg-version: "12"
+          pg-version: "12.10"
           gemfile: gemfiles/rails_6_0.gemfile
       - test:
           name: 'ruby 2.7 rails 6.1 pg 12'
           ruby-version: "2.7"
-          pg-version: "12"
+          pg-version: "12.10"
           gemfile: gemfiles/rails_6_1.gemfile
       - test:
           name: 'ruby 3.0 rails 6.1 pg 12'
           ruby-version: "3.0"
-          pg-version: "12"
+          pg-version: "12.10"
           gemfile: gemfiles/rails_6_1.gemfile
+      - test:
+          name: 'ruby 3.1 rails 7.0 pg 13'
+          ruby-version: "3.1"
+          pg-version: "13.6"
+          gemfile: gemfiles/rails_7_0.gemfile
       - publish:
           filters:
             branches:

data/Appraisals

@@ -12,3 +12,8 @@ appraise 'rails-6-1' do
   gem 'activerecord', '~> 6.1.0'
   gem 'activesupport', '~> 6.1.0'
 end
+
+appraise 'rails-7-0' do
+  gem 'activerecord', '~> 7.0.0'
+  gem 'activesupport', '~> 7.0.0'
+end

data/Gemfile

@@ -11,5 +11,6 @@ gem 'rubocop-iknow'
 gem 'minitest-ci'
 
 # Override gemspec for development version preferences
-gem 'activerecord',  '~> 6.0.0'
-gem 'activesupport', '~> 6.0.0'
+gem 'activerecord',  '~> 7.0.0'
+gem 'activesupport', '~> 7.0.0'
+gem 'actionpack',    '~> 7.0.0'

data/gemfiles/rails_5_2.gemfile

@@ -5,5 +5,6 @@ source 'https://rubygems.org'
 gem 'minitest-ci'
 gem 'activerecord', '~> 5.2.0'
 gem 'activesupport', '~> 5.2.0'
+gem 'actionpack', '~> 5.2.0'
 
 gemspec path: '../'

data/gemfiles/rails_6_0.gemfile

@@ -5,5 +5,6 @@ source 'https://rubygems.org'
 gem 'minitest-ci'
 gem 'activerecord', '~> 6.0.0'
 gem 'activesupport', '~> 6.0.0'
+gem 'actionpack', '~> 6.0.0'
 
 gemspec path: '../'

data/gemfiles/rails_6_1.gemfile

@@ -5,5 +5,6 @@ source 'https://rubygems.org'
 gem 'minitest-ci'
 gem 'activerecord', '~> 6.1.0'
 gem 'activesupport', '~> 6.1.0'
+gem 'actionpack', '~> 6.1.0'
 
 gemspec path: '../'

data/gemfiles/rails_7_0.gemfile

@@ -0,0 +1,10 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem 'minitest-ci'
+gem "activerecord", "~> 7.0.0"
+gem "activesupport", "~> 7.0.0"
+gem "actionpack", "~> 7.0.0"
+
+gemspec path: '../'

data/iknow_view_models.gemspec

@@ -21,6 +21,7 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = '>= 2.7'
 
+  spec.add_dependency 'actionpack', '>= 5.0'
   spec.add_dependency 'activerecord', '>= 5.0'
   spec.add_dependency 'activesupport', '>= 5.0'
 

data/lib/iknow_view_models/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module IknowViewModels
-  VERSION = '3.6.0'
+  VERSION = '3.6.3'
 end

data/lib/view_model/access_control/composed.rb

@@ -39,15 +39,31 @@ class ViewModel::AccessControl::Composed < ViewModel::AccessControl
           case
           when new_allow
             nil
-          when self.allow_error && other.allow_error
-            self.allow_error.merge(other.allow_error)
+          when self.allow_error.nil?
+            other.allow_error
+          when other.allow_error.nil?
+            self.allow_error
+          # Mergeable (standard) errors should be merged if possible; if not
+          # possible, we should take the first non-standard error.
+          when mergeable_error?(self.allow_error)
+            if mergeable_error?(other.allow_error)
+              self.allow_error.merge(other.allow_error)
+            else
+              other.allow_error
+            end
           else
-            self.allow_error || other.allow_error
+            self.allow_error
           end
 
         ComposedResult.new(new_allow, other.veto, new_allow_error, other.veto_error)
       end
     end
+
+    private
+
+    def mergeable_error?(err)
+      err.is_a?(NoRequiredConditionsError)
+    end
   end
 
   PermissionsCheck = Struct.new(:location, :reason, :error_type, :checker) do
@@ -196,22 +212,54 @@ class ViewModel::AccessControl::Composed < ViewModel::AccessControl
   protected
 
   def check_delegates(env, ifs, unlesses)
-    vetoed_checker = unlesses.detect { |checker| checker.check(env) }
+    veto, veto_error = detect_veto(env, unlesses)
+    allow, allow_error = detect_allow(env, ifs)
+
+    ComposedResult.new(allow, veto, allow_error, veto_error)
+  end
+
+  private
+
+  def detect_veto(env, checkers)
+    checkers.each do |checker|
+      result = checker.check(env)
+      next unless result
 
-    veto = vetoed_checker.present?
-    if veto
-      veto_error = vetoed_checker.error_type.new('Action not permitted because: ' +
-                                                 vetoed_checker.reason,
-                                                 env.view.blame_reference)
+      error =
+        if result.is_a?(StandardError)
+          result
+        else
+          checker.error_type.new('Action not permitted because: ' +
+                                 checker.reason,
+                                 env.view.blame_reference)
+        end
+
+      # short-circuit exit with failure
+      return true, error
     end
 
-    allow = ifs.any? { |checker| checker.check(env) }
+    return false, nil
+  end
+
+  def detect_allow(env, checkers)
+    error = nil
+
+    checkers.each do |checker|
+      result = checker.check(env)
+      next unless result
 
-    unless allow
-      allow_error = NoRequiredConditionsError.new(env.view.blame_reference,
-                                                  ifs.map(&:name))
+      if result.is_a?(StandardError)
+        error ||= result
+      else
+        # short-circuit exit with success
+        return true, nil
+      end
     end
 
-    ComposedResult.new(allow, veto, allow_error, veto_error)
+    error ||= NoRequiredConditionsError.new(
+      env.view.blame_reference,
+      checkers.map(&:name))
+
+    return false, error
   end
 end

data/lib/view_model/access_control.rb

@@ -149,6 +149,14 @@ class ViewModel::AccessControl
   def raise_if_error!(result)
     raise (result.error || yield) unless result.permit?
   end
+
+  # Called from composed access controls via the `env`, this is used to make the
+  # if/unless DSL more readable when returning a custom failure error.
+  def failure(err)
+    raise ArgumentError.new("Unexpected failure type: #{err}") unless err.is_a?(StandardError)
+
+    err
+  end
 end
 
 require 'view_model/access_control/open'

data/lib/view_model/active_record/update_context.rb

@@ -14,9 +14,9 @@ class ViewModel::ActiveRecord
       def release!
         model = viewmodel.model
         case association_data.direct_reflection.options[:dependent]
-        when :delete
+        when :delete, :delete_all
           model.delete
-        when :destroy
+        when :destroy, :destroy_async
           model.destroy
         end
       end

data/lib/view_model/migration/no_path_error.rb

@@ -14,7 +14,7 @@ class ViewModel::Migration::NoPathError < ViewModel::AbstractError
   end
 
   def detail
-    "No migration path for #{vm_name} from #{from} to #{to}"
+    "No migration path for #{vm_name} between client version #{from} and server version #{to}"
   end
 
   def meta

data/lib/view_model/test_helpers/arvm_builder.rb

@@ -65,8 +65,11 @@ class ViewModel::TestHelpers::ARVMBuilder
     ActiveRecord::Base.connection.execute("DROP TABLE IF EXISTS #{name.underscore.pluralize} CASCADE")
     namespace.send(:remove_const, name)
     namespace.send(:remove_const, viewmodel_name) if viewmodel
+
     # prevent cached old class from being used to resolve associations
-    ActiveSupport::Dependencies::Reference.clear!
+    if ActiveSupport::VERSION::MAJOR < 7
+      ActiveSupport::Dependencies::Reference.clear!
+    end
   end
 
   private

data/lib/view_model.rb

@@ -3,6 +3,7 @@
 # A ViewModel encapsulates a particular aggregation of data calculated via the
 # underlying models and provides a means of serializing it into views.
 require 'jbuilder'
+require 'base64'
 require 'deep_preloader'
 
 class ViewModel

data/nix/dependencies.nix

@@ -1,5 +1,5 @@
 {pkgs}:
 {
-  ruby = pkgs.ruby_2_7;
+  ruby = pkgs.ruby_3_0;
   postgresql = pkgs.postgresql_12;
 }

data/test/helpers/controller_test_helpers.rb

@@ -323,7 +323,11 @@ module ControllerTestControllers
     CONTROLLER_NAMES.each do |name|
       Object.send(:remove_const, name)
     end
-    ActiveSupport::Dependencies::Reference.clear!
+
+    if ActiveSupport::VERSION::MAJOR < 7
+      ActiveSupport::Dependencies::Reference.clear!
+    end
+
     super
   end
 end

data/test/unit/view_model/access_control_test.rb

@@ -156,6 +156,54 @@ class ViewModel::AccessControlTest < ActiveSupport::TestCase
       assert_equal(2, ex.reasons.count)
     end
 
+    def test_veto_ordering
+      TestAccessControl.visible_if!('always') { true }
+
+      TestAccessControl.visible_unless!('car starts with i') do
+        view.car =~ /^i/
+      end
+
+      TestAccessControl.visible_unless!('car ends with e') do
+        view.car =~ /e$/
+      end
+
+      assert_serializes(ListView, List.create!(car: 'ok'))
+      refute_serializes(ListView, List.create!(car: 'invisible'), /not permitted.*car starts with i/)
+    end
+
+    def test_custom_error_if
+      TestAccessControl.visible_if!('car is visible1') do
+        view.car == 'visible1' ||
+          # In principle a failure() may return any error, but by returning an
+          # AccessControlError we make it possible to test with refute_serializes
+          failure(ViewModel::AccessControlError.new('Custom Error Message', view.blame_reference))
+      end
+
+      TestAccessControl.visible_if!('car is visible2') do
+        view.car == 'visible2' ||
+          # Only the first failure() recorded by a failed if check will be
+          # raised as the error.
+          failure(ViewModel::AccessControlError.new('Should not be seen', view.blame_reference))
+      end
+
+      assert_serializes(ListView, List.create!(car: 'visible1'))
+      assert_serializes(ListView, List.create!(car: 'visible2'))
+      refute_serializes(ListView, List.create!(car: 'bad'), /Custom Error Message/)
+    end
+
+    def test_custom_error_unless
+      TestAccessControl.visible_if!('always') { true }
+
+      TestAccessControl.visible_unless!('car is invisible') do
+        if view.car == 'invisible'
+          failure(ViewModel::AccessControlError.new('Custom Error Message', view.blame_reference))
+        end
+      end
+
+      assert_serializes(ListView, List.create!(car: 'ok'))
+      refute_serializes(ListView, List.create!(car: 'invisible'), /Custom Error Message/)
+    end
+
     def test_inheritance
       child_access_control = Class.new(ViewModel::AccessControl::Composed)
       child_access_control.include_from(TestAccessControl)

data/test/unit/view_model/active_record/cache_test.rb

@@ -15,16 +15,11 @@ require 'view_model/active_record'
 
 DUMMY_RAILS_CACHE = ActiveSupport::Cache::MemoryStore.new
 
-module RailsDummyCache
-  def cache
-    DUMMY_RAILS_CACHE
-  end
+IknowCache.configure! do
+  logger ::ActiveRecord::Base.logger
+  cache DUMMY_RAILS_CACHE
 end
 
-# Ensure we have a dummy Rails, and then prepend our dummy cache
-module Rails; end
-Rails.singleton_class.prepend(RailsDummyCache)
-
 class ViewModel::ActiveRecord
   class CacheTest < ActiveSupport::TestCase
     using ViewModel::Utils::Collections

data/test/unit/view_model/record_test.rb

@@ -472,7 +472,11 @@ class ViewModel::RecordTest < ActiveSupport::TestCase
       def teardown
         Object.send(:remove_const, :Nested)
         Object.send(:remove_const, :NestedView)
-        ActiveSupport::Dependencies::Reference.clear!
+
+        if ActiveSupport::VERSION::MAJOR < 7
+          ActiveSupport::Dependencies::Reference.clear!
+        end
+
         super
       end
 

metadata

@@ -1,15 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: iknow_view_models
 version: !ruby/object:Gem::Version
-  version: 3.6.0
+  version: 3.6.3
 platform: ruby
 authors:
 - iKnow Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-12-17 00:00:00.000000000 Z
+date: 2022-05-23 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: actionpack
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '5.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
@@ -380,6 +394,7 @@ files:
 - gemfiles/rails_5_2.gemfile
 - gemfiles/rails_6_0.gemfile
 - gemfiles/rails_6_1.gemfile
+- gemfiles/rails_7_0.gemfile
 - iknow_view_models.gemspec
 - lib/iknow_view_models.rb
 - lib/iknow_view_models/railtie.rb
@@ -494,7 +509,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.3.11
 signing_key: 
 specification_version: 4
 summary: ViewModels provide a means of encapsulating a collection of related data