/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/tests/blackbox/test_add.py

  • Committer: John Arbash Meinel
  • Date: 2010-08-10 20:03:44 UTC
  • mto: This revision was merged to the branch mainline in revision 5376.
  • Revision ID: john@arbash-meinel.com-20100810200344-6muerwvkafqu7w47
Rework things a bit so the logic can be shared.

It turns out that some of the peak memory is actually during the inventory
to string to bundle translations. So re-use the refcount logic there.
This actually does show a decrease in peak memory.
Specifically 'cd bzr.dev; bzr send ../2.2' drops from 221MB peak to 156MB.

We don't speed anything up (16.5s both ways), but peak memory is quite
a bit better.

Show diffs side-by-side

added added

removed removed

Lines of Context:
23
23
    osutils,
24
24
    tests,
25
25
    )
 
26
from bzrlib.tests import (
 
27
    script,
 
28
    )
26
29
 
27
30
 
28
31
def load_tests(standard_tests, module, loader):
217
220
        os.symlink(osutils.abspath('target'), 'tree/link')
218
221
        out = self.run_bzr(['add', 'tree/link'])[0]
219
222
        self.assertEquals(out, 'adding link\n')
 
223
 
 
224
    def test_add_not_child(self):
 
225
        # https://bugs.launchpad.net/bzr/+bug/98735
 
226
        sr = script.ScriptRunner()
 
227
        self.make_branch_and_tree('tree1')
 
228
        self.make_branch_and_tree('tree2')
 
229
        self.build_tree(['tree1/a', 'tree2/b'])
 
230
        sr.run_script(self, '''
 
231
        $ bzr add tree1/a tree2/b
 
232
        2>bzr: ERROR: Path "...tree2/b" is not a child of path "...tree1"
 
233
        ''')