Issue #6634 has been updated by Andrew Parker.
A possible test for this:
<pre><code class="ruby">
it "should output resource documentation if needed" do
Puppet.settings[:document_all] = true
byline = sequence('documentation outputs in line order')
Puppet::Util::RDoc.expects(:puts).with("im a
class\n").in_sequence(byline)
Puppet::Util::RDoc.expects(:puts).with("im a
node\n").in_sequence(byline)
Puppet::Util::RDoc.expects(:puts).with("im a
define\n").in_sequence(byline)
Puppet::Util::RDoc.expects(:puts).with("im a
resource\n").in_sequence(byline)
# any other output must fail
Puppet::Util::RDoc.manifestdoc([my_fixture('basic.pp')])
end
</code></pre>
basic.pp
<pre>
# im a class
class foo {
file { '/tmp/foo' :
ensure => present,
}
}
# im a node
node gar {
}
# im a define
define baz { }
# im a resource
host { 'cow' : }
</pre>
----------------------------------------
Bug #6634: puppet doc doesn't document resources in manifests other than class,
node, define
https://projects.puppetlabs.com/issues/6634#change-63315
Author: Matt Robinson
Status: Accepted
Priority: Normal
Assignee:
Category:
Target version:
Affected Puppet version:
Keywords:
Branch:
This started happening in commit: b27e9b4bd79c30c7910a02122069695044f05477
You can reproduce this by running:
puppet doc filename.pp --all
with a file that looks like
# this should show up
file {'foo' : ensure => present}
Showed up as part of #5081 which was reverting #4349, so even though this was
working at some point, the code that made it work caused other problems.
--
You have received this notification because you have either subscribed to it,
or are involved in it.
To change your notification preferences, please click here:
http://projects.puppetlabs.com/my/account
--
You received this message because you are subscribed to the Google Groups
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/puppet-bugs?hl=en.
