diff options
author | Jeff McCune <jeff@puppetlabs.com> | 2012-11-26 11:37:08 -0800 |
---|---|---|
committer | Jeff McCune <jeff@puppetlabs.com> | 2012-11-26 11:37:08 -0800 |
commit | d8cde952956e80eeae617469c59a3bbd61336e70 (patch) | |
tree | f96e3c5a7c72ee528473cecf10e2439db49539ba | |
parent | dfcee63afb19d72677f177e83ce237765cccf0f7 (diff) | |
download | puppet-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.rb | 13 |
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 |