/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/test_revisiontree.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/test_revisiontree.py
1
 
# Copyright (C) 2006, 2008, 2009, 2010 Canonical Ltd
 
1
# Copyright (C) 2006, 2008-2011 Canonical Ltd
2
2
# Authors:  Robert Collins <robert.collins@canonical.com>
3
3
#
4
4
# This program is free software; you can redistribute it and/or modify
21
21
    errors,
22
22
    revision,
23
23
    )
24
 
import bzrlib
25
 
from bzrlib.inventory import ROOT_ID
26
24
from bzrlib.tests import TestCaseWithTransport
27
25
 
28
26
 
61
59
        null_tree = self.t.branch.repository.revision_tree(
62
60
            revision.NULL_REVISION)
63
61
        self.assertIs(None, null_tree.inventory.root)
 
62
        self.assertIs(None, null_tree.get_root_id())
 
63
 
 
64
    def test_get_file_revision_root(self):
 
65
        self.assertEquals(self.rev_id,
 
66
            self.rev_tree.get_file_revision(self.rev_tree.get_root_id()))
 
67
 
 
68
    def test_get_file_revision(self):
 
69
        self.build_tree_contents([('a', 'initial')])
 
70
        self.t.add(['a'])
 
71
        revid1 = self.t.commit('add a')
 
72
        revid2 = self.t.commit('another change', allow_pointless=True)
 
73
        tree = self.t.branch.repository.revision_tree(revid2)
 
74
        self.assertEquals(revid1,
 
75
            tree.get_file_revision(tree.path2id('a')))
64
76
 
65
77
    def test_get_file_mtime_ghost(self):
66
 
        file_id = iter(self.rev_tree).next()
 
78
        file_id = iter(self.rev_tree.all_file_ids()).next()
67
79
        self.rev_tree.inventory[file_id].revision = 'ghostrev'
68
80
        self.assertRaises(errors.FileTimestampUnavailable, 
69
81
            self.rev_tree.get_file_mtime, file_id)