scim_engine 2.1.1

data/spec/models/scim_engine/resources/base_validation_spec.rb

@@ -0,0 +1,61 @@
+require 'rails_helper'
+
+describe ScimEngine::Resources::Base do
+
+  describe '#valid?' do
+    MyCustomSchema = Class.new(ScimEngine::Schema::Base) do
+      def self.id
+        'custom-id'
+      end
+
+      def self.scim_attributes
+        [
+          ScimEngine::Schema::Attribute.new(
+            name: 'userName', type: 'string', required: false
+          ),
+          ScimEngine::Schema::Attribute.new(
+            name: 'enforce', type: 'boolean', required: true
+          ),
+          ScimEngine::Schema::Attribute.new(
+            name: 'complexName', complexType: ScimEngine::ComplexTypes::Name, required: false
+          ),
+          ScimEngine::Schema::Attribute.new(
+            name: 'complexNames', complexType: ScimEngine::ComplexTypes::Name, multiValued:true, required: false
+          )
+        ]
+      end
+    end
+
+    MyCustomResource =  Class.new(ScimEngine::Resources::Base) do
+      set_schema MyCustomSchema
+    end
+
+    it 'adds validation errors to the resource for simple attributes' do
+      resource = MyCustomResource.new(userName: 10)
+      expect(resource.valid?).to be(false)
+      expect(resource.errors.full_messages).to match_array(['Username has the wrong type. It has to be a(n) string.', 'Enforce is required'])
+    end
+
+    it 'adds validation errors to the resource for the complex attribute when the value does not match the schema' do
+      resource = MyCustomResource.new(complexName: 10, enforce: false)
+      expect(resource.valid?).to be(false)
+      expect(resource.errors.full_messages).to match_array(['Complexname has to follow the complexType format.'])
+    end
+
+    it 'adds validation errors to the resource from what the complex type schema returns' do
+      resource = MyCustomResource.new(complexName: { givenName: 10 }, enforce: false)
+      expect(resource.valid?).to be(false)
+      expect(resource.errors.full_messages).to match_array(["Complexname familyname is required", "Complexname givenname has the wrong type. It has to be a(n) string."])
+    end
+
+    it 'adds validation errors to the resource from what the complex type schema returns when it is multi-valued' do
+      resource = MyCustomResource.new(complexNames: [
+        "Jane Austen",
+        { givenName: 'Jane', familyName: true }
+      ],
+      enforce: false)
+      expect(resource.valid?).to be(false)
+      expect(resource.errors.full_messages).to match_array(["Complexnames has to follow the complexType format.", "Complexnames familyname has the wrong type. It has to be a(n) string."])
+    end
+  end
+end

data/spec/models/scim_engine/resources/user_spec.rb

@@ -0,0 +1,55 @@
+require 'rails_helper'
+
+describe ScimEngine::Resources::User do
+  describe '#name' do
+    it 'allows a setter for a valid name' do
+      user = described_class.new(name: ScimEngine::ComplexTypes::Name.new(
+        familyName: 'Smith',
+        givenName: 'John',
+        formatted: 'John Smith'
+      ))
+
+      expect(user.name.familyName).to eql('Smith')
+      expect(user.name.givenName).to eql('John')
+      expect(user.name.formatted).to eql('John Smith')
+      expect(user.as_json['name']['errors']).to be_nil
+    end
+
+    it 'validates that the provided name matches the name schema' do
+      user = described_class.new(name: ScimEngine::ComplexTypes::Email.new(
+        value: 'john@smoth.com',
+        primary: true
+      ))
+
+      expect(user.valid?).to be(false)
+    end
+  end
+
+  describe '#add_errors_from_hash' do
+    let(:user) { described_class.new }
+
+    it 'adds the error when the value is a string' do
+      user.add_errors_from_hash(key: 'some error')
+      expect(user.errors.messages).to eql({key: ['some error']})
+      expect(user.errors.full_messages).to eql(['Key some error'])
+    end
+
+    it 'adds the error when the value is an array' do
+      user.add_errors_from_hash(key: ['error1', 'error2'])
+      expect(user.errors.messages).to eql({key: ['error1', 'error2']})
+      expect(user.errors.full_messages).to eql(['Key error1', 'Key error2'])
+    end
+
+    it 'adds the error with prefix when the value is a string' do
+      user.add_errors_from_hash({key: 'some error'}, prefix: :pre)
+      expect(user.errors.messages).to eql({:'pre.key' => ['some error']})
+      expect(user.errors.full_messages).to eql(['Pre key some error'])
+    end
+
+    it 'adds the error wity prefix when the value is an array' do
+      user.add_errors_from_hash({key: ['error1', 'error2']}, prefix: :pre)
+      expect(user.errors.messages).to eql({:'pre.key' => ['error1', 'error2']})
+      expect(user.errors.full_messages).to eql(['Pre key error1', 'Pre key error2'])
+    end
+  end
+end

data/spec/models/scim_engine/schema/attribute_spec.rb

@@ -0,0 +1,80 @@
+require 'rails_helper'
+
+describe ScimEngine::Schema::Attribute do
+  describe '#initialize' do
+    it 'sets the default properties' do
+      attribute = described_class.new
+      expect(attribute.multiValued).to be(false)
+      expect(attribute.required).to be(true)
+      expect(attribute.caseExact).to be(false)
+      expect(attribute.mutability).to eql('readWrite')
+      expect(attribute.uniqueness).to eql('none')
+      expect(attribute.returned).to eql('default')
+    end
+    it 'lets override the default properties' do
+      attribute = described_class.new(multiValued: true)
+      expect(attribute.multiValued).to be(true)
+    end
+
+    it 'transforms complexTypes into subAttributes' do
+      name = described_class.new(name: 'name', complexType: ScimEngine::ComplexTypes::Name)
+      expect(name.type).to eql('complex')
+      expect(name.subAttributes).to eql(ScimEngine::Schema::Name.scim_attributes)
+    end
+
+  end
+
+
+  describe '#valid?' do
+    it 'is invalid if attribute is required but value is blank' do
+      attribute = described_class.new(name: 'userName', type: 'string', required: true)
+      expect(attribute.valid?(nil)).to be(false)
+      expect(attribute.errors.messages).to eql({userName: ['is required']})
+    end
+
+    it 'is valid if attribute is not required and  value is blank' do
+      attribute = described_class.new(name: 'userName', type: 'string', required: false)
+      expect(attribute.valid?(nil)).to be(true)
+      expect(attribute.errors.messages).to eql({})
+    end
+
+    it 'is valid if type is string and given value is string' do
+      expect(described_class.new(name: 'name', type: 'string').valid?('something')).to be(true)
+    end
+
+    it 'is invalid if type is string and given value is not string' do
+      attribute = described_class.new(name: 'userName', type: 'string')
+      expect(attribute.valid?(10)).to be(false)
+      expect(attribute.errors.messages).to eql({userName: ['has the wrong type. It has to be a(n) string.']})
+    end
+
+    it 'is valid if type is boolean and given value is boolean' do
+      expect(described_class.new(name: 'name', type: 'boolean').valid?(false)).to be(true)
+      expect(described_class.new(name: 'name', type: 'boolean').valid?(true)).to be(true)
+    end
+
+    it 'is valid if type is complex and the schema is same' do
+      expect(described_class.new(name: 'name', complexType: ScimEngine::ComplexTypes::Name).valid?(ScimEngine::ComplexTypes::Name.new(givenName: 'a', familyName: 'b'))).to be(true)
+    end
+
+    it 'is valid if type is integer and given value is integer (duh)' do
+      expect(described_class.new(name: 'quantity', type: 'integer').valid?(123)).to be(true)
+    end
+
+    it 'is not valid if type is integer and given value is not an integer' do
+      expect(described_class.new(name: 'quantity', type: 'integer').valid?(123.3)).to be(false)
+    end
+
+    it 'is valid if type is dateTime  and given value is an ISO8601 date time' do
+      expect(described_class.new(name: 'startDate', type: 'dateTime').valid?('2018-07-26T11:59:43-06:00')).to be(true)
+    end
+
+    it 'is not valid if type is dateTime  and given value is a valid date but not in  ISO8601 format' do
+      expect(described_class.new(name: 'startDate', type: 'dateTime').valid?('2018-07-26')).to be(false)
+    end
+    it 'is not valid if type is dateTime  and given value is not a valid date' do
+      expect(described_class.new(name: 'startDate', type: 'dateTime').valid?('gaga')).to be(false)
+    end
+  end
+
+end

data/spec/models/scim_engine/schema/base_spec.rb

@@ -0,0 +1,19 @@
+require 'rails_helper'
+
+describe ScimEngine::Schema::Base do
+  describe '#as_json' do
+    it 'converts the scim_attributes to attributes' do
+      attribute = ScimEngine::Schema::Attribute.new(name: 'nickName')
+      schema = described_class.new(scim_attributes: [attribute])
+      expect(schema.as_json['attributes']).to eql([attribute.as_json])
+      expect(schema.as_json['scim_attributes']).to be_nil
+    end
+  end
+
+  describe '#initialize' do
+    it 'creates a meta' do
+      schema = described_class.new
+      expect(schema.meta.resourceType).to eql('Schema')
+    end
+  end
+end

data/spec/models/scim_engine/schema/group_spec.rb

@@ -0,0 +1,66 @@
+require 'rails_helper'
+
+describe ScimEngine::Schema::Group do
+  it 'returns Group schema as JSON' do
+    expected_json = <<-EOJ
+    {
+    "name": "Group",
+    "id": "urn:ietf:params:scim:schemas:core:2.0:Group",
+    "description": "Represents a Group",
+    "meta": {
+      "resourceType": "Schema",
+      "location": "/scim_engine/Schemas?name=urn%3Aietf%3Aparams%3Ascim%3Aschemas%3Acore%3A2.0%3AGroup"
+    },
+    "attributes": [
+      {
+        "multiValued": false,
+        "required": true,
+        "caseExact": false,
+        "mutability": "readWrite",
+        "uniqueness": "none",
+        "returned": "default",
+        "name": "displayName",
+        "type": "string"
+      },
+      {
+        "multiValued": true,
+        "required": false,
+        "caseExact": false,
+        "mutability": "readOnly",
+        "uniqueness": "none",
+        "returned": "default",
+        "name": "members",
+        "type": "complex",
+        "subAttributes": [
+          {
+            "multiValued": false,
+            "required": true,
+            "caseExact": false,
+            "mutability": "readOnly",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "value",
+            "type": "string"
+          },
+          {
+            "multiValued": false,
+            "required": false,
+            "caseExact": false,
+            "mutability": "readOnly",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "display",
+            "type": "string"
+          }
+        ]
+      }
+    ]
+  }
+    EOJ
+
+    group_schema = ScimEngine::Schema::Group.new
+
+    expect(JSON.parse(expected_json)).to eql(JSON.parse(group_schema.to_json))
+
+  end
+end

data/spec/models/scim_engine/schema/user_spec.rb

@@ -0,0 +1,140 @@
+require 'rails_helper'
+
+describe ScimEngine::Schema::User do
+  it 'returns User schema as JSON' do
+    expected_json = <<-EOJ
+    [
+      {
+        "multiValued": false,
+        "required": true,
+        "caseExact": false,
+        "mutability": "readWrite",
+        "uniqueness": "server",
+        "returned": "default",
+        "name": "userName",
+        "type": "string"
+      },
+      {
+        "multiValued": false,
+        "required": true,
+        "caseExact": false,
+        "mutability": "readWrite",
+        "uniqueness": "none",
+        "returned": "default",
+        "name": "name",
+        "type": "complex",
+        "subAttributes": [
+          {
+            "multiValued": false,
+            "required": true,
+            "caseExact": false,
+            "mutability": "readWrite",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "familyName",
+            "type": "string"
+          },
+          {
+            "multiValued": false,
+            "required": true,
+            "caseExact": false,
+            "mutability": "readWrite",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "givenName",
+            "type": "string"
+          },
+          {
+            "multiValued": false,
+            "required": false,
+            "caseExact": false,
+            "mutability": "readWrite",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "formatted",
+            "type": "string"
+          }
+        ]
+      },
+      {
+        "multiValued": true,
+        "required": true,
+        "caseExact": false,
+        "mutability": "readWrite",
+        "uniqueness": "none",
+        "returned": "default",
+        "name": "emails",
+        "type": "complex",
+        "subAttributes": [
+          {
+            "multiValued": false,
+            "required": true,
+            "caseExact": false,
+            "mutability": "readWrite",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "value",
+            "type": "string"
+          },
+          {
+            "multiValued": false,
+            "required": false,
+            "caseExact": false,
+            "mutability": "readWrite",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "primary",
+            "type": "boolean"
+          },
+          {
+            "multiValued": false,
+            "required": false,
+            "caseExact": false,
+            "mutability": "readWrite",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "type",
+            "type": "string"
+          }
+        ]
+      },
+      {
+        "multiValued": true,
+        "required": true,
+        "caseExact": false,
+        "mutability": "immutable",
+        "uniqueness": "none",
+        "returned": "default",
+        "name": "groups",
+        "type": "complex",
+        "subAttributes": [
+          {
+            "multiValued": false,
+            "required": true,
+            "caseExact": false,
+            "mutability": "readOnly",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "value",
+            "type": "string"
+          },
+          {
+            "multiValued": false,
+            "required": false,
+            "caseExact": false,
+            "mutability": "readOnly",
+            "uniqueness": "none",
+            "returned": "default",
+            "name": "display",
+            "type": "string"
+          }
+        ]
+      }
+    ]
+    EOJ
+
+    expect(JSON.parse(expected_json)).to eql(JSON.parse(ScimEngine::Schema::User.scim_attributes.to_json))
+
+  end
+
+end

data/spec/rails_helper.rb

@@ -0,0 +1,57 @@
+# This file is copied to spec/ when you run 'rails generate rspec:install'
+require 'spec_helper'
+ENV['RAILS_ENV'] ||= 'test'
+require File.expand_path('../dummy/config/environment', __FILE__)
+# Prevent database truncation if the environment is production
+abort("The Rails environment is running in production mode!") if Rails.env.production?
+require 'rspec/rails'
+require 'byebug'
+# Add additional requires below this line. Rails is not loaded until this point!
+
+# Requires supporting ruby files with custom matchers and macros, etc, in
+# spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are
+# run as spec files by default. This means that files in spec/support that end
+# in _spec.rb will both be required and run as specs, causing the specs to be
+# run twice. It is recommended that you do not name files matching this glob to
+# end with _spec.rb. You can configure this pattern with the --pattern
+# option on the command line or in ~/.rspec, .rspec or `.rspec-local`.
+#
+# The following line is provided for convenience purposes. It has the downside
+# of increasing the boot-up time by auto-requiring all files in the support
+# directory. Alternatively, in the individual `*_spec.rb` files, manually
+# require only the support files necessary.
+#
+# Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f }
+
+# Checks for pending migrations and applies them before tests are run.
+# If you are not using ActiveRecord, you can remove this line.
+
+RSpec.configure do |config|
+  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
+  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+
+  # If you're not using ActiveRecord, or you'd prefer not to run each of your
+  # examples within a transaction, remove the following line or assign false
+  # instead of true.
+  config.use_transactional_fixtures = true
+
+  # RSpec Rails can automatically mix in different behaviours to your tests
+  # based on their file location, for example enabling you to call `get` and
+  # `post` in specs under `spec/controllers`.
+  #
+  # You can disable this behaviour by removing the line below, and instead
+  # explicitly tag your specs with their type, e.g.:
+  #
+  #     RSpec.describe UsersController, :type => :controller do
+  #       # ...
+  #     end
+  #
+  # The different available types are documented in the features, such as in
+  # https://relishapp.com/rspec/rspec-rails/docs
+  config.infer_spec_type_from_file_location!
+
+  # Filter lines from Rails gems in backtraces.
+  config.filter_rails_from_backtrace!
+  # arbitrary gems may also be filtered via:
+  # config.filter_gems_from_backtrace("gem name")
+end