Patchwork D6225: rebase: demonstrate bug in dry-run mode which causes cycles to not be reported

login
register
mail settings
Submitter phabricator
Date April 12, 2019, 4:07 p.m.
Message ID <differential-rev-PHID-DREV-a4fxoyqbmapdus4k62yn-req@phab.mercurial-scm.org>
Download mbox | patch
Permalink /patch/39568/
State Superseded
Headers show

Comments

phabricator - April 12, 2019, 4:07 p.m.
durin42 created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  tests/test-rebase-dest.t

CHANGE DETAILS




To: durin42, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/tests/test-rebase-dest.t b/tests/test-rebase-dest.t
--- a/tests/test-rebase-dest.t
+++ b/tests/test-rebase-dest.t
@@ -206,6 +206,18 @@ 
   abort: source and destination form a cycle
   [255]
 
+BUG: cycles aren't flagged correctly when --dry-run is set:
+  $ rebasewithdag -s B -d 'SRC' --dry-run <<'EOS'
+  > C
+  > |
+  > B
+  > |
+  > Z
+  > EOS
+  abort: no rebase in progress
+  starting dry-run rebase; repository will not be changed
+  [255]
+
 Switch roots:
 
   $ rebasewithdag -s 'all() - roots(all())' -d 'roots(all()) - ::SRC' <<'EOS'