Fixing #2574 - autoloading finds plugins in modules

We had some stupid errors that were preventing this
from happening; this fixes them and adds an
integration test.

Signed-off-by: Luke Kanies <luke@madstop.com>

1 files changed, 17 insertions, 3 deletions

diff --git a/spec/unit/util/autoload.rb b/spec/unit/util/autoload.rb
index c4a8642a0..18938125d 100755
--- a/spec/unit/util/autoload.rb
+++ b/spec/unit/util/autoload.rb
@@ -22,9 +22,9 @@ describe Puppet::Util::Autoload do
     describe "when building the search path" do
         it "should collect all of the plugins and lib directories that exist in the current environment's module path" do
             Puppet.settings.expects(:value).with(:environment).returns "foo"
-            Puppet.settings.expects(:value).with(:modulepath, "foo").returns %w{/a /b /c}
-            Dir.expects(:entries).with("/a").returns %w{/a/one /a/two}
-            Dir.expects(:entries).with("/b").returns %w{/b/one /b/two}
+            Puppet.settings.expects(:value).with(:modulepath, :foo).returns "/a:/b:/c"
+            Dir.expects(:entries).with("/a").returns %w{one two}
+            Dir.expects(:entries).with("/b").returns %w{one two}
 
             FileTest.stubs(:directory?).returns false
             FileTest.expects(:directory?).with("/a").returns true
@@ -36,6 +36,20 @@ describe Puppet::Util::Autoload do
             @autoload.module_directories.should == %w{/a/one/plugins /a/two/lib /b/one/plugins /b/two/lib}
         end
 
+        it "should not look for lib directories in directories starting with '.'" do
+            Puppet.settings.expects(:value).with(:environment).returns "foo"
+            Puppet.settings.expects(:value).with(:modulepath, :foo).returns "/a"
+            Dir.expects(:entries).with("/a").returns %w{. ..}
+
+            FileTest.expects(:directory?).with("/a").returns true
+            FileTest.expects(:directory?).with("/a/./lib").never
+            FileTest.expects(:directory?).with("/a/./plugins").never
+            FileTest.expects(:directory?).with("/a/../lib").never
+            FileTest.expects(:directory?).with("/a/../plugins").never
+
+            @autoload.module_directories
+        end
+
         it "should include the module directories, the Puppet libdir, and all of the Ruby load directories" do
             @autoload.expects(:module_directories).returns %w{/one /two}
             @autoload.search_directories.should == ["/one", "/two", Puppet[:libdir], $:].flatten