puppet 2.7.6 → 2.7.8

data/spec/integration/file_serving/terminus_helper_spec.rb

@@ -10,7 +10,7 @@ class TerminusHelperIntegrationTester
   end
 end
 
-describe Puppet::FileServing::TerminusHelper, :fails_on_windows => true do
+describe Puppet::FileServing::TerminusHelper do
   it "should be able to recurse on a single file" do
     @path = Tempfile.new("fileset_integration")
     request = Puppet::Indirector::Request.new(:metadata, :find, @path.path, :recurse => true)

data/spec/integration/indirector/direct_file_server_spec.rb

@@ -30,24 +30,18 @@ describe Puppet::Indirector::DirectFileServer, " when interacting with the files
   end
 end
 
-describe Puppet::Indirector::DirectFileServer, " when interacting with FileServing::Fileset and the model", :fails_on_windows => true do
-  before do
-    @terminus = Puppet::Indirector::FileContent::File.new
+describe Puppet::Indirector::DirectFileServer, " when interacting with FileServing::Fileset and the model" do
+  include PuppetSpec::Files
 
-    @path = Tempfile.new("direct_file_server_testing")
-    path = @path.path
-    @path.close!
-    @path = path
+  let(:path) { tmpdir('direct_file_server_testing') }
 
-    Dir.mkdir(@path)
-    File.open(File.join(@path, "one"), "w") { |f| f.print "one content" }
-    File.open(File.join(@path, "two"), "w") { |f| f.print "two content" }
+  before do
+    @terminus = Puppet::Indirector::FileContent::File.new
 
-    @request = @terminus.indirection.request(:search, "file:///#{@path}", :recurse => true)
-  end
+    File.open(File.join(path, "one"), "w") { |f| f.print "one content" }
+    File.open(File.join(path, "two"), "w") { |f| f.print "two content" }
 
-  after do
-    system("rm -rf #{@path}")
+    @request = @terminus.indirection.request(:search, "file:///#{path}", :recurse => true)
   end
 
   it "should return an instance for every file in the fileset" do
@@ -62,7 +56,7 @@ describe Puppet::Indirector::DirectFileServer, " when interacting with FileServi
       case instance.full_path
       when /one/; instance.content.should == "one content"
       when /two/; instance.content.should == "two content"
-      when @path
+      when path
       else
         raise "No valid key for #{instance.path.inspect}"
       end

data/spec/integration/indirector/file_content/file_server_spec.rb

@@ -6,7 +6,7 @@ require 'shared_behaviours/file_server_terminus'
 
 require 'puppet_spec/files'
 
-describe Puppet::Indirector::FileContent::FileServer, " when finding files", :fails_on_windows => true do
+describe Puppet::Indirector::FileContent::FileServer, " when finding files" do
   it_should_behave_like "Puppet::Indirector::FileServerTerminus"
   include PuppetSpec::Files
 

data/spec/integration/indirector/file_metadata/file_server_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 require 'puppet/indirector/file_metadata/file_server'
 require 'shared_behaviours/file_server_terminus'
 
-describe Puppet::Indirector::FileMetadata::FileServer, " when finding files", :fails_on_windows => true do
+describe Puppet::Indirector::FileMetadata::FileServer, " when finding files" do
   it_should_behave_like "Puppet::Indirector::FileServerTerminus"
 
   before do

data/spec/integration/provider/package_spec.rb

@@ -23,6 +23,10 @@ describe "Package provider", :'fails_on_ruby_1.9.2' => true do
       end
 
       it "should be able to get a list of existing packages", :fails_on_windows => true do
+        if provider.name == :msi
+          Puppet[:vardir] = tmpdir('msi_package_var_dir')
+        end
+
         provider.instances.each do |package|
           package.should be_instance_of(provider)
           package.properties[:provider].should == provider.name

data/spec/integration/provider/service/init_spec.rb

@@ -5,7 +5,7 @@ provider = Puppet::Type.type(:service).provider(:init)
 
 describe provider, :'fails_on_ruby_1.9.2' => true do
   describe "when running on FreeBSD", :if => (Facter.value(:operatingsystem) == "FreeBSD") do
-    it "should set its default path to include /etc/init.d and /usr/local/etc/init.d" do
+    it "should set its default path to include /etc/rc.d and /usr/local/etc/rc.d" do
       provider.defpath.should == ["/etc/rc.d", "/usr/local/etc/rc.d"]
     end
   end
@@ -16,7 +16,13 @@ describe provider, :'fails_on_ruby_1.9.2' => true do
     end
   end
 
-  describe "when not running on FreeBSD or HP-UX", :if => (! %w{HP-UX FreeBSD}.include?(Facter.value(:operatingsystem))) do
+  describe "when running on Archlinux", :if => (Facter.value(:operatingsystem) == "Archlinux") do
+    it "should set its default path to include /etc/rc.d" do
+      provider.defpath.should == "/etc/rc.d"
+    end
+  end
+
+  describe "when not running on FreeBSD, HP-UX or Archlinux", :if => (! %w{HP-UX FreeBSD Archlinux}.include?(Facter.value(:operatingsystem))) do
     it "should set its default path to include /etc/init.d" do
       provider.defpath.should == "/etc/init.d"
     end

data/spec/integration/reference/providers_spec.rb

@@ -11,6 +11,6 @@ describe reference do
   end
 
   it "should be able to be rendered as markdown" do
-    lambda { reference.to_markdown }.should_not raise_error
+    reference.to_markdown
   end
 end

data/spec/integration/ssl/certificate_request_spec.rb

@@ -3,8 +3,7 @@ require 'spec_helper'
 
 require 'puppet/ssl/certificate_request'
 
-# REMIND: Fails on windows because there is no user provider yet
-describe Puppet::SSL::CertificateRequest, :fails_on_windows => true do
+describe Puppet::SSL::CertificateRequest do
   include PuppetSpec::Files
 
   before do

data/spec/integration/ssl/certificate_revocation_list_spec.rb

@@ -3,8 +3,7 @@ require 'spec_helper'
 
 require 'puppet/ssl/certificate_revocation_list'
 
-# REMIND: Fails on windows because there is no user provider yet
-describe Puppet::SSL::CertificateRevocationList, :fails_on_windows => true do
+describe Puppet::SSL::CertificateRevocationList do
   include PuppetSpec::Files
 
   before do

data/spec/integration/ssl/host_spec.rb

@@ -3,8 +3,7 @@ require 'spec_helper'
 
 require 'puppet/ssl/host'
 
-# REMIND: Fails on windows because there is no user provider yet
-describe Puppet::SSL::Host, :fails_on_windows => true do
+describe Puppet::SSL::Host do
   include PuppetSpec::Files
 
   before do

data/spec/integration/transaction_spec.rb

@@ -16,6 +16,14 @@ describe Puppet::Transaction do
     catalog
   end
 
+  def usr_bin_touch(path)
+    Puppet.features.microsoft_windows? ? "#{ENV['windir']}/system32/cmd.exe /c \"type NUL >> \"#{path}\"\"" : "/usr/bin/touch #{path}"
+  end
+
+  def touch(path)
+    Puppet.features.microsoft_windows? ? "cmd.exe /c \"type NUL >> \"#{path}\"\"" : "touch #{path}"
+  end
+
   it "should not apply generated resources if the parent resource fails" do
     catalog = Puppet::Resource::Catalog.new
     resource = Puppet::Type.type(:file).new :path => make_absolute("/foo/bar"), :backup => false
@@ -131,11 +139,11 @@ describe Puppet::Transaction do
 
   # Verify that one component requiring another causes the contained
   # resources in the requiring component to get refreshed.
-  it "should propagate events from a contained resource through its container to its dependent container's contained resources", :fails_on_windows => true do
+  it "should propagate events from a contained resource through its container to its dependent container's contained resources" do
     transaction = nil
     file = Puppet::Type.type(:file).new :path => tmpfile("event_propagation"), :ensure => :present
     execfile = File.join(tmpdir("exec_event"), "exectestingness2")
-    exec = Puppet::Type.type(:exec).new :command => "touch #{execfile}", :path => ENV['PATH']
+    exec = Puppet::Type.type(:exec).new :command => touch(execfile), :path => ENV['PATH']
     catalog = mk_catalog(file)
 
     fcomp = Puppet::Type.type(:component).new(:name => "Foo[file]")
@@ -155,7 +163,7 @@ describe Puppet::Transaction do
   end
 
   # Make sure that multiple subscriptions get triggered.
-  it "should propagate events to all dependent resources", :fails_on_windows => true do
+  it "should propagate events to all dependent resources" do
     path = tmpfile("path")
     file1 = tmpfile("file1")
     file2 = tmpfile("file2")
@@ -167,14 +175,14 @@ describe Puppet::Transaction do
 
     exec1 = Puppet::Type.type(:exec).new(
       :path    => ENV["PATH"],
-      :command => "touch #{file1}",
+      :command => touch(file1),
       :refreshonly => true,
       :subscribe   => Puppet::Resource.new(:file, path)
     )
 
     exec2 = Puppet::Type.type(:exec).new(
       :path        => ENV["PATH"],
-      :command     => "touch #{file2}",
+      :command     => touch(file2),
       :refreshonly => true,
       :subscribe   => Puppet::Resource.new(:file, path)
     )
@@ -185,7 +193,7 @@ describe Puppet::Transaction do
     FileTest.should be_exist(file2)
   end
 
-  it "should not let one failed refresh result in other refreshes failing", :fails_on_windows => true do
+  it "should not let one failed refresh result in other refreshes failing" do
     path = tmpfile("path")
     newfile = tmpfile("file")
       file = Puppet::Type.type(:file).new(
@@ -195,7 +203,7 @@ describe Puppet::Transaction do
 
     exec1 = Puppet::Type.type(:exec).new(
       :path => ENV["PATH"],
-      :command => "touch /this/cannot/possibly/exist",
+      :command => touch(File.expand_path("/this/cannot/possibly/exist")),
       :logoutput => true,
       :refreshonly => true,
       :subscribe => file,
@@ -204,7 +212,7 @@ describe Puppet::Transaction do
 
     exec2 = Puppet::Type.type(:exec).new(
       :path => ENV["PATH"],
-      :command => "touch #{newfile}",
+      :command => touch(newfile),
       :logoutput => true,
       :refreshonly => true,
       :subscribe => [file, exec1],
@@ -218,7 +226,7 @@ describe Puppet::Transaction do
     FileTest.should be_exists(newfile)
   end
 
-  it "should still trigger skipped resources", :'fails_on_ruby_1.9.2' => true, :fails_on_windows => true do
+  it "should still trigger skipped resources", :'fails_on_ruby_1.9.2' => true do
     catalog = mk_catalog
     catalog.add_resource(*Puppet::Type.type(:schedule).mkdefaultschedules)
 
@@ -233,8 +241,8 @@ describe Puppet::Transaction do
     fname = tmpfile("exec")
 
     exec = Puppet::Type.type(:exec).new(
-      :name => "touch #{fname}",
-      :path => "/usr/bin:/bin",
+      :name => touch(fname),
+      :path => Puppet.features.microsoft_windows? ? "#{ENV['windir']}/system32" : "/usr/bin:/bin",
       :schedule => "monthly",
       :subscribe => Puppet::Resource.new("file", file.name)
     )
@@ -270,10 +278,10 @@ describe Puppet::Transaction do
     FileTest.should be_exists(fname)
   end
 
-  it "should not attempt to evaluate resources with failed dependencies", :fails_on_windows => true do
+  it "should not attempt to evaluate resources with failed dependencies" do
 
     exec = Puppet::Type.type(:exec).new(
-      :command => "/bin/mkdir /this/path/cannot/possibly/exist",
+      :command => "#{File.expand_path('/bin/mkdir')} /this/path/cannot/possibly/exist",
       :title => "mkdir"
     )
 
@@ -298,22 +306,22 @@ describe Puppet::Transaction do
     FileTest.should_not be_exists(file2[:path])
   end
 
-  it "should not trigger subscribing resources on failure", :fails_on_windows => true do
+  it "should not trigger subscribing resources on failure" do
     file1 = tmpfile("file1")
     file2 = tmpfile("file2")
 
     create_file1 = Puppet::Type.type(:exec).new(
-      :command => "/usr/bin/touch #{file1}"
+      :command => usr_bin_touch(file1)
     )
 
     exec = Puppet::Type.type(:exec).new(
-      :command => "/bin/mkdir /this/path/cannot/possibly/exist",
+      :command => "#{File.expand_path('/bin/mkdir')} /this/path/cannot/possibly/exist",
       :title => "mkdir",
       :notify => create_file1
     )
 
     create_file2 = Puppet::Type.type(:exec).new(
-      :command => "/usr/bin/touch #{file2}",
+      :command => usr_bin_touch(file2),
       :subscribe => exec
     )
 

data/spec/integration/type/exec_spec.rb

@@ -0,0 +1,77 @@
+#!/usr/bin/env rspec
+require 'spec_helper'
+
+require 'puppet_spec/files'
+
+describe Puppet::Type.type(:exec) do
+  include PuppetSpec::Files
+
+  let(:catalog) { Puppet::Resource::Catalog.new }
+  let(:path) { tmpfile('exec_provider') }
+  let(:command) { "ruby -e 'File.open(\"#{path}\", \"w\") { |f| f.print \"foo\" }'" }
+
+  before :each do
+    catalog.host_config = false
+  end
+
+  it "should execute the command" do
+    exec = described_class.new :command => command, :path => ENV['PATH']
+
+    catalog.add_resource exec
+    catalog.apply
+
+    File.read(path).should == 'foo'
+  end
+
+  it "should not execute the command if onlyif returns non-zero" do
+    exec = described_class.new(
+      :command => command,
+      :onlyif => "ruby -e 'exit 44'",
+      :path => ENV['PATH']
+    )
+
+    catalog.add_resource exec
+    catalog.apply
+
+    File.should_not be_exist(path)
+  end
+
+  it "should execute the command if onlyif returns zero" do
+    exec = described_class.new(
+      :command => command,
+      :onlyif => "ruby -e 'exit 0'",
+      :path => ENV['PATH']
+    )
+
+    catalog.add_resource exec
+    catalog.apply
+
+    File.read(path).should == 'foo'
+  end
+
+  it "should execute the command if unless returns non-zero" do
+    exec = described_class.new(
+      :command => command,
+      :unless => "ruby -e 'exit 45'",
+      :path => ENV['PATH']
+    )
+
+    catalog.add_resource exec
+    catalog.apply
+
+    File.read(path).should == 'foo'
+  end
+
+  it "should not execute the command if unless returns zero" do
+    exec = described_class.new(
+      :command => command,
+      :unless => "ruby -e 'exit 0'",
+      :path => ENV['PATH']
+    )
+
+    catalog.add_resource exec
+    catalog.apply
+
+    File.should_not be_exist(path)
+  end
+end

data/spec/integration/type/file_spec.rb

@@ -17,6 +17,10 @@ describe Puppet::Type.type(:file) do
   let(:path) { tmpfile('file_testing') }
 
   if Puppet.features.posix?
+    def set_mode(mode, file)
+      File.chmod(mode, file)
+    end
+
     def get_mode(file)
       File.lstat(file).mode
     end
@@ -33,6 +37,10 @@ describe Puppet::Type.type(:file) do
       extend Puppet::Util::Windows::Security
     end
 
+    def set_mode(mode, file)
+      SecurityHelper.set_mode(mode, file)
+    end
+
     def get_mode(file)
       SecurityHelper.get_mode(file)
     end
@@ -92,6 +100,255 @@ describe Puppet::Type.type(:file) do
 
       get_group(path).should == group
     end
+
+    describe "when setting mode" do
+      describe "for directories" do
+        let(:path) { tmpdir('dir_mode') }
+
+        it "should set executable bits for newly created directories" do
+          catalog.add_resource described_class.new(:path => path, :ensure => :directory, :mode => 0600)
+          catalog.apply
+
+          (get_mode(path) & 07777).should == 0700
+        end
+
+        it "should set executable bits for existing readable directories" do
+          File.should be_directory(path)
+          set_mode(0600, path)
+
+          catalog.add_resource described_class.new(:path => path, :ensure => :directory, :mode => 0644)
+          catalog.apply
+
+          (get_mode(path) & 07777).should == 0755
+        end
+
+        it "should not set executable bits for unreadable directories" do
+          begin
+            catalog.add_resource described_class.new(:path => path, :ensure => :directory, :mode => 0300)
+            catalog.apply
+
+            (get_mode(path) & 07777).should == 0300
+          ensure
+            # so we can cleanup
+            set_mode(0700, path)
+          end
+        end
+
+        it "should set user, group, and other executable bits" do
+          catalog.add_resource described_class.new(:path => path, :ensure => :directory, :mode => 0664)
+          catalog.apply
+
+          (get_mode(path) & 07777).should == 0775
+        end
+
+        it "should set executable bits when overwriting a non-executable file" do
+          FileUtils.rmdir(path)
+          FileUtils.touch(path)
+          set_mode(0444, path)
+
+          catalog.add_resource described_class.new(:path => path, :ensure => :directory, :mode => 0666, :backup => false)
+          catalog.apply
+
+          (get_mode(path) & 07777).should == 0777
+        end
+      end
+
+      describe "for files" do
+        let(:path) { tmpfile('file_mode') }
+
+        it "should not set executable bits" do
+          catalog.add_resource described_class.new(:path => path, :ensure => :file, :mode => 0666)
+          catalog.apply
+
+          (get_mode(path) & 07777).should == 0666
+        end
+
+        it "should not set executable bits when replacing an executable directory (#10365)" do
+          pending("bug #10365")
+
+          FileUtils.mkdir(path)
+          set_mode(0777, path)
+
+          catalog.add_resource described_class.new(:path => path, :ensure => :file, :mode => 0666, :backup => false, :force => true)
+          catalog.apply
+
+          (get_mode(path) & 07777).should == 0666
+        end
+      end
+
+      describe "for links", :unless => Puppet.features.microsoft_windows? do
+        let(:link) { tmpfile('link_mode') }
+
+        describe "when managing links" do
+          let(:target) { tmpfile('target') }
+
+          before :each do
+            FileUtils.touch(target)
+            File.chmod(0444, target)
+
+            File.symlink(target, link)
+          end
+
+          it "should not set the executable bit on the link nor the target" do
+            catalog.add_resource described_class.new(:path => link, :ensure => :link, :mode => 0666, :target => target, :links => :manage)
+            catalog.apply
+
+            (File.stat(link).mode & 07777) == 0666
+            (File.lstat(target).mode & 07777) == 0444
+          end
+
+          it "should ignore dangling symlinks (#6856)" do
+            File.delete(target)
+
+            catalog.add_resource described_class.new(:path => link, :ensure => :link, :mode => 0666, :target => target, :links => :manage)
+            catalog.apply
+
+            File.should_not be_exist(link)
+          end
+        end
+
+        describe "when following links" do
+          it "should ignore dangling symlinks (#6856)" do
+            target = tmpfile('dangling')
+
+            FileUtils.touch(target)
+            File.symlink(target, link)
+            File.delete(target)
+
+            catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0600, :links => :follow)
+            catalog.apply
+          end
+
+          describe "to a directory" do
+            let(:target) { tmpdir('dir_target') }
+
+            before :each do
+              File.chmod(0600, target)
+
+              File.symlink(target, link)
+            end
+
+            after :each do
+              File.chmod(0750, target)
+            end
+
+            describe "that is readable" do
+              it "should set the executable bits when creating the destination (#10315)" do
+                pending "bug #10315"
+
+                catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0666, :links => :follow)
+                catalog.apply
+
+                (get_mode(path) & 07777).should == 0777
+              end
+
+              it "should set the executable bits when overwriting the destination (#10315)" do
+                pending "bug #10315"
+
+                FileUtils.touch(path)
+
+                catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0666, :links => :follow)
+                catalog.apply
+
+                (get_mode(path) & 07777).should == 0777
+              end
+            end
+
+            describe "that is not readable" do
+              before :each do
+                set_mode(0300, target)
+              end
+
+              # so we can cleanup
+              after :each do
+                set_mode(0700, target)
+              end
+
+              it "should not set executable bits when creating the destination (#10315)" do
+                pending "bug #10315"
+
+                catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0666, :links => :follow)
+                catalog.apply
+
+                (get_mode(path) & 07777).should == 0666
+              end
+
+              it "should not set executable bits when overwriting the destination" do
+                FileUtils.touch(path)
+
+                catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0666, :links => :follow)
+                catalog.apply
+
+                (get_mode(path) & 07777).should == 0666
+              end
+            end
+          end
+
+          describe "to a file" do
+            let(:target) { tmpfile('file_target') }
+
+            it "should create the file, not a symlink (#2817, #10315)" do
+              pending "bug #2817, #10315"
+
+              catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0600, :links => :follow)
+              catalog.apply
+
+              File.should be_file(path)
+              (get_mode(path) & 07777) == 0600
+            end
+
+            it "should overwrite the file" do
+              FileUtils.touch(path)
+
+              catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0600, :links => :follow)
+              catalog.apply
+
+              File.should be_file(path)
+              (get_mode(path) & 07777) == 0600
+            end
+          end
+
+          describe "to a link to a directory" do
+            let(:real_target) { tmpdir('real_target') }
+            let(:target) { tmpfile('target') }
+
+            before :each do
+              File.chmod(0666, real_target)
+
+              # link -> target -> real_target
+              File.symlink(real_target, target)
+              File.symlink(target, link)
+            end
+
+            after :each do
+              File.chmod(0750, real_target)
+            end
+
+            describe "when following all links" do
+              it "should create the destination and apply executable bits (#10315)" do
+                pending "bug #10315"
+
+                catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0600, :links => :follow)
+                catalog.apply
+
+                File.should be_directory(path)
+                (get_mode(path) & 07777) == 0777
+              end
+
+              it "should overwrite the destination and apply executable bits" do
+                FileUtils.mkdir(path)
+
+                catalog.add_resource described_class.new(:path => path, :source => link, :mode => 0600, :links => :follow)
+                catalog.apply
+
+                File.should be_directory(path)
+                (get_mode(path) & 07777) == 0777
+              end
+            end
+          end
+        end
+      end
+    end
   end
 
   describe "when writing files" do
@@ -101,9 +358,9 @@ describe Puppet::Type.type(:file) do
       catalog.add_resource file
       catalog.add_resource filebucket
 
-      File.open(file[:path], "w") { |f| f.puts "bar" }
+      File.open(file[:path], "wb") { |f| f.puts "bar" }
 
-      md5 = Digest::MD5.hexdigest(File.read(file[:path]))
+      md5 = Digest::MD5.hexdigest(Puppet::Util.binread(file[:path]))
 
       catalog.apply
 
@@ -695,6 +952,69 @@ describe Puppet::Type.type(:file) do
     File.should_not be_exist(path)
   end
 
+  describe "when sourcing" do
+    let(:source) {
+      source = tmpfile("source_default_values")
+      File.open(source, "w") { |f| f.puts "yay" }
+      source
+    }
+
+    it "should apply the source metadata values" do
+      set_mode(0770, source)
+
+      file = described_class.new(
+        :path   => path,
+        :ensure => :file,
+        :source => source,
+        :backup => false
+      )
+
+      catalog.add_resource file
+      catalog.apply
+
+      get_owner(path).should == get_owner(source)
+      get_group(path).should == get_group(source)
+      (get_mode(path) & 07777).should == 0770
+    end
+
+    it "should override the default metadata values" do
+      set_mode(0770, source)
+
+      file = described_class.new(
+         :path   => path,
+         :ensure => :file,
+         :source => source,
+         :backup => false,
+         :mode => 0440
+       )
+
+      catalog.add_resource file
+      catalog.apply
+
+      (get_mode(path) & 07777).should == 0440
+    end
+
+    describe "on Windows systems", :if => Puppet.features.microsoft_windows? do
+      it "should provide valid default values when ACLs are not supported" do
+        Puppet::Util::Windows::Security.stubs(:supports_acl?).with(source).returns false
+
+        file = described_class.new(
+          :path   => path,
+          :ensure => :file,
+          :source => source,
+          :backup => false
+        )
+
+        catalog.add_resource file
+        catalog.apply
+
+        get_owner(path).should == 'S-1-5-32-544'
+        get_group(path).should == 'S-1-0-0'
+        get_mode(path).should == 0644
+      end
+    end
+  end
+
   describe "when purging files" do
     before do
       sourcedir = tmpfile("purge_source")