Merge pull request #23 from schwern/feat/raise_no_by

feat: Raise gracefully if #by is not set.

Author: schwern

lib/time_range/time.rb

@@ -15,6 +15,8 @@ def initialize(time, by = nil)
     end
 
     def succ
+      raise ArgumentError, 'by() not set for this TimeRange' if !@by || @by.empty?
+
       next_time = Advancer.advance_time(@time, @by)
       return self.class.new(next_time, @by)
     end

spec/time_range_spec.rb

@@ -92,6 +92,13 @@
   end
 
   shared_examples 'it has a begin' do
+    it 'raises gracefully if #by is not called' do
+      range = described_class.new(first, last)
+      expect {
+        range.first(1)
+      }.to raise_error ArgumentError, /by\(\) not set for this TimeRange/
+    end
+
     describe '#begin' do
       it 'has a begin' do
         expect(range.begin).to eq first