Page MenuHomePhabricator

test: adding test of issue4700

Authored by valentin.gatienbaron on Dec 18 2018, 8:58 AM.

Diff Detail

rHG Mercurial
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

yuja added a subscriber: yuja.Dec 19 2018, 7:09 AM

+ > def wrappedlookup(orig, repo, *args, kwargs):
+ > ret = orig(repo, *args,
+ > repo.hook('lookup')
+ > return ret
+ > for table in [wireprotov1server.commands, wireprotov2server.COMMANDS]:
+ > table['lookup'].func = functools.partial(wrappedlookup, table['lookup'].func)

Nit: has to be b'lookup' to support Python 3. Can be fixed in flight.

Ok. I added the b'', as this change created a small conflict later.

This revision was automatically updated to reflect the committed changes.
yuja added a comment.Dec 20 2018, 8:24 AM

Queued this, thanks.