Use LDAP::Config in all of the adapter methods

lib/gitlab/ldap/adapter.rb

@@ -31,7 +31,7 @@ module Gitlab
       #
       def groups(cn = "*", size = nil)
         options = {
-          base: config['group_base'],
+          base: config.group_base,
           filter: Net::LDAP::Filter.eq("cn", cn)
         }
 
@@ -54,13 +54,13 @@ module Gitlab
           }
         else
           options = {
-            base: config['base'],
+            base: config.base,
             filter: Net::LDAP::Filter.eq(field, value)
           }
         end
 
-        if config['user_filter'].present?
-          user_filter = Net::LDAP::Filter.construct(config['user_filter'])
+        if config.user_filter.present?
+          user_filter = Net::LDAP::Filter.construct(config.user_filter)
 
           options[:filter] = if options[:filter]
                                Net::LDAP::Filter.join(options[:filter], user_filter)

lib/gitlab/ldap/config.rb

@@ -23,6 +23,14 @@ module Gitlab
         end
       end
 
+      def base
+        options['base']
+      end
+
+      def uid
+        options['uid']
+      end
+
       def sync_ssh_keys?
         sync_ssh_keys.present?
       end

spec/lib/gitlab/ldap/access_spec.rb

@@ -34,7 +34,7 @@ describe Gitlab::LDAP::Access do
     subject { access.update_permissions }
 
     it "syncs ssh keys if enabled by configuration" do
-      access.stub sync_ssh_keys?: true
+      access.stub sync_ssh_keys?: 'sshpublickey'
       expect(access).to receive(:update_ssh_keys).once
 
       subject