/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_unknowns.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_unknowns.py
1
 
# Copyright (C) 2007 Canonical Ltd
 
1
# Copyright (C) 2007-2010 Canonical Ltd
2
2
# -*- coding: utf-8 -*-
3
3
#
4
4
# This program is free software; you can redistribute it and/or modify
18
18
 
19
19
"""Black-box tests for 'bzr unknowns', which shows unknown files."""
20
20
 
21
 
from bzrlib.tests.blackbox import ExternalBase
22
 
 
23
 
 
24
 
class TestUnknowns(ExternalBase):
 
21
from bzrlib.tests import TestCaseWithTransport
 
22
 
 
23
 
 
24
class TestUnknowns(TestCaseWithTransport):
25
25
 
26
26
    def test_unknowns(self):
27
27
        """Test that 'unknown' command reports unknown files"""
45
45
        # after all added, none shown
46
46
        tree.add(['b', 'c'])
47
47
        self.assertEquals(self.run_bzr('unknowns')[0], '')
 
48
 
 
49
    def test_unknowns_directory(self):
 
50
        """Test --directory option"""
 
51
        tree = self.make_branch_and_tree('a')
 
52
        self.build_tree(['a/README'])
 
53
        out, err = self.run_bzr(['unknowns', '--directory=a'])
 
54
        self.assertEquals('README\n', out)