1287500 - Mozreview and bugzilla can get out of sync as to who is supposed to review a patch when changing reviewers

Status: RESOLVED INVALID

(MozReview Graveyard :: Integration: Bugzilla, defect)

Description

[Boris Zbarsky [:bzbarsky]]

STEPS TO REPRODUCE:

1)  Person A pushes a changeset to mozreview, requests review from person B.
2)  Person B starts doing the review, writes some comments.
3)  Person B realizes that a different reviewer should really handle this,
    edits the reviewer via the mozreview UI to point to person C, publishes
    that change.
4)  Person B goes to Finish Review on the already-written comments.  In the
    Finish Review screen, he leaves the "r+/r-/r?" dropdown in its default
    state, which is "r?" (figuring that this now refers to person C's review).

EXPECTED RESULTS: Comments are published, patch now needs to be reviewed by person C.

ACTUAL RESULTS: In Mozreview, as expected results.  But in Bugzilla, step 4 shows up as person B requesting review from person B on the patch.  So now in Mozreview the patch needs review from C, but in Bugzilla it needs review from both B and C.

Comment 1

[BMO Automation]

MozReview is now obsolete. Please use Phabricator instead. Closing this bug.