1551.12.36
by Aaron Bentley
Fix failing tests |
1 |
# Copyright (C) 2007 Canonical Ltd
|
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
|
1551.12.36
by Aaron Bentley
Fix failing tests |
16 |
|
2520.7.2
by Aaron Bentley
Restore patch verification for CR, CRLF files |
17 |
import re |
1551.12.36
by Aaron Bentley
Fix failing tests |
18 |
|
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
19 |
from bzrlib import ( |
20 |
errors, |
|
1551.12.16
by Aaron Bentley
Enable signing merge directives |
21 |
gpg, |
4098.5.16
by Aaron Bentley
Move hook to MergeDirective, implement MergeDirective.compose_merge_request. |
22 |
mail_client, |
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
23 |
merge_directive, |
24 |
tests, |
|
4098.5.18
by Aaron Bentley
Gracefully handle mail clients that don't support bodies. |
25 |
trace, |
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
26 |
)
|
27 |
||
1551.12.4
by Aaron Bentley
Add failing test |
28 |
|
1551.12.45
by Aaron Bentley
Change format marker to not experimental |
29 |
OUTPUT1 = """# Bazaar merge directive format 1 |
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
30 |
# revision_id: example:
|
31 |
# target_branch: http://example.com
|
|
32 |
# testament_sha1: sha
|
|
33 |
# timestamp: 1970-01-01 00:09:33 +0002
|
|
34 |
#\x20 |
|
35 |
booga"""
|
|
36 |
||
2687.2.2
by Martin Pool
Fix up other references to 0.19 |
37 |
OUTPUT1_2 = """# Bazaar merge directive format 2 (Bazaar 0.90) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
38 |
# revision_id: example:
|
39 |
# target_branch: http://example.com
|
|
40 |
# testament_sha1: sha
|
|
41 |
# timestamp: 1970-01-01 00:09:33 +0002
|
|
2520.4.105
by Aaron Bentley
Implement patch verification |
42 |
# base_revision_id: null:
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
43 |
#\x20 |
44 |
# Begin bundle
|
|
45 |
booga"""
|
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
46 |
|
1551.12.45
by Aaron Bentley
Change format marker to not experimental |
47 |
OUTPUT2 = """# Bazaar merge directive format 1 |
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
48 |
# revision_id: example:
|
49 |
# target_branch: http://example.com
|
|
50 |
# testament_sha1: sha
|
|
51 |
# timestamp: 1970-01-01 00:09:33 +0002
|
|
52 |
# source_branch: http://example.org
|
|
53 |
# message: Hi mom!
|
|
54 |
#\x20 |
|
55 |
booga"""
|
|
56 |
||
2687.2.2
by Martin Pool
Fix up other references to 0.19 |
57 |
OUTPUT2_2 = """# Bazaar merge directive format 2 (Bazaar 0.90) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
58 |
# revision_id: example:
|
59 |
# target_branch: http://example.com
|
|
60 |
# testament_sha1: sha
|
|
61 |
# timestamp: 1970-01-01 00:09:33 +0002
|
|
62 |
# source_branch: http://example.org
|
|
63 |
# message: Hi mom!
|
|
2520.4.105
by Aaron Bentley
Implement patch verification |
64 |
# base_revision_id: null:
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
65 |
#\x20 |
66 |
# Begin patch
|
|
67 |
booga"""
|
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
68 |
|
1551.12.51
by Aaron Bentley
Allow leading junk before merge directive header |
69 |
INPUT1 = """ |
70 |
I was thinking today about creating a merge directive.
|
|
71 |
||
72 |
So I did.
|
|
73 |
||
74 |
Here it is.
|
|
75 |
||
76 |
(I've pasted it in the body of this message)
|
|
77 |
||
78 |
Aaron
|
|
79 |
||
80 |
# Bazaar merge directive format 1\r |
|
81 |
# revision_id: example:
|
|
82 |
# target_branch: http://example.com
|
|
83 |
# testament_sha1: sha
|
|
84 |
# timestamp: 1970-01-01 00:09:33 +0002
|
|
85 |
# source_branch: http://example.org
|
|
86 |
# message: Hi mom!
|
|
87 |
#\x20 |
|
88 |
booga""".splitlines(True) |
|
89 |
||
90 |
||
2520.4.73
by Aaron Bentley
Implement new merge directive format |
91 |
INPUT1_2 = """ |
92 |
I was thinking today about creating a merge directive.
|
|
93 |
||
94 |
So I did.
|
|
95 |
||
96 |
Here it is.
|
|
97 |
||
98 |
(I've pasted it in the body of this message)
|
|
99 |
||
100 |
Aaron
|
|
101 |
||
2687.2.2
by Martin Pool
Fix up other references to 0.19 |
102 |
# Bazaar merge directive format 2 (Bazaar 0.90)\r |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
103 |
# revision_id: example:
|
104 |
# target_branch: http://example.com
|
|
105 |
# testament_sha1: sha
|
|
106 |
# timestamp: 1970-01-01 00:09:33 +0002
|
|
107 |
# source_branch: http://example.org
|
|
2520.4.105
by Aaron Bentley
Implement patch verification |
108 |
# base_revision_id: null:
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
109 |
# message: Hi mom!
|
110 |
#\x20 |
|
111 |
# Begin patch
|
|
112 |
booga""".splitlines(True) |
|
113 |
||
114 |
||
2694.1.1
by Aaron Bentley
Restore support for Merge directive 2 / 0.19 |
115 |
INPUT1_2_OLD = """ |
116 |
I was thinking today about creating a merge directive.
|
|
117 |
||
118 |
So I did.
|
|
119 |
||
120 |
Here it is.
|
|
121 |
||
122 |
(I've pasted it in the body of this message)
|
|
123 |
||
124 |
Aaron
|
|
125 |
||
126 |
# Bazaar merge directive format 2 (Bazaar 0.19)\r |
|
127 |
# revision_id: example:
|
|
128 |
# target_branch: http://example.com
|
|
129 |
# testament_sha1: sha
|
|
130 |
# timestamp: 1970-01-01 00:09:33 +0002
|
|
131 |
# source_branch: http://example.org
|
|
132 |
# base_revision_id: null:
|
|
133 |
# message: Hi mom!
|
|
134 |
#\x20 |
|
135 |
# Begin patch
|
|
136 |
booga""".splitlines(True) |
|
137 |
||
2694.1.2
by Aaron Bentley
Fix whitespace |
138 |
|
2694.1.1
by Aaron Bentley
Restore support for Merge directive 2 / 0.19 |
139 |
OLD_DIRECTIVE_2 = """# Bazaar merge directive format 2 (Bazaar 0.19) |
140 |
# revision_id: abentley@panoramicfeedback.com-20070807234458-\ |
|
141 |
# nzhkoyza56lan7z5
|
|
142 |
# target_branch: http://panoramicfeedback.com/opensource/bzr/repo\ |
|
143 |
# /bzr.ab
|
|
144 |
# testament_sha1: d825a5cdb267a90ec2ba86b00895f3d8a9bed6bf
|
|
145 |
# timestamp: 2007-08-10 16:15:02 -0400
|
|
146 |
# source_branch: http://panoramicfeedback.com/opensource/bzr/repo\ |
|
147 |
# /bzr.ab
|
|
148 |
# base_revision_id: abentley@panoramicfeedback.com-20070731163346-\ |
|
149 |
# 623xwcycwij91xen
|
|
150 |
#
|
|
151 |
""".splitlines(True) |
|
152 |
||
153 |
||
2520.4.73
by Aaron Bentley
Implement new merge directive format |
154 |
class TestMergeDirective(object): |
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
155 |
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
156 |
def test_merge_source(self): |
2425.6.1
by Martin Pool
Fix formatting of timezones in bundles and merge directives. |
157 |
time = 500000.0 |
158 |
timezone = 5 * 3600 |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
159 |
self.assertRaises(errors.NoMergeSource, self.make_merge_directive, |
1551.12.3
by Aaron Bentley
Add timestamps to merge directives |
160 |
'example:', 'sha', time, timezone, 'http://example.com') |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
161 |
self.assertRaises(errors.NoMergeSource, self.make_merge_directive, |
1551.12.3
by Aaron Bentley
Add timestamps to merge directives |
162 |
'example:', 'sha', time, timezone, 'http://example.com', |
163 |
patch_type='diff') |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
164 |
self.make_merge_directive('example:', 'sha', time, timezone, |
1551.12.13
by Aaron Bentley
Rename fields |
165 |
'http://example.com', source_branch='http://example.org') |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
166 |
md = self.make_merge_directive('null:', 'sha', time, timezone, |
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
167 |
'http://example.com', patch='blah', patch_type='bundle') |
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
168 |
self.assertIs(None, md.source_branch) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
169 |
md2 = self.make_merge_directive('null:', 'sha', time, timezone, |
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
170 |
'http://example.com', patch='blah', patch_type='bundle', |
171 |
source_branch='bar') |
|
172 |
self.assertEqual('bar', md2.source_branch) |
|
173 |
||
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
174 |
def test_serialization(self): |
2425.6.1
by Martin Pool
Fix formatting of timezones in bundles and merge directives. |
175 |
time = 453 |
176 |
timezone = 120 |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
177 |
md = self.make_merge_directive('example:', 'sha', time, timezone, |
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
178 |
'http://example.com', patch='booga', patch_type='bundle') |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
179 |
self.assertEqualDiff(self.OUTPUT1, ''.join(md.to_lines())) |
180 |
md = self.make_merge_directive('example:', 'sha', time, timezone, |
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
181 |
'http://example.com', source_branch="http://example.org", |
182 |
patch='booga', patch_type='diff', message="Hi mom!") |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
183 |
self.assertEqualDiff(self.OUTPUT2, ''.join(md.to_lines())) |
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
184 |
|
1551.12.49
by Aaron Bentley
Proper error when deserializing junk |
185 |
def test_deserialize_junk(self): |
2425.6.1
by Martin Pool
Fix formatting of timezones in bundles and merge directives. |
186 |
time = 501 |
1551.12.49
by Aaron Bentley
Proper error when deserializing junk |
187 |
self.assertRaises(errors.NotAMergeDirective, |
188 |
merge_directive.MergeDirective.from_lines, 'lala') |
|
189 |
||
1551.12.59
by Aaron Bentley
Correctly handle empty merge directive texts |
190 |
def test_deserialize_empty(self): |
191 |
self.assertRaises(errors.NotAMergeDirective, |
|
192 |
merge_directive.MergeDirective.from_lines, []) |
|
193 |
||
1551.12.51
by Aaron Bentley
Allow leading junk before merge directive header |
194 |
def test_deserialize_leading_junk(self): |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
195 |
md = merge_directive.MergeDirective.from_lines(self.INPUT1) |
1551.12.51
by Aaron Bentley
Allow leading junk before merge directive header |
196 |
self.assertEqual('example:', md.revision_id) |
197 |
self.assertEqual('sha', md.testament_sha1) |
|
198 |
self.assertEqual('http://example.com', md.target_branch) |
|
199 |
self.assertEqual('http://example.org', md.source_branch) |
|
2425.6.1
by Martin Pool
Fix formatting of timezones in bundles and merge directives. |
200 |
self.assertEqual(453, md.time) |
201 |
self.assertEqual(120, md.timezone) |
|
1551.12.51
by Aaron Bentley
Allow leading junk before merge directive header |
202 |
self.assertEqual('booga', md.patch) |
203 |
self.assertEqual('diff', md.patch_type) |
|
204 |
self.assertEqual('Hi mom!', md.message) |
|
1551.12.49
by Aaron Bentley
Proper error when deserializing junk |
205 |
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
206 |
def test_roundtrip(self): |
2425.6.1
by Martin Pool
Fix formatting of timezones in bundles and merge directives. |
207 |
time = 500000 |
208 |
timezone = 7.5 * 3600 |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
209 |
md = self.make_merge_directive('example:', 'sha', time, timezone, |
1551.12.13
by Aaron Bentley
Rename fields |
210 |
'http://example.com', source_branch="http://example.org", |
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
211 |
patch='booga', patch_type='diff') |
212 |
md2 = merge_directive.MergeDirective.from_lines(md.to_lines()) |
|
1551.12.5
by Aaron Bentley
Get MergeDirective.from_objects working |
213 |
self.assertEqual('example:', md2.revision_id) |
1551.12.54
by Aaron Bentley
Decoded revision ids are utf-8 |
214 |
self.assertIsInstance(md2.revision_id, str) |
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
215 |
self.assertEqual('sha', md2.testament_sha1) |
1551.12.13
by Aaron Bentley
Rename fields |
216 |
self.assertEqual('http://example.com', md2.target_branch) |
217 |
self.assertEqual('http://example.org', md2.source_branch) |
|
1551.12.3
by Aaron Bentley
Add timestamps to merge directives |
218 |
self.assertEqual(time, md2.time) |
219 |
self.assertEqual(timezone, md2.timezone) |
|
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
220 |
self.assertEqual('diff', md2.patch_type) |
221 |
self.assertEqual('booga', md2.patch) |
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
222 |
self.assertEqual(None, md2.message) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
223 |
self.set_bundle(md, "# Bazaar revision bundle v0.9\n#\n") |
1551.12.26
by Aaron Bentley
Get email working, with optional message |
224 |
md.message = "Hi mom!" |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
225 |
lines = md.to_lines() |
226 |
md3 = merge_directive.MergeDirective.from_lines(lines) |
|
227 |
self.assertEqual("# Bazaar revision bundle v0.9\n#\n", md3.bundle) |
|
1551.12.2
by Aaron Bentley
Got directives round-tripping, with bundles and everything |
228 |
self.assertEqual("bundle", md3.patch_type) |
1551.12.12
by Aaron Bentley
Add format header |
229 |
self.assertContainsRe(md3.to_lines()[0], |
230 |
'^# Bazaar merge directive format ') |
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
231 |
self.assertEqual("Hi mom!", md3.message) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
232 |
md3.clear_payload() |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
233 |
self.assertIs(None, md3.get_raw_bundle()) |
1551.12.53
by Aaron Bentley
Fix deserialization of merge directives with no patch |
234 |
md4 = merge_directive.MergeDirective.from_lines(md3.to_lines()) |
235 |
self.assertIs(None, md4.patch_type) |
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
236 |
|
237 |
||
2520.4.73
by Aaron Bentley
Implement new merge directive format |
238 |
class TestMergeDirective1(tests.TestCase, TestMergeDirective): |
239 |
"""Test merge directive format 1"""
|
|
240 |
||
241 |
INPUT1 = INPUT1 |
|
242 |
||
243 |
OUTPUT1 = OUTPUT1 |
|
244 |
||
245 |
OUTPUT2 = OUTPUT2 |
|
246 |
||
247 |
def make_merge_directive(self, revision_id, testament_sha1, time, timezone, |
|
248 |
target_branch, patch=None, patch_type=None, |
|
249 |
source_branch=None, message=None): |
|
250 |
return merge_directive.MergeDirective(revision_id, testament_sha1, |
|
251 |
time, timezone, target_branch, patch, patch_type, |
|
252 |
source_branch, message) |
|
253 |
||
254 |
@staticmethod
|
|
255 |
def set_bundle(md, value): |
|
256 |
md.patch = value |
|
257 |
||
258 |
def test_require_patch(self): |
|
259 |
time = 500.0 |
|
260 |
timezone = 120 |
|
261 |
self.assertRaises(errors.PatchMissing, merge_directive.MergeDirective, |
|
262 |
'example:', 'sha', time, timezone, 'http://example.com', |
|
263 |
patch_type='bundle') |
|
264 |
md = merge_directive.MergeDirective('example:', 'sha1', time, timezone, |
|
265 |
'http://example.com', source_branch="http://example.org", |
|
266 |
patch='', patch_type='diff') |
|
267 |
self.assertEqual(md.patch, '') |
|
268 |
||
269 |
||
270 |
class TestMergeDirective2(tests.TestCase, TestMergeDirective): |
|
271 |
"""Test merge directive format 2"""
|
|
272 |
||
273 |
INPUT1 = INPUT1_2 |
|
274 |
||
275 |
OUTPUT1 = OUTPUT1_2 |
|
276 |
||
277 |
OUTPUT2 = OUTPUT2_2 |
|
278 |
||
279 |
def make_merge_directive(self, revision_id, testament_sha1, time, timezone, |
|
280 |
target_branch, patch=None, patch_type=None, |
|
2520.4.105
by Aaron Bentley
Implement patch verification |
281 |
source_branch=None, message=None, base_revision_id='null:'): |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
282 |
if patch_type == 'bundle': |
283 |
bundle = patch |
|
284 |
patch = None |
|
285 |
else: |
|
286 |
bundle = None |
|
287 |
return merge_directive.MergeDirective2(revision_id, testament_sha1, |
|
288 |
time, timezone, target_branch, patch, source_branch, message, |
|
2520.4.105
by Aaron Bentley
Implement patch verification |
289 |
bundle, base_revision_id) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
290 |
|
291 |
@staticmethod
|
|
292 |
def set_bundle(md, value): |
|
293 |
md.bundle = value |
|
294 |
||
295 |
||
2625.6.1
by Adeodato Simó
New EmailMessage class, façade around email.Message and MIMEMultipart. |
296 |
EMAIL1 = """From: "J. Random Hacker" <jrandom@example.com> |
1551.12.26
by Aaron Bentley
Get email working, with optional message |
297 |
Subject: Commit of rev2a
|
2625.6.1
by Adeodato Simó
New EmailMessage class, façade around email.Message and MIMEMultipart. |
298 |
To: pqm@example.com
|
299 |
User-Agent: Bazaar \(.*\)
|
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
300 |
|
1551.12.45
by Aaron Bentley
Change format marker to not experimental |
301 |
# Bazaar merge directive format 1
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
302 |
# revision_id: rev2a
|
303 |
# target_branch: (.|\n)* |
|
304 |
# testament_sha1: .*
|
|
1551.12.30
by Aaron Bentley
Use patch-style dates for timestamps in merge directives |
305 |
# timestamp: 1970-01-01 00:08:56 \\+0001 |
1551.12.26
by Aaron Bentley
Get email working, with optional message |
306 |
# source_branch: (.|\n)* |
307 |
"""
|
|
308 |
||
309 |
||
2625.6.2
by Adeodato Simó
Merge bzr.dev, resolving conflicts and updating test_merge_directive.py. |
310 |
EMAIL1_2 = """From: "J. Random Hacker" <jrandom@example.com> |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
311 |
Subject: Commit of rev2a
|
2625.6.2
by Adeodato Simó
Merge bzr.dev, resolving conflicts and updating test_merge_directive.py. |
312 |
To: pqm@example.com
|
313 |
User-Agent: Bazaar \(.*\)
|
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
314 |
|
2687.2.2
by Martin Pool
Fix up other references to 0.19 |
315 |
# Bazaar merge directive format 2 \\(Bazaar 0.90\\) |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
316 |
# revision_id: rev2a
|
317 |
# target_branch: (.|\n)* |
|
318 |
# testament_sha1: .*
|
|
319 |
# timestamp: 1970-01-01 00:08:56 \\+0001 |
|
320 |
# source_branch: (.|\n)* |
|
321 |
"""
|
|
322 |
||
323 |
||
2625.6.1
by Adeodato Simó
New EmailMessage class, façade around email.Message and MIMEMultipart. |
324 |
EMAIL2 = """From: "J. Random Hacker" <jrandom@example.com> |
1551.12.26
by Aaron Bentley
Get email working, with optional message |
325 |
Subject: Commit of rev2a with special message
|
2625.6.1
by Adeodato Simó
New EmailMessage class, façade around email.Message and MIMEMultipart. |
326 |
To: pqm@example.com
|
327 |
User-Agent: Bazaar \(.*\)
|
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
328 |
|
1551.12.45
by Aaron Bentley
Change format marker to not experimental |
329 |
# Bazaar merge directive format 1
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
330 |
# revision_id: rev2a
|
331 |
# target_branch: (.|\n)* |
|
332 |
# testament_sha1: .*
|
|
1551.12.30
by Aaron Bentley
Use patch-style dates for timestamps in merge directives |
333 |
# timestamp: 1970-01-01 00:08:56 \\+0001 |
1551.12.26
by Aaron Bentley
Get email working, with optional message |
334 |
# source_branch: (.|\n)* |
335 |
# message: Commit of rev2a with special message
|
|
336 |
"""
|
|
1551.12.4
by Aaron Bentley
Add failing test |
337 |
|
2625.6.2
by Adeodato Simó
Merge bzr.dev, resolving conflicts and updating test_merge_directive.py. |
338 |
EMAIL2_2 = """From: "J. Random Hacker" <jrandom@example.com> |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
339 |
Subject: Commit of rev2a with special message
|
2625.6.2
by Adeodato Simó
Merge bzr.dev, resolving conflicts and updating test_merge_directive.py. |
340 |
To: pqm@example.com
|
341 |
User-Agent: Bazaar \(.*\)
|
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
342 |
|
2687.2.2
by Martin Pool
Fix up other references to 0.19 |
343 |
# Bazaar merge directive format 2 \\(Bazaar 0.90\\) |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
344 |
# revision_id: rev2a
|
345 |
# target_branch: (.|\n)* |
|
346 |
# testament_sha1: .*
|
|
347 |
# timestamp: 1970-01-01 00:08:56 \\+0001 |
|
348 |
# source_branch: (.|\n)* |
|
349 |
# message: Commit of rev2a with special message
|
|
350 |
"""
|
|
1551.12.4
by Aaron Bentley
Add failing test |
351 |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
352 |
class TestMergeDirectiveBranch(object): |
1551.12.4
by Aaron Bentley
Add failing test |
353 |
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
354 |
def make_trees(self): |
1551.12.4
by Aaron Bentley
Add failing test |
355 |
tree_a = self.make_branch_and_tree('tree_a') |
1551.12.26
by Aaron Bentley
Get email working, with optional message |
356 |
tree_a.branch.get_config().set_user_option('email', |
357 |
'J. Random Hacker <jrandom@example.com>') |
|
2520.7.2
by Aaron Bentley
Restore patch verification for CR, CRLF files |
358 |
self.build_tree_contents([('tree_a/file', 'content_a\ncontent_b\n'), |
359 |
('tree_a/file_2', 'content_x\rcontent_y\r')]) |
|
360 |
tree_a.add(['file', 'file_2']) |
|
1551.12.4
by Aaron Bentley
Add failing test |
361 |
tree_a.commit('message', rev_id='rev1') |
362 |
tree_b = tree_a.bzrdir.sprout('tree_b').open_workingtree() |
|
1551.12.5
by Aaron Bentley
Get MergeDirective.from_objects working |
363 |
branch_c = tree_a.bzrdir.sprout('branch_c').open_branch() |
1551.12.4
by Aaron Bentley
Add failing test |
364 |
tree_b.commit('message', rev_id='rev2b') |
2520.7.2
by Aaron Bentley
Restore patch verification for CR, CRLF files |
365 |
self.build_tree_contents([('tree_a/file', 'content_a\ncontent_c \n'), |
366 |
('tree_a/file_2', 'content_x\rcontent_z\r')]) |
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
367 |
tree_a.commit('Commit of rev2a', rev_id='rev2a') |
368 |
return tree_a, tree_b, branch_c |
|
369 |
||
2490.2.28
by Aaron Bentley
Fix handling of null revision |
370 |
def test_empty_target(self): |
371 |
tree_a, tree_b, branch_c = self.make_trees() |
|
372 |
tree_d = self.make_branch_and_tree('tree_d') |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
373 |
md2 = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 120, |
374 |
tree_d.branch.base, patch_type='diff', |
|
375 |
public_branch=tree_a.branch.base) |
|
2490.2.28
by Aaron Bentley
Fix handling of null revision |
376 |
|
3251.2.1
by Aaron Bentley
Use nick/revno-based names for merge directives |
377 |
def test_disk_name(self): |
378 |
tree_a, tree_b, branch_c = self.make_trees() |
|
3449.4.1
by Lukáš Lalinský
Sanitize branch nick before using it as an attachment filename in ``bzr send`` |
379 |
tree_a.branch.nick = 'fancy <name>' |
3251.2.1
by Aaron Bentley
Use nick/revno-based names for merge directives |
380 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 120, |
381 |
tree_b.branch.base) |
|
382 |
self.assertEqual('fancy-name-2', md.get_disk_name(tree_a.branch)) |
|
383 |
||
384 |
def test_disk_name_old_revno(self): |
|
385 |
tree_a, tree_b, branch_c = self.make_trees() |
|
386 |
tree_a.branch.nick = 'fancy-name' |
|
387 |
md = self.from_objects(tree_a.branch.repository, 'rev1', 500, 120, |
|
388 |
tree_b.branch.base) |
|
389 |
self.assertEqual('fancy-name-1', md.get_disk_name(tree_a.branch)) |
|
390 |
||
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
391 |
def test_generate_patch(self): |
392 |
tree_a, tree_b, branch_c = self.make_trees() |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
393 |
md2 = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 120, |
394 |
tree_b.branch.base, patch_type='diff', |
|
395 |
public_branch=tree_a.branch.base) |
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
396 |
self.assertNotContainsRe(md2.patch, 'Bazaar revision bundle') |
397 |
self.assertContainsRe(md2.patch, '\\+content_c') |
|
398 |
self.assertNotContainsRe(md2.patch, '\\+\\+\\+ b/') |
|
399 |
self.assertContainsRe(md2.patch, '\\+\\+\\+ file') |
|
400 |
||
401 |
def test_public_branch(self): |
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
402 |
tree_a, tree_b, branch_c = self.make_trees() |
1551.12.5
by Aaron Bentley
Get MergeDirective.from_objects working |
403 |
self.assertRaises(errors.PublicBranchOutOfDate, |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
404 |
self.from_objects, tree_a.branch.repository, 'rev2a', 500, 144, |
405 |
tree_b.branch.base, public_branch=branch_c.base, patch_type='diff') |
|
2520.5.4
by Aaron Bentley
Replace 'bundle-revisions' with 'submit' command |
406 |
self.assertRaises(errors.PublicBranchOutOfDate, |
407 |
self.from_objects, tree_a.branch.repository, 'rev2a', 500, 144, |
|
408 |
tree_b.branch.base, public_branch=branch_c.base, patch_type=None) |
|
1551.12.34
by Aaron Bentley
Check public branch only if not using a bundle |
409 |
# public branch is not checked if patch format is bundle.
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
410 |
md1 = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 144, |
411 |
tree_b.branch.base, public_branch=branch_c.base) |
|
1551.12.34
by Aaron Bentley
Check public branch only if not using a bundle |
412 |
# public branch is provided with a bundle, despite possibly being out
|
413 |
# of date, because it's not required if a bundle is present.
|
|
414 |
self.assertEqual(md1.source_branch, branch_c.base) |
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
415 |
# Once we update the public branch, we can generate a diff.
|
1551.12.5
by Aaron Bentley
Get MergeDirective.from_objects working |
416 |
branch_c.pull(tree_a.branch) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
417 |
md3 = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 144, |
418 |
tree_b.branch.base, patch_type=None, public_branch=branch_c.base) |
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
419 |
|
1551.12.50
by Aaron Bentley
Use public location of submit branch if possible |
420 |
def test_use_public_submit_branch(self): |
421 |
tree_a, tree_b, branch_c = self.make_trees() |
|
422 |
branch_c.pull(tree_a.branch) |
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
423 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 144, |
424 |
tree_b.branch.base, patch_type=None, public_branch=branch_c.base) |
|
1551.12.50
by Aaron Bentley
Use public location of submit branch if possible |
425 |
self.assertEqual(md.target_branch, tree_b.branch.base) |
426 |
tree_b.branch.set_public_branch('http://example.com') |
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
427 |
md2 = self.from_objects( |
1551.12.50
by Aaron Bentley
Use public location of submit branch if possible |
428 |
tree_a.branch.repository, 'rev2a', 500, 144, tree_b.branch.base, |
429 |
patch_type=None, public_branch=branch_c.base) |
|
430 |
self.assertEqual(md2.target_branch, 'http://example.com') |
|
431 |
||
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
432 |
def test_message(self): |
433 |
tree_a, tree_b, branch_c = self.make_trees() |
|
2520.5.4
by Aaron Bentley
Replace 'bundle-revisions' with 'submit' command |
434 |
md3 = self.from_objects(tree_a.branch.repository, 'rev1', 500, 120, |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
435 |
tree_b.branch.base, patch_type=None, public_branch=branch_c.base, |
1551.12.33
by Aaron Bentley
Take public_branch as a string, not object |
436 |
message='Merge message') |
1551.12.7
by Aaron Bentley
Fix use of public location/branch |
437 |
md3.to_lines() |
1551.12.5
by Aaron Bentley
Get MergeDirective.from_objects working |
438 |
self.assertIs(None, md3.patch) |
1551.12.27
by Aaron Bentley
support custom message everywhere |
439 |
self.assertEqual('Merge message', md3.message) |
1551.12.16
by Aaron Bentley
Enable signing merge directives |
440 |
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
441 |
def test_generate_bundle(self): |
1551.12.40
by Aaron Bentley
Do not show prefixes in diffs |
442 |
tree_a, tree_b, branch_c = self.make_trees() |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
443 |
md1 = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 120, |
444 |
tree_b.branch.base, public_branch=branch_c.base) |
|
445 |
||
446 |
self.assertContainsRe(md1.get_raw_bundle(), 'Bazaar revision bundle') |
|
1551.12.41
by Aaron Bentley
Clean up tests, add serialization text test |
447 |
self.assertContainsRe(md1.patch, '\\+content_c') |
448 |
self.assertNotContainsRe(md1.patch, '\\+content_a') |
|
449 |
self.assertContainsRe(md1.patch, '\\+content_c') |
|
450 |
self.assertNotContainsRe(md1.patch, '\\+content_a') |
|
1551.12.40
by Aaron Bentley
Do not show prefixes in diffs |
451 |
|
1551.15.29
by Aaron Bentley
Make merge directives robust against broken bundles |
452 |
def test_broken_bundle(self): |
453 |
tree_a, tree_b, branch_c = self.make_trees() |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
454 |
md1 = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 120, |
455 |
tree_b.branch.base, public_branch=branch_c.base) |
|
1551.15.29
by Aaron Bentley
Make merge directives robust against broken bundles |
456 |
lines = md1.to_lines() |
457 |
lines = [l.replace('\n', '\r\n') for l in lines] |
|
458 |
md2 = merge_directive.MergeDirective.from_lines(lines) |
|
459 |
self.assertEqual('rev2a', md2.revision_id) |
|
460 |
||
1551.12.16
by Aaron Bentley
Enable signing merge directives |
461 |
def test_signing(self): |
2425.6.1
by Martin Pool
Fix formatting of timezones in bundles and merge directives. |
462 |
time = 453 |
463 |
timezone = 7200 |
|
1551.12.16
by Aaron Bentley
Enable signing merge directives |
464 |
class FakeBranch(object): |
465 |
def get_config(self): |
|
466 |
return self |
|
467 |
def gpg_signing_command(self): |
|
468 |
return 'loopback' |
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
469 |
md = self.make_merge_directive('example:', 'sha', time, timezone, |
1551.12.16
by Aaron Bentley
Enable signing merge directives |
470 |
'http://example.com', source_branch="http://example.org", |
471 |
patch='booga', patch_type='diff') |
|
472 |
old_strategy = gpg.GPGStrategy |
|
473 |
gpg.GPGStrategy = gpg.LoopbackGPGStrategy |
|
474 |
try: |
|
475 |
signed = md.to_signed(FakeBranch()) |
|
476 |
finally: |
|
477 |
gpg.GPGStrategy = old_strategy |
|
478 |
self.assertContainsRe(signed, '^-----BEGIN PSEUDO-SIGNED CONTENT') |
|
479 |
self.assertContainsRe(signed, 'example.org') |
|
480 |
self.assertContainsRe(signed, 'booga') |
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
481 |
|
482 |
def test_email(self): |
|
483 |
tree_a, tree_b, branch_c = self.make_trees() |
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
484 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 476, 60, |
485 |
tree_b.branch.base, patch_type=None, |
|
486 |
public_branch=tree_a.branch.base) |
|
1551.12.26
by Aaron Bentley
Get email working, with optional message |
487 |
message = md.to_email('pqm@example.com', tree_a.branch) |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
488 |
self.assertContainsRe(message.as_string(), self.EMAIL1) |
1551.12.26
by Aaron Bentley
Get email working, with optional message |
489 |
md.message = 'Commit of rev2a with special message' |
490 |
message = md.to_email('pqm@example.com', tree_a.branch) |
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
491 |
self.assertContainsRe(message.as_string(), self.EMAIL2) |
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
492 |
|
1551.14.9
by Aaron Bentley
rename get_target_revision to install_revisions |
493 |
def test_install_revisions_branch(self): |
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
494 |
tree_a, tree_b, branch_c = self.make_trees() |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
495 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 36, |
496 |
tree_b.branch.base, patch_type=None, |
|
497 |
public_branch=tree_a.branch.base) |
|
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
498 |
self.assertFalse(tree_b.branch.repository.has_revision('rev2a')) |
1551.14.9
by Aaron Bentley
rename get_target_revision to install_revisions |
499 |
revision = md.install_revisions(tree_b.branch.repository) |
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
500 |
self.assertEqual('rev2a', revision) |
501 |
self.assertTrue(tree_b.branch.repository.has_revision('rev2a')) |
|
502 |
||
2520.4.109
by Aaron Bentley
start work on directive cherry-picking |
503 |
def test_get_merge_request(self): |
2520.4.108
by Aaron Bentley
Start work on using merge base from directives |
504 |
tree_a, tree_b, branch_c = self.make_trees() |
505 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 36, |
|
2520.4.109
by Aaron Bentley
start work on directive cherry-picking |
506 |
tree_b.branch.base, patch_type='bundle', |
507 |
public_branch=tree_a.branch.base) |
|
508 |
self.assertFalse(tree_b.branch.repository.has_revision('rev2a')) |
|
509 |
md.install_revisions(tree_b.branch.repository) |
|
510 |
base, revision, verified = md.get_merge_request( |
|
511 |
tree_b.branch.repository) |
|
512 |
if isinstance(md, merge_directive.MergeDirective): |
|
513 |
self.assertIs(None, base) |
|
514 |
self.assertEqual('inapplicable', verified) |
|
515 |
else: |
|
516 |
self.assertEqual('rev1', base) |
|
517 |
self.assertEqual('verified', verified) |
|
518 |
self.assertEqual('rev2a', revision) |
|
519 |
self.assertTrue(tree_b.branch.repository.has_revision('rev2a')) |
|
520 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 36, |
|
2520.4.108
by Aaron Bentley
Start work on using merge base from directives |
521 |
tree_b.branch.base, patch_type=None, |
522 |
public_branch=tree_a.branch.base) |
|
2520.4.109
by Aaron Bentley
start work on directive cherry-picking |
523 |
base, revision, verified = md.get_merge_request( |
524 |
tree_b.branch.repository) |
|
525 |
if isinstance(md, merge_directive.MergeDirective): |
|
526 |
self.assertIs(None, base) |
|
527 |
self.assertEqual('inapplicable', verified) |
|
528 |
else: |
|
529 |
self.assertEqual('rev1', base) |
|
530 |
self.assertEqual('inapplicable', verified) |
|
531 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 36, |
|
532 |
tree_b.branch.base, patch_type='diff', |
|
533 |
public_branch=tree_a.branch.base) |
|
534 |
base, revision, verified = md.get_merge_request( |
|
2520.4.108
by Aaron Bentley
Start work on using merge base from directives |
535 |
tree_b.branch.repository) |
536 |
if isinstance(md, merge_directive.MergeDirective): |
|
537 |
self.assertIs(None, base) |
|
538 |
self.assertEqual('inapplicable', verified) |
|
539 |
else: |
|
540 |
self.assertEqual('rev1', base) |
|
541 |
self.assertEqual('verified', verified) |
|
2520.4.109
by Aaron Bentley
start work on directive cherry-picking |
542 |
md.patch='asdf' |
543 |
base, revision, verified = md.get_merge_request( |
|
544 |
tree_b.branch.repository) |
|
545 |
if isinstance(md, merge_directive.MergeDirective): |
|
546 |
self.assertIs(None, base) |
|
547 |
self.assertEqual('inapplicable', verified) |
|
548 |
else: |
|
549 |
self.assertEqual('rev1', base) |
|
2520.7.1
by Aaron Bentley
Reactivate patch verification |
550 |
self.assertEqual('failed', verified) |
2520.4.108
by Aaron Bentley
Start work on using merge base from directives |
551 |
|
1551.14.9
by Aaron Bentley
rename get_target_revision to install_revisions |
552 |
def test_install_revisions_bundle(self): |
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
553 |
tree_a, tree_b, branch_c = self.make_trees() |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
554 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 36, |
555 |
tree_b.branch.base, patch_type='bundle', |
|
556 |
public_branch=tree_a.branch.base) |
|
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
557 |
self.assertFalse(tree_b.branch.repository.has_revision('rev2a')) |
1551.14.9
by Aaron Bentley
rename get_target_revision to install_revisions |
558 |
revision = md.install_revisions(tree_b.branch.repository) |
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
559 |
self.assertEqual('rev2a', revision) |
560 |
self.assertTrue(tree_b.branch.repository.has_revision('rev2a')) |
|
561 |
||
562 |
def test_get_target_revision_nofetch(self): |
|
563 |
tree_a, tree_b, branch_c = self.make_trees() |
|
564 |
tree_b.branch.fetch(tree_a.branch) |
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
565 |
md = self.from_objects( tree_a.branch.repository, 'rev2a', 500, 36, |
566 |
tree_b.branch.base, patch_type=None, |
|
567 |
public_branch=tree_a.branch.base) |
|
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
568 |
md.source_branch = '/dev/null' |
1551.14.9
by Aaron Bentley
rename get_target_revision to install_revisions |
569 |
revision = md.install_revisions(tree_b.branch.repository) |
1551.14.4
by Aaron Bentley
Change bundle reader and merge directive to both be 'mergeables' |
570 |
self.assertEqual('rev2a', revision) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
571 |
|
1551.19.19
by Aaron Bentley
Merge directives can now fetch prerequisites from the target branch |
572 |
def test_use_submit_for_missing_dependency(self): |
573 |
tree_a, tree_b, branch_c = self.make_trees() |
|
574 |
branch_c.pull(tree_a.branch) |
|
575 |
self.build_tree_contents([('tree_a/file', 'content_q\ncontent_r\n')]) |
|
576 |
tree_a.commit('rev3a', rev_id='rev3a') |
|
577 |
md = self.from_objects(tree_a.branch.repository, 'rev3a', 500, 36, |
|
578 |
branch_c.base, base_revision_id='rev2a') |
|
579 |
revision = md.install_revisions(tree_b.branch.repository) |
|
580 |
||
3535.8.1
by James Westby
Handle something that isn't a branch being specified in target_branch. |
581 |
def test_handle_target_not_a_branch(self): |
582 |
tree_a, tree_b, branch_c = self.make_trees() |
|
583 |
branch_c.pull(tree_a.branch) |
|
584 |
self.build_tree_contents([('tree_a/file', 'content_q\ncontent_r\n')]) |
|
585 |
tree_a.commit('rev3a', rev_id='rev3a') |
|
586 |
md = self.from_objects(tree_a.branch.repository, 'rev3a', 500, 36, |
|
587 |
branch_c.base, base_revision_id='rev2a') |
|
3535.8.2
by James Westby
Incorporate spiv's feedback. |
588 |
md.target_branch = self.get_url('not-a-branch') |
3535.8.1
by James Westby
Handle something that isn't a branch being specified in target_branch. |
589 |
self.assertRaises(errors.TargetNotBranch, md.install_revisions, |
590 |
tree_b.branch.repository) |
|
591 |
||
2520.4.73
by Aaron Bentley
Implement new merge directive format |
592 |
|
593 |
class TestMergeDirective1Branch(tests.TestCaseWithTransport, |
|
594 |
TestMergeDirectiveBranch): |
|
595 |
"""Test merge directive format 1 with a branch"""
|
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
596 |
|
597 |
EMAIL1 = EMAIL1 |
|
598 |
||
599 |
EMAIL2 = EMAIL2 |
|
600 |
||
2520.4.73
by Aaron Bentley
Implement new merge directive format |
601 |
def from_objects(self, repository, revision_id, time, timezone, |
602 |
target_branch, patch_type='bundle', local_target_branch=None, |
|
1551.19.19
by Aaron Bentley
Merge directives can now fetch prerequisites from the target branch |
603 |
public_branch=None, message=None, base_revision_id=None): |
604 |
if base_revision_id is not None: |
|
605 |
raise tests.TestNotApplicable('This format does not support' |
|
606 |
' explicit bases.') |
|
3010.1.9
by Robert Collins
test_merge_directive locking correctness. |
607 |
repository.lock_write() |
608 |
try: |
|
609 |
return merge_directive.MergeDirective.from_objects( repository, |
|
610 |
revision_id, time, timezone, target_branch, patch_type, |
|
611 |
local_target_branch, public_branch, message) |
|
612 |
finally: |
|
613 |
repository.unlock() |
|
2520.4.73
by Aaron Bentley
Implement new merge directive format |
614 |
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
615 |
def make_merge_directive(self, revision_id, testament_sha1, time, timezone, |
616 |
target_branch, patch=None, patch_type=None, |
|
617 |
source_branch=None, message=None): |
|
618 |
return merge_directive.MergeDirective(revision_id, testament_sha1, |
|
619 |
time, timezone, target_branch, patch, patch_type, |
|
620 |
source_branch, message) |
|
621 |
||
2520.4.73
by Aaron Bentley
Implement new merge directive format |
622 |
|
623 |
class TestMergeDirective2Branch(tests.TestCaseWithTransport, |
|
624 |
TestMergeDirectiveBranch): |
|
625 |
"""Test merge directive format 2 with a branch"""
|
|
626 |
||
2520.4.80
by Aaron Bentley
Improve merge directive tests |
627 |
EMAIL1 = EMAIL1_2 |
628 |
||
629 |
EMAIL2 = EMAIL2_2 |
|
630 |
||
2520.4.73
by Aaron Bentley
Implement new merge directive format |
631 |
def from_objects(self, repository, revision_id, time, timezone, |
632 |
target_branch, patch_type='bundle', local_target_branch=None, |
|
2520.4.112
by Aaron Bentley
Make cherry-pick merge directives possible |
633 |
public_branch=None, message=None, base_revision_id=None): |
2520.5.4
by Aaron Bentley
Replace 'bundle-revisions' with 'submit' command |
634 |
include_patch = (patch_type in ('bundle', 'diff')) |
635 |
include_bundle = (patch_type == 'bundle') |
|
3376.2.4
by Martin Pool
Remove every assert statement from bzrlib! |
636 |
self.assertTrue(patch_type in ('bundle', 'diff', None)) |
2520.4.73
by Aaron Bentley
Implement new merge directive format |
637 |
return merge_directive.MergeDirective2.from_objects( |
638 |
repository, revision_id, time, timezone, target_branch, |
|
2520.5.4
by Aaron Bentley
Replace 'bundle-revisions' with 'submit' command |
639 |
include_patch, include_bundle, local_target_branch, public_branch, |
640 |
message, base_revision_id) |
|
2520.4.80
by Aaron Bentley
Improve merge directive tests |
641 |
|
642 |
def make_merge_directive(self, revision_id, testament_sha1, time, timezone, |
|
643 |
target_branch, patch=None, patch_type=None, |
|
2520.4.105
by Aaron Bentley
Implement patch verification |
644 |
source_branch=None, message=None, base_revision_id='null:'): |
2520.4.80
by Aaron Bentley
Improve merge directive tests |
645 |
if patch_type == 'bundle': |
646 |
bundle = patch |
|
647 |
patch = None |
|
648 |
else: |
|
649 |
bundle = None |
|
650 |
return merge_directive.MergeDirective2(revision_id, testament_sha1, |
|
651 |
time, timezone, target_branch, patch, source_branch, message, |
|
2520.4.105
by Aaron Bentley
Implement patch verification |
652 |
bundle, base_revision_id) |
653 |
||
654 |
def test_base_revision(self): |
|
655 |
tree_a, tree_b, branch_c = self.make_trees() |
|
656 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 60, |
|
657 |
tree_b.branch.base, patch_type='bundle', |
|
2520.4.112
by Aaron Bentley
Make cherry-pick merge directives possible |
658 |
public_branch=tree_a.branch.base, base_revision_id=None) |
2520.4.105
by Aaron Bentley
Implement patch verification |
659 |
self.assertEqual('rev1', md.base_revision_id) |
2520.4.112
by Aaron Bentley
Make cherry-pick merge directives possible |
660 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 60, |
661 |
tree_b.branch.base, patch_type='bundle', |
|
662 |
public_branch=tree_a.branch.base, base_revision_id='null:') |
|
663 |
self.assertEqual('null:', md.base_revision_id) |
|
2520.4.105
by Aaron Bentley
Implement patch verification |
664 |
lines = md.to_lines() |
665 |
md2 = merge_directive.MergeDirective.from_lines(lines) |
|
666 |
self.assertEqual(md2.base_revision_id, md.base_revision_id) |
|
667 |
||
668 |
def test_patch_verification(self): |
|
669 |
tree_a, tree_b, branch_c = self.make_trees() |
|
670 |
md = self.from_objects(tree_a.branch.repository, 'rev2a', 500, 60, |
|
671 |
tree_b.branch.base, patch_type='bundle', |
|
672 |
public_branch=tree_a.branch.base) |
|
673 |
lines = md.to_lines() |
|
674 |
md2 = merge_directive.MergeDirective.from_lines(lines) |
|
675 |
md2._verify_patch(tree_a.branch.repository) |
|
2520.7.2
by Aaron Bentley
Restore patch verification for CR, CRLF files |
676 |
# Strip trailing whitespace
|
2520.4.105
by Aaron Bentley
Implement patch verification |
677 |
md2.patch = md2.patch.replace(' \n', '\n') |
678 |
md2._verify_patch(tree_a.branch.repository) |
|
679 |
# Convert to Mac line-endings
|
|
2520.7.2
by Aaron Bentley
Restore patch verification for CR, CRLF files |
680 |
md2.patch = re.sub('(\r\n|\r|\n)', '\r', md2.patch) |
2520.4.108
by Aaron Bentley
Start work on using merge base from directives |
681 |
self.assertTrue(md2._verify_patch(tree_a.branch.repository)) |
2520.4.105
by Aaron Bentley
Implement patch verification |
682 |
# Convert to DOS line-endings
|
2520.7.2
by Aaron Bentley
Restore patch verification for CR, CRLF files |
683 |
md2.patch = re.sub('(\r\n|\r|\n)', '\r\n', md2.patch) |
2520.4.108
by Aaron Bentley
Start work on using merge base from directives |
684 |
self.assertTrue(md2._verify_patch(tree_a.branch.repository)) |
2520.4.105
by Aaron Bentley
Implement patch verification |
685 |
md2.patch = md2.patch.replace('content_c', 'content_d') |
2520.4.108
by Aaron Bentley
Start work on using merge base from directives |
686 |
self.assertFalse(md2._verify_patch(tree_a.branch.repository)) |
2694.1.1
by Aaron Bentley
Restore support for Merge directive 2 / 0.19 |
687 |
|
688 |
||
689 |
class TestParseOldMergeDirective2(tests.TestCase): |
|
690 |
||
691 |
def test_parse_old_merge_directive(self): |
|
692 |
md = merge_directive.MergeDirective.from_lines(INPUT1_2_OLD) |
|
693 |
self.assertEqual('example:', md.revision_id) |
|
694 |
self.assertEqual('sha', md.testament_sha1) |
|
695 |
self.assertEqual('http://example.com', md.target_branch) |
|
696 |
self.assertEqual('http://example.org', md.source_branch) |
|
697 |
self.assertEqual(453, md.time) |
|
698 |
self.assertEqual(120, md.timezone) |
|
699 |
self.assertEqual('booga', md.patch) |
|
700 |
self.assertEqual('diff', md.patch_type) |
|
701 |
self.assertEqual('Hi mom!', md.message) |
|
4098.5.16
by Aaron Bentley
Move hook to MergeDirective, implement MergeDirective.compose_merge_request. |
702 |
|
703 |
||
704 |
class TestHook(object): |
|
705 |
"""Hook callback for test purposes."""
|
|
706 |
||
707 |
def __init__(self, result=None): |
|
708 |
self.calls = [] |
|
709 |
self.result = result |
|
710 |
||
711 |
def __call__(self, params): |
|
712 |
self.calls.append(params) |
|
713 |
return self.result |
|
714 |
||
715 |
||
716 |
class HookMailClient(mail_client.MailClient): |
|
717 |
"""Mail client for testing hooks."""
|
|
718 |
||
719 |
def __init__(self, config): |
|
720 |
self.body = None |
|
721 |
self.config = config |
|
722 |
||
723 |
def compose(self, prompt, to, subject, attachment, mime_subtype, |
|
724 |
extension, basename=None, body=None): |
|
725 |
self.body = body |
|
726 |
||
727 |
||
728 |
class TestBodyHook(tests.TestCaseWithTransport): |
|
729 |
||
4098.5.18
by Aaron Bentley
Gracefully handle mail clients that don't support bodies. |
730 |
def compose_with_hooks(self, test_hooks, supports_body=True): |
4098.5.16
by Aaron Bentley
Move hook to MergeDirective, implement MergeDirective.compose_merge_request. |
731 |
client = HookMailClient({}) |
4098.5.18
by Aaron Bentley
Gracefully handle mail clients that don't support bodies. |
732 |
client.supports_body = supports_body |
4098.5.16
by Aaron Bentley
Move hook to MergeDirective, implement MergeDirective.compose_merge_request. |
733 |
for test_hook in test_hooks: |
734 |
merge_directive.MergeDirective.hooks.install_named_hook( |
|
735 |
'merge_request_body', test_hook, 'test') |
|
736 |
tree = self.make_branch_and_tree('foo') |
|
737 |
tree.commit('foo') |
|
738 |
directive = merge_directive.MergeDirective2( |
|
739 |
tree.branch.last_revision(), 'sha', 0, 0, 'sha', |
|
740 |
source_branch=tree.branch.base, |
|
741 |
base_revision_id=tree.branch.last_revision(), |
|
742 |
message='This code rox') |
|
743 |
directive.compose_merge_request(client, 'jrandom@example.com', |
|
744 |
None, tree.branch) |
|
745 |
return client, directive |
|
746 |
||
4098.5.18
by Aaron Bentley
Gracefully handle mail clients that don't support bodies. |
747 |
def test_no_supports_body(self): |
748 |
test_hook = TestHook('foo') |
|
749 |
old_warn = trace.warning |
|
750 |
warnings = [] |
|
751 |
def warn(*args): |
|
752 |
warnings.append(args) |
|
753 |
trace.warning = warn |
|
754 |
try: |
|
755 |
client, directive = self.compose_with_hooks([test_hook], |
|
756 |
supports_body=False) |
|
757 |
finally: |
|
758 |
trace.warning = old_warn |
|
759 |
self.assertEqual(0, len(test_hook.calls)) |
|
760 |
self.assertEqual(('Cannot run merge_request_body hooks because mail' |
|
761 |
' client %s does not support message bodies.', |
|
762 |
'HookMailClient'), warnings[0]) |
|
763 |
||
4098.5.16
by Aaron Bentley
Move hook to MergeDirective, implement MergeDirective.compose_merge_request. |
764 |
def test_body_hook(self): |
765 |
test_hook = TestHook('foo') |
|
766 |
client, directive = self.compose_with_hooks([test_hook]) |
|
767 |
self.assertEqual(1, len(test_hook.calls)) |
|
768 |
self.assertEqual('foo', client.body) |
|
769 |
params = test_hook.calls[0] |
|
770 |
self.assertIsInstance(params, |
|
771 |
merge_directive.MergeRequestBodyParams) |
|
772 |
self.assertIs(None, params.body) |
|
773 |
self.assertIs(None, params.orig_body) |
|
774 |
self.assertEqual('jrandom@example.com', params.to) |
|
775 |
self.assertEqual('[MERGE] This code rox', params.subject) |
|
776 |
self.assertEqual(directive, params.directive) |
|
777 |
self.assertEqual('foo-1', params.basename) |
|
778 |
||
779 |
def test_body_hook_chaining(self): |
|
780 |
test_hook1 = TestHook('foo') |
|
781 |
test_hook2 = TestHook('bar') |
|
782 |
client = self.compose_with_hooks([test_hook1, test_hook2])[0] |
|
783 |
self.assertEqual(None, test_hook1.calls[0].body) |
|
784 |
self.assertEqual(None, test_hook1.calls[0].orig_body) |
|
785 |
self.assertEqual('foo', test_hook2.calls[0].body) |
|
786 |
self.assertEqual(None, test_hook2.calls[0].orig_body) |
|
787 |
self.assertEqual('bar', client.body) |