fixed parsing of ical with multiple recurrence rules

Author: dimerman

lib/ice_cube/parsers/ical_parser.rb

@@ -16,7 +16,8 @@ def self.schedule_from_ical(ical_string, options = {})
         when 'DURATION'
           data[:duration] # FIXME
         when 'RRULE'
-          data[:rrules] = [rule_from_ical(value)]
+          data[:rrules] ||= []
+          data[:rrules] += [rule_from_ical(value)]
         end
       end
       Schedule.from_hash data

spec/examples/from_ical_spec.rb

@@ -102,7 +102,7 @@ module IceCube
   RRULE:FREQ=WEEKLY;BYDAY=TH;UNTIL=20130531T100000Z
   ICAL
 
-    ical_string_woth_multiple_exdates = <<-ICAL.gsub(/^\s*/, '')
+    ical_string_with_multiple_exdates = <<-ICAL.gsub(/^\s*/, '')
   DTSTART;TZID=America/Denver:20130731T143000
   DTEND;TZID=America/Denver:20130731T153000
   RRULE:FREQ=WEEKLY;UNTIL=20140730T203000Z;BYDAY=MO,WE,FR
@@ -111,6 +111,11 @@ module IceCube
   EXDATE;TZID=America/Denver:20130807T143000
   ICAL
 
+    ical_string_with_multiple_rules = <<-ICAL.gsub(/^\s*/, '' )
+  DTSTART;TZID=CDT:20151005T195541
+  RRULE:FREQ=WEEKLY;BYDAY=MO,TU
+  RRULE:FREQ=WEEKLY;INTERVAL=2;WKST=SU;BYDAY=FR
+    ICAL
 
     def sorted_ical(ical)
       ical.split(/\n/).sort.map { |field|
@@ -359,10 +364,17 @@ def sorted_ical(ical)
       end
 
       it 'handles multiple EXDATE lines' do
-        schedule = IceCube::Schedule.from_ical ical_string_woth_multiple_exdates
+        schedule = IceCube::Schedule.from_ical ical_string_with_multiple_exdates
         schedule.exception_times.count.should == 3
       end
     end
+
+    describe 'multiple rules' do
+      it 'handles multiple recurrence rules' do
+        schedule = IceCube::Schedule.from_ical ical_string_with_multiple_rules
+        schedule.recurrence_rules.count.should == 2
+      end
+    end
   end
 
 end