ski_binding_calculator 0.8.2 → 0.8.5

data/lib/ski_binding_calculator/calculator.rb

@@ -36,7 +36,13 @@ class SkiBinding::Calculator
         raise SkiBinding::Error.new(*[:weight, "is less than 10kg"])
       end
       hashy[:height] = attrs[:height].to_f
+      if hashy[:height] <= 0.0
+        raise SkiBinding::Error.new(*[:height, "must be bigger than 0"])
+      end
       hashy[:sole_length] = attrs[:sole_length].to_f
+      if hashy[:sole_length] <= 0.0
+        raise SkiBinding::Error.new(*[:sole_length, "must be bigger than 0"])
+      end 
       hashy[:birthday_year] = attrs[:birthday_year].to_i 
       hashy[:birthday_month] = attrs[:birthday_month].to_i 
       hashy[:birthday_day] = attrs[:birthday_day].to_i

data/lib/ski_binding_calculator/version.rb

@@ -1,3 +1,3 @@
 module SkiBinding
-  VERSION = "0.8.2"
+  VERSION = "0.8.5"
 end

data/spec/lib/calculator_spec.rb

@@ -93,14 +93,36 @@ describe SkiBinding::Calculator do
       end
     end
     
-    context "when weight is less than 9kg" do
+    context "when weight is less than 10kg" do
       let(:parameters) do
         skiers_parameter[:weight] = 9 
         skiers_parameter
       end
       it "raises SkiBinding::Error" do
-          expect { calculated_preped }.to raise_error(SkiBinding::Error) { |e|
-            e.messages.should == { :weight => "is less than 10kg" } }
+        expect { calculated_preped }.to raise_error(SkiBinding::Error) { |e|
+          e.messages.should == { :weight => "is less than 10kg" } }
+      end
+    end
+    
+    context "when height is less than one" do
+      let(:parameters) do
+        skiers_parameter[:height] = 0
+        skiers_parameter
+      end
+      it "raises SkiBinding::Error" do
+        expect { calculated_preped }.to raise_error(SkiBinding::Error) { |e|
+          e.messages.should == { :height => "must be bigger than 0" } }
+      end
+    end
+    
+    context "when sole_length is less than one" do
+      let(:parameters) do
+        skiers_parameter[:sole_length] = 0
+        skiers_parameter
+      end
+      it "raises SkiBinding::Error" do
+        expect { calculated_preped }.to raise_error(SkiBinding::Error) { |e|
+          e.messages.should == { :sole_length => "must be bigger than 0" } }
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ski_binding_calculator
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.8.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-18 00:00:00.000000000 Z
+date: 2014-02-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport