Merge lp://qastaging/~michael.nelson/rnr-server/709172-reviews-per-distroseries into lp://qastaging/rnr-server
Proposed by
Michael Nelson
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Michael Nelson | ||||
| Approved revision: | 154 | ||||
| Merged at revision: | 153 | ||||
| Proposed branch: | lp://qastaging/~michael.nelson/rnr-server/709172-reviews-per-distroseries | ||||
| Merge into: | lp://qastaging/rnr-server | ||||
| Diff against target: |
561 lines (+357/-41) 7 files modified
src/reviewsapp/management/commands/populate.py (+4/-4) src/reviewsapp/migrations/0004_add_per_repo_and_origin_stats.py (+194/-0) src/reviewsapp/models/__init__.py (+4/-0) src/reviewsapp/models/reviews.py (+65/-30) src/reviewsapp/tests/factory.py (+22/-0) src/reviewsapp/tests/test_handlers.py (+7/-7) src/reviewsapp/tests/test_models.py (+61/-0) |
||||
| To merge this branch: | bzr merge lp://qastaging/~michael.nelson/rnr-server/709172-reviews-per-distroseries | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Danny Tamez (community) | Approve | ||
|
Review via email:
|
|||
Commit message
[r=zematynnad][bug=709172] Add aggregate stats per repository and origin.
Description of the change
Overview
========
Part 1 of the fix for bug 709172, this branch adds the new models for aggregating stats for a software item per repository, and per origin.
Details
=======
I pulled out the stats field from SoftwareItem into a base-class that could be re-used for the new models also.
Also updated a few tests that were assuming database ids (which is normally fine, until a different test fails and leaves data in the database which screws things up).
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Very nice. Ship it!