Mercurial > public > mercurial-scm > hg-stable
diff tests/test-lfs-test-server.t @ 37244:67db84842356
lfs: improve the client message when the server signals an object error
Two things here. First, the previous message included a snippet of JSON, which
tends to be long (and in the case of lfs-test-server, has no error message).
Instead, give a concise message where possible, and leave the JSON to a debug
output. Second, the server can signal issues other than a missing individual
file. This change shows a corrupt file, but I'm debating letting the corrupt
file get downloaded, because 1) the error code doesn't really fit, and 2) having
it locally makes forensics easier. Maybe need a config knob for that.
author | Matt Harbison <matt_harbison@yahoo.com> |
---|---|
date | Sun, 25 Feb 2018 23:44:02 -0500 |
parents | f51c2780db3a |
children | 435481393198 |
line wrap: on
line diff
--- a/tests/test-lfs-test-server.t Sat Mar 31 15:54:26 2018 -0400 +++ b/tests/test-lfs-test-server.t Sun Feb 25 23:44:02 2018 -0500 @@ -449,7 +449,7 @@ Content-Type: text/plain; charset=utf-8 (git-server !) Date: $HTTP_DATE$ (git-server !) abort: corrupt remote lfs object: d11e1a642b60813aee592094109b406089b8dff4cb157157f753418ec7857998 (git-server !) - abort: LFS server error. Remote object for "c" not found: *! (glob) (hg-server !) + abort: LFS server error for "c": Validation error! (hg-server !) [255] The corrupted blob is not added to the usercache or local store @@ -807,7 +807,7 @@ ] "transfer": "basic" (hg-server !) } - abort: LFS server error. Remote object for "b" not found:(.*)! (re) + abort: LFS server error for "b": The object does not exist! [255] Check error message when object does not exist: @@ -918,7 +918,7 @@ ] "transfer": "basic" (hg-server !) } - abort: LFS server error. Remote object for "a" not found:(.*)! (re) + abort: LFS server error for "a": The object does not exist! [255] $ $PYTHON $RUNTESTDIR/killdaemons.py $DAEMON_PIDS