D5961: copies: add test that makes both the merging csets dirty and run w/o error

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Mon Feb 25 12:27:07 EST 2019


martinvonz added inline comments.

INLINE COMMENTS

> test-copytrace-heuristics.t:724-728
> +  $ cat >> $HGRCPATH << EOF
> +  > [experimental]
> +  > evolution.createmarkers=True
> +  > evolution.allowunstable=True
> +  > EOF

Since this does not seem to set `experimental.copytrace = heuristics` (right?), this is probably not the right file for the test case. I recently added `test-copies.t`, which may be a better place for this. That file doesn't yet have any tests for `mergecopies()` (only for `pathcopies()`), but I was probably going to add that later anyway. Depending on how many tests I would add, I might have put all `mergecopies()` tests in a separate file, but since there isn't much at this point (only these few that you're adding), it probably makes sense for them to live in `test-copies.t`.

REPOSITORY
  rHG Mercurial

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

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


More information about the Mercurial-devel mailing list