Merge pull request #255 from code-mancers/fix-issue-with-passing-time-format-to-active-support

Allow -1 arity in Occurrence#to_s to comply with ActiveSupport::TimeWithZone

Author: avit

lib/ice_cube/occurrence.rb

@@ -84,7 +84,7 @@ def to_time
     # time formats and is only used when ActiveSupport is available.
     #
     def to_s(format=nil)
-      if format && to_time.public_method(:to_s).arity > 0
+      if format && to_time.public_method(:to_s).arity != 0
         t0, t1 = start_time.to_s(format), end_time.to_s(format)
       else
         t0, t1 = start_time.to_s, end_time.to_s

spec/examples/occurrence_spec.rb

@@ -28,9 +28,11 @@
     end
 
     it "accepts a format option to comply with ActiveSupport" do
-      occurrence = Occurrence.new(Time.now)
+      require 'active_support/core_ext/time'
+      time_now = Time.current
+      occurrence = Occurrence.new(time_now)
 
-      expect { occurrence.to_s(:short) }.not_to raise_error
+      expect(occurrence.to_s(:short)).to eq time_now.to_s(:short)
     end
   end