Merge lp:~charlesk/indicator-datetime/lp-1411171-strengthen-recurrence-rule-test-rtm-14.09 into lp:indicator-datetime/rtm-14.09

[Charles Kerr]

Fix bug that misidentified repeating alarms as one-time alarms, causing them to be disabled after triggering.

Fix bug that misidentified repeating alarms as one-time alarms, causing them to be disabled after triggering.

=== Change Description

Fix bug that misidentified repeating alarms as one-time alarms, causing them to be disabled after triggering.

trunk MP @ https://code.launchpad.net/~charlesk/indicator-datetime/lp-1411171-strengthen-recurrence-rule-test/+merge/246629

=== MP Checklist

> Are there any related MPs required for this MP to build/function as expected? Please list.

No prerequisites

> Is your branch in sync with latest trunk? (e.g. bzr pull lp:trunk -> no changes)

Yes, for latest rtm-14.09

> Did the code build without warnings?

Yes

> Did the tests run successfully?

Yes

> Did you perform an exploratory manual test run of your code change and any related functionality?

Yes

> If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?

N/A

> What device (or emulator) has your component test plan been executed successfully on?

Krillin 198

> What manual tests are relevant for this MP?

indicator-datetime/disable-one-time-alarms-after-notification

> Did you include a link to the MR Review Checklist Template to make your reviewer's life easier?

https://wiki.ubuntu.com/Process/Merges/Checklists/indicator-datetime

[dobey (dobey)]

review: Approve

=== modified file 'src/engine-eds.cpp'
--- src/engine-eds.cpp	2014-12-08 02:52:50 +0000
+++ src/engine-eds.cpp	2015-01-15 19:48:41 +0000
@@ -534,8 +534,11 @@
         icalcomponent * icc = nullptr;
         if (e_cal_client_get_object_finish (E_CAL_CLIENT(client), result, &icc, nullptr))
         {
-            struct icaltimetype itt = icalcomponent_get_recurrenceid(icc);
-            if (icaltime_is_null_time(itt))
+            auto rrule_property = icalcomponent_get_first_property (icc, ICAL_RRULE_PROPERTY); // transfer none
+            auto rdate_property = icalcomponent_get_first_property (icc, ICAL_RDATE_PROPERTY); // transfer none
+            const bool is_nonrepeating = (rrule_property == nullptr) && (rdate_property == nullptr);
+
+            if (is_nonrepeating)
             {
                 g_debug("'%s' appears to be a one-time alarm... adding 'disabled' tag.",
                         icalcomponent_as_ical_string(icc));
 
=== modified file 'tests/manual'
--- tests/manual	2014-12-08 02:52:50 +0000
+++ tests/manual	2015-01-15 19:48:41 +0000
@@ -56,6 +56,12 @@
 		<dd>Confirm that the alarm notification is shown</dd>
 		<dd>Confirm that the alarm's sound is played while the alarm notification is present</dd>
 		<dd>Confirm that the one-time alarm is disabled after the notification is shown. NOTE: due to a refresh bug in clock-app you may need to refresh its alarms page (by swapping back to the main page, then the alarm page again, this is tracked in #1362341) in order to see the alarm change from enabled to disabled.</dd>
+	<dt>Create and save an upcoming repeating alarm in ubuntu-clock-app</dt>
+		<dd>Confirm that the alarm icon appears next to the current time in unity's indicator display</dd>
+	<dt>Wait until the alarm time is reached</dt>
+		<dd>Confirm that the alarm notification is shown</dd>
+		<dd>Confirm that the alarm's sound is played while the alarm notification is present</dd>
+		<dd>Confirm that the repeating alarm is not disabled after the notification is shown. NOTE: due to a refresh bug in clock-app you may need to refresh its alarms page (by swapping back to the main page, then the alarm page again, this is tracked in #1362341) in order to see the alarm change from enabled to disabled.</dd>
 </dl>
 
 Test-case indicator-datetime/calendar-event-notification