diff options
author | Charles Kerr <charles.kerr@canonical.com> | 2014-06-11 12:36:41 +0000 |
---|---|---|
committer | CI bot <ps-jenkins@lists.canonical.com> | 2014-06-11 12:36:41 +0000 |
commit | 9eee6f013103e856f66f16e17e156b1f6103695a (patch) | |
tree | 3678963f3d3520e8703732d7d0bb8489f23b7f49 /MERGE-REVIEW | |
parent | 5e6083e752f7dd50571f87e5910dca6302ce8113 (diff) | |
parent | db2898b2da7231490fe77ebcef0fb373ce1f2776 (diff) | |
download | ayatana-indicator-datetime-9eee6f013103e856f66f16e17e156b1f6103695a.tar.gz ayatana-indicator-datetime-9eee6f013103e856f66f16e17e156b1f6103695a.tar.bz2 ayatana-indicator-datetime-9eee6f013103e856f66f16e17e156b1f6103695a.zip |
Prefer to use ubuntu-platform-hardware-api for wakeups when possible s.t. user-defined alarms/appointments can wake up the phone from sleep to give a snap decision. Fixes: 1299916
Diffstat (limited to 'MERGE-REVIEW')
-rw-r--r-- | MERGE-REVIEW | 35 |
1 files changed, 30 insertions, 5 deletions
diff --git a/MERGE-REVIEW b/MERGE-REVIEW index 5e40f45..1a5815c 100644 --- a/MERGE-REVIEW +++ b/MERGE-REVIEW @@ -6,14 +6,39 @@ and reviewers should ensure that they've done for a merge into the project. * Ensure the project compiles and the test suite executes without error * Ensure that non-obvious code has comments explaining it - * If the change works on specific profiles, please include those in the merge description. + * If the change affects specific features, please reference the appropriate + tags in the merge description so reviewers can test appropriately: + [phone profile], [desktop profile], [alarms] == Reviewer Responsibilities == * Did the Jenkins build compile? Pass? Run unit tests successfully? * Are there appropriate tests to cover any new functionality? - * If the description says this effects the phone profile: - * Run tests indicator-datetime/unity8* - * If the description says this effects the desktop profile: - * Run tests indicator-datetime/unity7* + * Do the tag-specific tests pass? + +== Phone Profile Tests == + + * Run tests indicator-datetime/unity8* + +== Desktop Profile Tests == + + * Run tests indicator-datetime/unity7* + +== Alarm Tests == + + * Hardware wakeups for new alarms: + 1. Create and save an upcoming alarm in ubuntu-clock-app. + 2. Unplug the phone from any USB connection and put it to sleep. + 3. Confirm that the alarm sounds on time even if the phone is asleep. + (Note: if in doubt about sleep you can see in the syslog whether the + device actually suspended or whether the suspend was aborted) + + * Hardware wakeups for edited alarms: + 1. Edit an alarm that's already passed. (see previous test) + 2. Unplug the phone from any USB connection and put it to sleep. + 3. Confirm that the alarm sounds on time even if the phone is asleep. + (Note: if in doubt about sleep you can see in the syslog whether the + device actually suspended or whether the suspend was aborted.) + + |