nicer error messages

nickelser · May 7, 2015 · 9c90ebc · 9c90ebc
1 parent a13f086
commit 9c90ebc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/zhong/job.rb b/lib/zhong/job.rb
@@ -13,7 +13,7 @@ def initialize(name, config = {}, &block)
         @logger.error "warning: #{self} has `at` but no `every`; could run far more often than expected!"
       end
 
-      fail "must specific either `at` or `every` for a job" unless @at || @every
+      fail "must specific either `at` or `every` for job: #{self}" unless @at || @every
 
       @block = block
 

diff --git a/lib/zhong/scheduler.rb b/lib/zhong/scheduler.rb
@@ -19,7 +19,7 @@ def initialize(config = {})
     end
 
     def category(name)
-      fail "cannot nest categories: #{name} would be nested in #{@category}" if @category
+      fail "cannot nest categories: #{name} would be nested in #{@category} (#{caller.first})" if @category
 
       @category = name.to_s
 
@@ -29,6 +29,7 @@ def category(name)
     end
 
     def every(period, name, opts = {}, &block)
+      fail "must specify a period for #{name} (#{caller.first})" unless period
       job = Job.new(name, opts.merge(@config).merge(every: period, category: @category), &block)
       add(job)
     end