Fixes to the mixin tests from the review

Signed-off-by: Tim Smith <tsmith@chef.io>

1 files changed, 6 insertions, 2 deletions

diff --git a/spec/unit/mixin/cron_validations_spec.rb b/spec/unit/mixin/cron_validations_spec.rb
index 409df2da1c..b37a523237 100644
--- a/spec/unit/mixin/cron_validations_spec.rb
+++ b/spec/unit/mixin/cron_validations_spec.rb
@@ -56,7 +56,7 @@ describe Chef::Mixin::CronValidations do
       expect(Chef::Mixin::CronValidations.validate_month(13)).to be false
     end
 
-    it "returns false for an invalid string" do
+    it "returns false for an invalid string (typo)" do
       expect(Chef::Mixin::CronValidations.validate_month("janurary")).to be false
     end
   end
@@ -66,8 +66,12 @@ describe Chef::Mixin::CronValidations do
       expect(Chef::Mixin::CronValidations.validate_numeric(5, 1, 100)).to be true
     end
 
-    it "returns false if the value is out of the allowed range" do
+    it "returns false if the value less than the allowed range" do
       expect(Chef::Mixin::CronValidations.validate_numeric(-1, 1, 100)).to be false
     end
+
+    it "returns false if the value more than the allowed range" do
+      expect(Chef::Mixin::CronValidations.validate_numeric(101, 1, 100)).to be false
+    end
   end
 end