bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
|
4988.10.3
by John Arbash Meinel
Merge bzr.dev 5007, resolve conflict, update NEWS |
1 |
# Copyright (C) 2006-2010 Canonical Ltd
|
|
1887.1.1
by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines, |
2 |
#
|
|
1540.2.6
by Robey Pointer
make 'log' and 'status' treat '-r N..' as implicitly '-r N..-1' |
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.
|
|
|
1887.1.1
by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines, |
7 |
#
|
|
1540.2.6
by Robey Pointer
make 'log' and 'status' treat '-r N..' as implicitly '-r N..-1' |
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.
|
|
|
1887.1.1
by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines, |
12 |
#
|
|
1540.2.6
by Robey Pointer
make 'log' and 'status' treat '-r N..' as implicitly '-r N..-1' |
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
|
|
1540.2.6
by Robey Pointer
make 'log' and 'status' treat '-r N..' as implicitly '-r N..-1' |
16 |
|
17 |
||
|
1624.1.3
by Robert Collins
Convert log to use the new tsort.merge_sort routine. |
18 |
"""Black-box tests for bzr log."""
|
|
1540.2.6
by Robey Pointer
make 'log' and 'status' treat '-r N..' as implicitly '-r N..-1' |
19 |
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
20 |
from itertools import izip |
|
4325.4.3
by Vincent Ladeuil
More cleanups. |
21 |
import os |
22 |
import re |
|
|
1540.2.6
by Robey Pointer
make 'log' and 'status' treat '-r N..' as implicitly '-r N..-1' |
23 |
|
|
4325.4.3
by Vincent Ladeuil
More cleanups. |
24 |
from bzrlib import ( |
|
4955.4.6
by Vincent Ladeuil
Switch to branchbuilder to avoid the cost of test.script. |
25 |
branchbuilder, |
|
4955.7.3
by Vincent Ladeuil
Check ancestry so we don't output random revisions. |
26 |
errors, |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
27 |
log, |
|
4325.4.3
by Vincent Ladeuil
More cleanups. |
28 |
osutils, |
29 |
tests, |
|
|
2978.6.1
by Kent Gibson
Use normalize_log to more accurately test blackbox log output. |
30 |
)
|
|
4955.4.5
by Vincent Ladeuil
Start reproducing the problems reported in the bug. |
31 |
from bzrlib.tests import ( |
32 |
script, |
|
33 |
test_log, |
|
34 |
)
|
|
|
3144.7.9
by Guillermo Gonzalez
* bzrlib.log.show_roperties don't hide handler errors |
35 |
|
36 |
||
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
37 |
class TestLog(tests.TestCaseWithTransport, test_log.TestLogMixin): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
38 |
|
39 |
def make_minimal_branch(self, path='.', format=None): |
|
40 |
tree = self.make_branch_and_tree(path, format=format) |
|
41 |
self.build_tree([path + '/hello.txt']) |
|
42 |
tree.add('hello.txt') |
|
43 |
tree.commit(message='message1') |
|
44 |
return tree |
|
45 |
||
46 |
def make_linear_branch(self, path='.', format=None): |
|
|
2809.1.2
by Daniel Watkins
Removed unnecessary check as per abentley's on-list comments. |
47 |
tree = self.make_branch_and_tree(path, format=format) |
|
2664.14.1
by Daniel Watkins
Fixed tests.blackbox.test_log to use internals where appropriate. |
48 |
self.build_tree( |
49 |
[path + '/hello.txt', path + '/goodbye.txt', path + '/meep.txt']) |
|
50 |
tree.add('hello.txt') |
|
51 |
tree.commit(message='message1') |
|
52 |
tree.add('goodbye.txt') |
|
53 |
tree.commit(message='message2') |
|
54 |
tree.add('meep.txt') |
|
55 |
tree.commit(message='message3') |
|
56 |
return tree |
|
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
57 |
|
|
4369.2.1
by Marius Kruger
add some leftover log tests |
58 |
def make_merged_branch(self, path='.', format=None): |
|
4369.2.2
by Marius Kruger
use make_linear_branch in make_merged_branch |
59 |
tree = self.make_linear_branch(path, format) |
60 |
tree2 = tree.bzrdir.sprout('tree2', |
|
61 |
revision_id=tree.branch.get_rev_id(1)).open_workingtree() |
|
|
4369.2.1
by Marius Kruger
add some leftover log tests |
62 |
tree2.commit(message='tree2 message2') |
63 |
tree2.commit(message='tree2 message3') |
|
64 |
tree.merge_from_branch(tree2.branch) |
|
65 |
tree.commit(message='merge') |
|
66 |
return tree |
|
67 |
||
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
68 |
|
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
69 |
class TestLogWithLogCatcher(TestLog): |
70 |
||
71 |
def setUp(self): |
|
72 |
super(TestLogWithLogCatcher, self).setUp() |
|
|
4955.4.5
by Vincent Ladeuil
Start reproducing the problems reported in the bug. |
73 |
# Capture log formatter creations
|
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
74 |
class MyLogFormatter(test_log.LogCatcher): |
75 |
||
76 |
def __new__(klass, *args, **kwargs): |
|
|
4955.4.5
by Vincent Ladeuil
Start reproducing the problems reported in the bug. |
77 |
self.log_catcher = test_log.LogCatcher(*args, **kwargs) |
78 |
# Always return our own log formatter
|
|
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
79 |
return self.log_catcher |
80 |
||
81 |
def getme(branch): |
|
82 |
# Always return our own log formatter class hijacking the
|
|
83 |
# default behavior (which requires setting up a config
|
|
84 |
# variable)
|
|
85 |
return MyLogFormatter |
|
|
4985.1.5
by Vincent Ladeuil
Deploying the new overrideAttr facility further reduces the complexity |
86 |
self.overrideAttr(log.log_formatter_registry, 'get_default', getme) |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
87 |
|
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
88 |
def get_captured_revisions(self): |
89 |
return self.log_catcher.revisions |
|
90 |
||
91 |
def assertLogRevnos(self, args, expected_revnos, working_dir='.'): |
|
92 |
self.run_bzr(['log'] + args, working_dir=working_dir) |
|
93 |
self.assertEqual(expected_revnos, |
|
94 |
[r.revno for r in self.get_captured_revisions()]) |
|
95 |
||
96 |
def assertLogRevnosAndDepths(self, args, expected_revnos_and_depths, |
|
97 |
working_dir='.'): |
|
98 |
self.run_bzr(['log'] + args, working_dir=working_dir) |
|
99 |
self.assertEqual(expected_revnos_and_depths, |
|
100 |
[(r.revno, r.merge_depth) |
|
101 |
for r in self.get_captured_revisions()]) |
|
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
102 |
|
103 |
||
104 |
class TestLogRevSpecs(TestLogWithLogCatcher): |
|
105 |
||
106 |
def test_log_no_revspec(self): |
|
107 |
self.make_linear_branch() |
|
108 |
self.assertLogRevnos([], ['3', '2', '1']) |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
109 |
|
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
110 |
def test_log_null_end_revspec(self): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
111 |
self.make_linear_branch() |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
112 |
self.assertLogRevnos(['-r1..'], ['3', '2', '1']) |
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
113 |
|
114 |
def test_log_null_begin_revspec(self): |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
115 |
self.make_linear_branch() |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
116 |
self.assertLogRevnos(['-r..3'], ['3', '2', '1']) |
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
117 |
|
118 |
def test_log_null_both_revspecs(self): |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
119 |
self.make_linear_branch() |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
120 |
self.assertLogRevnos(['-r..'], ['3', '2', '1']) |
|
2978.4.1
by Kent Gibson
Logging revision 0 returns error. |
121 |
|
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
122 |
def test_log_negative_begin_revspec_full_log(self): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
123 |
self.make_linear_branch() |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
124 |
self.assertLogRevnos(['-r-3..'], ['3', '2', '1']) |
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
125 |
|
126 |
def test_log_negative_both_revspec_full_log(self): |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
127 |
self.make_linear_branch() |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
128 |
self.assertLogRevnos(['-r-3..-1'], ['3', '2', '1']) |
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
129 |
|
130 |
def test_log_negative_both_revspec_partial(self): |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
131 |
self.make_linear_branch() |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
132 |
self.assertLogRevnos(['-r-3..-2'], ['2', '1']) |
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
133 |
|
134 |
def test_log_negative_begin_revspec(self): |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
135 |
self.make_linear_branch() |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
136 |
self.assertLogRevnos(['-r-2..'], ['3', '2']) |
|
1553.4.2
by Michael Ellerman
Make bzr log -r .. work, fixes bug #4609. Add a bunch of tests to make sure |
137 |
|
|
2978.4.1
by Kent Gibson
Logging revision 0 returns error. |
138 |
def test_log_positive_revspecs(self): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
139 |
self.make_linear_branch() |
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
140 |
self.assertLogRevnos(['-r1..3'], ['3', '2', '1']) |
|
1624.1.3
by Robert Collins
Convert log to use the new tsort.merge_sort routine. |
141 |
|
|
4369.2.1
by Marius Kruger
add some leftover log tests |
142 |
def test_log_dotted_revspecs(self): |
143 |
self.make_merged_branch() |
|
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
144 |
self.assertLogRevnos(['-n0', '-r1..1.1.1'], ['1.1.1', '1']) |
|
3734.1.1
by Vincent Ladeuil
Fix bug #248427 by adding a --change option to log. |
145 |
|
|
4955.5.1
by Vincent Ladeuil
Split some tests to better characterize the classes |
146 |
def test_log_limit(self): |
147 |
tree = self.make_branch_and_tree('.') |
|
148 |
# We want more commits than our batch size starts at
|
|
149 |
for pos in range(10): |
|
150 |
tree.commit("%s" % pos) |
|
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
151 |
self.assertLogRevnos(['--limit', '2'], ['10', '9']) |
|
4955.5.1
by Vincent Ladeuil
Split some tests to better characterize the classes |
152 |
|
153 |
def test_log_limit_short(self): |
|
154 |
self.make_linear_branch() |
|
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
155 |
self.assertLogRevnos(['-l', '2'], ['3', '2']) |
156 |
||
|
4955.4.7
by Vincent Ladeuil
Some more cleanup. |
157 |
def test_log_change_revno(self): |
158 |
self.make_linear_branch() |
|
159 |
self.assertLogRevnos(['-c1'], ['1']) |
|
160 |
||
|
4955.4.6
by Vincent Ladeuil
Switch to branchbuilder to avoid the cost of test.script. |
161 |
|
|
4955.7.5
by Vincent Ladeuil
Fixed as per Ian's review. |
162 |
class TestLogMergedLinearAncestry(TestLogWithLogCatcher): |
|
4955.4.5
by Vincent Ladeuil
Start reproducing the problems reported in the bug. |
163 |
|
164 |
def setUp(self): |
|
|
4955.7.5
by Vincent Ladeuil
Fixed as per Ian's review. |
165 |
super(TestLogMergedLinearAncestry, self).setUp() |
|
4955.4.6
by Vincent Ladeuil
Switch to branchbuilder to avoid the cost of test.script. |
166 |
# FIXME: Using a MemoryTree would be even better here (but until we
|
167 |
# stop calling run_bzr, there is no point) --vila 100118.
|
|
168 |
builder = branchbuilder.BranchBuilder(self.get_transport()) |
|
169 |
builder.start_series() |
|
170 |
# mainline
|
|
171 |
builder.build_snapshot('1', None, [ |
|
172 |
('add', ('', 'root-id', 'directory', ''))]) |
|
173 |
builder.build_snapshot('2', ['1'], []) |
|
174 |
# branch
|
|
175 |
builder.build_snapshot('1.1.1', ['1'], []) |
|
176 |
# merge branch into mainline
|
|
177 |
builder.build_snapshot('3', ['2', '1.1.1'], []) |
|
178 |
# new commits in branch
|
|
179 |
builder.build_snapshot('1.1.2', ['1.1.1'], []) |
|
180 |
builder.build_snapshot('1.1.3', ['1.1.2'], []) |
|
181 |
# merge branch into mainline
|
|
182 |
builder.build_snapshot('4', ['3', '1.1.3'], []) |
|
183 |
# merge mainline into branch
|
|
184 |
builder.build_snapshot('1.1.4', ['1.1.3', '4'], []) |
|
185 |
# merge branch into mainline
|
|
186 |
builder.build_snapshot('5', ['4', '1.1.4'], []) |
|
187 |
builder.finish_series() |
|
|
4955.4.5
by Vincent Ladeuil
Start reproducing the problems reported in the bug. |
188 |
|
189 |
def test_n0(self): |
|
190 |
self.assertLogRevnos(['-n0', '-r1.1.1..1.1.4'], |
|
191 |
['1.1.4', '4', '1.1.3', '1.1.2', '3', '1.1.1']) |
|
192 |
def test_n0_forward(self): |
|
193 |
self.assertLogRevnos(['-n0', '-r1.1.1..1.1.4', '--forward'], |
|
194 |
['3', '1.1.1', '4', '1.1.2', '1.1.3', '1.1.4']) |
|
195 |
||
196 |
def test_n1(self): |
|
|
4955.4.6
by Vincent Ladeuil
Switch to branchbuilder to avoid the cost of test.script. |
197 |
# starting from 1.1.4 we follow the left-hand ancestry
|
|
4955.4.5
by Vincent Ladeuil
Start reproducing the problems reported in the bug. |
198 |
self.assertLogRevnos(['-n1', '-r1.1.1..1.1.4'], |
199 |
['1.1.4', '1.1.3', '1.1.2', '1.1.1']) |
|
200 |
||
201 |
def test_n1_forward(self): |
|
202 |
self.assertLogRevnos(['-n1', '-r1.1.1..1.1.4', '--forward'], |
|
203 |
['1.1.1', '1.1.2', '1.1.3', '1.1.4']) |
|
204 |
||
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
205 |
|
|
4955.7.5
by Vincent Ladeuil
Fixed as per Ian's review. |
206 |
class Test_GenerateAllRevisions(TestLogWithLogCatcher): |
|
4955.7.3
by Vincent Ladeuil
Check ancestry so we don't output random revisions. |
207 |
|
|
4955.7.5
by Vincent Ladeuil
Fixed as per Ian's review. |
208 |
def make_branch_with_many_merges(self, path='.', format=None): |
|
4955.7.3
by Vincent Ladeuil
Check ancestry so we don't output random revisions. |
209 |
builder = branchbuilder.BranchBuilder(self.get_transport()) |
210 |
builder.start_series() |
|
211 |
# The graph below may look a bit complicated (and it may be but I've
|
|
212 |
# banged my head enough on it) but the bug requires at least dotted
|
|
213 |
# revnos *and* merged revisions below that.
|
|
214 |
builder.build_snapshot('1', None, [ |
|
215 |
('add', ('', 'root-id', 'directory', ''))]) |
|
216 |
builder.build_snapshot('2', ['1'], []) |
|
217 |
builder.build_snapshot('1.1.1', ['1'], []) |
|
218 |
builder.build_snapshot('2.1.1', ['2'], []) |
|
219 |
builder.build_snapshot('3', ['2', '1.1.1'], []) |
|
220 |
builder.build_snapshot('2.1.2', ['2.1.1'], []) |
|
221 |
builder.build_snapshot('2.2.1', ['2.1.1'], []) |
|
222 |
builder.build_snapshot('2.1.3', ['2.1.2', '2.2.1'], []) |
|
223 |
builder.build_snapshot('4', ['3', '2.1.3'], []) |
|
224 |
builder.build_snapshot('5', ['4', '2.1.2'], []) |
|
225 |
builder.finish_series() |
|
226 |
return builder |
|
227 |
||
228 |
def test_not_an_ancestor(self): |
|
|
4955.7.5
by Vincent Ladeuil
Fixed as per Ian's review. |
229 |
builder = self.make_branch_with_many_merges() |
|
4955.7.3
by Vincent Ladeuil
Check ancestry so we don't output random revisions. |
230 |
b = builder.get_branch() |
231 |
b.lock_read() |
|
232 |
self.addCleanup(b.unlock) |
|
233 |
self.assertRaises(errors.BzrCommandError, |
|
234 |
log._generate_all_revisions, |
|
235 |
b, '1.1.1', '2.1.3', 'reverse', |
|
236 |
delayed_graph_generation=True) |
|
237 |
||
238 |
def test_wrong_order(self): |
|
|
4955.7.5
by Vincent Ladeuil
Fixed as per Ian's review. |
239 |
builder = self.make_branch_with_many_merges() |
|
4955.7.3
by Vincent Ladeuil
Check ancestry so we don't output random revisions. |
240 |
b = builder.get_branch() |
241 |
b.lock_read() |
|
242 |
self.addCleanup(b.unlock) |
|
243 |
self.assertRaises(errors.BzrCommandError, |
|
244 |
log._generate_all_revisions, |
|
245 |
b, '5', '2.1.3', 'reverse', |
|
246 |
delayed_graph_generation=True) |
|
247 |
||
248 |
||
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
249 |
class TestLogRevSpecsWithPaths(TestLogWithLogCatcher): |
250 |
||
251 |
def test_log_revno_n_path_wrong_namespace(self): |
|
252 |
self.make_linear_branch('branch1') |
|
253 |
self.make_linear_branch('branch2') |
|
254 |
# There is no guarantee that a path exist between two arbitrary
|
|
255 |
# revisions.
|
|
256 |
self.run_bzr("log -r revno:2:branch1..revno:3:branch2", retcode=3) |
|
257 |
||
258 |
def test_log_revno_n_path_correct_order(self): |
|
259 |
self.make_linear_branch('branch2') |
|
260 |
self.assertLogRevnos(['-rrevno:1:branch2..revno:3:branch2'], |
|
261 |
['3', '2','1']) |
|
262 |
||
263 |
def test_log_revno_n_path(self): |
|
264 |
self.make_linear_branch('branch2') |
|
265 |
self.assertLogRevnos(['-rrevno:1:branch2'], |
|
266 |
['1']) |
|
267 |
rev_props = self.log_catcher.revisions[0].rev.properties |
|
268 |
self.assertEqual('branch2', rev_props['branch-nick']) |
|
|
4955.5.1
by Vincent Ladeuil
Split some tests to better characterize the classes |
269 |
|
270 |
||
271 |
class TestLogErrors(TestLog): |
|
272 |
||
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
273 |
def test_log_zero_revspec(self): |
274 |
self.make_minimal_branch() |
|
275 |
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'], |
|
276 |
['log', '-r0']) |
|
277 |
||
278 |
def test_log_zero_begin_revspec(self): |
|
279 |
self.make_linear_branch() |
|
280 |
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'], |
|
281 |
['log', '-r0..2']) |
|
282 |
||
283 |
def test_log_zero_end_revspec(self): |
|
284 |
self.make_linear_branch() |
|
285 |
self.run_bzr_error(['bzr: ERROR: Logging revision 0 is invalid.'], |
|
286 |
['log', '-r-2..0']) |
|
287 |
||
|
3878.3.3
by Marius Kruger
Add tests for log -r with non-exising revno's |
288 |
def test_log_nonexistent_revno(self): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
289 |
self.make_minimal_branch() |
|
4955.4.7
by Vincent Ladeuil
Some more cleanup. |
290 |
self.run_bzr_error(["bzr: ERROR: Requested revision: '1234' " |
291 |
"does not exist in branch:"], |
|
292 |
['log', '-r1234']) |
|
|
3878.3.3
by Marius Kruger
Add tests for log -r with non-exising revno's |
293 |
|
294 |
def test_log_nonexistent_dotted_revno(self): |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
295 |
self.make_minimal_branch() |
|
4955.4.7
by Vincent Ladeuil
Some more cleanup. |
296 |
self.run_bzr_error(["bzr: ERROR: Requested revision: '123.123' " |
297 |
"does not exist in branch:"], |
|
298 |
['log', '-r123.123']) |
|
|
3734.1.1
by Vincent Ladeuil
Fix bug #248427 by adding a --change option to log. |
299 |
|
|
3878.3.2
by Marius Kruger
Add tests for log -c with non-exising revno's |
300 |
def test_log_change_nonexistent_revno(self): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
301 |
self.make_minimal_branch() |
|
4955.4.7
by Vincent Ladeuil
Some more cleanup. |
302 |
self.run_bzr_error(["bzr: ERROR: Requested revision: '1234' " |
303 |
"does not exist in branch:"], |
|
304 |
['log', '-c1234']) |
|
|
3878.3.2
by Marius Kruger
Add tests for log -c with non-exising revno's |
305 |
|
306 |
def test_log_change_nonexistent_dotted_revno(self): |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
307 |
self.make_minimal_branch() |
|
4955.4.7
by Vincent Ladeuil
Some more cleanup. |
308 |
self.run_bzr_error(["bzr: ERROR: Requested revision: '123.123' " |
309 |
"does not exist in branch:"], |
|
310 |
['log', '-c123.123']) |
|
|
3878.3.2
by Marius Kruger
Add tests for log -c with non-exising revno's |
311 |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
312 |
def test_log_change_single_revno_only(self): |
313 |
self.make_minimal_branch() |
|
|
4325.4.6
by Vincent Ladeuil
Fixed as per John's and Markus reviews. |
314 |
self.run_bzr_error(['bzr: ERROR: Option --change does not' |
315 |
' accept revision ranges'], |
|
|
3734.1.1
by Vincent Ladeuil
Fix bug #248427 by adding a --change option to log. |
316 |
['log', '--change', '2..3']) |
317 |
||
318 |
def test_log_change_incompatible_with_revision(self): |
|
|
4325.4.6
by Vincent Ladeuil
Fixed as per John's and Markus reviews. |
319 |
self.run_bzr_error(['bzr: ERROR: --revision and --change' |
320 |
' are mutually exclusive'], |
|
|
3734.1.1
by Vincent Ladeuil
Fix bug #248427 by adding a --change option to log. |
321 |
['log', '--change', '2', '--revision', '3']) |
322 |
||
|
2100.1.1
by wang
Running ``bzr log`` on nonexistent file gives an error instead of the |
323 |
def test_log_nonexistent_file(self): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
324 |
self.make_minimal_branch() |
|
2100.1.1
by wang
Running ``bzr log`` on nonexistent file gives an error instead of the |
325 |
# files that don't exist in either the basis tree or working tree
|
326 |
# should give an error
|
|
|
2552.2.3
by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests. |
327 |
out, err = self.run_bzr('log does-not-exist', retcode=3) |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
328 |
self.assertContainsRe(err, |
329 |
'Path unknown at end or start of revision range: '
|
|
330 |
'does-not-exist') |
|
|
2388.1.11
by Alexander Belchenko
changes after John's review |
331 |
|
|
4955.5.2
by Vincent Ladeuil
Simplify tests. |
332 |
def test_log_reversed_revspecs(self): |
333 |
self.make_linear_branch() |
|
334 |
self.run_bzr_error(('bzr: ERROR: Start revision must be older than ' |
|
335 |
'the end revision.\n',), |
|
336 |
['log', '-r3..1']) |
|
337 |
||
338 |
def test_log_reversed_dotted_revspecs(self): |
|
339 |
self.make_merged_branch() |
|
340 |
self.run_bzr_error(('bzr: ERROR: Start revision not found in ' |
|
341 |
'left-hand history of end revision.\n',), |
|
342 |
"log -r 1.1.1..1") |
|
343 |
||
|
4955.5.1
by Vincent Ladeuil
Split some tests to better characterize the classes |
344 |
def test_log_bad_message_re(self): |
345 |
"""Bad --message argument gives a sensible message |
|
346 |
|
|
347 |
See https://bugs.launchpad.net/bzr/+bug/251352
|
|
348 |
"""
|
|
349 |
self.make_minimal_branch() |
|
350 |
out, err = self.run_bzr(['log', '-m', '*'], retcode=3) |
|
351 |
self.assertEqual("bzr: ERROR: Invalid regular expression" |
|
352 |
" in log message filter"
|
|
353 |
": '*'"
|
|
354 |
": nothing to repeat\n", err) |
|
355 |
self.assertEqual('', out) |
|
356 |
||
|
4955.4.7
by Vincent Ladeuil
Some more cleanup. |
357 |
def test_log_unsupported_timezone(self): |
358 |
self.make_linear_branch() |
|
359 |
self.run_bzr_error(['bzr: ERROR: Unsupported timezone format "foo", ' |
|
360 |
'options are "utc", "original", "local".'], |
|
361 |
['log', '--timezone', 'foo']) |
|
|
4955.5.1
by Vincent Ladeuil
Split some tests to better characterize the classes |
362 |
|
363 |
||
364 |
class TestLogTags(TestLog): |
|
365 |
||
|
2388.1.3
by Erik Bagfors
tests for tags in log output |
366 |
def test_log_with_tags(self): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
367 |
tree = self.make_linear_branch(format='dirstate-tags') |
|
2664.14.1
by Daniel Watkins
Fixed tests.blackbox.test_log to use internals where appropriate. |
368 |
branch = tree.branch |
369 |
branch.tags.set_tag('tag1', branch.get_rev_id(1)) |
|
370 |
branch.tags.set_tag('tag1.1', branch.get_rev_id(1)) |
|
|
3842.2.5
by Vincent Ladeuil
Better fix for bug #300055. |
371 |
branch.tags.set_tag('tag3', branch.last_revision()) |
372 |
||
|
2530.3.1
by Martin Pool
Cleanup old variations on run_bzr in the test suite |
373 |
log = self.run_bzr("log -r-1")[0] |
|
2388.1.3
by Erik Bagfors
tests for tags in log output |
374 |
self.assertTrue('tags: tag3' in log) |
375 |
||
|
2530.3.1
by Martin Pool
Cleanup old variations on run_bzr in the test suite |
376 |
log = self.run_bzr("log -r1")[0] |
|
2388.1.3
by Erik Bagfors
tests for tags in log output |
377 |
# I guess that we can't know the order of tags in the output
|
378 |
# since dicts are unordered, need to check both possibilities
|
|
|
2388.1.11
by Alexander Belchenko
changes after John's review |
379 |
self.assertContainsRe(log, r'tags: (tag1, tag1\.1|tag1\.1, tag1)') |
380 |
||
|
2388.1.9
by Erik Bagfors
test for merges with tags in log |
381 |
def test_merged_log_with_tags(self): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
382 |
branch1_tree = self.make_linear_branch('branch1', |
383 |
format='dirstate-tags') |
|
|
2664.14.1
by Daniel Watkins
Fixed tests.blackbox.test_log to use internals where appropriate. |
384 |
branch1 = branch1_tree.branch |
385 |
branch2_tree = branch1_tree.bzrdir.sprout('branch2').open_workingtree() |
|
386 |
branch1_tree.commit(message='foobar', allow_pointless=True) |
|
387 |
branch1.tags.set_tag('tag1', branch1.last_revision()) |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
388 |
# tags don't propagate if we don't merge
|
389 |
self.run_bzr('merge ../branch1', working_dir='branch2') |
|
|
2664.14.1
by Daniel Watkins
Fixed tests.blackbox.test_log to use internals where appropriate. |
390 |
branch2_tree.commit(message='merge branch 1') |
|
4511.3.11
by Marius Kruger
* fix tests again |
391 |
log = self.run_bzr("log -n0 -r-1", working_dir='branch2')[0] |
|
2388.1.11
by Alexander Belchenko
changes after John's review |
392 |
self.assertContainsRe(log, r' tags: tag1') |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
393 |
log = self.run_bzr("log -n0 -r3.1.1", working_dir='branch2')[0] |
|
2466.12.2
by Kent Gibson
shift log output with only merge revisions to the left margin |
394 |
self.assertContainsRe(log, r'tags: tag1') |
|
2388.1.3
by Erik Bagfors
tests for tags in log output |
395 |
|
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
396 |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
397 |
class TestLogVerbose(TestLog): |
|
3874.1.8
by Vincent Ladeuil
Fixed as ber Robert's review. |
398 |
|
399 |
def setUp(self): |
|
400 |
super(TestLogVerbose, self).setUp() |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
401 |
self.make_minimal_branch() |
|
3874.1.8
by Vincent Ladeuil
Fixed as ber Robert's review. |
402 |
|
403 |
def assertUseShortDeltaFormat(self, cmd): |
|
404 |
log = self.run_bzr(cmd)[0] |
|
405 |
# Check that we use the short status format
|
|
|
3947.1.7
by Ian Clatworthy
tweak indenting/offsetting for --short given dotted revno lengths |
406 |
self.assertContainsRe(log, '(?m)^\s*A hello.txt$') |
407 |
self.assertNotContainsRe(log, '(?m)^\s*added:$') |
|
|
3874.1.4
by Vincent Ladeuil
Fixed as per Aarons' comment. |
408 |
|
|
3874.1.8
by Vincent Ladeuil
Fixed as ber Robert's review. |
409 |
def assertUseLongDeltaFormat(self, cmd): |
410 |
log = self.run_bzr(cmd)[0] |
|
411 |
# Check that we use the long status format
|
|
|
3947.1.7
by Ian Clatworthy
tweak indenting/offsetting for --short given dotted revno lengths |
412 |
self.assertNotContainsRe(log, '(?m)^\s*A hello.txt$') |
413 |
self.assertContainsRe(log, '(?m)^\s*added:$') |
|
|
3874.1.8
by Vincent Ladeuil
Fixed as ber Robert's review. |
414 |
|
415 |
def test_log_short_verbose(self): |
|
416 |
self.assertUseShortDeltaFormat(['log', '--short', '-v']) |
|
417 |
||
|
3874.1.4
by Vincent Ladeuil
Fixed as per Aarons' comment. |
418 |
def test_log_short_verbose_verbose(self): |
|
3874.1.8
by Vincent Ladeuil
Fixed as ber Robert's review. |
419 |
self.assertUseLongDeltaFormat(['log', '--short', '-vv']) |
|
3874.1.4
by Vincent Ladeuil
Fixed as per Aarons' comment. |
420 |
|
421 |
def test_log_long_verbose(self): |
|
|
3874.1.7
by Vincent Ladeuil
Restrict '-v' change to log --short only. |
422 |
# Check that we use the long status format, ignoring the verbosity
|
423 |
# level
|
|
|
3874.1.8
by Vincent Ladeuil
Fixed as ber Robert's review. |
424 |
self.assertUseLongDeltaFormat(['log', '--long', '-v']) |
|
3874.1.4
by Vincent Ladeuil
Fixed as per Aarons' comment. |
425 |
|
426 |
def test_log_long_verbose_verbose(self): |
|
|
3874.1.7
by Vincent Ladeuil
Restrict '-v' change to log --short only. |
427 |
# Check that we use the long status format, ignoring the verbosity
|
428 |
# level
|
|
|
3874.1.8
by Vincent Ladeuil
Fixed as ber Robert's review. |
429 |
self.assertUseLongDeltaFormat(['log', '--long', '-vv']) |
|
3874.1.1
by Vincent Ladeuil
Fix #87179 by using the short status format when the short format is used for log. |
430 |
|
|
3144.7.9
by Guillermo Gonzalez
* bzrlib.log.show_roperties don't hide handler errors |
431 |
|
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
432 |
class TestLogMerges(TestLogWithLogCatcher): |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
433 |
|
434 |
def setUp(self): |
|
435 |
super(TestLogMerges, self).setUp() |
|
436 |
self.make_branches_with_merges() |
|
437 |
||
438 |
def make_branches_with_merges(self): |
|
439 |
level0 = self.make_branch_and_tree('level0') |
|
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
440 |
self.wt_commit(level0, 'in branch level0') |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
441 |
level1 = level0.bzrdir.sprout('level1').open_workingtree() |
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
442 |
self.wt_commit(level1, 'in branch level1') |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
443 |
level2 = level1.bzrdir.sprout('level2').open_workingtree() |
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
444 |
self.wt_commit(level2, 'in branch level2') |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
445 |
level1.merge_from_branch(level2.branch) |
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
446 |
self.wt_commit(level1, 'merge branch level2') |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
447 |
level0.merge_from_branch(level1.branch) |
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
448 |
self.wt_commit(level0, 'merge branch level1') |
|
3947.1.3
by Ian Clatworthy
blackbox tests |
449 |
|
|
2466.12.1
by Kent Gibson
Fix ``bzr log -r`` to support selecting merge revisions. |
450 |
def test_merges_are_indented_by_level(self): |
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
451 |
self.run_bzr(['log', '-n0'], working_dir='level0') |
452 |
revnos_and_depth = [(r.revno, r.merge_depth) |
|
453 |
for r in self.get_captured_revisions()] |
|
454 |
self.assertEqual([('2', 0), ('1.1.2', 1), ('1.2.1', 2), ('1.1.1', 1), |
|
455 |
('1', 0)], |
|
456 |
[(r.revno, r.merge_depth) |
|
457 |
for r in self.get_captured_revisions()]) |
|
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
458 |
|
|
3947.1.3
by Ian Clatworthy
blackbox tests |
459 |
def test_force_merge_revisions_off(self): |
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
460 |
self.assertLogRevnos(['-n1'], ['2', '1'], working_dir='level0') |
|
3947.1.3
by Ian Clatworthy
blackbox tests |
461 |
|
462 |
def test_force_merge_revisions_on(self): |
|
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
463 |
self.assertLogRevnos(['-n0'], ['2', '1.1.2', '1.2.1', '1.1.1', '1'], |
464 |
working_dir='level0') |
|
|
3947.1.7
by Ian Clatworthy
tweak indenting/offsetting for --short given dotted revno lengths |
465 |
|
|
4221.2.1
by Ian Clatworthy
--include-merges as an alias for --levels 0 in log |
466 |
def test_include_merges(self): |
467 |
# Confirm --include-merges gives the same output as -n0
|
|
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
468 |
self.assertLogRevnos(['--include-merges'], |
469 |
['2', '1.1.2', '1.2.1', '1.1.1', '1'], |
|
470 |
working_dir='level0') |
|
471 |
self.assertLogRevnos(['--include-merges'], |
|
472 |
['2', '1.1.2', '1.2.1', '1.1.1', '1'], |
|
473 |
working_dir='level0') |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
474 |
out_im, err_im = self.run_bzr('log --include-merges', |
475 |
working_dir='level0') |
|
476 |
out_n0, err_n0 = self.run_bzr('log -n0', working_dir='level0') |
|
477 |
self.assertEqual('', err_im) |
|
478 |
self.assertEqual('', err_n0) |
|
|
4221.2.1
by Ian Clatworthy
--include-merges as an alias for --levels 0 in log |
479 |
self.assertEqual(out_im, out_n0) |
480 |
||
|
3947.1.7
by Ian Clatworthy
tweak indenting/offsetting for --short given dotted revno lengths |
481 |
def test_force_merge_revisions_N(self): |
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
482 |
self.assertLogRevnos(['-n2'], |
483 |
['2', '1.1.2', '1.1.1', '1'], |
|
484 |
working_dir='level0') |
|
|
3947.1.3
by Ian Clatworthy
blackbox tests |
485 |
|
|
2466.12.1
by Kent Gibson
Fix ``bzr log -r`` to support selecting merge revisions. |
486 |
def test_merges_single_merge_rev(self): |
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
487 |
self.assertLogRevnosAndDepths(['-n0', '-r1.1.2'], |
488 |
[('1.1.2', 0), ('1.2.1', 1)], |
|
489 |
working_dir='level0') |
|
|
2466.12.1
by Kent Gibson
Fix ``bzr log -r`` to support selecting merge revisions. |
490 |
|
491 |
def test_merges_partial_range(self): |
|
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
492 |
self.assertLogRevnosAndDepths( |
493 |
['-n0', '-r1.1.1..1.1.2'], |
|
494 |
[('1.1.2', 0), ('1.2.1', 1), ('1.1.1', 0)], |
|
495 |
working_dir='level0') |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
496 |
|
|
4511.3.1
by Marius Kruger
add test_merges_partial_range_ignore_before_lower_bound |
497 |
def test_merges_partial_range_ignore_before_lower_bound(self): |
|
4511.3.11
by Marius Kruger
* fix tests again |
498 |
"""Dont show revisions before the lower bound's merged revs""" |
|
4955.4.8
by Vincent Ladeuil
Simplify more tests. |
499 |
self.assertLogRevnosAndDepths( |
500 |
['-n0', '-r1.1.2..2'], |
|
501 |
[('2', 0), ('1.1.2', 1), ('1.2.1', 2)], |
|
502 |
working_dir='level0') |
|
|
4511.3.1
by Marius Kruger
add test_merges_partial_range_ignore_before_lower_bound |
503 |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
504 |
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
505 |
class TestLogDiff(TestLogWithLogCatcher): |
506 |
||
507 |
# FIXME: We need specific tests for each LogFormatter about how the diffs
|
|
508 |
# are displayed: --long indent them by depth, --short use a fixed
|
|
509 |
# indent and --line does't display them. -- vila 10019
|
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
510 |
|
511 |
def setUp(self): |
|
512 |
super(TestLogDiff, self).setUp() |
|
513 |
self.make_branch_with_diffs() |
|
514 |
||
515 |
def make_branch_with_diffs(self): |
|
516 |
level0 = self.make_branch_and_tree('level0') |
|
517 |
self.build_tree(['level0/file1', 'level0/file2']) |
|
518 |
level0.add('file1') |
|
519 |
level0.add('file2') |
|
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
520 |
self.wt_commit(level0, 'in branch level0') |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
521 |
|
522 |
level1 = level0.bzrdir.sprout('level1').open_workingtree() |
|
523 |
self.build_tree_contents([('level1/file2', 'hello\n')]) |
|
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
524 |
self.wt_commit(level1, 'in branch level1') |
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
525 |
level0.merge_from_branch(level1.branch) |
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
526 |
self.wt_commit(level0, 'merge branch level1') |
|
3943.5.3
by Ian Clatworthy
add tests |
527 |
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
528 |
def _diff_file1_revno1(self): |
529 |
return """=== added file 'file1' |
|
|
4325.4.6
by Vincent Ladeuil
Fixed as per John's and Markus reviews. |
530 |
--- file1\t1970-01-01 00:00:00 +0000 |
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
531 |
+++ file1\t2005-11-22 00:00:00 +0000 |
|
3943.5.6
by Ian Clatworthy
feedback from jam's review |
532 |
@@ -0,0 +1,1 @@
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
533 |
+contents of level0/file1
|
|
3943.5.6
by Ian Clatworthy
feedback from jam's review |
534 |
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
535 |
"""
|
536 |
||
537 |
def _diff_file2_revno2(self): |
|
538 |
return """=== modified file 'file2' |
|
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
539 |
--- file2\t2005-11-22 00:00:00 +0000 |
540 |
+++ file2\t2005-11-22 00:00:01 +0000 |
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
541 |
@@ -1,1 +1,1 @@
|
542 |
-contents of level0/file2
|
|
543 |
+hello
|
|
544 |
||
545 |
"""
|
|
546 |
||
547 |
def _diff_file2_revno1_1_1(self): |
|
548 |
return """=== modified file 'file2' |
|
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
549 |
--- file2\t2005-11-22 00:00:00 +0000 |
550 |
+++ file2\t2005-11-22 00:00:01 +0000 |
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
551 |
@@ -1,1 +1,1 @@
|
552 |
-contents of level0/file2
|
|
553 |
+hello
|
|
554 |
||
555 |
"""
|
|
556 |
||
557 |
def _diff_file2_revno1(self): |
|
558 |
return """=== added file 'file2' |
|
|
4325.4.6
by Vincent Ladeuil
Fixed as per John's and Markus reviews. |
559 |
--- file2\t1970-01-01 00:00:00 +0000 |
|
4955.4.17
by Vincent Ladeuil
Use a data factory for commit properties and reduce code duplication. |
560 |
+++ file2\t2005-11-22 00:00:00 +0000 |
|
3943.5.6
by Ian Clatworthy
feedback from jam's review |
561 |
@@ -0,0 +1,1 @@
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
562 |
+contents of level0/file2
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
563 |
|
564 |
"""
|
|
565 |
||
|
4955.4.13
by Vincent Ladeuil
Remove dead code. |
566 |
def assertLogRevnosAndDiff(self, args, expected, |
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
567 |
working_dir='.'): |
568 |
self.run_bzr(['log', '-p'] + args, working_dir=working_dir) |
|
569 |
expected_revnos_and_depths = [ |
|
570 |
(revno, depth) for revno, depth, diff in expected] |
|
571 |
# Check the revnos and depths first to make debugging easier
|
|
572 |
self.assertEqual(expected_revnos_and_depths, |
|
573 |
[(r.revno, r.merge_depth) |
|
574 |
for r in self.get_captured_revisions()]) |
|
575 |
# Now check the diffs, adding the revno in case of failure
|
|
576 |
fmt = 'In revno %s\n%s' |
|
577 |
for expected_rev, actual_rev in izip(expected, |
|
578 |
self.get_captured_revisions()): |
|
579 |
revno, depth, expected_diff = expected_rev |
|
580 |
actual_diff = actual_rev.diff |
|
581 |
self.assertEqualDiff(fmt % (revno, expected_diff), |
|
582 |
fmt % (revno, actual_diff)) |
|
583 |
||
584 |
def test_log_diff_with_merges(self): |
|
|
4955.4.13
by Vincent Ladeuil
Remove dead code. |
585 |
self.assertLogRevnosAndDiff( |
586 |
['-n0'], |
|
587 |
[('2', 0, self._diff_file2_revno2()), |
|
588 |
('1.1.1', 1, self._diff_file2_revno1_1_1()), |
|
589 |
('1', 0, self._diff_file1_revno1() |
|
590 |
+ self._diff_file2_revno1())], |
|
591 |
working_dir='level0') |
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
592 |
|
593 |
||
594 |
def test_log_diff_file1(self): |
|
|
4955.4.13
by Vincent Ladeuil
Remove dead code. |
595 |
self.assertLogRevnosAndDiff(['-n0', 'file1'], |
596 |
[('1', 0, self._diff_file1_revno1())], |
|
597 |
working_dir='level0') |
|
|
4955.4.11
by Vincent Ladeuil
Give some tests a better focus and simplify them accordingly. |
598 |
|
599 |
def test_log_diff_file2(self): |
|
|
4955.4.13
by Vincent Ladeuil
Remove dead code. |
600 |
self.assertLogRevnosAndDiff(['-n1', 'file2'], |
601 |
[('2', 0, self._diff_file2_revno2()), |
|
602 |
('1', 0, self._diff_file2_revno1())], |
|
603 |
working_dir='level0') |
|
|
4325.4.5
by Vincent Ladeuil
Some cleanup in blackbox log tests. |
604 |
|
605 |
||
606 |
class TestLogUnicodeDiff(TestLog): |
|
|
3943.5.4
by Ian Clatworthy
filter diff by file |
607 |
|
|
4110.1.1
by Alexander Belchenko
Fixed problem with `log -p` and non-ascii content of files: show_diff should write the diff to exact [stdout] stream. |
608 |
def test_log_show_diff_non_ascii(self): |
609 |
# Smoke test for bug #328007 UnicodeDecodeError on 'log -p'
|
|
610 |
message = u'Message with \xb5' |
|
611 |
body = 'Body with \xb5\n' |
|
612 |
wt = self.make_branch_and_tree('.') |
|
613 |
self.build_tree_contents([('foo', body)]) |
|
614 |
wt.add('foo') |
|
615 |
wt.commit(message=message) |
|
616 |
# check that command won't fail with unicode error
|
|
617 |
# don't care about exact output because we have other tests for this
|
|
618 |
out,err = self.run_bzr('log -p --long') |
|
619 |
self.assertNotEqual('', out) |
|
620 |
self.assertEqual('', err) |
|
621 |
out,err = self.run_bzr('log -p --short') |
|
622 |
self.assertNotEqual('', out) |
|
623 |
self.assertEqual('', err) |
|
624 |
out,err = self.run_bzr('log -p --line') |
|
625 |
self.assertNotEqual('', out) |
|
626 |
self.assertEqual('', err) |
|
627 |
||
|
3943.5.3
by Ian Clatworthy
add tests |
628 |
|
|
4325.4.3
by Vincent Ladeuil
More cleanups. |
629 |
class TestLogEncodings(tests.TestCaseInTempDir): |
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
630 |
|
631 |
_mu = u'\xb5' |
|
632 |
_message = u'Message with \xb5' |
|
633 |
||
634 |
# Encodings which can encode mu
|
|
635 |
good_encodings = [ |
|
636 |
'utf-8', |
|
637 |
'latin-1', |
|
638 |
'iso-8859-1', |
|
639 |
'cp437', # Common windows encoding |
|
|
4110.1.1
by Alexander Belchenko
Fixed problem with `log -p` and non-ascii content of files: show_diff should write the diff to exact [stdout] stream. |
640 |
'cp1251', # Russian windows encoding |
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
641 |
'cp1258', # Common windows encoding |
642 |
]
|
|
643 |
# Encodings which cannot encode mu
|
|
644 |
bad_encodings = [ |
|
645 |
'ascii', |
|
646 |
'iso-8859-2', |
|
647 |
'koi8_r', |
|
648 |
]
|
|
649 |
||
650 |
def setUp(self): |
|
|
4325.4.3
by Vincent Ladeuil
More cleanups. |
651 |
super(TestLogEncodings, self).setUp() |
|
4985.1.5
by Vincent Ladeuil
Deploying the new overrideAttr facility further reduces the complexity |
652 |
self.overrideAttr(osutils, '_cached_user_encoding') |
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
653 |
|
654 |
def create_branch(self): |
|
655 |
bzr = self.run_bzr |
|
656 |
bzr('init') |
|
|
4955.4.16
by Vincent Ladeuil
Be windows-friendly and don't left opened files behind. |
657 |
self.build_tree_contents([('a', 'some stuff\n')]) |
|
2552.2.3
by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests. |
658 |
bzr('add a') |
659 |
bzr(['commit', '-m', self._message]) |
|
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
660 |
|
661 |
def try_encoding(self, encoding, fail=False): |
|
662 |
bzr = self.run_bzr |
|
663 |
if fail: |
|
664 |
self.assertRaises(UnicodeEncodeError, |
|
665 |
self._mu.encode, encoding) |
|
666 |
encoded_msg = self._message.encode(encoding, 'replace') |
|
667 |
else: |
|
668 |
encoded_msg = self._message.encode(encoding) |
|
669 |
||
|
3224.5.4
by Andrew Bennetts
Fix test suite, mainly weeding out uses of bzrlib.user_encoding. |
670 |
old_encoding = osutils._cached_user_encoding |
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
671 |
# This test requires that 'run_bzr' uses the current
|
672 |
# bzrlib, because we override user_encoding, and expect
|
|
673 |
# it to be used
|
|
674 |
try: |
|
|
3224.5.4
by Andrew Bennetts
Fix test suite, mainly weeding out uses of bzrlib.user_encoding. |
675 |
osutils._cached_user_encoding = 'ascii' |
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
676 |
# We should be able to handle any encoding
|
677 |
out, err = bzr('log', encoding=encoding) |
|
678 |
if not fail: |
|
679 |
# Make sure we wrote mu as we expected it to exist
|
|
680 |
self.assertNotEqual(-1, out.find(encoded_msg)) |
|
681 |
out_unicode = out.decode(encoding) |
|
682 |
self.assertNotEqual(-1, out_unicode.find(self._message)) |
|
683 |
else: |
|
684 |
self.assertNotEqual(-1, out.find('Message with ?')) |
|
685 |
finally: |
|
|
3224.5.4
by Andrew Bennetts
Fix test suite, mainly weeding out uses of bzrlib.user_encoding. |
686 |
osutils._cached_user_encoding = old_encoding |
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
687 |
|
688 |
def test_log_handles_encoding(self): |
|
689 |
self.create_branch() |
|
690 |
||
691 |
for encoding in self.good_encodings: |
|
692 |
self.try_encoding(encoding) |
|
693 |
||
694 |
def test_log_handles_bad_encoding(self): |
|
695 |
self.create_branch() |
|
696 |
||
697 |
for encoding in self.bad_encodings: |
|
698 |
self.try_encoding(encoding, fail=True) |
|
699 |
||
700 |
def test_stdout_encoding(self): |
|
701 |
bzr = self.run_bzr |
|
|
3224.5.4
by Andrew Bennetts
Fix test suite, mainly weeding out uses of bzrlib.user_encoding. |
702 |
osutils._cached_user_encoding = "cp1251" |
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
703 |
|
704 |
bzr('init') |
|
705 |
self.build_tree(['a']) |
|
|
2552.2.3
by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests. |
706 |
bzr('add a') |
707 |
bzr(['commit', '-m', u'\u0422\u0435\u0441\u0442']) |
|
|
1685.1.5
by John Arbash Meinel
Merged test_log.py.moved into test_log.py |
708 |
stdout, stderr = self.run_bzr('log', encoding='cp866') |
709 |
||
710 |
message = stdout.splitlines()[-1] |
|
711 |
||
712 |
# explanation of the check:
|
|
713 |
# u'\u0422\u0435\u0441\u0442' is word 'Test' in russian
|
|
714 |
# in cp866 encoding this is string '\x92\xa5\xe1\xe2'
|
|
715 |
# in cp1251 encoding this is string '\xd2\xe5\xf1\xf2'
|
|
716 |
# This test should check that output of log command
|
|
717 |
# encoded to sys.stdout.encoding
|
|
718 |
test_in_cp866 = '\x92\xa5\xe1\xe2' |
|
719 |
test_in_cp1251 = '\xd2\xe5\xf1\xf2' |
|
720 |
# Make sure the log string is encoded in cp866
|
|
721 |
self.assertEquals(test_in_cp866, message[2:]) |
|
722 |
# Make sure the cp1251 string is not found anywhere
|
|
723 |
self.assertEquals(-1, stdout.find(test_in_cp1251)) |
|
724 |
||
|
1551.10.18
by Aaron Bentley
Log works in local treeless branches (#84247) |
725 |
|
|
4955.4.9
by Vincent Ladeuil
More simplifications. |
726 |
class TestLogFile(TestLogWithLogCatcher): |
|
1551.10.18
by Aaron Bentley
Log works in local treeless branches (#84247) |
727 |
|
728 |
def test_log_local_branch_file(self): |
|
729 |
"""We should be able to log files in local treeless branches""" |
|
730 |
tree = self.make_branch_and_tree('tree') |
|
731 |
self.build_tree(['tree/file']) |
|
732 |
tree.add('file') |
|
733 |
tree.commit('revision 1') |
|
734 |
tree.bzrdir.destroy_workingtree() |
|
|
2552.2.3
by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests. |
735 |
self.run_bzr('log tree/file') |
|
2359.1.1
by Kent Gibson
Fix ``bzr log <file>`` so it only logs the revisions that changed the file, and does it faster. |
736 |
|
|
3943.6.1
by Ian Clatworthy
find file using the end revision |
737 |
def prepare_tree(self, complex=False): |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
738 |
# The complex configuration includes deletes and renames
|
|
2359.1.1
by Kent Gibson
Fix ``bzr log <file>`` so it only logs the revisions that changed the file, and does it faster. |
739 |
tree = self.make_branch_and_tree('parent') |
740 |
self.build_tree(['parent/file1', 'parent/file2', 'parent/file3']) |
|
741 |
tree.add('file1') |
|
742 |
tree.commit('add file1') |
|
743 |
tree.add('file2') |
|
744 |
tree.commit('add file2') |
|
745 |
tree.add('file3') |
|
746 |
tree.commit('add file3') |
|
|
2664.14.1
by Daniel Watkins
Fixed tests.blackbox.test_log to use internals where appropriate. |
747 |
child_tree = tree.bzrdir.sprout('child').open_workingtree() |
748 |
self.build_tree_contents([('child/file2', 'hello')]) |
|
749 |
child_tree.commit(message='branch 1') |
|
750 |
tree.merge_from_branch(child_tree.branch) |
|
751 |
tree.commit(message='merge child branch') |
|
|
3943.6.1
by Ian Clatworthy
find file using the end revision |
752 |
if complex: |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
753 |
tree.remove('file2') |
754 |
tree.commit('remove file2') |
|
755 |
tree.rename_one('file3', 'file4') |
|
756 |
tree.commit('file3 is now called file4') |
|
|
3943.6.1
by Ian Clatworthy
find file using the end revision |
757 |
tree.remove('file1') |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
758 |
tree.commit('remove file1') |
|
2359.1.1
by Kent Gibson
Fix ``bzr log <file>`` so it only logs the revisions that changed the file, and does it faster. |
759 |
os.chdir('parent') |
|
3940.1.2
by Ian Clatworthy
add test |
760 |
|
|
4955.4.9
by Vincent Ladeuil
More simplifications. |
761 |
# FIXME: It would be good to parametrize the following tests against all
|
762 |
# formatters. But the revisions selection is not *currently* part of the
|
|
763 |
# LogFormatter contract, so using LogCatcher is sufficient -- vila 100118
|
|
764 |
def test_log_file1(self): |
|
765 |
self.prepare_tree() |
|
766 |
self.assertLogRevnos(['-n0', 'file1'], ['1']) |
|
767 |
||
768 |
def test_log_file2(self): |
|
769 |
self.prepare_tree() |
|
770 |
# file2 full history
|
|
771 |
self.assertLogRevnos(['-n0', 'file2'], ['4', '3.1.1', '2']) |
|
772 |
# file2 in a merge revision
|
|
773 |
self.assertLogRevnos(['-n0', '-r3.1.1', 'file2'], ['3.1.1']) |
|
774 |
# file2 in a mainline revision
|
|
775 |
self.assertLogRevnos(['-n0', '-r4', 'file2'], ['4', '3.1.1']) |
|
776 |
# file2 since a revision
|
|
777 |
self.assertLogRevnos(['-n0', '-r3..', 'file2'], ['4', '3.1.1']) |
|
778 |
# file2 up to a revision
|
|
779 |
self.assertLogRevnos(['-n0', '-r..3', 'file2'], ['2']) |
|
780 |
||
781 |
def test_log_file3(self): |
|
782 |
self.prepare_tree() |
|
783 |
self.assertLogRevnos(['-n0', 'file3'], ['3']) |
|
|
3940.1.2
by Ian Clatworthy
add test |
784 |
|
|
3943.6.4
by Ian Clatworthy
review feedback from vila |
785 |
def test_log_file_historical_missing(self): |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
786 |
# Check logging a deleted file gives an error if the
|
787 |
# file isn't found at the end or start of the revision range
|
|
|
3943.6.4
by Ian Clatworthy
review feedback from vila |
788 |
self.prepare_tree(complex=True) |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
789 |
err_msg = "Path unknown at end or start of revision range: file2" |
790 |
err = self.run_bzr('log file2', retcode=3)[1] |
|
|
3943.6.1
by Ian Clatworthy
find file using the end revision |
791 |
self.assertContainsRe(err, err_msg) |
792 |
||
|
3943.6.4
by Ian Clatworthy
review feedback from vila |
793 |
def test_log_file_historical_end(self): |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
794 |
# Check logging a deleted file is ok if the file existed
|
795 |
# at the end the revision range
|
|
|
3943.6.4
by Ian Clatworthy
review feedback from vila |
796 |
self.prepare_tree(complex=True) |
|
4955.4.9
by Vincent Ladeuil
More simplifications. |
797 |
self.assertLogRevnos(['-n0', '-r..4', 'file2'], ['4', '3.1.1', '2']) |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
798 |
|
|
3943.6.4
by Ian Clatworthy
review feedback from vila |
799 |
def test_log_file_historical_start(self): |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
800 |
# Check logging a deleted file is ok if the file existed
|
801 |
# at the start of the revision range
|
|
|
3943.6.4
by Ian Clatworthy
review feedback from vila |
802 |
self.prepare_tree(complex=True) |
|
4955.4.9
by Vincent Ladeuil
More simplifications. |
803 |
self.assertLogRevnos(['file1'], ['1']) |
|
3943.6.1
by Ian Clatworthy
find file using the end revision |
804 |
|
805 |
def test_log_file_renamed(self): |
|
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
806 |
"""File matched against revision range, not current tree.""" |
|
3943.6.1
by Ian Clatworthy
find file using the end revision |
807 |
self.prepare_tree(complex=True) |
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
808 |
|
|
3943.6.1
by Ian Clatworthy
find file using the end revision |
809 |
# Check logging a renamed file gives an error by default
|
|
3943.6.3
by Ian Clatworthy
search the start tree if the end tree doesn't have a file |
810 |
err_msg = "Path unknown at end or start of revision range: file3" |
|
3943.6.1
by Ian Clatworthy
find file using the end revision |
811 |
err = self.run_bzr('log file3', retcode=3)[1] |
812 |
self.assertContainsRe(err, err_msg) |
|
813 |
||
814 |
# Check we can see a renamed file if we give the right end revision
|
|
|
4955.4.9
by Vincent Ladeuil
More simplifications. |
815 |
self.assertLogRevnos(['-r..4', 'file3'], ['3']) |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
816 |
|
817 |
||
|
4955.4.10
by Vincent Ladeuil
More simplification. |
818 |
class TestLogMultiple(TestLogWithLogCatcher): |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
819 |
|
820 |
def prepare_tree(self): |
|
821 |
tree = self.make_branch_and_tree('parent') |
|
822 |
self.build_tree([ |
|
823 |
'parent/file1', |
|
824 |
'parent/file2', |
|
825 |
'parent/dir1/', |
|
826 |
'parent/dir1/file5', |
|
827 |
'parent/dir1/dir2/', |
|
828 |
'parent/dir1/dir2/file3', |
|
829 |
'parent/file4']) |
|
830 |
tree.add('file1') |
|
|
4202.2.2
by Ian Clatworthy
improve signal-to-noise ratio in tests |
831 |
tree.commit('add file1') |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
832 |
tree.add('file2') |
|
4202.2.2
by Ian Clatworthy
improve signal-to-noise ratio in tests |
833 |
tree.commit('add file2') |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
834 |
tree.add(['dir1', 'dir1/dir2', 'dir1/dir2/file3']) |
|
4202.2.2
by Ian Clatworthy
improve signal-to-noise ratio in tests |
835 |
tree.commit('add file3') |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
836 |
tree.add('file4') |
|
4202.2.2
by Ian Clatworthy
improve signal-to-noise ratio in tests |
837 |
tree.commit('add file4') |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
838 |
tree.add('dir1/file5') |
|
4202.2.2
by Ian Clatworthy
improve signal-to-noise ratio in tests |
839 |
tree.commit('add file5') |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
840 |
child_tree = tree.bzrdir.sprout('child').open_workingtree() |
841 |
self.build_tree_contents([('child/file2', 'hello')]) |
|
|
4202.2.2
by Ian Clatworthy
improve signal-to-noise ratio in tests |
842 |
child_tree.commit(message='branch 1') |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
843 |
tree.merge_from_branch(child_tree.branch) |
|
4202.2.2
by Ian Clatworthy
improve signal-to-noise ratio in tests |
844 |
tree.commit(message='merge child branch') |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
845 |
os.chdir('parent') |
846 |
||
847 |
def test_log_files(self): |
|
848 |
"""The log for multiple file should only list revs for those files""" |
|
849 |
self.prepare_tree() |
|
|
4955.4.10
by Vincent Ladeuil
More simplification. |
850 |
self.assertLogRevnos(['file1', 'file2', 'dir1/dir2/file3'], |
851 |
['6', '5.1.1', '3', '2', '1']) |
|
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
852 |
|
853 |
def test_log_directory(self): |
|
854 |
"""The log for a directory should show all nested files.""" |
|
855 |
self.prepare_tree() |
|
|
4955.4.10
by Vincent Ladeuil
More simplification. |
856 |
self.assertLogRevnos(['dir1'], ['5', '3']) |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
857 |
|
858 |
def test_log_nested_directory(self): |
|
859 |
"""The log for a directory should show all nested files.""" |
|
860 |
self.prepare_tree() |
|
|
4955.4.10
by Vincent Ladeuil
More simplification. |
861 |
self.assertLogRevnos(['dir1/dir2'], ['3']) |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
862 |
|
863 |
def test_log_in_nested_directory(self): |
|
864 |
"""The log for a directory should show all nested files.""" |
|
865 |
self.prepare_tree() |
|
866 |
os.chdir("dir1") |
|
|
4955.4.10
by Vincent Ladeuil
More simplification. |
867 |
self.assertLogRevnos(['.'], ['5', '3']) |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
868 |
|
869 |
def test_log_files_and_directories(self): |
|
870 |
"""Logging files and directories together should be fine.""" |
|
871 |
self.prepare_tree() |
|
|
4955.4.10
by Vincent Ladeuil
More simplification. |
872 |
self.assertLogRevnos(['file4', 'dir1/dir2'], ['4', '3']) |
|
4202.2.1
by Ian Clatworthy
get directory logging working again |
873 |
|
874 |
def test_log_files_and_dirs_in_nested_directory(self): |
|
875 |
"""The log for a directory should show all nested files.""" |
|
876 |
self.prepare_tree() |
|
877 |
os.chdir("dir1") |
|
|
4955.4.10
by Vincent Ladeuil
More simplification. |
878 |
self.assertLogRevnos(['dir2', 'file5'], ['5', '3']) |