activeldap 6.0.0 → 6.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ddf65edda308e8db6ca8c35e3ef87c3299f048c0088b2f0836bab3f194709d11
-  data.tar.gz: fb21442b49251ff549b0f8c9189443b29830230eb77b71fd9cd44da14a04afd0
+  metadata.gz: fd73013b75dc4f7aa629d12ba1b144ea34458f64f69245f1e78a399f5a5a7072
+  data.tar.gz: 8ee47f89c8e5e9316bdbe7f9fdc620657a336dceb968751cf15ad8aae0d39e6f
 SHA512:
-  metadata.gz: ee133139964a025b4b55e294c3601ee7b686016718cf347a70f8beb1832a1b8a8f7867171dca577366bdf09a3e2777e4d61daf371570e7718eb0ef89c1423c9a
-  data.tar.gz: 6fa4404370ecfcbb9509db4628165f9a41a0dc58ef420bc6809ba2ddb7d57c03761d6330bffbb1fc05d9a3ab5dbc2f336f1f7449d6d0df9fcea042792a840a15
+  metadata.gz: caaa12bfb35a8cf5838de135583e0165c7ef7a08e825b553f75dee95a26074430365f112cfb78cc11b42460ce039857af144c3e2d87da15426f97ade72c17eec
+  data.tar.gz: a148359f6c91dafb62b3883cd3bd38e95ab0650f68243cc5a0d69229a591111b8625d816cf8db75ee4d128ad69b6352e054a1d5170542e38031272ca3434d623

data/doc/text/news.md

@@ -1,5 +1,73 @@
 # News
 
+## 6.1.0: 2020-12-24 {#release-6-1-0}
+
+### Improvements
+
+  * Changed to use `:use_paged_results` option value by default.
+    [GitHub#189][Reported by Kevin McCormack]
+
+### Thanks
+
+  * Kevin McCormack
+
+## 6.0.4: 2020-12-06 {#release-6-0-4}
+
+### Improvements
+
+  * Enabled concurrency by default.
+    [GitHub#188][Reported by Kevin McCormack]
+
+### Thanks
+
+  * Kevin McCormack
+
+## 6.0.3: 2020-08-17 {#release-6-0-3}
+
+### Improvements
+
+  * Added support for `save(validate: false)`.
+    [GitHub#180][Reported by Kevin McCormack]
+
+  * jndi: Added support for follow referrals.
+    [GitHub#182][Patch by Kevin McCormack]
+
+### Fixes
+
+  * Fixed a bug that sub base is ignored in DN specified by `new`.
+    [GitHub#185][Reported by Kevin McCormack]
+
+### Thanks
+
+  * Kevin McCormack
+
+## 6.0.2: 2020-05-19 {#release-6-0-2}
+
+### Improvements
+
+  * Added `options` to {ActiveLdap::Persistance#reload}.
+    [GitHub#176][Reported by Kevin McCormack]
+
+  * jndi: Improved DN escaping.
+    [GitHub#178][Patch by Kevin McCormack]
+
+### Thanks
+
+  * Kevin McCormack
+
+## 6.0.1: 2020-04-21 {#release-6-0-1}
+
+### Improvements
+
+  * Dropped support for Ruby 2.4.
+
+  * Stopped using paged results when we need only one entry.
+    [GitHub#173][Patch by Kevin McCormack]
+
+### Thanks
+
+  * Kevin McCormack
+
 ## 6.0.0: 2020-04-16 {#release-6-0-0}
 
 ### Improvements

data/doc/text/rails.md

@@ -60,6 +60,15 @@ to skip `active_ldap prefix` in `config/application.rb`:
 config.app_generators.orm :active_ldap
 ```
 
+Concurrency is now enabled by default to ensure thread safe searches and modifications. This can
+still be disabled if desired.
+
+```ruby
+# config/initializers/active_ldap.rb
+
+ActiveLdap::Base.allow_concurrency = false
+```
+
 ## Model
 
 You can generate a User model that represents entries under

data/doc/text/tutorial.md

@@ -641,7 +641,6 @@ base = File.expand_path(File.join(File.dirname(__FILE__), ".."))
 $LOAD_PATH << File.join(base, "lib")
 $LOAD_PATH << File.join(base, "examples")
 
-require 'rubygems'
 require 'active_ldap'
 require 'objects/user'
 require 'objects/group'
@@ -694,7 +693,6 @@ base = File.expand_path(File.join(File.dirname(__FILE__), ".."))
 $LOAD_PATH << File.join(base, "lib")
 $LOAD_PATH << File.join(base, "examples")
 
-require 'rubygems'
 require 'active_ldap'
 require 'objects/user'
 require 'objects/group'
@@ -744,7 +742,6 @@ base = File.expand_path(File.join(File.dirname(__FILE__), ".."))
 $LOAD_PATH << File.join(base, "lib")
 $LOAD_PATH << File.join(base, "examples")
 
-require 'rubygems'
 require 'active_ldap'
 require 'objects/user'
 require 'objects/group'

data/lib/active_ldap.rb

@@ -1,4 +1,3 @@
-require "rubygems"
 require "active_model"
 require "active_support/core_ext"
 

data/lib/active_ldap/adapter/base.rb

@@ -180,8 +180,9 @@ module ActiveLdap
         limit = options[:limit] || 0
         limit = nil if limit <= 0
         use_paged_results = options[:use_paged_results]
-        if use_paged_results or use_paged_results.nil?
-          use_paged_results = supported_control.paged_results?
+        use_paged_results = @use_paged_results if use_paged_results.nil?
+        if use_paged_results
+          use_paged_results = limit != 1 && supported_control.paged_results?
         end
         search_options = {
           base: base,
@@ -682,8 +683,7 @@ module ActiveLdap
                :scope => :base,
                :attributes => attrs,
                :limit => 1,
-               :try_reconnect => try_reconnect,
-               :use_paged_results => false) do |dn, attributes|
+               :try_reconnect => try_reconnect) do |dn, attributes|
           found_attributes = attributes
         end
         found_attributes

data/lib/active_ldap/adapter/jndi.rb

@@ -22,9 +22,23 @@ module ActiveLdap
         super do |host, port, method|
           uri = construct_uri(host, port, method == :ssl)
           with_start_tls = method == :start_tls
-          info = {:uri => uri, :with_start_tls => with_start_tls}
-          [log("connect", info) {JndiConnection.new(host, port, method, @timeout)},
-           uri, with_start_tls]
+          follow_referrals = follow_referrals?(options)
+          info = {
+            :uri => uri,
+            :with_start_tls => with_start_tls,
+            :follow_referrals => follow_referrals,
+          }
+          [
+            log("connect", info) {
+              JndiConnection.new(host,
+                                 port,
+                                 method,
+                                 @timeout,
+                                 follow_referrals)
+            },
+            uri,
+            with_start_tls,
+          ]
         end
       end
 

data/lib/active_ldap/adapter/jndi_connection.rb

@@ -75,13 +75,14 @@ module ActiveLdap
         end
       end
 
-      def initialize(host, port, method, timeout)
+      def initialize(host, port, method, timeout, follow_referrals)
         @host = host
         @port = port
         @method = method
         @timeout = timeout
         @context = nil
         @tls = nil
+        @follow_referrals = follow_referrals
       end
 
       def unbind
@@ -136,6 +137,7 @@ module ActiveLdap
         end
 
         escaped_base = escape_dn(base)
+
         loop do
           @context.search(escaped_base, filter, controls).each do |search_result|
             yield(build_raw_search_result(search_result))
@@ -166,26 +168,23 @@ module ActiveLdap
         records.each do |record|
           attributes.put(record.to_java_attribute)
         end
-        escaped_dn = escape_dn(dn)
         @context.set_request_controls([])
-        @context.create_subcontext(escaped_dn, attributes)
+        @context.create_subcontext(escape_dn(dn), attributes)
       end
 
       def modify(dn, records)
-        escaped_dn = escape_dn(dn)
         items = records.collect(&:to_java_modification_item)
         @context.set_request_controls([])
-        @context.modify_attributes(escaped_dn, items.to_java(ModificationItem))
+        @context.modify_attributes(escape_dn(dn), items.to_java(ModificationItem))
       end
 
       def modify_rdn(dn, new_rdn, delete_old_rdn)
-        escaped_dn = escape_dn(dn)
         # should use mutex
         delete_rdn_key = "java.naming.ldap.deleteRDN"
         @context.set_request_controls([])
         begin
           @context.add_to_environment(delete_rdn_key, delete_old_rdn.to_s)
-          @context.rename(escaped_dn, new_rdn)
+          @context.rename(escape_dn(dn), escape_dn(new_rdn))
         ensure
           @context.remove_from_environment(delete_rdn_key)
         end
@@ -205,9 +204,10 @@ module ActiveLdap
           Context::PROVIDER_URL => ldap_uri,
           'com.sun.jndi.ldap.connect.timeout' => (@timeout * 1000).to_i.to_s,
           'com.sun.jndi.ldap.read.timeout' => (@timeout * 1000).to_i.to_s,
+          'java.naming.ldap.derefAliases' => 'never',
+          'java.naming.referral' => @follow_referrals ? 'follow' : 'ignore',
         }
-        environment = HashTable.new(environment)
-        context = InitialLdapContext.new(environment, nil)
+        context = InitialLdapContext.new(HashTable.new(environment), nil)
         if @method == :start_tls
           @tls = context.extended_operation(StartTlsRequest.new)
           @tls.negotiate
@@ -230,34 +230,9 @@ module ActiveLdap
       end
 
       def escape_dn(dn)
-        parsed_dn = nil
-        begin
-          parsed_dn = DN.parse(dn)
-        rescue DistinguishedNameInvalid
-          return dn
-        end
-
-        escaped_rdns = parsed_dn.rdns.collect do |rdn|
-          escaped_rdn_strings = rdn.collect do |key, value|
-            escaped_value = DN.escape_value(value)
-            # We may need to escape the followings too:
-            #   * ,
-            #   * =
-            #   * +
-            #   * <
-            #   * >
-            #   * #
-            #   * ;
-            #
-            # See javax.naming.ldap.Rdn.unescapeValue()
-            escaped_value = escaped_value.gsub(/\\\\/) do
-              "\\5C"
-            end
-            "#{key}=#{escaped_value}"
-          end
-          escaped_rdn_strings.join("+")
-        end
-        escaped_rdns.join(",")
+        javax.naming.ldap.LdapName.new(dn)
+      rescue Java::JavaLang::IllegalArgumentException, Java::JavaxNaming::InvalidNameException
+        dn
       end
 
       def build_paged_results_control(page_size, page_cookie=nil)

data/lib/active_ldap/adapter/ldap.rb

@@ -209,15 +209,17 @@ module ActiveLdap
       def prepare_connection(options={})
         operation(options) do
           @connection.set_option(LDAP::LDAP_OPT_PROTOCOL_VERSION, 3)
-          unless follow_referrals?(options)
-            @connection.set_option(LDAP::LDAP_OPT_REFERRALS, 0)
-          end
+          @ldap_follow_referrals = follow_referrals?(options) ? 1 : 0
+          @connection.set_option(LDAP::LDAP_OPT_REFERRALS,
+                                 @ldap_follow_referrals)
         end
       end
 
       def execute(method, info=nil, *args, &block)
         begin
           name = (info || {}).delete(:name) || method
+          @connection.set_option(LDAP::LDAP_OPT_REFERRALS,
+                                 @ldap_follow_referrals)
           log(name, info) {@connection.send(method, *args, &block)}
         rescue LDAP::ResultError
           @connection.assert_error_code

data/lib/active_ldap/base.rb

@@ -851,6 +851,8 @@ module ActiveLdap
 
       _schema = _local_entry_attribute = nil
       targets = sanitize_for_mass_assignment(new_attributes)
+      have_dn = false
+      dn_value = nil
       targets.each do |key, value|
         setter = "#{key}="
         unless respond_to?(setter)
@@ -860,8 +862,15 @@ module ActiveLdap
           _local_entry_attribute ||= local_entry_attribute
           _local_entry_attribute.register(attribute)
         end
-        send(setter, value)
+        case setter
+        when "dn=", "id="
+          have_dn = true
+          dn_value = value
+        else
+          send(setter, value)
+        end
       end
+      self.dn = dn_value if have_dn
     end
 
     def to_ldif_record
@@ -1281,6 +1290,7 @@ module ActiveLdap
     end
 
     def compute_base
+      ensure_update_dn
       base_of_class = self.class.base
       if @base_value.nil?
         base_of_class

data/lib/active_ldap/connection.rb

@@ -6,7 +6,7 @@ module ActiveLdap
 
     module ClassMethods
       @@active_connections = {}
-      @@allow_concurrency = false
+      @@allow_concurrency = true
 
       def thread_safe_active_connections
         @@active_connections[Thread.current.object_id] ||= {}

data/lib/active_ldap/persistence.rb

@@ -81,9 +81,10 @@ module ActiveLdap
       end
     end
 
-    def reload
+    def reload(options={})
       clear_association_cache
-      _, attributes = search(:value => id).find do |_dn, _attributes|
+      search_options = options.merge(value: id)
+      _, attributes = search(search_options).find do |_dn, _attributes|
         dn == _dn
       end
       if attributes.nil?

data/lib/active_ldap/validations.rb

@@ -53,15 +53,23 @@ module ActiveLdap
       errors.empty? && output
     end
 
-    def save(*)
-      valid? ? super : false
+    def save(**options)
+      perform_validations(options) ? super : false
     end
 
-    def save!(*)
-      valid? ? super : raise(EntryInvalid.new(self))
+    def save!(**options)
+      perform_validations(options) ? super : raise(EntryInvalid.new(self))
     end
 
     private
+    def perform_validations(options)
+      if options[:validate] == false
+        true
+      else
+        valid?(options[:context])
+      end
+    end
+
     def format_validation_message(format, parameters)
       format % parameters
     end

data/lib/active_ldap/version.rb

@@ -1,3 +1,3 @@
 module ActiveLdap
-  VERSION = "6.0.0"
+  VERSION = "6.1.0"
 end

data/test/add-phonetic-attribute-options-to-slapd.ldif

@@ -1,6 +1,6 @@
 # Your LDAP server needs to accept 'phonetic' attribute option for test.
 # This is a LDIF file for OpenLDAP to do the configuration.
-# You can use this file by the following command linne on Debian GNU/Linux
+# You can use this file by the following command line on Debian GNU/Linux
 # or Ubuntu:
 #   % sudo -H ldapmodify -Y EXTERNAL -H ldapi:/// -f test/add-phonetic-attribute-options-to-slapd.ldif
 version: 1

data/test/al-test-utils.rb

@@ -182,6 +182,7 @@ module AlTestUtils
       populate_ou
       populate_user_class
       populate_group_class
+      populate_group_of_urls_class
       populate_associations
     end
 
@@ -215,7 +216,7 @@ module AlTestUtils
     end
 
     def populate_ou
-      %w(Users Groups).each do |name|
+      %w(Users Groups GroupOfURLsSet).each do |name|
         make_ou(name)
       end
     end
@@ -246,6 +247,14 @@ module AlTestUtils
       assign_class_name(@group_class, "Group")
     end
 
+    def populate_group_of_urls_class
+      @group_of_urls_class = Class.new(ActiveLdap::Base)
+      @group_of_urls_class.ldap_mapping :prefix => "ou=GroupOfURLsSet",
+                                        :scope => :sub,
+                                        :classes => ["groupOfURLs"]
+      assign_class_name(@group_of_urls_class, "GroupOfURLs")
+    end
+
     def populate_associations
       @user_class.belongs_to :groups, :many => "memberUid"
       @user_class.belongs_to :primary_group,
@@ -280,6 +289,7 @@ module AlTestUtils
       super
       @user_index = 0
       @group_index = 0
+      @group_of_urls_index = 0
       @temporary_uids = []
     end
 
@@ -366,6 +376,25 @@ module AlTestUtils
       end
     end
 
+    def make_temporary_group_of_urls(config={})
+      @group_of_urls_index += 1
+      cn = config[:cn] || "temp-group-of-urls-#{@group_of_urls_index}"
+      ensure_delete_group_of_urls(cn) do
+        _wrap_assertion do
+          assert(!@group_of_urls_class.exists?(cn))
+          assert_raise(ActiveLdap::EntryNotFound) do
+            @group_of_urls_class.find(cn)
+          end
+          group_of_urls = @group_of_urls_class.new(cn)
+          assert(group_of_urls.new_entry?)
+          group_of_urls.member_url = config[:member_url]
+          assert(group_of_urls.save!)
+          assert(!group_of_urls.new_entry?)
+          yield(@group_of_urls_class.find(group_of_urls.cn))
+        end
+      end
+    end
+
     def ensure_delete_user(uid)
       yield(uid)
     ensure
@@ -379,6 +408,12 @@ module AlTestUtils
       @group_class.delete(cn) if @group_class.exists?(cn)
     end
 
+    def ensure_delete_group_of_urls(cn)
+      yield(cn)
+    ensure
+      @group_of_urls_class.delete(cn) if @group_of_urls_class.exists?(cn)
+    end
+
     def default_uid
       "10000#{@user_index}"
     end
@@ -467,8 +502,13 @@ module AlTestUtils
       omit(message || "This test is not for JRuby")
     end
 
+    def omit_unless_jruby(message=nil)
+      return if RUBY_PLATFORM == "java"
+      omit(message || "This test is only for JRuby")
+    end
+
     def omit_if_ldap(message=nil)
-      return unless current_configuration[:adapter] == "ldap"
+      return if current_configuration[:adapter] == "ldap"
       omit(message || "This test is not for ruby-ldap")
     end
   end

data/test/enable-dynamic-groups.ldif

@@ -0,0 +1,22 @@
+# Your LDAP server needs to support dynamic list for test.
+# This is a LDIF file for OpenLDAP to do the configuration.
+# You can use this file by the following command line on Debian GNU/Linux
+# or Ubuntu:
+#
+#   % sudo -H ldapadd -Y EXTERNAL -H ldapi:/// -f /etc/ldap/schema/dyngroup.ldif
+#   % sudo -H ldapmodify -Y EXTERNAL -H ldapi:/// -f test/enable-dynamic-groups.ldif
+version: 1
+
+# Enable dynlist module
+dn: cn=module{0},cn=config
+changetype: modify
+add: olcModuleLoad
+olcModuleLoad: dynlist
+
+# Set up dynlist overlay
+dn: olcOverlay=dynlist,olcDatabase={1}mdb,cn=config
+changetype: add
+objectClass: olcOverlayConfig
+objectClass: olcDynamicList
+olcOverlay: dynlist
+olcDlAttrSet: groupOfURLs memberURL member

data/test/enable-start-tls.ldif

@@ -1,6 +1,6 @@
 # Your LDAP server needs to support StartTLS when you test StartTLS related
 # feature. This is a LDIF file for OpenLDAP to do the configuration.
-# You can use this file by the following command linne on Debian GNU/Linux
+# You can use this file by the following command line on Debian GNU/Linux
 # or Ubuntu:
 #
 #   % sudo usermod -a -G ssl-cert openldap

data/test/run-test.rb

@@ -2,8 +2,6 @@
 
 $VERBOSE = true
 
-$KCODE = 'u' if RUBY_VERSION < "1.9"
-
 base_dir = File.expand_path(File.dirname(__FILE__))
 top_dir = File.expand_path(File.join(base_dir, ".."))
 lib_dir = File.join(top_dir, "lib")
@@ -11,11 +9,9 @@ test_dir = File.join(top_dir, "test")
 $LOAD_PATH.unshift(lib_dir)
 $LOAD_PATH.unshift(test_dir)
 
-require "rubygems"
 require "bundler/setup"
 
 require "test/unit"
-require "test/unit/notify"
 Test::Unit::Priority.enable
 
 Dir.glob(File.join(test_dir, "**", "test_*.rb")) do |test_file|

data/test/test_base.rb

@@ -5,7 +5,52 @@ require 'al-test-utils'
 class TestBase < Test::Unit::TestCase
   include AlTestUtils
 
+  sub_test_case("follow_referrals") do
+    def test_default
+      make_temporary_user do |user1,|
+        make_temporary_user do |user2,|
+          member_url = ["ldap:///#{user1.base.to_s}??one?(objectClass=person)"]
+          make_temporary_group_of_urls(member_url: member_url) do |group_of_urls|
+            assert_equal([user1.dn, user2.dn],
+                         group_of_urls.attributes["member"])
+          end
+        end
+      end
+    end
+
+    def test_connection_false
+      omit_unless_jruby
+      @group_of_urls_class.setup_connection(
+        current_configuration.merge(follow_referrals: false)
+      )
+      make_temporary_user do |user1,|
+        make_temporary_user do |user2,|
+          member_url = ["ldap:///#{user1.base.to_s}??one?(objectClass=person)"]
+          make_temporary_group_of_urls(member_url: member_url) do |group_of_urls|
+            assert_nil(group_of_urls.attributes["member"])
+          end
+        end
+      end
+    end
+
+    def test_connect_false
+      omit_unless_jruby
+      connection = @group_of_urls_class.connection
+      connection.disconnect!
+      connection.connect(follow_referrals: false)
+      make_temporary_user do |user1,|
+        make_temporary_user do |user2,|
+          member_url = ["ldap:///#{user1.base.to_s}??one?(objectClass=person)"]
+          make_temporary_group_of_urls(member_url: member_url) do |group_of_urls|
+            assert_nil(group_of_urls.attributes["member"])
+          end
+        end
+      end
+    end
+  end
+
   priority :must
+  priority :normal
   def test_search_colon_value
     make_temporary_group(:cn => "temp:group") do |group|
       assert_equal("temp:group", group.cn)
@@ -13,7 +58,6 @@ class TestBase < Test::Unit::TestCase
     end
   end
 
-  priority :normal
   def test_lower_case_object_class
     fixture_file = fixture("lower_case_object_class_schema.rb")
     schema_entries = eval(File.read(fixture_file))
@@ -122,9 +166,9 @@ class TestBase < Test::Unit::TestCase
 
   def test_operational_attributes
     make_temporary_group do |group|
-      dn, attributes = @group_class.search(:attributes => ["*"])[0]
+      _dn, attributes = @group_class.search(:attributes => ["*"])[0]
       normal_attributes = attributes.keys
-      dn, attributes = @group_class.search(:attributes => ["*", "+"])[0]
+      _dn, attributes = @group_class.search(:attributes => ["*", "+"])[0]
       operational_attributes = attributes.keys - normal_attributes
       operational_attribute = operational_attributes[0]
 
@@ -383,6 +427,19 @@ class TestBase < Test::Unit::TestCase
     end
   end
 
+  def test_set_dn_with_unnormalized_dn_attribute_with_forward_slash
+    make_temporary_user do |user,|
+      new_dn = "uid=temp/user1,#{user.class.base}"
+      assert_not_equal(user.dn.to_s, new_dn)
+
+      user.uid = 'temp/user1'
+      assert_equal(user.dn.to_s, new_dn)
+
+      assert_true(user.save!)
+      assert_true(user.class.find(user.uid).update_attributes!(gidNumber: 100069))
+    end
+  end
+
   def test_destroy_with_empty_base_and_prefix_of_class
     make_temporary_user do |user,|
       base = user.class.base
@@ -945,7 +1002,8 @@ class TestBase < Test::Unit::TestCase
     ou_class.ldap_mapping(:dn_attribute => :ou,
                           :prefix => "",
                           :classes => ["top", "organizationalUnit"])
-    assert_equal(["ou=Groups,#{current_configuration['base']}",
+    assert_equal(["ou=GroupOfURLsSet,#{current_configuration['base']}",
+                  "ou=Groups,#{current_configuration['base']}",
                   "ou=Users,#{current_configuration['base']}"],
                  ou_class.find(:all).collect(&:dn).collect(&:to_s).sort)
   end

data/test/test_base_per_instance.rb

@@ -9,13 +9,45 @@ class TestBasePerInstance < Test::Unit::TestCase
   end
 
   priority :must
+  def test_dn_with_sub_base_first
+    sub_user = @user_class.new(dn: "uid=user1,ou=Sub,#{@user_class.base}",
+                               uid: "user1")
+    # Order is important. #base should be called before #dn.
+    base = sub_user.base.to_s
+    dn = sub_user.dn.to_s
+    assert_equal([
+                   "ou=Sub,#{@user_class.base}",
+                   "uid=user1,ou=Sub,#{@user_class.base}",
+                 ],
+                 [
+                   base,
+                   dn,
+                 ])
+  end
+
+  def test_dn_with_sub_base_last
+    sub_user = @user_class.new(uid: "user1",
+                               dn: "uid=user1,ou=Sub,#{@user_class.base}")
+    # Order is important. #base should be called before #dn.
+    base = sub_user.base.to_s
+    dn = sub_user.dn.to_s
+    assert_equal([
+                   "ou=Sub,#{@user_class.base}",
+                   "uid=user1,ou=Sub,#{@user_class.base}",
+                 ],
+                 [
+                   base,
+                   dn,
+                 ])
+  end
+
+  priority :normal
   def test_set_base
     guest = @user_class.new("guest")
     guest.base = "ou=Sub"
     assert_equal("uid=guest,ou=Sub,#{@user_class.base}", guest.dn)
   end
 
-  priority :normal
   def test_dn_is_base
     entry_class = Class.new(ActiveLdap::Base)
     entry_class.ldap_mapping :prefix => "",

data/test/test_entry.rb

@@ -11,6 +11,7 @@ class TestEntry < Test::Unit::TestCase
         all_entries = [ActiveLdap::Base.base]
         all_entries += [user.dn, user.base]
         all_entries += [group.dn, group.base]
+        all_entries += [@group_of_urls_class.base]
         assert_equal(all_entries.sort,
                      ActiveLdap::Entry.all.collect(&:dn).sort)
       end

data/test/test_validation.rb

@@ -37,6 +37,15 @@ class TestValidation < Test::Unit::TestCase
   priority :must
 
   priority :normal
+  def test_validate_false
+    make_temporary_user(:simple => true) do |user,|
+      user.sn = nil
+      assert_raise(ActiveLdap::RequiredAttributeMissed) do
+        user.save(validate: false)
+      end
+    end
+  end
+
   def test_octet_string
     make_temporary_user(:simple => true) do |user,|
       utf8_encoded_binary_value = "\xff".force_encoding("UTF-8")

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activeldap
 version: !ruby/object:Gem::Version
-  version: 6.0.0
+  version: 6.1.0
 platform: ruby
 authors:
 - Will Drewry
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-15 00:00:00.000000000 Z
+date: 2020-12-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '5.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: locale
   requirement: !ruby/object:Gem::Requirement
@@ -151,20 +151,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: test-unit-notify
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -293,6 +279,7 @@ files:
 - test/al-test-utils.rb
 - test/command.rb
 - test/config.yaml.sample
+- test/enable-dynamic-groups.ldif
 - test/enable-start-tls.ldif
 - test/fixtures/lower_case_object_class_schema.rb
 - test/run-test.rb
@@ -358,7 +345,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.0.pre1
+rubygems_version: 3.2.0.rc.2
 signing_key: 
 specification_version: 4
 summary: ActiveLdap is a object-oriented API to LDAP
@@ -367,6 +354,7 @@ test_files:
 - test/al-test-utils.rb
 - test/command.rb
 - test/config.yaml.sample
+- test/enable-dynamic-groups.ldif
 - test/enable-start-tls.ldif
 - test/fixtures/lower_case_object_class_schema.rb
 - test/run-test.rb