Patchwork D11854: tests: remove unnecessary explicit args from `failfilemerge.py`

login
register
mail settings
Submitter phabricator
Date Dec. 4, 2021, 7:06 a.m.
Message ID <differential-rev-PHID-DREV-4zdd5lh2leexhvcs75f5-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/50180/
State Superseded
Headers show

Comments

phabricator - Dec. 4, 2021, 7:06 a.m.
martinvonz created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  The `failfilemerge()` function always raises an exception without
  looking at its arguments, so let's use `*args` and `**kwargs`.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D11854

AFFECTED FILES
  tests/failfilemerge.py

CHANGE DETAILS




To: martinvonz, #hg-reviewers
Cc: mercurial-patches, mercurial-devel

Patch

diff --git a/tests/failfilemerge.py b/tests/failfilemerge.py
--- a/tests/failfilemerge.py
+++ b/tests/failfilemerge.py
@@ -9,11 +9,8 @@ 
 )
 
 
-def failfilemerge(
-    filemergefn, premerge, repo, wctx, mynode, orig, fcd, fco, fca, labels=None
-):
+def failfilemerge(*args, **kwargs):
     raise error.Abort(b"^C")
-    return filemergefn(premerge, repo, mynode, orig, fcd, fco, fca, labels)
 
 
 def extsetup(ui):