190
146
# Now that we have a repository with shared files, make sure
191
147
# that things aren't copied out by a 'push'
192
self.run_bzr('push ../../push-b', working_dir='repo/b')
193
pushed_tree = workingtree.WorkingTree.open('push-b')
149
self.run_bzr('push', '../../push-b')
150
pushed_tree = WorkingTree.open('../../push-b')
194
151
pushed_repo = pushed_tree.branch.repository
195
152
self.assertFalse(pushed_repo.has_revision('a-1'))
196
153
self.assertFalse(pushed_repo.has_revision('a-2'))
197
154
self.assertTrue(pushed_repo.has_revision('b-1'))
199
def test_push_funky_id(self):
200
t = self.make_branch_and_tree('tree')
201
self.build_tree(['tree/filename'])
202
t.add('filename', 'funky-chars<>%&;"\'')
203
t.commit('commit filename')
204
self.run_bzr('push -d tree new-tree')
206
def test_push_dash_d(self):
207
t = self.make_branch_and_tree('from')
208
t.commit(allow_pointless=True,
209
message='first commit')
210
self.run_bzr('push -d from to-one')
211
self.failUnlessExists('to-one')
212
self.run_bzr('push -d %s %s'
213
% tuple(map(urlutils.local_path_to_url, ['from', 'to-two'])))
214
self.failUnlessExists('to-two')
216
def test_push_smart_non_stacked_streaming_acceptance(self):
217
self.setup_smart_server_with_call_log()
218
t = self.make_branch_and_tree('from')
219
t.commit(allow_pointless=True, message='first commit')
220
self.reset_smart_call_log()
221
self.run_bzr(['push', self.get_url('to-one')], working_dir='from')
222
# This figure represent the amount of work to perform this use case. It
223
# is entirely ok to reduce this number if a test fails due to rpc_count
224
# being too low. If rpc_count increases, more network roundtrips have
225
# become necessary for this use case. Please do not adjust this number
226
# upwards without agreement from bzr's network support maintainers.
227
self.assertLength(9, self.hpss_calls)
229
def test_push_smart_stacked_streaming_acceptance(self):
230
self.setup_smart_server_with_call_log()
231
parent = self.make_branch_and_tree('parent', format='1.9')
232
parent.commit(message='first commit')
233
local = parent.bzrdir.sprout('local').open_workingtree()
234
local.commit(message='local commit')
235
self.reset_smart_call_log()
236
self.run_bzr(['push', '--stacked', '--stacked-on', '../parent',
237
self.get_url('public')], working_dir='local')
238
# This figure represent the amount of work to perform this use case. It
239
# is entirely ok to reduce this number if a test fails due to rpc_count
240
# being too low. If rpc_count increases, more network roundtrips have
241
# become necessary for this use case. Please do not adjust this number
242
# upwards without agreement from bzr's network support maintainers.
243
self.assertLength(14, self.hpss_calls)
244
remote = branch.Branch.open('public')
245
self.assertEndsWith(remote.get_stacked_on_url(), '/parent')
247
def test_push_smart_tags_streaming_acceptance(self):
248
self.setup_smart_server_with_call_log()
249
t = self.make_branch_and_tree('from')
250
rev_id = t.commit(allow_pointless=True, message='first commit')
251
t.branch.tags.set_tag('new-tag', rev_id)
252
self.reset_smart_call_log()
253
self.run_bzr(['push', self.get_url('to-one')], working_dir='from')
254
# This figure represent the amount of work to perform this use case. It
255
# is entirely ok to reduce this number if a test fails due to rpc_count
256
# being too low. If rpc_count increases, more network roundtrips have
257
# become necessary for this use case. Please do not adjust this number
258
# upwards without agreement from bzr's network support maintainers.
259
self.assertLength(11, self.hpss_calls)
261
def test_push_smart_incremental_acceptance(self):
262
self.setup_smart_server_with_call_log()
263
t = self.make_branch_and_tree('from')
264
rev_id1 = t.commit(allow_pointless=True, message='first commit')
265
rev_id2 = t.commit(allow_pointless=True, message='second commit')
267
['push', self.get_url('to-one'), '-r1'], working_dir='from')
268
self.reset_smart_call_log()
269
self.run_bzr(['push', self.get_url('to-one')], working_dir='from')
270
# This figure represent the amount of work to perform this use case. It
271
# is entirely ok to reduce this number if a test fails due to rpc_count
272
# being too low. If rpc_count increases, more network roundtrips have
273
# become necessary for this use case. Please do not adjust this number
274
# upwards without agreement from bzr's network support maintainers.
275
self.assertLength(11, self.hpss_calls)
277
def test_push_smart_with_default_stacking_url_path_segment(self):
278
# If the default stacked-on location is a path element then branches
279
# we push there over the smart server are stacked and their
280
# stacked_on_url is that exact path segment. Added to nail bug 385132.
281
self.setup_smart_server_with_call_log()
282
self.make_branch('stack-on', format='1.9')
283
self.make_bzrdir('.').get_config().set_default_stack_on(
285
self.make_branch('from', format='1.9')
286
out, err = self.run_bzr(['push', '-d', 'from', self.get_url('to')])
287
b = branch.Branch.open(self.get_url('to'))
288
self.assertEqual('/extra/stack-on', b.get_stacked_on_url())
290
def test_push_smart_with_default_stacking_relative_path(self):
291
# If the default stacked-on location is a relative path then branches
292
# we push there over the smart server are stacked and their
293
# stacked_on_url is a relative path. Added to nail bug 385132.
294
self.setup_smart_server_with_call_log()
295
self.make_branch('stack-on', format='1.9')
296
self.make_bzrdir('.').get_config().set_default_stack_on('stack-on')
297
self.make_branch('from', format='1.9')
298
out, err = self.run_bzr(['push', '-d', 'from', self.get_url('to')])
299
b = branch.Branch.open(self.get_url('to'))
300
self.assertEqual('../stack-on', b.get_stacked_on_url())
302
def create_simple_tree(self):
303
tree = self.make_branch_and_tree('tree')
304
self.build_tree(['tree/a'])
305
tree.add(['a'], ['a-id'])
306
tree.commit('one', rev_id='r1')
309
def test_push_create_prefix(self):
310
"""'bzr push --create-prefix' will create leading directories."""
311
tree = self.create_simple_tree()
313
self.run_bzr_error(['Parent directory of ../new/tree does not exist'],
316
self.run_bzr('push ../new/tree --create-prefix',
318
new_tree = workingtree.WorkingTree.open('new/tree')
319
self.assertEqual(tree.last_revision(), new_tree.last_revision())
320
self.failUnlessExists('new/tree/a')
322
def test_push_use_existing(self):
323
"""'bzr push --use-existing-dir' can push into an existing dir.
325
By default, 'bzr push' will not use an existing, non-versioned dir.
327
tree = self.create_simple_tree()
328
self.build_tree(['target/'])
330
self.run_bzr_error(['Target directory ../target already exists',
331
'Supply --use-existing-dir',
333
'push ../target', working_dir='tree')
335
self.run_bzr('push --use-existing-dir ../target',
338
new_tree = workingtree.WorkingTree.open('target')
339
self.assertEqual(tree.last_revision(), new_tree.last_revision())
340
# The push should have created target/a
341
self.failUnlessExists('target/a')
343
def test_push_use_existing_into_empty_bzrdir(self):
344
"""'bzr push --use-existing-dir' into a dir with an empty .bzr dir
347
tree = self.create_simple_tree()
348
self.build_tree(['target/', 'target/.bzr/'])
350
['Target directory ../target already contains a .bzr directory, '
351
'but it is not valid.'],
352
'push ../target --use-existing-dir', working_dir='tree')
354
def test_push_onto_repo(self):
355
"""We should be able to 'bzr push' into an existing bzrdir."""
356
tree = self.create_simple_tree()
357
repo = self.make_repository('repo', shared=True)
359
self.run_bzr('push ../repo',
362
# Pushing onto an existing bzrdir will create a repository and
363
# branch as needed, but will only create a working tree if there was
365
self.assertRaises(errors.NoWorkingTree,
366
workingtree.WorkingTree.open, 'repo')
367
new_branch = branch.Branch.open('repo')
368
self.assertEqual(tree.last_revision(), new_branch.last_revision())
370
def test_push_onto_just_bzrdir(self):
371
"""We don't handle when the target is just a bzrdir.
373
Because you shouldn't be able to create *just* a bzrdir in the wild.
375
# TODO: jam 20070109 Maybe it would be better to create the repository
377
tree = self.create_simple_tree()
378
a_bzrdir = self.make_bzrdir('dir')
380
self.run_bzr_error(['At ../dir you have a valid .bzr control'],
384
def test_push_with_revisionspec(self):
385
"""We should be able to push a revision older than the tip."""
386
tree_from = self.make_branch_and_tree('from')
387
tree_from.commit("One.", rev_id="from-1")
388
tree_from.commit("Two.", rev_id="from-2")
390
self.run_bzr('push -r1 ../to', working_dir='from')
392
tree_to = workingtree.WorkingTree.open('to')
393
repo_to = tree_to.branch.repository
394
self.assertTrue(repo_to.has_revision('from-1'))
395
self.assertFalse(repo_to.has_revision('from-2'))
396
self.assertEqual(tree_to.branch.last_revision_info()[1], 'from-1')
399
['bzr: ERROR: bzr push --revision '
400
'takes exactly one revision identifier\n'],
401
'push -r0..2 ../to', working_dir='from')
403
def create_trunk_and_feature_branch(self):
405
trunk_tree = self.make_branch_and_tree('target',
407
trunk_tree.commit('mainline')
408
# and a branch from it
409
branch_tree = self.make_branch_and_tree('branch',
411
branch_tree.pull(trunk_tree.branch)
412
branch_tree.branch.set_parent(trunk_tree.branch.base)
413
# with some work on it
414
branch_tree.commit('moar work plz')
415
return trunk_tree, branch_tree
417
def assertPublished(self, branch_revid, stacked_on):
418
"""Assert that the branch 'published' has been published correctly."""
419
published_branch = branch.Branch.open('published')
420
# The published branch refers to the mainline
421
self.assertEqual(stacked_on, published_branch.get_stacked_on_url())
422
# and the branch's work was pushed
423
self.assertTrue(published_branch.repository.has_revision(branch_revid))
425
def test_push_new_branch_stacked_on(self):
426
"""Pushing a new branch with --stacked-on creates a stacked branch."""
427
trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
428
# we publish branch_tree with a reference to the mainline.
429
out, err = self.run_bzr(['push', '--stacked-on', trunk_tree.branch.base,
430
self.get_url('published')], working_dir='branch')
431
self.assertEqual('', out)
432
self.assertEqual('Created new stacked branch referring to %s.\n' %
433
trunk_tree.branch.base, err)
434
self.assertPublished(branch_tree.last_revision(),
435
trunk_tree.branch.base)
437
def test_push_new_branch_stacked_uses_parent_when_no_public_url(self):
438
"""When the parent has no public url the parent is used as-is."""
439
trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
440
# now we do a stacked push, which should determine the public location
442
out, err = self.run_bzr(['push', '--stacked',
443
self.get_url('published')], working_dir='branch')
444
self.assertEqual('', out)
445
self.assertEqual('Created new stacked branch referring to %s.\n' %
446
trunk_tree.branch.base, err)
447
self.assertPublished(branch_tree.last_revision(),
448
trunk_tree.branch.base)
450
def test_push_new_branch_stacked_uses_parent_public(self):
451
"""Pushing a new branch with --stacked creates a stacked branch."""
452
trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
453
# the trunk is published on a web server
454
self.transport_readonly_server = http_server.HttpServer
455
trunk_public = self.make_branch('public_trunk', format='1.9')
456
trunk_public.pull(trunk_tree.branch)
457
trunk_public_url = self.get_readonly_url('public_trunk')
458
trunk_tree.branch.set_public_branch(trunk_public_url)
459
# now we do a stacked push, which should determine the public location
461
out, err = self.run_bzr(['push', '--stacked',
462
self.get_url('published')], working_dir='branch')
463
self.assertEqual('', out)
464
self.assertEqual('Created new stacked branch referring to %s.\n' %
465
trunk_public_url, err)
466
self.assertPublished(branch_tree.last_revision(), trunk_public_url)
468
def test_push_new_branch_stacked_no_parent(self):
469
"""Pushing with --stacked and no parent branch errors."""
470
branch = self.make_branch_and_tree('branch', format='1.9')
471
# now we do a stacked push, which should fail as the place to refer too
472
# cannot be determined.
473
out, err = self.run_bzr_error(
474
['Could not determine branch to refer to\\.'], ['push', '--stacked',
475
self.get_url('published')], working_dir='branch')
476
self.assertEqual('', out)
477
self.assertFalse(self.get_transport('published').has('.'))
479
def test_push_notifies_default_stacking(self):
480
self.make_branch('stack_on', format='1.6')
481
self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')
482
self.make_branch('from', format='1.6')
483
out, err = self.run_bzr('push -d from to')
484
self.assertContainsRe(err,
485
'Using default stacking branch stack_on at .*')
487
def test_push_stacks_with_default_stacking_if_target_is_stackable(self):
488
self.make_branch('stack_on', format='1.6')
489
self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')
490
self.make_branch('from', format='pack-0.92')
491
out, err = self.run_bzr('push -d from to')
492
b = branch.Branch.open('to')
493
self.assertEqual('../stack_on', b.get_stacked_on_url())
495
def test_push_does_not_change_format_with_default_if_target_cannot(self):
496
self.make_branch('stack_on', format='pack-0.92')
497
self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')
498
self.make_branch('from', format='pack-0.92')
499
out, err = self.run_bzr('push -d from to')
500
b = branch.Branch.open('to')
501
self.assertRaises(errors.UnstackableBranchFormat, b.get_stacked_on_url)
503
def test_push_doesnt_create_broken_branch(self):
504
"""Pushing a new standalone branch works even when there's a default
505
stacking policy at the destination.
507
The new branch will preserve the repo format (even if it isn't the
508
default for the branch), and will be stacked when the repo format
509
allows (which means that the branch format isn't necessarly preserved).
511
self.make_repository('repo', shared=True, format='1.6')
512
builder = self.make_branch_builder('repo/local', format='pack-0.92')
513
builder.start_series()
514
builder.build_snapshot('rev-1', None, [
515
('add', ('', 'root-id', 'directory', '')),
516
('add', ('filename', 'f-id', 'file', 'content\n'))])
517
builder.build_snapshot('rev-2', ['rev-1'], [])
518
builder.build_snapshot('rev-3', ['rev-2'],
519
[('modify', ('f-id', 'new-content\n'))])
520
builder.finish_series()
521
branch = builder.get_branch()
522
# Push rev-1 to "trunk", so that we can stack on it.
523
self.run_bzr('push -d repo/local trunk -r 1')
524
# Set a default stacking policy so that new branches will automatically
526
self.make_bzrdir('.').get_config().set_default_stack_on('trunk')
527
# Push rev-2 to a new branch "remote". It will be stacked on "trunk".
528
out, err = self.run_bzr('push -d repo/local remote -r 2')
529
self.assertContainsRe(
530
err, 'Using default stacking branch trunk at .*')
531
# Push rev-3 onto "remote". If "remote" not stacked and is missing the
532
# fulltext record for f-id @ rev-1, then this will fail.
533
out, err = self.run_bzr('push -d repo/local remote -r 3')
535
def test_push_verbose_shows_log(self):
536
tree = self.make_branch_and_tree('source')
538
out, err = self.run_bzr('push -v -d source target')
539
# initial push contains log
540
self.assertContainsRe(out, 'rev1')
542
out, err = self.run_bzr('push -v -d source target')
543
# subsequent push contains log
544
self.assertContainsRe(out, 'rev2')
545
# subsequent log is accurate
546
self.assertNotContainsRe(out, 'rev1')
548
def test_push_from_subdir(self):
549
t = self.make_branch_and_tree('tree')
550
self.build_tree(['tree/dir/', 'tree/dir/file'])
551
t.add('dir', 'dir/file')
553
out, err = self.run_bzr('push ../../pushloc', working_dir='tree/dir')
554
self.assertEqual('', out)
555
self.assertEqual('Created new branch.\n', err)
558
class RedirectingMemoryTransport(memory.MemoryTransport):
560
def mkdir(self, relpath, mode=None):
561
if self._cwd == '/source/':
562
raise errors.RedirectRequested(self.abspath(relpath),
563
self.abspath('../target'),
565
elif self._cwd == '/infinite-loop/':
566
raise errors.RedirectRequested(self.abspath(relpath),
567
self.abspath('../infinite-loop'),
570
return super(RedirectingMemoryTransport, self).mkdir(
573
def get(self, relpath):
574
if self.clone(relpath)._cwd == '/infinite-loop/':
575
raise errors.RedirectRequested(self.abspath(relpath),
576
self.abspath('../infinite-loop'),
579
return super(RedirectingMemoryTransport, self).get(relpath)
581
def _redirected_to(self, source, target):
582
# We do accept redirections
583
return transport.get_transport(target)
586
class RedirectingMemoryServer(memory.MemoryServer):
588
def start_server(self):
589
self._dirs = {'/': None}
592
self._scheme = 'redirecting-memory+%s:///' % id(self)
593
transport.register_transport(self._scheme, self._memory_factory)
595
def _memory_factory(self, url):
596
result = RedirectingMemoryTransport(url)
597
result._dirs = self._dirs
598
result._files = self._files
599
result._locks = self._locks
602
def stop_server(self):
603
transport.unregister_transport(self._scheme, self._memory_factory)
606
class TestPushRedirect(tests.TestCaseWithTransport):
609
tests.TestCaseWithTransport.setUp(self)
610
self.memory_server = RedirectingMemoryServer()
611
self.start_server(self.memory_server)
612
# Make the branch and tree that we'll be pushing.
613
t = self.make_branch_and_tree('tree')
614
self.build_tree(['tree/file'])
618
def test_push_redirects_on_mkdir(self):
619
"""If the push requires a mkdir, push respects redirect requests.
621
This is added primarily to handle lp:/ URI support, so that users can
622
push to new branches by specifying lp:/ URIs.
624
destination_url = self.memory_server.get_url() + 'source'
625
self.run_bzr(['push', '-d', 'tree', destination_url])
627
local_revision = branch.Branch.open('tree').last_revision()
628
remote_revision = branch.Branch.open(
629
self.memory_server.get_url() + 'target').last_revision()
630
self.assertEqual(remote_revision, local_revision)
632
def test_push_gracefully_handles_too_many_redirects(self):
633
"""Push fails gracefully if the mkdir generates a large number of
636
destination_url = self.memory_server.get_url() + 'infinite-loop'
637
out, err = self.run_bzr_error(
638
['Too many redirections trying to make %s\\.\n'
639
% re.escape(destination_url)],
640
['push', '-d', 'tree', destination_url], retcode=3)
641
self.assertEqual('', out)
644
class TestPushStrictMixin(object):
646
def make_local_branch_and_tree(self):
647
self.tree = self.make_branch_and_tree('local')
648
self.build_tree_contents([('local/file', 'initial')])
649
self.tree.add('file')
650
self.tree.commit('adding file', rev_id='added')
651
self.build_tree_contents([('local/file', 'modified')])
652
self.tree.commit('modify file', rev_id='modified')
654
def set_config_push_strict(self, value):
655
# set config var (any of bazaar.conf, locations.conf, branch.conf
657
conf = self.tree.branch.get_config()
658
conf.set_user_option('push_strict', value)
660
_default_command = ['push', '../to']
661
_default_wd = 'local'
662
_default_errors = ['Working tree ".*/local/" has uncommitted '
663
'changes \(See bzr status\)\.',]
664
_default_pushed_revid = 'modified'
666
def assertPushFails(self, args):
667
self.run_bzr_error(self._default_errors, self._default_command + args,
668
working_dir=self._default_wd, retcode=3)
670
def assertPushSucceeds(self, args, pushed_revid=None):
671
self.run_bzr(self._default_command + args,
672
working_dir=self._default_wd)
673
if pushed_revid is None:
674
pushed_revid = self._default_pushed_revid
675
tree_to = workingtree.WorkingTree.open('to')
676
repo_to = tree_to.branch.repository
677
self.assertTrue(repo_to.has_revision(pushed_revid))
678
self.assertEqual(tree_to.branch.last_revision_info()[1], pushed_revid)
682
class TestPushStrictWithoutChanges(tests.TestCaseWithTransport,
683
TestPushStrictMixin):
686
super(TestPushStrictWithoutChanges, self).setUp()
687
self.make_local_branch_and_tree()
689
def test_push_default(self):
690
self.assertPushSucceeds([])
692
def test_push_strict(self):
693
self.assertPushSucceeds(['--strict'])
695
def test_push_no_strict(self):
696
self.assertPushSucceeds(['--no-strict'])
698
def test_push_config_var_strict(self):
699
self.set_config_push_strict('true')
700
self.assertPushSucceeds([])
702
def test_push_config_var_no_strict(self):
703
self.set_config_push_strict('false')
704
self.assertPushSucceeds([])
707
class TestPushStrictWithChanges(tests.TestCaseWithTransport,
708
TestPushStrictMixin):
710
_changes_type = None # Set by load_tests
713
super(TestPushStrictWithChanges, self).setUp()
714
# Apply the changes defined in load_tests: one of _uncommitted_changes,
715
# _pending_merges or _out_of_sync_trees
716
getattr(self, self._changes_type)()
718
def _uncommitted_changes(self):
719
self.make_local_branch_and_tree()
720
# Make a change without committing it
721
self.build_tree_contents([('local/file', 'in progress')])
723
def _pending_merges(self):
724
self.make_local_branch_and_tree()
725
# Create 'other' branch containing a new file
726
other_bzrdir = self.tree.bzrdir.sprout('other')
727
other_tree = other_bzrdir.open_workingtree()
728
self.build_tree_contents([('other/other-file', 'other')])
729
other_tree.add('other-file')
730
other_tree.commit('other commit', rev_id='other')
731
# Merge and revert, leaving a pending merge
732
self.tree.merge_from_branch(other_tree.branch)
733
self.tree.revert(filenames=['other-file'], backups=False)
735
def _out_of_sync_trees(self):
736
self.make_local_branch_and_tree()
737
self.run_bzr(['checkout', '--lightweight', 'local', 'checkout'])
738
# Make a change and commit it
739
self.build_tree_contents([('local/file', 'modified in local')])
740
self.tree.commit('modify file', rev_id='modified-in-local')
741
# Exercise commands from the checkout directory
742
self._default_wd = 'checkout'
743
self._default_errors = ["Working tree is out of date, please run"
745
self._default_pushed_revid = 'modified-in-local'
747
def test_push_default(self):
748
self.assertPushFails([])
750
def test_push_with_revision(self):
751
self.assertPushSucceeds(['-r', 'revid:added'], pushed_revid='added')
753
def test_push_no_strict(self):
754
self.assertPushSucceeds(['--no-strict'])
756
def test_push_strict_with_changes(self):
757
self.assertPushFails(['--strict'])
759
def test_push_respect_config_var_strict(self):
760
self.set_config_push_strict('true')
761
self.assertPushFails([])
763
def test_push_bogus_config_var_ignored(self):
764
self.set_config_push_strict("I don't want you to be strict")
765
self.assertPushFails([])
767
def test_push_no_strict_command_line_override_config(self):
768
self.set_config_push_strict('yES')
769
self.assertPushFails([])
770
self.assertPushSucceeds(['--no-strict'])
772
def test_push_strict_command_line_override_config(self):
773
self.set_config_push_strict('oFF')
774
self.assertPushFails(['--strict'])
775
self.assertPushSucceeds([])
778
class TestPushForeign(blackbox.ExternalBase):
781
super(TestPushForeign, self).setUp()
782
test_foreign.register_dummy_foreign_for_test(self)
784
def make_dummy_builder(self, relpath):
785
builder = self.make_branch_builder(
786
relpath, format=test_foreign.DummyForeignVcsDirFormat())
787
builder.build_snapshot('revid', None,
788
[('add', ('', 'TREE_ROOT', 'directory', None)),
789
('add', ('foo', 'fooid', 'file', 'bar'))])
792
def test_no_roundtripping(self):
793
target_branch = self.make_dummy_builder('dp').get_branch()
794
source_tree = self.make_branch_and_tree("dc")
795
output, error = self.run_bzr("push -d dc dp", retcode=3)
796
self.assertEquals("", output)
797
self.assertEquals(error, "bzr: ERROR: It is not possible to losslessly"
798
" push to dummy. You may want to use dpush instead.\n")