scim-kit 0.2.15 → 0.2.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 29f7f31b1bd1883c5596ae65c31154ccc5a677e3544b99bc1b094a5d642cabe3
-  data.tar.gz: 7410afff1d62abb9707de138964af6c05cfb144be5b81391f6c8d16a36bbb730
+  metadata.gz: 888e429b11091880152b63a98fffb2606b91d9b2fa318295ba4fff4e6cf5ab5f
+  data.tar.gz: e4a53aabe13796a5f5eedd50971a3b5b9fcc01c6f8e5508b3be6590ce5dacec6
 SHA512:
-  metadata.gz: e160792a08bd353d7e7b087a3bccb1ee858bfc1b049c56f74d00d5965a37b7a71f5b7761ba5173a67bba7e51489eca37e8fafd67c2489fe73aafb3b9196ad9c2
-  data.tar.gz: 218155f337f9d0f2ce1a989208ec9368be426d25f691bcda9bb437a130251bd53e9f0c3e09bda6743bc604311cdb6385fd9604c4b60ffa680607473ab9d32448
+  metadata.gz: 810fce60e3daae9fe523b5162030a56aa42d446b86c881e30dbc02782392371f08cce1023c37fa065e62cc88e3817a435f07efb280c372a700d0c4f21fdd79db
+  data.tar.gz: 0f232783f9f5f37d01032827ce7c619529417e8571759363698107ea03fd61182c409520edd171296d2dcea232fce8987aba7cc2d8ed3ba5cb7a373dc2f7f31e

data/.rubocop.yml

@@ -9,6 +9,9 @@ AllCops:
     - 'vendor/**/*'
   TargetRubyVersion: 2.5
 
+Layout/IndentArray:
+  EnforcedStyle: consistent
+
 Metrics/BlockLength:
   Exclude:
     - '*.gemspec'
@@ -24,3 +27,6 @@ Naming/FileName:
 
 RSpec/NamedSubject:
   Enabled: false
+
+RSpec/NestedGroups:
+  Max: 4

data/CHANGELOG.md

@@ -0,0 +1,42 @@
+Version 0.2.16
+# Changelog
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [Unreleased]
+### Changed
+- nil
+
+## [0.2.16] - 2019-02-03
+### Added
+- Default logger
+- Attributes now implement Enumerable
+- Attributable#attribute\_for now returns a null object instead of nil.
+- Validations for multi valued attributes
+- Validations for complex attributes
+- rescue errors from type coercion.
+
+### Changed
+- \_assign does not coerce values by default.
+- errors are merged together instead of overwritten during attribute validation.
+
+[Unreleased]: https://github.com/mokhan/scim-kit/compare/v0.2.16...HEAD
+[0.2.16]: https://github.com/mokhan/scim-kit/compare/v0.2.15...v0.2.16
+[0.2.15]: https://github.com/mokhan/scim-kit/compare/v0.2.14...v0.2.15
+[0.2.14]: https://github.com/mokhan/scim-kit/compare/v0.2.13...v0.2.14
+[0.2.13]: https://github.com/mokhan/scim-kit/compare/v0.2.12...v0.2.13
+[0.2.12]: https://github.com/mokhan/scim-kit/compare/v0.2.11...v0.2.12
+[0.2.11]: https://github.com/mokhan/scim-kit/compare/v0.2.10...v0.2.11
+[0.2.10]: https://github.com/mokhan/scim-kit/compare/v0.2.9...v0.2.10
+[0.2.9]: https://github.com/mokhan/scim-kit/compare/v0.2.8...v0.2.9
+[0.2.8]: https://github.com/mokhan/scim-kit/compare/v0.2.7...v0.2.8
+[0.2.7]: https://github.com/mokhan/scim-kit/compare/v0.2.6...v0.2.7
+[0.2.6]: https://github.com/mokhan/scim-kit/compare/v0.2.5...v0.2.6
+[0.2.5]: https://github.com/mokhan/scim-kit/compare/v0.2.4...v0.2.5
+[0.2.4]: https://github.com/mokhan/scim-kit/compare/v0.2.3...v0.2.4
+[0.2.3]: https://github.com/mokhan/scim-kit/compare/v0.2.2...v0.2.3
+[0.2.2]: https://github.com/mokhan/scim-kit/compare/v0.2.1...v0.2.2
+[0.2.1]: https://github.com/mokhan/scim-kit/compare/v0.2.0...v0.2.1
+[0.2.0]: https://github.com/mokhan/scim-kit/compare/v0.1.0...v0.2.0

data/bin/test

@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/bash
 
 # script/test: Run test suite for application. Optionally pass in a path to an
 #              individual test file to run a single test.
@@ -10,8 +10,12 @@ cd "$(dirname "$0")/.."
 
 [ -z "$DEBUG" ] || set -x
 
-echo [$(date "+%H:%M:%S")] "==> Running setup…"
+echo ["$(date "+%H:%M:%S")"] "==> Running setup…"
 bin/setup
 
-echo [$(date "+%H:%M:%S")] "==> Running tests…"
-bundle exec rake spec
+echo ["$(date "+%H:%M:%S")"] "==> Running tests…"
+if [[ $# -eq 0 ]]; then
+  bundle exec rake spec
+else
+  bundle exec rspec "$1"
+fi

data/lib/scim/kit.rb

@@ -3,6 +3,7 @@
 require 'active_model'
 require 'active_support/core_ext/hash/indifferent_access'
 require 'json'
+require 'logger'
 require 'pathname'
 require 'tilt'
 require 'tilt/jbuilder'
@@ -14,8 +15,17 @@ require 'scim/kit/v2'
 require 'scim/kit/version'
 
 module Scim
+  # @api
   module Kit
     class Error < StandardError; end
     class UnknownAttributeError < Error; end
+
+    def self.logger
+      @logger ||= Logger.new(STDOUT)
+    end
+
+    def self.logger=(logger)
+      @logger = logger
+    end
   end
 end

data/lib/scim/kit/v2.rb

@@ -4,6 +4,7 @@ require 'scim/kit/v2/attributable'
 require 'scim/kit/v2/attribute'
 require 'scim/kit/v2/attribute_type'
 require 'scim/kit/v2/authentication_scheme'
+require 'scim/kit/v2/complex_attribute_validator'
 require 'scim/kit/v2/configuration'
 require 'scim/kit/v2/messages'
 require 'scim/kit/v2/meta'
@@ -17,6 +18,7 @@ require 'scim/kit/v2/schemas'
 require 'scim/kit/v2/service_provider_configuration'
 require 'scim/kit/v2/supportable'
 require 'scim/kit/v2/uniqueness'
+require 'scim/kit/v2/unknown_attribute'
 
 module Scim
   module Kit

data/lib/scim/kit/v2/attributable.rb

@@ -5,6 +5,8 @@ module Scim
     module V2
       # Represents a dynamic attribute that corresponds to a SCIM type
       module Attributable
+        include Enumerable
+
         def dynamic_attributes
           @dynamic_attributes ||= {}.with_indifferent_access
         end
@@ -25,10 +27,8 @@ module Scim
           end
         end
 
-        private
-
         def attribute_for(name)
-          dynamic_attributes[name.to_s.underscore]
+          dynamic_attributes[name.to_s.underscore] || UnknownAttribute.new(name)
         end
 
         def read_attribute(name)
@@ -42,13 +42,18 @@ module Scim
           if value.is_a?(Hash)
             attribute_for(name)&.assign_attributes(value)
           else
-            attribute = attribute_for(name)
-            raise Scim::Kit::UnknownAttributeError, name unless attribute
+            attribute_for(name)._value = value
+          end
+        end
 
-            attribute._value = value
+        def each
+          dynamic_attributes.each do |_name, attribute|
+            yield attribute
           end
         end
 
+        private
+
         def create_module_for(type)
           name = type.name.to_sym
           Module.new do

data/lib/scim/kit/v2/attribute.rb

@@ -14,16 +14,21 @@ module Scim
 
         validate :presence_of_value, if: proc { |x| x._type.required }
         validate :inclusion_of_value, if: proc { |x| x._type.canonical_values }
-        validate :validate_type
+        validate :validate_type, unless: proc { |x| x.complex? }
+        validate :validate_complex, if: proc { |x| x.complex? }
+        validate :multiple, if: proc { |x| x.multi_valued && !x.complex? }
+
+        delegate :complex?, :multi_valued, to: :_type
 
         def initialize(resource:, type:, value: nil)
           @_type = type
           @_value = value || type.multi_valued ? [] : nil
           @_resource = resource
+
           define_attributes_for(resource, type.attributes)
         end
 
-        def _assign(new_value, coerce: true)
+        def _assign(new_value, coerce: false)
           @_value = coerce ? _type.coerce(new_value) : new_value
         end
 
@@ -39,6 +44,10 @@ module Scim
           true
         end
 
+        def each_value(&block)
+          Array(_value).each(&block)
+        end
+
         private
 
         def server_only?
@@ -66,11 +75,28 @@ module Scim
         end
 
         def validate_type
+          return if _value.nil?
           return if _type.valid?(_value)
 
           errors.add(_type.name, I18n.t('errors.messages.invalid'))
         end
 
+        def validate_complex
+          validates_with ComplexAttributeValidator
+        end
+
+        def multiple
+          return unless _value.respond_to?(:to_a)
+
+          duped_type = _type.dup
+          duped_type.multi_valued = false
+          _value.to_a.each do |x|
+            unless duped_type.valid?(x)
+              errors.add(duped_type.name, I18n.t('errors.messages.invalid'))
+            end
+          end
+        end
+
         def read_only?
           _type.mutability == Mutability::READ_ONLY
         end

data/lib/scim/kit/v2/attribute_type.rb

@@ -69,11 +69,9 @@ module Scim
           if multi_valued
             return value unless value.respond_to?(:to_a)
 
-            value.to_a.map do |x|
-              COERCION.fetch(type, ->(y) { y }).call(x)
-            end
+            value.to_a.map { |x| coerce_single(x) }
           else
-            COERCION.fetch(type, ->(x) { x }).call(value)
+            coerce_single(value)
           end
         end
 
@@ -89,6 +87,13 @@ module Scim
 
         private
 
+        def coerce_single(value)
+          COERCION.fetch(type, ->(x) { x }).call(value)
+        rescue StandardError => error
+          Scim::Kit.logger.error(error)
+          value
+        end
+
         def validate(value)
           complex? ? valid_complex?(value) : valid_simple?(value)
         end

data/lib/scim/kit/v2/complex_attribute_validator.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module Scim
+  module Kit
+    module V2
+      # Validates a complex attribute
+      class ComplexAttributeValidator < ::ActiveModel::Validator
+        def validate(item)
+          if item._type.multi_valued
+            multi_valued_validation(item)
+          else
+            item.each do |attribute|
+              item.errors.merge!(attribute.errors) unless attribute.valid?
+            end
+          end
+        end
+
+        private
+
+        def multi_valued_validation(item)
+          item.each_value do |hash|
+            validated = hash.map do |key, value|
+              attribute = item.attribute_for(key)
+              attribute._assign(value)
+              item.errors.merge!(attribute.errors) unless attribute.valid?
+
+              key.to_sym
+            end
+            validate_missing(item, hash, validated)
+          end
+        end
+
+        def validate_missing(item, hash, validated)
+          not_validated = item.map { |x| x._type.name.to_sym } - validated
+          not_validated.each do |key|
+            attribute = item.attribute_for(key)
+            attribute._assign(hash[key])
+            item.errors.merge!(attribute.errors) unless attribute.valid?
+          end
+        end
+      end
+    end
+  end
+end

data/lib/scim/kit/v2/resource.rb

@@ -52,7 +52,7 @@ module Scim
 
         def validate_attribute(type)
           attribute = attribute_for(type.name)
-          errors.copy!(attribute.errors) unless attribute.valid?
+          errors.merge!(attribute.errors) unless attribute.valid?
         end
       end
     end

data/lib/scim/kit/v2/unknown_attribute.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Scim
+  module Kit
+    module V2
+      # Represents an Unknown/Unrecognized Attribute
+      class UnknownAttribute
+        include ::ActiveModel::Validations
+        validate :unknown
+        attr_reader :name
+
+        def initialize(name)
+          @name = name
+        end
+
+        def _assign(*_args)
+          valid?
+        end
+
+        def _value=(*_args)
+          raise Scim::Kit::UnknownAttributeError, name
+        end
+
+        def unknown
+          errors.add(name, I18n.t('errors.messages.invalid'))
+        end
+      end
+    end
+  end
+end

data/lib/scim/kit/version.rb

@@ -2,6 +2,6 @@
 
 module Scim
   module Kit
-    VERSION = '0.2.15'
+    VERSION = '0.2.16'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: scim-kit
 version: !ruby/object:Gem::Version
-  version: 0.2.15
+  version: 0.2.16
 platform: ruby
 authors:
 - mo
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-01-28 00:00:00.000000000 Z
+date: 2019-02-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -177,6 +177,7 @@ files:
 - ".rspec"
 - ".rubocop.yml"
 - ".travis.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -197,6 +198,7 @@ files:
 - lib/scim/kit/v2/attribute.rb
 - lib/scim/kit/v2/attribute_type.rb
 - lib/scim/kit/v2/authentication_scheme.rb
+- lib/scim/kit/v2/complex_attribute_validator.rb
 - lib/scim/kit/v2/configuration.rb
 - lib/scim/kit/v2/error.rb
 - lib/scim/kit/v2/messages.rb
@@ -220,6 +222,7 @@ files:
 - lib/scim/kit/v2/templates/service_provider_configuration.json.jbuilder
 - lib/scim/kit/v2/templates/supportable.json.jbuilder
 - lib/scim/kit/v2/uniqueness.rb
+- lib/scim/kit/v2/unknown_attribute.rb
 - lib/scim/kit/version.rb
 - scim-kit.gemspec
 homepage: https://www.github.com/mokhan/scim-kit