/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_pack.py

  • Committer: Vincent Ladeuil
  • Date: 2012-01-05 14:26:58 UTC
  • mto: This revision was merged to the branch mainline in revision 6468.
  • Revision ID: v.ladeuil+lp@free.fr-20120105142658-vek3v6pzlxb751s2
Tests passing for a first rough version of a cached branch config store. The changes here are too invasive and several parallel proposals have been made. 

@only_raises is evil and gave a hard time since any exception during
save_changes() was swallowed.

Possible improvements: 

- add some needs_write_lock decorators to crucial
  methods (_set_config_location ?) but keep locking the branch at higher levels

- decorate branch.unlock to call stack.save if last_lock() it True
  outside of @only_raises scope (evil decorator)

- add @needs_write_lock to stack.set and stack.remove (will probably get
  rid of most testing issues) we probably need a specialized decorator
  that can relay to the store and from there to the branch or whatever is
  needed. This will also helps bzr config to get it right. The
  get_mutable_section trick should not be needed anymore either.

- decorate branch.unlock to call stack.save if last_lock() it True outside
  of @only_raises scope (evil decorator)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_pack.py
19
19
import os
20
20
 
21
21
from bzrlib import tests
 
22
from bzrlib.tests.matchers import ContainsNoVfsCalls
22
23
 
23
24
 
24
25
class TestPack(tests.TestCaseWithTransport):
38
39
 
39
40
    def _update_file(self, path, text, checkin=True):
40
41
        """append text to file 'path' and check it in"""
41
 
        open(path, 'a').write(text)
 
42
        with open(path, 'a') as f:
 
43
            f.write(text)
 
44
 
42
45
        if checkin:
43
46
            self.run_bzr(['ci', path, '-m', '"' + path + '"'])
44
47
 
81
84
 
82
85
        pack_names = transport.list_dir('repository/obsolete_packs')
83
86
        self.assertTrue(len(pack_names) == 0)
 
87
 
 
88
 
 
89
class TestSmartServerPack(tests.TestCaseWithTransport):
 
90
 
 
91
    def test_simple_pack(self):
 
92
        self.setup_smart_server_with_call_log()
 
93
        t = self.make_branch_and_tree('branch')
 
94
        self.build_tree_contents([('branch/foo', 'thecontents')])
 
95
        t.add("foo")
 
96
        t.commit("message")
 
97
        self.reset_smart_call_log()
 
98
        out, err = self.run_bzr(['pack', self.get_url('branch')])
 
99
        # This figure represent the amount of HPSS calls to perform this use
 
100
        # case. It is entirely ok to reduce this number if a test fails due to
 
101
        # rpc_count # being too low. If rpc_count increases, more network
 
102
        # roundtrips have become necessary for this use case. Please do not
 
103
        # adjust this number upwards without agreement from bzr's network
 
104
        # support maintainers.
 
105
        self.assertLength(6, self.hpss_calls)
 
106
        self.assertThat(self.hpss_calls, ContainsNoVfsCalls)