summaryrefslogtreecommitdiffstats
path: root/spec/unit
diff options
context:
space:
mode:
authorLuke Kanies <luke@madstop.com>2008-03-18 18:16:03 -0500
committerLuke Kanies <luke@madstop.com>2008-03-18 18:16:03 -0500
commita40e9b7dc58eef2e965e81f6c7be2dc7c14910d9 (patch)
tree133ce47fcf4de7107fd8a351d3bc3eaed9daf66f /spec/unit
parent7d35ae8fed989ef3edb8a304f625786a04ee5faa (diff)
downloadpuppet-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-xspec/unit/parser/resource.rb4
-rwxr-xr-xspec/unit/ral/type/file.rb11
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