add a bit more coverage

lib/cloudwatchtographite/exception.rb

@@ -28,6 +28,9 @@ class ArgumentTypeError < ArgumentError; end
   # Raised when too many dimensions are specified
   class TooManyDimensionError < Exception; end
 
+  # Raised when parsing fails
+  class ParseError < Exception; end
+
   # Raised when an unknown protocol is used to try to send to graphite
   class ProtocolError < Exception; end
 end

spec/metricdefinition_spec.rb

@@ -22,6 +22,9 @@
     it "should be a MetricDefinition" do
       @definition.should be_an_instance_of CloudwatchToGraphite::MetricDefinition
     end
+    it "should require valid arguments" do
+      expect { CloudwatchToGraphite::MetricDefinition.create_and_fill({'namespace' => 'blah'}) }.to raise_error(CloudwatchToGraphite::ParseError)
+    end
   end
 
   describe ".new" do

spec/metricdimension_spec.rb

@@ -13,6 +13,16 @@
     end
   end
 
+  describe ".create_and_fill" do
+    it "should return a MetricDimension" do
+      d= CloudwatchToGraphite::MetricDimension.create_and_fill({'name' => 'a', 'value' => 'b'})
+      d.should be_an_instance_of CloudwatchToGraphite::MetricDimension
+    end
+    it "should require valid arguments" do
+      expect { CloudwatchToGraphite::MetricDefinition.create_and_fill({'name' => 'blah'}) }.to raise_error(CloudwatchToGraphite::ParseError)
+    end
+  end
+
   describe ".Name" do
     it "returns the correct name" do
       @dimension.Name.should eql 'testname'