adauth 2.0.2 → 2.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 48e8b82dd827473a19acccdc250b66bcdf00e44a
-  data.tar.gz: af6839234b1405eecbb6551dcf91acecc6d76c2b
+  metadata.gz: 1a01855a1bb516984482d2cbdaa7020445431786
+  data.tar.gz: 077068256ee217f616c28322931be7500e8fb967
 SHA512:
-  metadata.gz: 0fc542e5a2f1392253738852897817726bdb2b1bfb697dc82fe287a775729cd75131b85f1135727a3d7e9e869fe4b3131829417c828bd625ff0e09303f45b41b
-  data.tar.gz: 1a08d5744964257fd2284f7c52de1fd853f5c3783f1c182d0e77b5c6821a32f82f793fd48beb0ba753de391228c8cbbadbe3c3e3ece4e07e7ad7d854b66152c0
+  metadata.gz: 6e9fd1e2c0af98fff7ce34b6bfe0300490a718c5408d4d7da9f49f84d709070c4ec1163a9fa4ebc90f579295ffa10b7068ba819f3822d4d39241561790af6db5
+  data.tar.gz: 120341150092ad1327e8cac8a20ba4c321215c504c2964265920cd33762a4670b87bbeaf9b66c86cce5c99f6ce5827673531f4aaaa91d7f2a2d743fad48838cc

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    adauth (2.0.1)
+    adauth (2.0.2)
       expects (~> 0.0.2)
       net-ldap
 

data/lib/adauth.rb

@@ -10,6 +10,7 @@ require 'adauth/ad_object'
 require 'adauth/authenticate'
 require 'adauth/config'
 require 'adauth/connection'
+require 'adauth/search_results'
 # AdObjects
 require 'adauth/ad_objects/computer'
 require 'adauth/ad_objects/folder'

data/lib/adauth/ad_object.rb

@@ -16,11 +16,33 @@ module Adauth
     # Provides all the common functions for Active Directory.
     class AdObject
         include Expects
-      
+        
+        def self.method_missing(method, *args)
+          return super unless method =~ /^find_by_/
+          method_field = method.to_s.split("_").last
+          field = self::Fields[method_field.to_sym]
+          return super unless field
+          self.where(field, args.first)
+        end
+        
+        def method_missing(method, *args)
+          field = self.class::Fields[method]
+          return handle_field(field) if field
+          return super
+        end
+        
+        def self.reverse_field(search)
+          hash = {}
+          self::Fields.each do |k, v|
+            hash[v] = k
+          end
+          return hash[search]
+        end
+        
         # Returns all objects which have the ObjectClass of the inherited class
         def self.all
             Adauth.logger.info(self.class.inspect) { "Searching for all objects matching filter \"#{self::ObjectFilter}\"" }
-            self.filter(self::ObjectFilter)
+            Adauth::SearchResults.new(self.filter(self::ObjectFilter))
         end
         
         # Returns all the objects which match the supplied query
@@ -29,7 +51,7 @@ module Adauth
         def self.where(field, value)
             search_filter = Net::LDAP::Filter.eq(field, value)
             Adauth.logger.info(self.class.inspect) { "Searching for all \"#{self::ObjectFilter}\" where #{field} = #{value}" }
-            filter(add_object_filter(search_filter))
+            Adauth::SearchResults.new(filter(add_object_filter(search_filter)))
         end
         
         # Returns all LDAP objects that match the given filter
@@ -65,13 +87,6 @@ module Adauth
             @ldap_object
         end
         
-        # Over ride method missing to see if the object has a field by that name
-        def method_missing(method, *args)
-          field = self.class::Fields[method]
-          return handle_field(field) if field
-          return super
-        end
-        
         # Handle the output for the given field
         def handle_field(field)
           case field

data/lib/adauth/ad_objects/group.rb

@@ -18,7 +18,7 @@ module Adauth
             Fields = {
                     :name => :samaccountname,
                     :cn_members => [ :member,
-                        Proc.new {|g| g.sub(/.*?CN=(.*?),.*/, '\1')} ],
+                        Proc.new {|g| g.sub(/.*?CN=(.*?),.*/, '\1').to_s} ],
                     :memberof => :member
                 }
             

data/lib/adauth/ad_objects/user.rb

@@ -21,7 +21,7 @@ module Adauth
                     :email => :mail,
                     :name => :name,
                     :cn_groups => [ :memberof,
-                        Proc.new {|g| g.sub(/.*?CN=(.*?),.*/, '\1')} ]
+                        Proc.new {|g| g.sub(/.*?CN=(.*?),.*/, '\1').to_s} ]
                     }
               
             # Object Net::LDAP filter

data/lib/adauth/search_results.rb

@@ -0,0 +1,18 @@
+module Adauth
+  class SearchResults < Array
+    def limit(x)
+      return self[0..(x-1)]
+    end
+    
+    def order(field, direction = :asc)
+      case direction
+      when :asc
+        return sort! { |x, y| x.send(field) <=> y.send(field) }
+      when :desc
+        return order(field, :asc).reverse!
+      else
+        raise "Invalid Order Provided, please use :asc or :desc"
+      end
+    end
+  end
+end

data/lib/adauth/version.rb

@@ -1,4 +1,4 @@
 module Adauth
     # Adauths Version Number
-    Version = '2.0.2'
+    Version = '2.0.3'
 end

data/spec/adauth_ad_object_user_spec.rb

@@ -46,9 +46,9 @@ describe Adauth::AdObjects::User do
     it "should allow for additional methods" do
         default_config
         Adauth.add_field(Adauth::AdObjects::User, :description, :description)
-        administrator.description.should be_a String
+        user.description.should be_a String
         Adauth.add_field(Adauth::AdObjects::User, :objectguid, :objectguid)
-        administrator.objectguid.should be_a String
+        user.objectguid.should be_a String
     end
     
     it "should allow you to reset the password" do
@@ -86,4 +86,10 @@ describe Adauth::AdObjects::User do
       rq_user.member_of?("Adauth Test Group").should be_false
       new_group.delete
     end
+    
+    it "should have find_by methods (and not break method_missing)" do
+      default_config
+      lambda { Adauth::AdObjects::User.fooooooooo }.should raise_exception
+      Adauth::AdObjects::User.find_by_login(test_data("domain", "breakable_user")).should be_a Adauth::SearchResults
+    end
 end

data/spec/adauth_rails_model_bridge_spec.rb

@@ -21,17 +21,21 @@ class TestUserModel
 end
 
 describe Adauth::Rails::ModelBridge do
+  let(:user) do
+    Adauth::AdObjects::User.where('sAMAccountName', test_data("domain", "breakable_user")).first
+  end
+  
       it "should extend", :no_ad => true do
         TestUserModel.should respond_to :create_from_adauth
     end
     
     it "should create the model" do
         default_config
-        TestUserModel.create_from_adauth(administrator)
+        TestUserModel.create_from_adauth(user)
     end
     
     it "should return and create the model" do
         default_config
-        TestUserModel.return_and_create_from_adauth(administrator)
+        TestUserModel.return_and_create_from_adauth(user)
     end
 end

data/spec/adauth_search_results_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe Adauth::SearchResults, :no_ad => true do
+  let(:test_array) do
+    [OpenStruct.new({name: "foo"}), OpenStruct.new({name: "bar"}), OpenStruct.new({name: "widget"})]
+  end
+  
+  let(:sorted_array) do
+    [OpenStruct.new({name: "bar"}), OpenStruct.new({name: "foo"}), OpenStruct.new({name: "widget"})]
+  end
+  
+  let(:search_results) do
+    Adauth::SearchResults.new(test_array)
+  end
+  
+  it "should create self from_array" do
+    Adauth::SearchResults.new(test_array).should be_a Adauth::SearchResults
+  end
+  
+  it "should have the limit function" do
+    search_results.limit(2).length.should eq 2
+    search_results.limit(2).last.should_not eq test_array.last
+    search_results.limit(2).should be_a Adauth::SearchResults
+  end
+  
+  it "should have the order function" do
+    search_results.order(:name, :asc).should eq sorted_array
+    search_results.order(:name, :asc).should be_a Adauth::SearchResults
+    search_results.order(:name, :desc).should eq sorted_array.reverse
+    search_results.order(:name, :desc).should be_a Adauth::SearchResults
+  end
+  
+  it "should handle having a wrong direction passed to it" do
+    lambda { search_results.order(:name, :foo) }.should raise_exception
+  end
+  
+  it "should default to :asc for order" do
+    search_results.order(:name).should eq sorted_array
+    search_results.order(:name).should be_a Adauth::SearchResults
+  end
+end

data/spec/spec_helper.rb

@@ -23,15 +23,3 @@ def test_data(set, key)
     @yaml ||= YAML::load(File.open('spec/test_data.yml'))
     @yaml[set][key]
 end
-
-def administrator
-    Adauth::AdObjects::User.where('sAMAccountName', "administrator").first
-end
-
-#def breakable_user
-#  Adauth::AdObjects::User.where('sAMAccountName', test_data("domain", "breakable_user")).first
-#end
-
-def query_user
-  Adauth::AdObjects::User.where('sAMAccountName', test_data("domain", "query_user")).first
-end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: adauth
 version: !ruby/object:Gem::Version
-  version: 2.0.2
+  version: 2.0.3
 platform: ruby
 authors:
 - Adam "Arcath" Laycock
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-28 00:00:00.000000000 Z
+date: 2013-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -124,6 +124,7 @@ files:
 - lib/adauth/rails.rb
 - lib/adauth/rails/helpers.rb
 - lib/adauth/rails/model_bridge.rb
+- lib/adauth/search_results.rb
 - lib/adauth/version.rb
 - lib/generators/adauth/config/USAGE
 - lib/generators/adauth/config/config_generator.rb
@@ -144,6 +145,7 @@ files:
 - spec/adauth_connection_spec.rb
 - spec/adauth_issue_spec.rb
 - spec/adauth_rails_model_bridge_spec.rb
+- spec/adauth_search_results_spec.rb
 - spec/adauth_spec.rb
 - spec/spec_helper.rb
 - spec/test_data.example.yml
@@ -183,6 +185,7 @@ test_files:
 - spec/adauth_connection_spec.rb
 - spec/adauth_issue_spec.rb
 - spec/adauth_rails_model_bridge_spec.rb
+- spec/adauth_search_results_spec.rb
 - spec/adauth_spec.rb
 - spec/spec_helper.rb
 - spec/test_data.example.yml