summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeff McCune <jeff@puppetlabs.com>2012-11-26 11:37:08 -0800
committerJeff McCune <jeff@puppetlabs.com>2012-11-26 11:37:08 -0800
commitd8cde952956e80eeae617469c59a3bbd61336e70 (patch)
treef96e3c5a7c72ee528473cecf10e2439db49539ba
parentdfcee63afb19d72677f177e83ce237765cccf0f7 (diff)
downloadpuppet-stdlib-d8cde952956e80eeae617469c59a3bbd61336e70.tar.gz
puppet-stdlib-d8cde952956e80eeae617469c59a3bbd61336e70.tar.bz2
(#14670) Fixup file_line autorequire specs
Without this patch applied the file_line autorequire examples are failing. This is a problem because the failures are false positives and should be passing given the implementation. This patch fixes the problem by changing the examples to directly test the existence of the relationship by finding it in the list of autorequire relationships.
-rw-r--r--spec/unit/puppet/type/file_line_spec.rb13
1 files changed, 6 insertions, 7 deletions
diff --git a/spec/unit/puppet/type/file_line_spec.rb b/spec/unit/puppet/type/file_line_spec.rb
index 1fa8e84..0cd8a26 100644
--- a/spec/unit/puppet/type/file_line_spec.rb
+++ b/spec/unit/puppet/type/file_line_spec.rb
@@ -54,17 +54,16 @@ describe Puppet::Type.type(:file_line) do
file = Puppet::Type.type(:file).new(:name => "/tmp/path")
catalog.add_resource file
catalog.add_resource file_line
- reqs = file_line.autorequire
- reqs.size.should eq 1
- reqs[0].source.should eq file
- reqs[0].target.should eq file_line
+
+ relationship = file_line.autorequire.find do |rel|
+ (rel.source.to_s == "File[/tmp/path]") and (rel.target.to_s == file_line.to_s)
+ end
+ relationship.should be_a Puppet::Relationship
end
it "should not autorequire the file it manages if it is not managed" do
catalog = Puppet::Resource::Catalog.new
catalog.add_resource file_line
- reqs = file_line.autorequire
- reqs.size.should eq 0
+ file_line.autorequire.should be_empty
end
-
end