/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
2530.3.1 by Martin Pool
Cleanup old variations on run_bzr in the test suite
1
# Copyright (C) 2005, 2006, 2007 Canonical Ltd
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
2
#
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
7
#
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
12
#
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
16
#
17
18
"""Tests of the 'bzr add' command."""
19
20
import os
21
4301.2.4 by Aaron Bentley
Further cleanups
22
from bzrlib import osutils
4163.2.2 by Ian Clatworthy
use multiply_tests rather than subclassing
23
from bzrlib.tests import (
24
    condition_isinstance,
25
    split_suite_by_condition,
26
    multiply_tests,
4301.1.1 by Geoff Bache
Fixing bug 183831, where 'bzr add' fails with a python stack if the path contains a symbolic link
27
    SymlinkFeature
4163.2.2 by Ian Clatworthy
use multiply_tests rather than subclassing
28
    )
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
29
from bzrlib.tests.blackbox import ExternalBase
2617.5.8 by Kuno Meyer
Extended tests for unicode chars outside of the iso-8859-* range
30
from bzrlib.tests.test_win32utils import NeedsGlobExpansionFeature
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
31
32
4163.2.2 by Ian Clatworthy
use multiply_tests rather than subclassing
33
def load_tests(standard_tests, module, loader):
34
    """Parameterize tests for view-aware vs not."""
35
    to_adapt, result = split_suite_by_condition(
36
        standard_tests, condition_isinstance(TestAdd))
37
    scenarios = [
38
        ('pre-views', {'branch_tree_format': 'pack-0.92'}),
4241.6.8 by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)
39
        ('view-aware', {'branch_tree_format': 'development6-rich-root'}),
4163.2.2 by Ian Clatworthy
use multiply_tests rather than subclassing
40
        ]
41
    return multiply_tests(to_adapt, scenarios, result)
42
43
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
44
class TestAdd(ExternalBase):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
45
4163.2.1 by Ian Clatworthy
Fix add in trees supports views
46
    def make_branch_and_tree(self, dir):
47
        return ExternalBase.make_branch_and_tree(self, dir,
48
            format=self.branch_tree_format)
49
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
50
    def test_add_reports(self):
51
        """add command prints the names of added files."""
2664.3.1 by Daniel Watkins
tests.blackbox.test_add now uses internals where appropriate.
52
        tree = self.make_branch_and_tree('.')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
53
        self.build_tree(['top.txt', 'dir/', 'dir/sub.txt', 'CVS'])
1765.1.1 by Robert Collins
Remove the default ignores list from bzr, lowering the minimum overhead in bzr add.
54
        self.build_tree_contents([('.bzrignore', 'CVS\n')])
2581.1.2 by Martin Pool
Remove unnecessary retcode=0 to run_bzr calls
55
        out = self.run_bzr('add')[0]
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
56
        # the ordering is not defined at the moment
57
        results = sorted(out.rstrip('\n').split('\n'))
58
        self.assertEquals(['If you wish to add some of these files, please'\
59
                           ' add them by name.',
3985.2.1 by Daniel Watkins
Updated tests for new behaviour.
60
                           'adding .bzrignore',
61
                           'adding dir',
62
                           'adding dir/sub.txt',
63
                           'adding top.txt',
1987.1.1 by John Arbash Meinel
Update the test suite to put HOME in a different directory
64
                           'ignored 1 file(s).'],
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
65
                          results)
2581.1.2 by Martin Pool
Remove unnecessary retcode=0 to run_bzr calls
66
        out = self.run_bzr('add -v')[0]
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
67
        results = sorted(out.rstrip('\n').split('\n'))
68
        self.assertEquals(['If you wish to add some of these files, please'\
69
                           ' add them by name.',
70
                           'ignored CVS matching "CVS"'],
71
                          results)
72
73
    def test_add_quiet_is(self):
74
        """add -q does not print the names of added files."""
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
75
        tree = self.make_branch_and_tree('.')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
76
        self.build_tree(['top.txt', 'dir/', 'dir/sub.txt'])
2581.1.2 by Martin Pool
Remove unnecessary retcode=0 to run_bzr calls
77
        out = self.run_bzr('add -q')[0]
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
78
        # the ordering is not defined at the moment
79
        results = sorted(out.rstrip('\n').split('\n'))
80
        self.assertEquals([''], results)
81
82
    def test_add_in_unversioned(self):
83
        """Try to add a file in an unversioned directory.
84
85
        "bzr add" should add the parent(s) as necessary.
86
        """
2664.3.1 by Daniel Watkins
tests.blackbox.test_add now uses internals where appropriate.
87
        tree = self.make_branch_and_tree('.')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
88
        self.build_tree(['inertiatic/', 'inertiatic/esp'])
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
89
        self.assertEquals(self.run_bzr('unknowns')[0], 'inertiatic\n')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
90
        self.run_bzr('add inertiatic/esp')
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
91
        self.assertEquals(self.run_bzr('unknowns')[0], '')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
92
93
        # Multiple unversioned parents
94
        self.build_tree(['veil/', 'veil/cerpin/', 'veil/cerpin/taxt'])
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
95
        self.assertEquals(self.run_bzr('unknowns')[0], 'veil\n')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
96
        self.run_bzr('add veil/cerpin/taxt')
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
97
        self.assertEquals(self.run_bzr('unknowns')[0], '')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
98
99
        # Check whacky paths work
100
        self.build_tree(['cicatriz/', 'cicatriz/esp'])
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
101
        self.assertEquals(self.run_bzr('unknowns')[0], 'cicatriz\n')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
102
        self.run_bzr('add inertiatic/../cicatriz/esp')
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
103
        self.assertEquals(self.run_bzr('unknowns')[0], '')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
104
105
    def test_add_in_versioned(self):
106
        """Try to add a file in a versioned directory.
107
108
        "bzr add" should do this happily.
109
        """
2664.3.1 by Daniel Watkins
tests.blackbox.test_add now uses internals where appropriate.
110
        tree = self.make_branch_and_tree('.')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
111
        self.build_tree(['inertiatic/', 'inertiatic/esp'])
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
112
        self.assertEquals(self.run_bzr('unknowns')[0], 'inertiatic\n')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
113
        self.run_bzr('add --no-recurse inertiatic')
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
114
        self.assertEquals(self.run_bzr('unknowns')[0], 'inertiatic/esp\n')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
115
        self.run_bzr('add inertiatic/esp')
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
116
        self.assertEquals(self.run_bzr('unknowns')[0], '')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
117
118
    def test_subdir_add(self):
119
        """Add in subdirectory should add only things from there down"""
120
        from bzrlib.workingtree import WorkingTree
1836.1.16 by John Arbash Meinel
Cleanup some tests which don't expect .bazaar/ to show up. Some still fail.
121
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
122
        eq = self.assertEqual
1836.1.16 by John Arbash Meinel
Cleanup some tests which don't expect .bazaar/ to show up. Some still fail.
123
        ass = self.assertTrue
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
124
        chdir = os.chdir
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
125
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
126
        t = self.make_branch_and_tree('.')
127
        b = t.branch
128
        self.build_tree(['src/', 'README'])
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
129
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
130
        eq(sorted(t.unknowns()),
131
           ['README', 'src'])
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
132
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
133
        self.run_bzr('add src')
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
134
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
135
        self.build_tree(['src/foo.c'])
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
136
2255.2.171 by Martin Pool
Fix up blackbox test_add to avoid depending on inventory not being held in memory
137
        # add with no arguments in a subdirectory gets only files below that
138
        # subdirectory
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
139
        chdir('src')
140
        self.run_bzr('add')
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
141
        self.assertEquals(self.run_bzr('unknowns')[0], 'README\n')
2255.2.176 by Martin Pool
Merge dirstate and some small cleanups
142
        # reopen to see the new changes
143
        t = t.bzrdir.open_workingtree()
2255.2.171 by Martin Pool
Fix up blackbox test_add to avoid depending on inventory not being held in memory
144
        versioned = [path for path, entry in t.iter_entries_by_dir()]
145
        self.assertEquals(versioned,
146
            ['', 'src', 'src/foo.c'])
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
147
2255.2.171 by Martin Pool
Fix up blackbox test_add to avoid depending on inventory not being held in memory
148
        # add from the parent directory should pick up all file names
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
149
        chdir('..')
150
        self.run_bzr('add')
2552.2.2 by Vincent Ladeuil
Enforce run_bzr(string) where possible.
151
        self.assertEquals(self.run_bzr('unknowns')[0], '')
1711.1.3 by Robert Collins
Add new test_add file - should have been in last commit.
152
        self.run_bzr('check')
1757.2.1 by Robert Collins
Add an explicit test that adding a missing file barfs.
153
154
    def test_add_missing(self):
155
        """bzr add foo where foo is missing should error."""
156
        self.make_branch_and_tree('.')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
157
        self.run_bzr('add missing-file', retcode=3)
1911.3.2 by John Arbash Meinel
Adding the AddFromBaseAction, which tries to reuse file ids from another tree
158
159
    def test_add_from(self):
160
        base_tree = self.make_branch_and_tree('base')
161
        self.build_tree(['base/a', 'base/b/', 'base/b/c'])
162
        base_tree.add(['a', 'b', 'b/c'])
163
        base_tree.commit('foo')
164
165
        new_tree = self.make_branch_and_tree('new')
166
        self.build_tree(['new/a', 'new/b/', 'new/b/c', 'd'])
167
168
        os.chdir('new')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
169
        out, err = self.run_bzr('add --file-ids-from ../base')
1911.3.2 by John Arbash Meinel
Adding the AddFromBaseAction, which tries to reuse file ids from another tree
170
        self.assertEqual('', err)
3985.2.1 by Daniel Watkins
Updated tests for new behaviour.
171
        self.assertEqualDiff('adding a w/ file id from a\n'
172
                             'adding b w/ file id from b\n'
4075.1.1 by Robert Collins
add should not print 'add completed' unnecessarily.
173
                             'adding b/c w/ file id from b/c\n',
1911.3.2 by John Arbash Meinel
Adding the AddFromBaseAction, which tries to reuse file ids from another tree
174
                             out)
2255.2.171 by Martin Pool
Fix up blackbox test_add to avoid depending on inventory not being held in memory
175
        new_tree = new_tree.bzrdir.open_workingtree()
1911.3.2 by John Arbash Meinel
Adding the AddFromBaseAction, which tries to reuse file ids from another tree
176
        self.assertEqual(base_tree.path2id('a'), new_tree.path2id('a'))
177
        self.assertEqual(base_tree.path2id('b'), new_tree.path2id('b'))
178
        self.assertEqual(base_tree.path2id('b/c'), new_tree.path2id('b/c'))
179
180
    def test_add_from_subdir(self):
181
        base_tree = self.make_branch_and_tree('base')
182
        self.build_tree(['base/a', 'base/b/', 'base/b/c', 'base/b/d'])
183
        base_tree.add(['a', 'b', 'b/c', 'b/d'])
184
        base_tree.commit('foo')
185
186
        new_tree = self.make_branch_and_tree('new')
187
        self.build_tree(['new/c', 'new/d'])
188
189
        os.chdir('new')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
190
        out, err = self.run_bzr('add --file-ids-from ../base/b')
1911.3.2 by John Arbash Meinel
Adding the AddFromBaseAction, which tries to reuse file ids from another tree
191
        self.assertEqual('', err)
3985.2.1 by Daniel Watkins
Updated tests for new behaviour.
192
        self.assertEqualDiff('adding c w/ file id from b/c\n'
4075.1.1 by Robert Collins
add should not print 'add completed' unnecessarily.
193
                             'adding d w/ file id from b/d\n',
1911.3.2 by John Arbash Meinel
Adding the AddFromBaseAction, which tries to reuse file ids from another tree
194
                             out)
195
2255.2.171 by Martin Pool
Fix up blackbox test_add to avoid depending on inventory not being held in memory
196
        new_tree = new_tree.bzrdir.open_workingtree()
1911.3.2 by John Arbash Meinel
Adding the AddFromBaseAction, which tries to reuse file ids from another tree
197
        self.assertEqual(base_tree.path2id('b/c'), new_tree.path2id('c'))
198
        self.assertEqual(base_tree.path2id('b/d'), new_tree.path2id('d'))
1928.1.1 by Alexander Belchenko
blackbox test for 'bzr add --dry-run'
199
200
    def test_add_dry_run(self):
2568.2.4 by Robert Collins
* ``bzrlib.add.smart_add`` and ``bzrlib.add.smart_add_tree`` are now
201
        """Test a dry run add, make sure nothing is added."""
202
        wt = self.make_branch_and_tree('.')
203
        self.build_tree(['inertiatic/', 'inertiatic/esp'])
204
        self.assertEqual(list(wt.unknowns()), ['inertiatic'])
205
        self.run_bzr('add --dry-run')
206
        self.assertEqual(list(wt.unknowns()), ['inertiatic'])
2279.5.1 by Matthias Rahlf
FastPath objects can now be printed nicely
207
208
    def test_add_control_dir(self):
209
        """The control dir and its content should be refused."""
2279.6.1 by Alexander Belchenko
Instead of __str__ method for FastPath object we use .raw_path attribute (note from Aaron).
210
        self.make_branch_and_tree('.')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
211
        err = self.run_bzr('add .bzr', retcode=3)[1]
2279.5.1 by Matthias Rahlf
FastPath objects can now be printed nicely
212
        self.assertContainsRe(err, r'ERROR:.*\.bzr.*control file')
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
213
        err = self.run_bzr('add .bzr/README', retcode=3)[1]
2279.5.1 by Matthias Rahlf
FastPath objects can now be printed nicely
214
        self.assertContainsRe(err, r'ERROR:.*\.bzr.*control file')
215
        self.build_tree(['.bzr/crescent'])
2530.3.3 by Martin Pool
Clean up some callers that use varargs syntax for run_bzr, but don't
216
        err = self.run_bzr('add .bzr/crescent', retcode=3)[1]
2279.5.1 by Matthias Rahlf
FastPath objects can now be printed nicely
217
        self.assertContainsRe(err, r'ERROR:.*\.bzr.*control file')
2617.5.3 by Kuno Meyer
Blackbox test for adding with wildcards (Win32).
218
219
    def test_add_with_wildcards(self):
2617.5.8 by Kuno Meyer
Extended tests for unicode chars outside of the iso-8859-* range
220
        self.requireFeature(NeedsGlobExpansionFeature)
2617.5.4 by Kuno Meyer
Included feedback on initial patch.
221
        self.make_branch_and_tree('.')
2617.5.3 by Kuno Meyer
Blackbox test for adding with wildcards (Win32).
222
        self.build_tree(['a1', 'a2', 'b', 'c33'])
2617.5.4 by Kuno Meyer
Included feedback on initial patch.
223
        self.run_bzr(['add', 'a?', 'c*'])
2617.5.3 by Kuno Meyer
Blackbox test for adding with wildcards (Win32).
224
        self.assertEquals(self.run_bzr('unknowns')[0], 'b\n')
2617.5.8 by Kuno Meyer
Extended tests for unicode chars outside of the iso-8859-* range
225
226
    def test_add_with_wildcards_unicode(self):
227
        self.requireFeature(NeedsGlobExpansionFeature)
228
        self.make_branch_and_tree('.')
229
        self.build_tree([u'\u1234A', u'\u1235A', u'\u1235AA', 'cc'])
230
        self.run_bzr(['add', u'\u1234?', u'\u1235*'])
231
        self.assertEquals(self.run_bzr('unknowns')[0], 'cc\n')
4301.1.1 by Geoff Bache
Fixing bug 183831, where 'bzr add' fails with a python stack if the path contains a symbolic link
232
233
    def test_add_via_symlink(self):
234
        self.requireFeature(SymlinkFeature)
235
        self.make_branch_and_tree('source')
236
        self.build_tree(['source/top.txt'])
237
        os.symlink('source', 'link')
238
        out = self.run_bzr(['add', 'link/top.txt'])[0]
239
        self.assertEquals(out, 'adding top.txt\n')
240
241
    def test_add_symlink_to_abspath(self):
242
        self.requireFeature(SymlinkFeature)
4301.2.4 by Aaron Bentley
Further cleanups
243
        self.make_branch_and_tree('tree')
244
        os.symlink(osutils.abspath('target'), 'tree/link')
245
        out = self.run_bzr(['add', 'tree/link'])[0]
4301.1.1 by Geoff Bache
Fixing bug 183831, where 'bzr add' fails with a python stack if the path contains a symbolic link
246
        self.assertEquals(out, 'adding link\n')