This is an archive of the discontinued Mercurial Phabricator instance.

Differential D8257

copies-tests: remove the final summary
ClosedPublic

Authored by marmoute on Mar 6 2020, 6:51 PM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHG7f5735639360: copies-tests: remove the final summary
Summary

The more we add tests, the less useful it is. So after discussion with Martin,
we decided to drop it.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

marmoute created this revision.Mar 6 2020, 6:51 PM

series update after Martin feedback

marmoute added a commit: rHG7f5735639360: copies-tests: remove the final summary.Mar 6 2020, 7:30 PM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHG7f5735639360: copies-tests: remove the final summary (authored by marmoute). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/test-copies-chain-merge.t (46 lines)

Diff 20590

tests/test-copies-chain-merge.t

| |
o 0 i-0 initial commit: a b] o 0 i-0 initial commit: a b]
Note: Note:
| In this case, one of the merge wrongly record a merge while there is none. | In this case, one of the merge wrongly record a merge while there is none.
| This lead to bad copy tracing information to be dug up. | This lead to bad copy tracing information to be dug up.
final summary
$ hg update null --quiet
$ hg log -G
o 20 mEAm-0 simple merge - the other way]
|\
+---o 19 mAEm-0 simple merge - one way]
| |/
| o 18 e-2 g -move-> f]
| |
| o 17 e-1 b -move-> g]
| |
| | o 16 mDBm-0 simple merge - the other way]
| | |\
| | +---o 15 mBDm-0 simple merge - one way]
| | | |/
| | | o 14 d-2 re-add d]
| | | |
| +---o 13 d-1 delete d]
| | |
| | | o 12 mCBm-1 re-add d]
| | | |
| | | o 11 mCBm-0 simple merge - the other way]
| | |/|
| | | | o 10 mBCm-1 re-add d]
| | | | |
| | +---o 9 mBCm-0 simple merge - one way]
| | | |/
| +---o 8 c-1 delete d]
| | |
+-----o 7 mABm-0 simple merge - the other way]
| | |/
+-----o 6 mBAm-0 simple merge - one way]
| | |/
| | o 5 b-1: b update]
| |/
o | 4 a-2: e -move-> f]
| |
o | 3 a-1: d -move-> e]
|/
o 2 i-2: c -move-> d]
|
o 1 i-1: a -move-> c]
|
o 0 i-0 initial commit: a b]
Check results Check results
============= =============
merging with unrelated change does not interfer wit the renames merging with unrelated change does not interfer wit the renames
--------------------------------------------------------------- ---------------------------------------------------------------
- rename on one side - rename on one side