summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNick Lewis <nick@puppetlabs.com>2011-05-03 16:31:37 -0700
committerNick Lewis <nick@puppetlabs.com>2011-05-03 16:31:37 -0700
commit71af776efc45daaaf14ca66db41261d6906bfd1b (patch)
treed7ba3529c0cbc64f4cc1cebb36cf9b784fac5374
parent18b3584e16515cfc45aeaa8d0913de8e8bcb3e95 (diff)
parent1f112cd61ff9e9a3b4a27314533ea4a91dcca76e (diff)
downloadpuppet-71af776efc45daaaf14ca66db41261d6906bfd1b.tar.gz
puppet-71af776efc45daaaf14ca66db41261d6906bfd1b.tar.xz
puppet-71af776efc45daaaf14ca66db41261d6906bfd1b.zip
Merge branch 'ticket/2.7.x/7139' into 2.7.x
-rw-r--r--lib/puppet/file_serving/fileset.rb2
-rwxr-xr-xspec/unit/file_serving/fileset_spec.rb7
2 files changed, 8 insertions, 1 deletions
diff --git a/lib/puppet/file_serving/fileset.rb b/lib/puppet/file_serving/fileset.rb
index c020f036d..f29f70a53 100644
--- a/lib/puppet/file_serving/fileset.rb
+++ b/lib/puppet/file_serving/fileset.rb
@@ -59,7 +59,7 @@ class Puppet::FileServing::Fileset
end
def initialize(path, options = {})
- path = path.chomp(File::SEPARATOR)
+ path = path.chomp(File::SEPARATOR) unless path == File::SEPARATOR
raise ArgumentError.new("Fileset paths must be fully qualified") unless File.expand_path(path) == path
@path = path
diff --git a/spec/unit/file_serving/fileset_spec.rb b/spec/unit/file_serving/fileset_spec.rb
index a369ad39c..41810650a 100755
--- a/spec/unit/file_serving/fileset_spec.rb
+++ b/spec/unit/file_serving/fileset_spec.rb
@@ -20,6 +20,13 @@ describe Puppet::FileServing::Fileset, " when initializing" do
fileset.path.should == path
end
+ it "should not fail if the path is just the file separator" do
+ path = File::SEPARATOR
+ File.stubs(:lstat).with(path).returns stub('stat')
+ fileset = Puppet::FileServing::Fileset.new(path)
+ fileset.path.should == path
+ end
+
it "should fail if its path does not exist" do
File.expects(:lstat).with("/some/file").returns nil
proc { Puppet::FileServing::Fileset.new("/some/file") }.should raise_error(ArgumentError)