diff options
author | Luke Kanies <luke@madstop.com> | 2008-03-18 18:16:03 -0500 |
---|---|---|
committer | Luke Kanies <luke@madstop.com> | 2008-03-18 18:16:03 -0500 |
commit | a40e9b7dc58eef2e965e81f6c7be2dc7c14910d9 (patch) | |
tree | 133ce47fcf4de7107fd8a351d3bc3eaed9daf66f /spec/unit | |
parent | 7d35ae8fed989ef3edb8a304f625786a04ee5faa (diff) | |
download | puppet-a40e9b7dc58eef2e965e81f6c7be2dc7c14910d9.tar.gz puppet-a40e9b7dc58eef2e965e81f6c7be2dc7c14910d9.tar.xz puppet-a40e9b7dc58eef2e965e81f6c7be2dc7c14910d9.zip |
Fixing some tests that only failed under certain
circumstances (mostly, when loaded with other files, or
when loaded from rake or autotest rather than separately).
Diffstat (limited to 'spec/unit')
-rwxr-xr-x | spec/unit/parser/resource.rb | 4 | ||||
-rwxr-xr-x | spec/unit/ral/type/file.rb | 11 |
2 files changed, 8 insertions, 7 deletions
diff --git a/spec/unit/parser/resource.rb b/spec/unit/parser/resource.rb index 1948a3c07..776e9c742 100755 --- a/spec/unit/parser/resource.rb +++ b/spec/unit/parser/resource.rb @@ -257,9 +257,7 @@ describe Puppet::Parser::Resource do it "should fail when the override was not created by a parent class" do @override.source = "source2" @override.source.expects(:child_of?).with("source1").returns(false) - assert_raise(Puppet::ParseError, "Allowed unrelated resources to override") do - @resource.merge(@override) - end + lambda { @resource.merge(@override) }.should raise_error(Puppet::ParseError) end it "should succeed when the override was created in the current scope" do diff --git a/spec/unit/ral/type/file.rb b/spec/unit/ral/type/file.rb index 1ef924569..e1a597434 100755 --- a/spec/unit/ral/type/file.rb +++ b/spec/unit/ral/type/file.rb @@ -57,9 +57,10 @@ describe Puppet::Type.type(:file) do describe "when managing links" do require 'puppettest/support/assertions' include PuppetTest + require 'tempfile' before do - @basedir = tempfile() + @basedir = tempfile Dir.mkdir(@basedir) @file = File.join(@basedir, "file") @link = File.join(@basedir, "link") @@ -71,21 +72,23 @@ describe Puppet::Type.type(:file) do :path => @link, :mode => "755" ) + @catalog = Puppet::Node::Catalog.new + @catalog.add_resource @resource end after do - teardown + remove_tmp_files end it "should default to managing the link" do - assert_events([], @resource) + @catalog.apply # I convert them to strings so they display correctly if there's an error. ("%o" % (File.stat(@file).mode & 007777)).should == "%o" % 0644 end it "should be able to follow links" do @resource[:links] = :follow - assert_events([:file_changed], @resource) + @catalog.apply ("%o" % (File.stat(@file).mode & 007777)).should == "%o" % 0755 end |