/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_weave.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_weave.py
1
 
# Copyright (C) 2005 Canonical Ltd
 
1
# Copyright (C) 2005-2009, 2011 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
28
28
    )
29
29
from bzrlib.osutils import sha_string
30
30
from bzrlib.tests import TestCase, TestCaseInTempDir
31
 
from bzrlib.weave import Weave, WeaveFormatError, WeaveError
 
31
from bzrlib.weave import Weave, WeaveFormatError
32
32
from bzrlib.weavefile import write_weave, read_weave
33
33
 
34
34
 
64
64
 
65
65
class WeaveContains(TestBase):
66
66
    """Weave __contains__ operator"""
 
67
 
67
68
    def runTest(self):
68
69
        k = Weave(get_scope=lambda:None)
69
70
        self.assertFalse('foo' in k)
72
73
 
73
74
 
74
75
class Easy(TestBase):
 
76
 
75
77
    def runTest(self):
76
78
        k = Weave()
77
79
 
78
80
 
79
81
class AnnotateOne(TestBase):
 
82
 
80
83
    def runTest(self):
81
84
        k = Weave()
82
85
        k.add_lines('text0', [], TEXT_0)
86
89
 
87
90
class InvalidAdd(TestBase):
88
91
    """Try to use invalid version number during add."""
 
92
 
89
93
    def runTest(self):
90
94
        k = Weave()
91
95
 
107
111
 
108
112
 
109
113
class InvalidRepeatedAdd(TestBase):
 
114
 
110
115
    def runTest(self):
111
116
        k = Weave()
112
117
        k.add_lines('basis', [], TEXT_0)
533
538
 
534
539
class Merge(TestBase):
535
540
    """Storage of versions that merge diverged parents"""
 
541
 
536
542
    def runTest(self):
537
543
        k = Weave()
538
544
 
649
655
 
650
656
 
651
657
class JoinWeavesTests(TestBase):
 
658
 
652
659
    def setUp(self):
653
660
        super(JoinWeavesTests, self).setUp()
654
661
        self.weave1 = Weave()