diff options
author | Jesse Wolfe <jes5199@gmail.com> | 2010-10-04 14:01:49 -0700 |
---|---|---|
committer | Markus Roberts <Markus@reality.com> | 2010-11-16 12:22:04 -0800 |
commit | 178c2a628256d22b6bec39ee6f9022b38c9aa4f4 (patch) | |
tree | 05170d52aba75b0a3afe4e0f984e38c58c35564f /test | |
parent | 74b6c09e3b074e0ffc8679dcf31b5fa900020e39 (diff) | |
download | puppet-178c2a628256d22b6bec39ee6f9022b38c9aa4f4.tar.gz puppet-178c2a628256d22b6bec39ee6f9022b38c9aa4f4.tar.xz puppet-178c2a628256d22b6bec39ee6f9022b38c9aa4f4.zip |
Fix test failures that fixing #4726 exposed.
The patch for #4726 causes old unit tests of the rrd reporting
infrastructure to run on my machine. These tests were calling the old
report api, which does not succeed.
Also, the rrd settings had unintentionally been moved out of the
:metrics section, making it possible that the rrd report directory would
fail to get created during testing.
Diffstat (limited to 'test')
-rwxr-xr-x | test/other/report.rb | 7 | ||||
-rwxr-xr-x | test/util/metrics.rb | 4 |
2 files changed, 4 insertions, 7 deletions
diff --git a/test/other/report.rb b/test/other/report.rb index b3b41da19..8a909b41c 100755 --- a/test/other/report.rb +++ b/test/other/report.rb @@ -35,10 +35,7 @@ class TestReports < Test::Unit::TestCase config.retrieval_duration = 0.001 trans = config.apply - report = Puppet::Transaction::Report.new - trans.add_metrics_to_report(report) - - report + trans.generate_report end # Make sure we can use reports as log destinations. @@ -95,7 +92,7 @@ class TestReports < Test::Unit::TestCase assert_equal(yaml, File.read(file), "File did not get written") end - if Puppet.features.rrd? + if Puppet.features.rrd? || Puppet.features.rrd_legacy? def test_rrdgraph_report Puppet.settings.use(:main, :metrics) report = mkreport diff --git a/test/util/metrics.rb b/test/util/metrics.rb index 1fd57f2f1..82e792d0b 100755 --- a/test/util/metrics.rb +++ b/test/util/metrics.rb @@ -8,7 +8,7 @@ require 'puppettest' require 'puppet/type' class TestMetric < PuppetTest::TestCase - confine "Missing RRDtool library" => Puppet.features.rrd? + confine "Missing RRDtool library" => (Puppet.features.rrd? || Puppet.features.rrd_legacy?) include PuppetTest def gendata @@ -43,7 +43,7 @@ class TestMetric < PuppetTest::TestCase def rundata(report, time) assert_nothing_raised { gendata.each do |name, data| - report.newmetric(name, data) + report.add_metric(name, data) end report.metrics.each { |n, m| m.store(time) } } |