fix: repeat 0 causes infinite loop

Fixes #49

CHANGELOG.adoc

@@ -167,6 +167,7 @@ Fourth Ctrl-C sends SIGKILL to active jobs.
 * Fixed typo of `--verbose` option
 * Fixed missing `--version` option
 * Fixed coloring output when STDOUT is not a TTY
+* Fixed a job repeat value of `0` causes an infinite loop
 
 === Removed
 

lib/jobrnr/dsl/job_builder.rb

@@ -25,8 +25,8 @@ def execute(command = nil, &block)
       end
 
       def repeat(times)
-        if !times.is_a?(Integer) || times.negative?
-          raise Jobrnr::TypeError, "'repeat' expects a positive Integer " \
+        if !times.is_a?(Integer) || times <= 0
+          raise Jobrnr::TypeError, "'repeat' expects an Integer greater than zero " \
                                    "but was given value of '#{times}' of type " \
                                    "'#{times.class.name}' @ #{caller_source}"
         end

test/dsl_command_error_test.rb

@@ -104,7 +104,7 @@ def clear
           end
         end
         assert_equal(Jobrnr::Util.strip_heredoc(<<-EOF).strip, e.message)
-          'repeat' expects a positive Integer but was given value of '5' of type 'String' @ file:line
+          'repeat' expects an Integer greater than zero but was given value of '5' of type 'String' @ file:line
         EOF
       end
 
@@ -116,7 +116,19 @@ def clear
             end
           end
         end
-        assert_match(/'repeat' expects a positive Integer but was given value of '-1' of type '\w+' @ file:line$/,
+        assert_match(/'repeat' expects an Integer greater than zero but was given value of '-1' of type '\w+' @ file:line$/,
+                     e.message)
+      end
+
+      it "requires a non-zero Integer" do
+        e = assert_raises(Jobrnr::TypeError) do
+          @obj.job :id do
+            stub :caller_source, "file:line" do
+              repeat(0)
+            end
+          end
+        end
+        assert_match(/'repeat' expects an Integer greater than zero but was given value of '0' of type '\w+' @ file:line$/,
                      e.message)
       end
     end