Merge lp://staging/~gmb/launchpad/not-nullify-bwa.result-bug-546774 into lp://staging/launchpad/db-devel
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Graham Binns | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp://staging/~gmb/launchpad/not-nullify-bwa.result-bug-546774 | ||||
Merge into: | lp://staging/launchpad/db-devel | ||||
Diff against target: |
185 lines (+51/-14) 7 files modified
database/schema/patch-2207-47-0.sql (+6/-0) lib/lp/bugs/doc/bug-watch-activity.txt (+9/-5) lib/lp/bugs/interfaces/bugwatch.py (+14/-0) lib/lp/bugs/model/bugwatch.py (+6/-1) lib/lp/bugs/scripts/checkwatches/scheduler.py (+4/-4) lib/lp/bugs/stories/bugwatches/xx-bugwatch-errors.txt (+8/-2) lib/lp/bugs/tests/test_bugwatch.py (+4/-2) |
||||
To merge this branch: | bzr merge lp://staging/~gmb/launchpad/not-nullify-bwa.result-bug-546774 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Björn Tillenius (community) | db | Approve | |
Gavin Panella (community) | code | Approve | |
Stuart Bishop (community) | db | Approve | |
Review via email: mp+22934@code.staging.launchpad.net |
Commit message
Make BugWatchActivit
Description of the change
This branch adds a DB patch to make BugWatchActivit
updates all existing BugWatchActivity's with a NULL result to have a
result of BugWatchActivit
success status.
== database/
- This patch updates all existing BugWatchActivity records with a NULL
result to have a result value of 9, which corresponds with
BugWatchActi
- The patch also makes BugWatchActivit
== lib/lp/
- I've updated the tests to account for the fact that
BugWatchActi
== lib/lp/
- I've added a SYNC_SUCCESS status to BugWatchActivit
- I've added a list of BugWatchActivit
which we can use in queries and so on. We plan to expand this later.
== lib/lp/
- I've updated addActivity() so that it now sets the result of the new
BugWatchActivity entry to SYNC_SUCCESS if no other value is
specified.
== lib/lp/
- I've updated the scheduler query to query for activity without a
success status rather than just querying for result is not null.
== lib/lp/
- I've updated the test to ensure that the scheduler treats success
results correctly (i.e. by ignoring them when it calculates when next
to check the watch).
Looks good. Approved as patch-2207-47-0.sql