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') |