diff options
| author | Luke Kanies <luke@madstop.com> | 2008-10-06 19:03:40 -0500 |
|---|---|---|
| committer | James Turnbull <james@lovedthanlost.net> | 2008-10-08 10:56:09 +1100 |
| commit | 765db307f875a52b7dec386b0af90f8e5b4590bb (patch) | |
| tree | 52d245b9521003bc3548059f23aed8e5cb2a565d /spec | |
| parent | fb14e91226e494210c3b6c88d8553a745e4ac3ed (diff) | |
| download | puppet-765db307f875a52b7dec386b0af90f8e5b4590bb.tar.gz puppet-765db307f875a52b7dec386b0af90f8e5b4590bb.tar.xz puppet-765db307f875a52b7dec386b0af90f8e5b4590bb.zip | |
Adding partial spec tests for Puppet::Util::Metric.
Signed-off-by: Luke Kanies <luke@madstop.com>
Diffstat (limited to 'spec')
| -rwxr-xr-x | spec/unit/util/metric.rb | 86 |
1 files changed, 86 insertions, 0 deletions
diff --git a/spec/unit/util/metric.rb b/spec/unit/util/metric.rb new file mode 100755 index 000000000..7aa04486c --- /dev/null +++ b/spec/unit/util/metric.rb @@ -0,0 +1,86 @@ +#!/usr/bin/env ruby + +Dir.chdir(File.dirname(__FILE__)) { (s = lambda { |f| File.exist?(f) ? require(f) : Dir.chdir("..") { s.call(f) } }).call("spec/spec_helper.rb") } + +require 'puppet/util/metric' + +describe Puppet::Util::Metric do + before do + @metric = Puppet::Util::Metric.new("foo") + end + + it "should be aliased to Puppet::Metric" do + Puppet::Util::Metric.should equal(Puppet::Metric) + end + + [:type, :name, :value, :label, :basedir].each do |name| + it "should have a #{name} attribute" do + @metric.should respond_to(name) + @metric.should respond_to(name.to_s + "=") + end + end + + it "should default to the :rrdir as the basedir "do + Puppet.settings.expects(:value).with(:rrddir).returns "myrrd" + @metric.basedir.should == "myrrd" + end + + it "should use any provided basedir" do + @metric.basedir = "foo" + @metric.basedir.should == "foo" + end + + it "should require a name at initialization" do + lambda { Puppet::Util::Metric.new }.should raise_error(ArgumentError) + end + + it "should always convert its name to a string" do + Puppet::Util::Metric.new(:foo).name.should == "foo" + end + + it "should support a label" do + Puppet::Util::Metric.new("foo", "mylabel").label.should == "mylabel" + end + + it "should autogenerate a label if none is provided" do + Puppet::Util::Metric.new("foo_bar").label.should == "Foo bar" + end + + it "should have a method for adding values" do + @metric.should respond_to(:newvalue) + end + + it "should have a method for returning values" do + @metric.should respond_to(:values) + end + + it "should require a name and value for its values" do + lambda { @metric.newvalue }.should raise_error(ArgumentError) + end + + it "should support a label for values" do + @metric.newvalue(:foo, 10, "label") + @metric.values[0][1].should == "label" + end + + it "should autogenerate value labels if none is provided" do + @metric.newvalue("foo_bar", 10) + @metric.values[0][1].should == "Foo bar" + end + + it "should return its values sorted by label" do + @metric.newvalue(:foo, 10, "b") + @metric.newvalue(:bar, 10, "a") + + @metric.values.should == [[:bar, "a", 10], [:foo, "b", 10]] + end + + # LAK: I'm not taking the time to develop these tests right now. + # I expect they should actually be extracted into a separate class + # anyway. + it "should be able to graph metrics using RRDTool" + + it "should be able to create a new RRDTool database" + + it "should be able to store metrics into an RRDTool database" +end |
