Add Classic members search strategy

mtodd · mtodd · commit 9f48809ca3ae · 2014-11-24T22:13:50.000-08:00
diff --git a/lib/github/ldap/members.rb b/lib/github/ldap/members.rb
@@ -1,3 +1,4 @@
+require 'github/ldap/members/classic'
 require 'github/ldap/members/recursive'
 
 module GitHub
@@ -13,6 +14,7 @@ class Ldap
     module Members
       # Internal: Mapping of strategy name to class.
       STRATEGIES = {
+        :classic   => GitHub::Ldap::Members::Classic,
         :recursive => GitHub::Ldap::Members::Recursive
       }
     end
diff --git a/lib/github/ldap/members/classic.rb b/lib/github/ldap/members/classic.rb
@@ -0,0 +1,30 @@
+module GitHub
+  class Ldap
+    module Members
+      # Look up group members using the existing `Group#members` and
+      # `Group#subgroups` API.
+      class Classic
+        # Internal: The GitHub::Ldap object to search domains with.
+        attr_reader :ldap
+
+        # Public: Instantiate new search strategy.
+        #
+        # - ldap:    GitHub::Ldap object
+        # - options: Hash of options (unused)
+        def initialize(ldap, options = {})
+          @ldap    = ldap
+          @options = options
+        end
+
+        # Public: Performs search for group members, including groups and
+        # members of subgroups recursively.
+        #
+        # Returns Array of Net::LDAP::Entry objects.
+        def perform(group_entry)
+          group = ldap.load_group(group_entry)
+          group.members + group.subgroups
+        end
+      end
+    end
+  end
+end
