Skip to content

Commit 5791e45

Browse files
committed
fixed bug that caused merge not to apply uncommitted saved changes
1 parent 360bed9 commit 5791e45

File tree

3 files changed

+23
-3
lines changed

3 files changed

+23
-3
lines changed

gitless/cli/gl_commit.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ def _op_continue(op, fn):
6868
pprint.blank()
6969
try:
7070
op(op_cb=pprint.OP_CB)
71-
pprint.ok('{0} succeeded'.format(op))
71+
pprint.ok('{0} succeeded'.format(fn))
7272
except core.ApplyFailedError as e:
73-
pprint.ok('{0} succeeded'.format(op))
73+
pprint.ok('{0} succeeded'.format(fn))
7474
raise e

gitless/core.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -792,7 +792,7 @@ def merge(self, src, op_cb=None):
792792
raise GlError(stdout(e) + err)
793793
if op_cb and op_cb.save:
794794
op_cb.save()
795-
git.stash.save('--', _stash_msg_merge)
795+
git.stash.save('--', _stash_msg_merge(self))
796796
try:
797797
git.merge(src, '--no-ff')
798798
except ErrorReturnCode as e:

gitless/tests/test_e2e.py

+20
Original file line numberDiff line numberDiff line change
@@ -604,6 +604,26 @@ def test_uncommitted_changes(self):
604604
self.assertEqual('uncommitted', utils.read_file(self.MASTER_FILE))
605605
self.assertEqual('uncommitted', utils.read_file('master_untracked'))
606606

607+
def test_uncommitted_tracked_changes_that_conflict(self):
608+
gl.branch(c='tmp', divergent_point='HEAD~1')
609+
gl.switch('tmp')
610+
utils.write_file(self.MASTER_FILE, contents='uncommitted')
611+
self.assertRaisesRegexp(
612+
ErrorReturnCode, 'failed to apply', gl.merge, 'master')
613+
contents = utils.read_file(self.MASTER_FILE)
614+
self.assertTrue('uncommitted' in contents)
615+
self.assertTrue('contents 2' in contents)
616+
617+
def test_uncommitted_tracked_changes_that_conflict_append(self):
618+
gl.branch(c='tmp', divergent_point='HEAD~1')
619+
gl.switch('tmp')
620+
utils.append_to_file(self.MASTER_FILE, contents='uncommitted')
621+
self.assertRaisesRegexp(
622+
ErrorReturnCode, 'failed to apply', gl.merge, 'master')
623+
contents = utils.read_file(self.MASTER_FILE)
624+
self.assertTrue('uncommitted' in contents)
625+
self.assertTrue('contents 2' in contents)
626+
607627

608628
class TestPerformance(TestEndToEnd):
609629

0 commit comments

Comments
 (0)