errors: raise StateError when push fails because it creates new heads

I decided to raise `StateError` here because the local and remote
repos are in an incompatible state. I think remote errors (exit code
100) should be when something goes wrong on the remote and there's
nothing the user can do.

Differential Revision: https://phab.mercurial-scm.org/D9391

mercurial/discovery.py

@@ -362,7 +362,7 b' def _nowarnheads(pushop):'
 def checkheads(pushop):
     """Check that a push won't add any outgoing head
 
-    raise Abort error and display ui message as needed.
+    raise StateError error and display ui message as needed.
     """
 
     repo = pushop.repo.unfiltered()
@@ -409,7 +409,7 b' def checkheads(pushop):'
         else:
             errmsg = _(b"push creates new remote branches: %s") % branchnames
         hint = _(b"use 'hg push --new-branch' to create new remote branches")
-        raise error.Abort(errmsg, hint=hint)
+        raise error.StateError(errmsg, hint=hint)
 
     # 2. Find heads that we need not warn about
     nowarnheads = _nowarnheads(pushop)
@@ -505,7 +505,7 b' def checkheads(pushop):'
             for h in dhs:
                 repo.ui.note(b" %s\n" % short(h))
     if errormsg:
-        raise error.Abort(errormsg, hint=hint)
+        raise error.StateError(errormsg, hint=hint)
 
 
 def _postprocessobsolete(pushop, futurecommon, candidate_newhs):

tests/test-bookmarks-pushpull.t

@@ -742,7 +742,7 b' diverging a remote bookmark fails'
   searching for changes
   abort: push creates new remote head c922c0139ca0 with bookmark 'Y'
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ hg -R ../a book
      @                         1:0d2164f0ce0d
    * X                         1:0d2164f0ce0d
@@ -759,7 +759,7 b' Unrelated marker does not alter the deci'
   searching for changes
   abort: push creates new remote head c922c0139ca0 with bookmark 'Y'
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ hg -R ../a book
      @                         1:0d2164f0ce0d
    * X                         1:0d2164f0ce0d
@@ -1055,7 +1055,7 b' pushing an existing but divergent bookma'
   remote has heads on branch 'default' that are not known locally: a2a606d9ff1b
   abort: push creates new remote head 54694f811df9 with bookmark 'X'
   (pull and merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ cd ../addmarks
 
 Check summary output for incoming/outgoing bookmarks

tests/test-exchange-obsmarkers-case-A3.t

@@ -201,7 +201,7 b' check it complains about multiple heads'
   searching for changes
   abort: push creates new remote head e5ea8f9c7314
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ cd ..
 
 test obsmarkers exchange.

tests/test-obsolete-checkheads.t

@@ -97,7 +97,7 b' Abort: old will still be an head because'
   searching for changes
   abort: push creates new remote head 71e3228bffe1
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
 old head is now public (public remote version)
 ==============================================
@@ -157,7 +157,7 b' Push should abort on new head'
   searching for changes
   abort: push creates new remote head d7d41ccbd4de
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
 
 
@@ -269,7 +269,7 b' We do not have enought data to take the '
   remote has heads on branch 'default' that are not known locally: c70b08862e08
   abort: push creates new remote head 71e3228bffe1
   (pull and merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
 Pulling the missing data makes it work
 
@@ -319,4 +319,4 b' setup'
   searching for changes
   abort: push creates new remote head 350a93b716be
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]

tests/test-phases-exchange.t

@@ -1017,7 +1017,7 b' appear on the remote side.'
   searching for changes
   abort: push creates new remote head 435b5d83910c
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ hg push -fr 435b5d83910c ../mu # because the push will create new visible head
   pushing to ../mu
   searching for changes

tests/test-push-checkheads-multibranches-E3.t

@@ -89,6 +89,6 b' Actual testing'
   searching for changes
   abort: push creates new remote head dc44c53142f0
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..

tests/test-push-checkheads-partial-C1.t

@@ -80,6 +80,6 b' Actual testing'
   searching for changes
   abort: push creates new remote head 25c56d33e4c4
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..

tests/test-push-checkheads-partial-C2.t

@@ -81,6 +81,6 b' Actual testing'
   searching for changes
   abort: push creates new remote head f6082bc4ffef
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..

tests/test-push-checkheads-partial-C3.t

@@ -80,6 +80,6 b' Actual testing'
   searching for changes
   abort: push creates new remote head 0f88766e02d6
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..

tests/test-push-checkheads-partial-C4.t

@@ -81,6 +81,6 b' Actual testing'
   searching for changes
   abort: push creates new remote head 0f88766e02d6
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..

tests/test-push-checkheads-unpushed-D1.t

@@ -73,7 +73,7 b' Actual testing'
   searching for changes
   abort: push creates new remote head 74ff5441d343
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..
 

tests/test-push-checkheads-unpushed-D2.t

@@ -94,6 +94,6 b' Actual testing'
   searching for changes
   abort: push creates new remote head 0f88766e02d6
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..

tests/test-push-checkheads-unpushed-D3.t

@@ -93,13 +93,13 b' Actual testing'
   searching for changes
   abort: push creates new remote head f6082bc4ffef
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ hg push --rev 'desc(B1)'
   pushing to $TESTTMP/D3/server
   searching for changes
   abort: push creates new remote head 25c56d33e4c4
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
 Extra testing
 -------------
@@ -111,6 +111,6 b' In this case, even a bare push is creati'
   searching for changes
   abort: push creates new remote head 25c56d33e4c4
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..

tests/test-push-checkheads-unpushed-D4.t

@@ -111,7 +111,7 b' Actual testing (new branch only)'
   searching for changes
   abort: push creates new remote head f6082bc4ffef
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
 Actual testing (existing branch only)
 ------------------------------------

tests/test-push-checkheads-unpushed-D5.t

@@ -100,7 +100,7 b' Actual testing'
   searching for changes
   abort: push creates new remote head 25c56d33e4c4
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ hg push --rev 'desc(A1)'
   pushing to $TESTTMP/D5/server
   searching for changes

tests/test-push-checkheads-unpushed-D6.t

@@ -81,6 +81,6 b' Actual testing'
   searching for changes
   abort: push creates new remote head 0f88766e02d6
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ../..

tests/test-push-warn.t

@@ -33,7 +33,7 b' Specifying a revset that evaluates to nu'
   remote has heads on branch 'default' that are not known locally: 1c9246a22a0a
   abort: push creates new remote head 1e108cc5548c
   (pull and merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ hg push --debug ../a
   pushing to ../a
@@ -51,7 +51,7 b' Specifying a revset that evaluates to nu'
    1e108cc5548c
   abort: push creates new remote head 1e108cc5548c
   (pull and merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ hg pull ../a
   pulling from ../a
@@ -68,7 +68,7 b' Specifying a revset that evaluates to nu'
   searching for changes
   abort: push creates new remote head 1e108cc5548c
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ hg merge
   1 files updated, 0 files merged, 0 files removed, 0 files unresolved
@@ -121,7 +121,7 b' Specifying a revset that evaluates to nu'
   searching for changes
   abort: push creates new remote head 6346d66eb9f5
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ hg push -r 2 ../c
   pushing to ../c
@@ -134,7 +134,7 b' Specifying a revset that evaluates to nu'
   searching for changes
   abort: push creates new remote head a5dda829a167
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ hg push -v -r 3 -r 4 ../c
   pushing to ../c
@@ -144,7 +144,7 b' Specifying a revset that evaluates to nu'
    ee8fbc7a0295
   abort: push creates new remote head a5dda829a167
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ hg push -v -f -r 3 -r 4 ../c
   pushing to ../c
@@ -234,14 +234,14 b' Push on existing branch and new branch:'
   searching for changes
   abort: push creates new remote branches: c
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
   $ hg push -r 4 -r 5 ../f
   pushing to ../f
   searching for changes
   abort: push creates new remote branches: c
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
 
 Multiple new branches:
@@ -255,14 +255,14 b' Multiple new branches:'
   searching for changes
   abort: push creates new remote branches: c, d
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
   $ hg push -r 4 -r 6 ../f
   pushing to ../f
   searching for changes
   abort: push creates new remote branches: c, d
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
   $ cd ../g
 
@@ -278,7 +278,7 b' Fail on multiple head push:'
   searching for changes
   abort: push creates new remote head 0b715ef6ff8f on branch 'a'
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
 Push replacement head on existing branches:
 
@@ -355,7 +355,7 b' Failed push of new named branch:'
   searching for changes
   abort: push creates new remote branches: e
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
 
 Using --new-branch to push new named branch:
@@ -383,7 +383,7 b' Pushing multi headed new branch:'
   searching for changes
   abort: push creates new branch 'f' with multiple heads
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ hg push --branch f --new-branch --force ../f
   pushing to ../f
   searching for changes
@@ -427,7 +427,7 b" multiple new heads but also doesn't repo"
   remote has heads on branch 'default' that are not known locally: 534543e22c29 764f8ec07b96 afe7cc7679f5 ce4212fc8847
   abort: push creates new remote head 97bd0c84d346
   (pull and merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ hg -R h up -q 0; echo x > h/b; hg -R h ci -qAmx
   $ hg -R i push h
   pushing to h
@@ -435,7 +435,7 b" multiple new heads but also doesn't repo"
   remote has heads on branch 'default' that are not known locally: 18ddb72c4590 534543e22c29 764f8ec07b96 afe7cc7679f5 and 1 others
   abort: push creates new remote head 97bd0c84d346
   (pull and merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
   $ hg -R i push h -v
   pushing to h
   searching for changes
@@ -444,7 +444,7 b" multiple new heads but also doesn't repo"
    97bd0c84d346
   abort: push creates new remote head 97bd0c84d346
   (pull and merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
 
 Check prepush logic with merged branches:
@@ -479,7 +479,7 b' Check prepush logic with merged branches'
   searching for changes
   abort: push creates new remote branches: b
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
 
 Prepush -r should not allow you to sneak in new heads:
@@ -515,7 +515,7 b' Prepush -r should not allow you to sneak'
   searching for changes
   abort: push creates new remote head 451211cc22b0 on branch 'a'
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ..
 
@@ -765,14 +765,14 b' outgoing:'
   searching for changes
   abort: push creates new remote head 7d0f4fb6cf04 on branch 'A'
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ hg push inner -r4 -r5
   pushing to inner
   searching for changes
   abort: push creates new remote head 7d0f4fb6cf04 on branch 'A'
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ hg in inner
   comparing with inner
@@ -788,7 +788,7 b' Test fail hook'
   running fail-push hook
   abort: push creates new remote head 7d0f4fb6cf04 on branch 'A'
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 
   $ cd ..
 
@@ -817,7 +817,7 b' When there is a single closed branch'
   searching for changes
   abort: push creates new remote branches: foo (1 closed)
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
 When there is more than one closed branches
   $ hg -q branch bar
@@ -830,7 +830,7 b' When there is more than one closed branc'
   searching for changes
   abort: push creates new remote branches: bar, foo (2 closed)
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
 When there are more than one new branches and not all are closed
   $ hg -q branch bar1
@@ -842,6 +842,6 b' When there are more than one new branche'
   searching for changes
   abort: push creates new remote branches: bar, bar1, foo (2 closed)
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
 
   $ cd ..

tests/test-push.t

@@ -387,7 +387,7 b' Test `commands.push.require-revs`'
   searching for changes
   abort: push creates new remote head [0-9a-f]+ (re)
   (merge or see 'hg help push' for details about pushing new heads)
-  [255]
+  [20]
 (demonstrate that even though we don't have anything to exchange, we're still
 showing the error)
   $ hg push

tests/test-treediscovery-legacy.t

@@ -287,7 +287,7 b' Local is superset:'
   searching for changes
   abort: push creates new remote branches: both, name1
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
   $ hg push $remote --new-branch
   pushing to http://localhost:$HGPORT/
   searching for changes

tests/test-treediscovery.t

@@ -271,7 +271,7 b' Local is superset:'
   searching for changes
   abort: push creates new remote branches: both, name1
   (use 'hg push --new-branch' to create new remote branches)
-  [255]
+  [20]
   $ hg push $remote --new-branch
   pushing to http://localhost:$HGPORT/
   searching for changes