reviewboard: add test for pushing to submitted review request; r=smacleod draft
authorGregory Szorc <gps@mozilla.com>
Tue, 12 Jan 2016 14:01:20 -0800
changeset 6708 bac671496fbf8a1857b6b2a3ad28a5a6d89bc9a3
parent 6707 6c164172d1ecdf2c8b0acc20eda349716f78be5e
child 6709 9324845f90fa68c7322123f2be9f80f4ff3884f0
push id513
push usergszorc@mozilla.com
push dateWed, 13 Jan 2016 02:13:15 +0000
reviewerssmacleod
reviewboard: add test for pushing to submitted review request; r=smacleod We swallow output because it currently spews an exception stack, which is a bug already on file.
hgext/reviewboard/tests/test-review-request-closed-submitted.t
--- a/hgext/reviewboard/tests/test-review-request-closed-submitted.t
+++ b/hgext/reviewboard/tests/test-review-request-closed-submitted.t
@@ -162,16 +162,25 @@ Child review request with ID 2 should be
     - +++ b/foo
     - '@@ -1,1 +1,1 @@'
     - -foo1
     - +foo2
     - ''
   approved: false
   approval_failure: A suitable reviewer has not given a "Ship It!"
 
+Submitting against a published review request results in error
+TODO Fix the error output (bug 1169664)
+
+  $ echo foo3 > foo
+  $ hg commit -m 'Bug 1 - Foo 3'
+  $ hg push > /dev/null
+  abort: unexpected response: empty string
+  [255]
+
 Re-opening the parent review request should re-open all of the children.
 
   $ rbmanage reopen 1
 
 Squashed review request with ID 1 should be re-opened...
 
   $ rbmanage dumpreview 1
   id: 1