/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to breezy/tests/test_bundle.py

  • Committer: Jelmer Vernooij
  • Date: 2018-08-23 01:36:15 UTC
  • mto: This revision was merged to the branch mainline in revision 7089.
  • Revision ID: jelmer@jelmer.uk-20180823013615-ndckfn0vqyvjmgqi
Fix some stream tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005-2013, 2016 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
 
from cStringIO import StringIO
 
17
import bz2
 
18
from io import BytesIO
18
19
import os
19
 
import socket
 
20
try:
 
21
    import socketserver
 
22
except ImportError:
 
23
    import SocketServer as socketserver
20
24
import sys
21
 
import threading
22
25
 
23
 
from bzrlib import (
24
 
    bzrdir,
 
26
from .. import (
25
27
    diff,
26
28
    errors,
27
 
    inventory,
28
29
    merge,
29
30
    osutils,
30
 
    repository,
31
31
    revision as _mod_revision,
32
32
    tests,
33
33
    treebuilder,
34
34
    )
35
 
from bzrlib.bundle import read_mergeable_from_url
36
 
from bzrlib.bundle.apply_bundle import install_bundle, merge_bundle
37
 
from bzrlib.bundle.bundle_data import BundleTree
38
 
from bzrlib.bzrdir import BzrDir
39
 
from bzrlib.directory_service import directories
40
 
from bzrlib.bundle.serializer import write_bundle, read_bundle, v09, v4
41
 
from bzrlib.bundle.serializer.v08 import BundleSerializerV08
42
 
from bzrlib.bundle.serializer.v09 import BundleSerializerV09
43
 
from bzrlib.bundle.serializer.v4 import BundleSerializerV4
44
 
from bzrlib.branch import Branch
45
 
from bzrlib.repofmt import knitrepo
46
 
from bzrlib.tests import (
 
35
from ..bzr import (
 
36
    bzrdir,
 
37
    inventory,
 
38
    )
 
39
from ..bundle import read_mergeable_from_url
 
40
from ..bundle.apply_bundle import install_bundle, merge_bundle
 
41
from ..bundle.bundle_data import BundleTree
 
42
from ..directory_service import directories
 
43
from ..bundle.serializer import write_bundle, read_bundle, v09, v4
 
44
from ..bundle.serializer.v08 import BundleSerializerV08
 
45
from ..bundle.serializer.v09 import BundleSerializerV09
 
46
from ..bundle.serializer.v4 import BundleSerializerV4
 
47
from ..bzr import knitrepo
 
48
from . import (
 
49
    features,
 
50
    test_commit,
47
51
    test_read_bundle,
48
 
    test_commit,
 
52
    test_server,
49
53
    )
50
 
from bzrlib.transform import TreeTransform
 
54
from ..transform import TreeTransform
51
55
 
52
56
 
53
57
def get_text(vf, key):
54
58
    """Get the fulltext for a given revision id that is present in the vf"""
55
59
    stream = vf.get_record_stream([key], 'unordered', True)
56
 
    record = stream.next()
 
60
    record = next(stream)
57
61
    return record.get_bytes_as('fulltext')
58
62
 
59
63
 
67
71
 
68
72
 
69
73
class MockTree(object):
 
74
 
70
75
    def __init__(self):
71
 
        from bzrlib.inventory import InventoryDirectory, ROOT_ID
 
76
        from ..bzr.inventory import InventoryDirectory, ROOT_ID
72
77
        object.__init__(self)
73
78
        self.paths = {ROOT_ID: ""}
74
79
        self.ids = {"": ROOT_ID}
76
81
        self.root = InventoryDirectory(ROOT_ID, '', None)
77
82
 
78
83
    inventory = property(lambda x:x)
79
 
 
80
 
    def __iter__(self):
81
 
        return self.paths.iterkeys()
 
84
    root_inventory = property(lambda x:x)
 
85
 
 
86
    def get_root_id(self):
 
87
        return self.root.file_id
 
88
 
 
89
    def all_file_ids(self):
 
90
        return set(self.paths.keys())
 
91
 
 
92
    def all_versioned_paths(self):
 
93
        return set(self.paths.values())
 
94
 
 
95
    def is_executable(self, path, file_id):
 
96
        # Not all the files are executable.
 
97
        return False
82
98
 
83
99
    def __getitem__(self, file_id):
84
100
        if file_id == self.root.file_id:
93
109
        return self.ids[parent_dir]
94
110
 
95
111
    def iter_entries(self):
96
 
        for path, file_id in self.ids.iteritems():
 
112
        for path, file_id in self.ids.items():
97
113
            yield path, self[file_id]
98
114
 
99
 
    def get_file_kind(self, file_id):
 
115
    def kind(self, path, file_id=None):
 
116
        if file_id is None:
 
117
            file_id = self.path2id(path)
100
118
        if file_id in self.contents:
101
119
            kind = 'file'
102
120
        else:
104
122
        return kind
105
123
 
106
124
    def make_entry(self, file_id, path):
107
 
        from bzrlib.inventory import (InventoryEntry, InventoryFile
108
 
                                    , InventoryDirectory, InventoryLink)
 
125
        from ..bzr.inventory import (InventoryFile, InventoryDirectory,
 
126
            InventoryLink)
 
127
        if not isinstance(file_id, bytes):
 
128
            raise TypeError(file_id)
109
129
        name = os.path.basename(path)
110
 
        kind = self.get_file_kind(file_id)
 
130
        kind = self.kind(path, file_id)
111
131
        parent_id = self.parent_id(file_id)
112
 
        text_sha_1, text_size = self.contents_stats(file_id)
 
132
        text_sha_1, text_size = self.contents_stats(path, file_id)
113
133
        if kind == 'directory':
114
134
            ie = InventoryDirectory(file_id, name, parent_id)
115
135
        elif kind == 'file':
116
136
            ie = InventoryFile(file_id, name, parent_id)
 
137
            ie.text_sha1 = text_sha_1
 
138
            ie.text_size = text_size
117
139
        elif kind == 'symlink':
118
140
            ie = InventoryLink(file_id, name, parent_id)
119
141
        else:
120
142
            raise errors.BzrError('unknown kind %r' % kind)
121
 
        ie.text_sha1 = text_sha_1
122
 
        ie.text_size = text_size
123
143
        return ie
124
144
 
125
145
    def add_dir(self, file_id, path):
 
146
        if not isinstance(file_id, bytes):
 
147
            raise TypeError(file_id)
126
148
        self.paths[file_id] = path
127
149
        self.ids[path] = file_id
128
150
 
129
151
    def add_file(self, file_id, path, contents):
 
152
        if not isinstance(file_id, bytes):
 
153
            raise TypeError(file_id)
130
154
        self.add_dir(file_id, path)
131
155
        self.contents[file_id] = contents
132
156
 
139
163
    def has_id(self, file_id):
140
164
        return self.id2path(file_id) is not None
141
165
 
142
 
    def get_file(self, file_id):
143
 
        result = StringIO()
144
 
        result.write(self.contents[file_id])
145
 
        result.seek(0,0)
 
166
    def get_file(self, path, file_id=None):
 
167
        if file_id is None:
 
168
            file_id = self.path2id(path)
 
169
        result = BytesIO()
 
170
        try:
 
171
            result.write(self.contents[file_id])
 
172
        except KeyError:
 
173
            raise errors.NoSuchFile(path)
 
174
        result.seek(0, 0)
146
175
        return result
147
176
 
148
 
    def contents_stats(self, file_id):
 
177
    def get_file_revision(self, path, file_id=None):
 
178
        if file_id is None:
 
179
            file_id = self.path2id(path)
 
180
        return self.inventory[file_id].revision
 
181
 
 
182
    def get_file_size(self, path, file_id=None):
 
183
        if file_id is None:
 
184
            file_id = self.path2id(path)
 
185
        return self.inventory[file_id].text_size
 
186
 
 
187
    def get_file_sha1(self, path, file_id=None):
 
188
        if file_id is None:
 
189
            file_id = self.path2id(path)
 
190
        return self.inventory[file_id].text_sha1
 
191
 
 
192
    def contents_stats(self, path, file_id):
149
193
        if file_id not in self.contents:
150
194
            return None, None
151
 
        text_sha1 = osutils.sha_file(self.get_file(file_id))
 
195
        text_sha1 = osutils.sha_file(self.get_file(path, file_id))
152
196
        return text_sha1, len(self.contents[file_id])
153
197
 
154
198
 
157
201
 
158
202
    def make_tree_1(self):
159
203
        mtree = MockTree()
160
 
        mtree.add_dir("a", "grandparent")
161
 
        mtree.add_dir("b", "grandparent/parent")
162
 
        mtree.add_file("c", "grandparent/parent/file", "Hello\n")
163
 
        mtree.add_dir("d", "grandparent/alt_parent")
 
204
        mtree.add_dir(b"a", "grandparent")
 
205
        mtree.add_dir(b"b", "grandparent/parent")
 
206
        mtree.add_file(b"c", "grandparent/parent/file", b"Hello\n")
 
207
        mtree.add_dir(b"d", "grandparent/alt_parent")
164
208
        return BundleTree(mtree, ''), mtree
165
209
 
166
210
    def test_renames(self):
172
216
        self.assertEqual(btree.old_path("grandparent/parent/file"),
173
217
                         "grandparent/parent/file")
174
218
 
175
 
        self.assertEqual(btree.id2path("a"), "grandparent")
176
 
        self.assertEqual(btree.id2path("b"), "grandparent/parent")
177
 
        self.assertEqual(btree.id2path("c"), "grandparent/parent/file")
178
 
 
179
 
        self.assertEqual(btree.path2id("grandparent"), "a")
180
 
        self.assertEqual(btree.path2id("grandparent/parent"), "b")
181
 
        self.assertEqual(btree.path2id("grandparent/parent/file"), "c")
182
 
 
183
 
        self.assertTrue(btree.path2id("grandparent2") is None)
184
 
        self.assertTrue(btree.path2id("grandparent2/parent") is None)
185
 
        self.assertTrue(btree.path2id("grandparent2/parent/file") is None)
 
219
        self.assertEqual(btree.id2path(b"a"), "grandparent")
 
220
        self.assertEqual(btree.id2path(b"b"), "grandparent/parent")
 
221
        self.assertEqual(btree.id2path(b"c"), "grandparent/parent/file")
 
222
 
 
223
        self.assertEqual(btree.path2id("grandparent"), b"a")
 
224
        self.assertEqual(btree.path2id("grandparent/parent"), b"b")
 
225
        self.assertEqual(btree.path2id("grandparent/parent/file"), b"c")
 
226
 
 
227
        self.assertIs(btree.path2id("grandparent2"), None)
 
228
        self.assertIs(btree.path2id("grandparent2/parent"), None)
 
229
        self.assertIs(btree.path2id("grandparent2/parent/file"), None)
186
230
 
187
231
        btree.note_rename("grandparent", "grandparent2")
188
 
        self.assertTrue(btree.old_path("grandparent") is None)
189
 
        self.assertTrue(btree.old_path("grandparent/parent") is None)
190
 
        self.assertTrue(btree.old_path("grandparent/parent/file") is None)
191
 
 
192
 
        self.assertEqual(btree.id2path("a"), "grandparent2")
193
 
        self.assertEqual(btree.id2path("b"), "grandparent2/parent")
194
 
        self.assertEqual(btree.id2path("c"), "grandparent2/parent/file")
195
 
 
196
 
        self.assertEqual(btree.path2id("grandparent2"), "a")
197
 
        self.assertEqual(btree.path2id("grandparent2/parent"), "b")
198
 
        self.assertEqual(btree.path2id("grandparent2/parent/file"), "c")
 
232
        self.assertIs(btree.old_path("grandparent"), None)
 
233
        self.assertIs(btree.old_path("grandparent/parent"), None)
 
234
        self.assertIs(btree.old_path("grandparent/parent/file"), None)
 
235
 
 
236
        self.assertEqual(btree.id2path(b"a"), "grandparent2")
 
237
        self.assertEqual(btree.id2path(b"b"), "grandparent2/parent")
 
238
        self.assertEqual(btree.id2path(b"c"), "grandparent2/parent/file")
 
239
 
 
240
        self.assertEqual(btree.path2id("grandparent2"), b"a")
 
241
        self.assertEqual(btree.path2id("grandparent2/parent"), b"b")
 
242
        self.assertEqual(btree.path2id("grandparent2/parent/file"), b"c")
199
243
 
200
244
        self.assertTrue(btree.path2id("grandparent") is None)
201
245
        self.assertTrue(btree.path2id("grandparent/parent") is None)
202
246
        self.assertTrue(btree.path2id("grandparent/parent/file") is None)
203
247
 
204
248
        btree.note_rename("grandparent/parent", "grandparent2/parent2")
205
 
        self.assertEqual(btree.id2path("a"), "grandparent2")
206
 
        self.assertEqual(btree.id2path("b"), "grandparent2/parent2")
207
 
        self.assertEqual(btree.id2path("c"), "grandparent2/parent2/file")
 
249
        self.assertEqual(btree.id2path(b"a"), "grandparent2")
 
250
        self.assertEqual(btree.id2path(b"b"), "grandparent2/parent2")
 
251
        self.assertEqual(btree.id2path(b"c"), "grandparent2/parent2/file")
208
252
 
209
 
        self.assertEqual(btree.path2id("grandparent2"), "a")
210
 
        self.assertEqual(btree.path2id("grandparent2/parent2"), "b")
211
 
        self.assertEqual(btree.path2id("grandparent2/parent2/file"), "c")
 
253
        self.assertEqual(btree.path2id("grandparent2"), b"a")
 
254
        self.assertEqual(btree.path2id("grandparent2/parent2"), b"b")
 
255
        self.assertEqual(btree.path2id("grandparent2/parent2/file"), b"c")
212
256
 
213
257
        self.assertTrue(btree.path2id("grandparent2/parent") is None)
214
258
        self.assertTrue(btree.path2id("grandparent2/parent/file") is None)
215
259
 
216
260
        btree.note_rename("grandparent/parent/file",
217
261
                          "grandparent2/parent2/file2")
218
 
        self.assertEqual(btree.id2path("a"), "grandparent2")
219
 
        self.assertEqual(btree.id2path("b"), "grandparent2/parent2")
220
 
        self.assertEqual(btree.id2path("c"), "grandparent2/parent2/file2")
 
262
        self.assertEqual(btree.id2path(b"a"), "grandparent2")
 
263
        self.assertEqual(btree.id2path(b"b"), "grandparent2/parent2")
 
264
        self.assertEqual(btree.id2path(b"c"), "grandparent2/parent2/file2")
221
265
 
222
 
        self.assertEqual(btree.path2id("grandparent2"), "a")
223
 
        self.assertEqual(btree.path2id("grandparent2/parent2"), "b")
224
 
        self.assertEqual(btree.path2id("grandparent2/parent2/file2"), "c")
 
266
        self.assertEqual(btree.path2id("grandparent2"), b"a")
 
267
        self.assertEqual(btree.path2id("grandparent2/parent2"), b"b")
 
268
        self.assertEqual(btree.path2id("grandparent2/parent2/file2"), b"c")
225
269
 
226
270
        self.assertTrue(btree.path2id("grandparent2/parent2/file") is None)
227
271
 
230
274
        btree = self.make_tree_1()[0]
231
275
        btree.note_rename("grandparent/parent/file",
232
276
                          "grandparent/alt_parent/file")
233
 
        self.assertEqual(btree.id2path("c"), "grandparent/alt_parent/file")
234
 
        self.assertEqual(btree.path2id("grandparent/alt_parent/file"), "c")
 
277
        self.assertEqual(btree.id2path(b"c"), "grandparent/alt_parent/file")
 
278
        self.assertEqual(btree.path2id("grandparent/alt_parent/file"), b"c")
235
279
        self.assertTrue(btree.path2id("grandparent/parent/file") is None)
236
280
 
237
281
    def unified_diff(self, old, new):
238
 
        out = StringIO()
 
282
        out = BytesIO()
239
283
        diff.internal_diff("old", old, "new", new, out)
240
 
        out.seek(0,0)
 
284
        out.seek(0, 0)
241
285
        return out.read()
242
286
 
243
287
    def make_tree_2(self):
244
288
        btree = self.make_tree_1()[0]
245
289
        btree.note_rename("grandparent/parent/file",
246
290
                          "grandparent/alt_parent/file")
247
 
        self.assertTrue(btree.id2path("e") is None)
248
 
        self.assertTrue(btree.path2id("grandparent/parent/file") is None)
249
 
        btree.note_id("e", "grandparent/parent/file")
 
291
        self.assertTrue(btree.id2path(b"e") is None)
 
292
        self.assertFalse(btree.is_versioned("grandparent/parent/file"))
 
293
        btree.note_id(b"e", "grandparent/parent/file")
250
294
        return btree
251
295
 
252
296
    def test_adds(self):
253
297
        """File/inventory adds"""
254
298
        btree = self.make_tree_2()
255
 
        add_patch = self.unified_diff([], ["Extra cheese\n"])
 
299
        add_patch = self.unified_diff([], [b"Extra cheese\n"])
256
300
        btree.note_patch("grandparent/parent/file", add_patch)
257
 
        btree.note_id('f', 'grandparent/parent/symlink', kind='symlink')
 
301
        btree.note_id(b'f', 'grandparent/parent/symlink', kind='symlink')
258
302
        btree.note_target('grandparent/parent/symlink', 'venus')
259
303
        self.adds_test(btree)
260
304
 
261
305
    def adds_test(self, btree):
262
 
        self.assertEqual(btree.id2path("e"), "grandparent/parent/file")
263
 
        self.assertEqual(btree.path2id("grandparent/parent/file"), "e")
264
 
        self.assertEqual(btree.get_file("e").read(), "Extra cheese\n")
265
 
        self.assertEqual(btree.get_symlink_target('f'), 'venus')
 
306
        self.assertEqual(btree.id2path(b"e"), "grandparent/parent/file")
 
307
        self.assertEqual(btree.path2id("grandparent/parent/file"), b"e")
 
308
        with btree.get_file("grandparent/parent/file") as f:
 
309
            self.assertEqual(f.read(), b"Extra cheese\n")
 
310
        self.assertEqual(
 
311
            btree.get_symlink_target('grandparent/parent/symlink'), 'venus')
266
312
 
267
313
    def test_adds2(self):
268
314
        """File/inventory adds, with patch-compatibile renames"""
269
315
        btree = self.make_tree_2()
270
316
        btree.contents_by_id = False
271
 
        add_patch = self.unified_diff(["Hello\n"], ["Extra cheese\n"])
 
317
        add_patch = self.unified_diff([b"Hello\n"], [b"Extra cheese\n"])
272
318
        btree.note_patch("grandparent/parent/file", add_patch)
273
 
        btree.note_id('f', 'grandparent/parent/symlink', kind='symlink')
 
319
        btree.note_id(b'f', 'grandparent/parent/symlink', kind='symlink')
274
320
        btree.note_target('grandparent/parent/symlink', 'venus')
275
321
        self.adds_test(btree)
276
322
 
277
323
    def make_tree_3(self):
278
324
        btree, mtree = self.make_tree_1()
279
 
        mtree.add_file("e", "grandparent/parent/topping", "Anchovies\n")
 
325
        mtree.add_file(b"e", "grandparent/parent/topping", b"Anchovies\n")
280
326
        btree.note_rename("grandparent/parent/file",
281
327
                          "grandparent/alt_parent/file")
282
328
        btree.note_rename("grandparent/parent/topping",
284
330
        return btree
285
331
 
286
332
    def get_file_test(self, btree):
287
 
        self.assertEqual(btree.get_file("e").read(), "Lemon\n")
288
 
        self.assertEqual(btree.get_file("c").read(), "Hello\n")
 
333
        with btree.get_file(btree.id2path(b"e")) as f:
 
334
            self.assertEqual(f.read(), b"Lemon\n")
 
335
        with btree.get_file(btree.id2path(b"c")) as f:
 
336
            self.assertEqual(f.read(), b"Hello\n")
289
337
 
290
338
    def test_get_file(self):
291
339
        """Get file contents"""
292
340
        btree = self.make_tree_3()
293
 
        mod_patch = self.unified_diff(["Anchovies\n"], ["Lemon\n"])
 
341
        mod_patch = self.unified_diff([b"Anchovies\n"], [b"Lemon\n"])
294
342
        btree.note_patch("grandparent/alt_parent/stopping", mod_patch)
295
343
        self.get_file_test(btree)
296
344
 
297
345
    def test_get_file2(self):
298
 
        """Get file contents, with patch-compatibile renames"""
 
346
        """Get file contents, with patch-compatible renames"""
299
347
        btree = self.make_tree_3()
300
348
        btree.contents_by_id = False
301
 
        mod_patch = self.unified_diff([], ["Lemon\n"])
 
349
        mod_patch = self.unified_diff([], [b"Lemon\n"])
302
350
        btree.note_patch("grandparent/alt_parent/stopping", mod_patch)
303
 
        mod_patch = self.unified_diff([], ["Hello\n"])
 
351
        mod_patch = self.unified_diff([], [b"Hello\n"])
304
352
        btree.note_patch("grandparent/alt_parent/file", mod_patch)
305
353
        self.get_file_test(btree)
306
354
 
307
355
    def test_delete(self):
308
356
        "Deletion by bundle"
309
357
        btree = self.make_tree_1()[0]
310
 
        self.assertEqual(btree.get_file("c").read(), "Hello\n")
 
358
        with btree.get_file(btree.id2path(b"c")) as f:
 
359
            self.assertEqual(f.read(), b"Hello\n")
311
360
        btree.note_deletion("grandparent/parent/file")
312
 
        self.assertTrue(btree.id2path("c") is None)
313
 
        self.assertTrue(btree.path2id("grandparent/parent/file") is None)
 
361
        self.assertTrue(btree.id2path(b"c") is None)
 
362
        self.assertFalse(btree.is_versioned("grandparent/parent/file"))
314
363
 
315
364
    def sorted_ids(self, tree):
316
 
        ids = list(tree)
317
 
        ids.sort()
 
365
        ids = sorted(tree.all_file_ids())
318
366
        return ids
319
367
 
320
368
    def test_iteration(self):
321
369
        """Ensure that iteration through ids works properly"""
322
370
        btree = self.make_tree_1()[0]
323
371
        self.assertEqual(self.sorted_ids(btree),
324
 
            [inventory.ROOT_ID, 'a', 'b', 'c', 'd'])
 
372
            [inventory.ROOT_ID, b'a', b'b', b'c', b'd'])
325
373
        btree.note_deletion("grandparent/parent/file")
326
 
        btree.note_id("e", "grandparent/alt_parent/fool", kind="directory")
 
374
        btree.note_id(b"e", "grandparent/alt_parent/fool", kind="directory")
327
375
        btree.note_last_changed("grandparent/alt_parent/fool",
328
376
                                "revisionidiguess")
329
377
        self.assertEqual(self.sorted_ids(btree),
330
 
            [inventory.ROOT_ID, 'a', 'b', 'd', 'e'])
 
378
            [inventory.ROOT_ID, b'a', b'b', b'd', b'e'])
331
379
 
332
380
 
333
381
class BundleTester1(tests.TestCaseWithTransport):
338
386
        serializer = BundleSerializerV08('0.8')
339
387
        b = self.make_branch('.', format=format)
340
388
        self.assertRaises(errors.IncompatibleBundleFormat, serializer.write,
341
 
                          b.repository, [], {}, StringIO())
 
389
                          b.repository, [], {}, BytesIO())
342
390
 
343
391
    def test_matched_bundle(self):
344
392
        """Don't raise IncompatibleBundleFormat for knit2 and bundle0.9"""
346
394
        format.repository_format = knitrepo.RepositoryFormatKnit3()
347
395
        serializer = BundleSerializerV09('0.9')
348
396
        b = self.make_branch('.', format=format)
349
 
        serializer.write(b.repository, [], {}, StringIO())
 
397
        serializer.write(b.repository, [], {}, BytesIO())
350
398
 
351
399
    def test_mismatched_model(self):
352
400
        """Try copying a bundle from knit2 to knit1"""
353
401
        format = bzrdir.BzrDirMetaFormat1()
354
402
        format.repository_format = knitrepo.RepositoryFormatKnit3()
355
403
        source = self.make_branch_and_tree('source', format=format)
356
 
        source.commit('one', rev_id='one-id')
357
 
        source.commit('two', rev_id='two-id')
358
 
        text = StringIO()
359
 
        write_bundle(source.branch.repository, 'two-id', 'null:', text,
 
404
        source.commit('one', rev_id=b'one-id')
 
405
        source.commit('two', rev_id=b'two-id')
 
406
        text = BytesIO()
 
407
        write_bundle(source.branch.repository, b'two-id', b'null:', text,
360
408
                     format='0.9')
361
409
        text.seek(0)
362
410
 
386
434
        return tests.TestCaseWithTransport.make_branch(self, path, format)
387
435
 
388
436
    def create_bundle_text(self, base_rev_id, rev_id):
389
 
        bundle_txt = StringIO()
 
437
        bundle_txt = BytesIO()
390
438
        rev_ids = write_bundle(self.b1.repository, rev_id, base_rev_id,
391
439
                               bundle_txt, format=self.format)
392
440
        bundle_txt.seek(0)
393
441
        self.assertEqual(bundle_txt.readline(),
394
 
                         '# Bazaar revision bundle v%s\n' % self.format)
395
 
        self.assertEqual(bundle_txt.readline(), '#\n')
 
442
                         b'# Bazaar revision bundle v%s\n' % self.format.encode('ascii'))
 
443
        self.assertEqual(bundle_txt.readline(), b'#\n')
396
444
 
397
445
        rev = self.b1.repository.get_revision(rev_id)
398
446
        self.assertEqual(bundle_txt.readline().decode('utf-8'),
427
475
                             len(bundle_rev.parent_ids))
428
476
        self.assertEqual(rev_ids,
429
477
                         [r.revision_id for r in bundle.real_revisions])
430
 
        self.valid_apply_bundle(base_rev_id, bundle,
431
 
                                   checkout_dir=checkout_dir)
 
478
        self.valid_apply_bundle(base_rev_id, bundle, checkout_dir=checkout_dir)
432
479
 
433
480
        return bundle
434
481
 
439
486
        :return: The in-memory bundle
440
487
        """
441
488
        bundle_txt, rev_ids = self.create_bundle_text(base_rev_id, rev_id)
442
 
        new_text = bundle_txt.getvalue().replace('executable:no',
443
 
                                               'executable:yes')
444
 
        bundle_txt = StringIO(new_text)
 
489
        new_text = bundle_txt.getvalue().replace(b'executable:no',
 
490
                                                 b'executable:yes')
 
491
        bundle_txt = BytesIO(new_text)
445
492
        bundle = read_bundle(bundle_txt)
446
493
        self.valid_apply_bundle(base_rev_id, bundle)
447
494
        return bundle
448
495
 
449
496
    def test_non_bundle(self):
450
497
        self.assertRaises(errors.NotABundle,
451
 
                          read_bundle, StringIO('#!/bin/sh\n'))
 
498
                          read_bundle, BytesIO(b'#!/bin/sh\n'))
452
499
 
453
500
    def test_malformed(self):
454
501
        self.assertRaises(errors.BadBundle, read_bundle,
455
 
                          StringIO('# Bazaar revision bundle v'))
 
502
                          BytesIO(b'# Bazaar revision bundle v'))
456
503
 
457
504
    def test_crlf_bundle(self):
458
505
        try:
459
 
            read_bundle(StringIO('# Bazaar revision bundle v0.8\r\n'))
 
506
            read_bundle(BytesIO(b'# Bazaar revision bundle v0.8\r\n'))
460
507
        except errors.BadBundle:
461
508
            # It is currently permitted for bundles with crlf line endings to
462
509
            # make read_bundle raise a BadBundle, but this should be fixed.
473
520
            if not os.path.exists(checkout_dir):
474
521
                os.mkdir(checkout_dir)
475
522
        tree = self.make_branch_and_tree(checkout_dir)
476
 
        s = StringIO()
477
 
        ancestors = write_bundle(self.b1.repository, rev_id, 'null:', s,
 
523
        s = BytesIO()
 
524
        ancestors = write_bundle(self.b1.repository, rev_id, b'null:', s,
478
525
                                 format=self.format)
479
526
        s.seek(0)
480
 
        self.assertIsInstance(s.getvalue(), str)
 
527
        self.assertIsInstance(s.getvalue(), bytes)
481
528
        install_bundle(tree.branch.repository, read_bundle(s))
482
529
        for ancestor in ancestors:
483
530
            old = self.b1.repository.revision_tree(ancestor)
492
539
                                 % (ancestor,))
493
540
 
494
541
                # Now check that the file contents are all correct
495
 
                for inventory_id in old:
 
542
                for path in old.all_versioned_paths():
496
543
                    try:
497
 
                        old_file = old.get_file(inventory_id)
 
544
                        old_file = old.get_file(path)
498
545
                    except errors.NoSuchFile:
499
546
                        continue
500
 
                    if old_file is None:
501
 
                        continue
502
 
                    self.assertEqual(old_file.read(),
503
 
                                     new.get_file(inventory_id).read())
 
547
                    self.assertEqual(
 
548
                            old_file.read(), new.get_file(path).read())
504
549
            finally:
505
550
                new.unlock()
506
551
                old.unlock()
507
552
        if not _mod_revision.is_null(rev_id):
508
 
            rh = self.b1.revision_history()
509
 
            tree.branch.set_revision_history(rh[:rh.index(rev_id)+1])
 
553
            tree.branch.generate_revision_history(rev_id)
510
554
            tree.update()
511
555
            delta = tree.changes_from(self.b1.repository.revision_tree(rev_id))
512
556
            self.assertFalse(delta.has_changed(),
530
574
        original_parents = to_tree.get_parent_ids()
531
575
        self.assertIs(repository.has_revision(base_rev_id), True)
532
576
        for rev in info.real_revisions:
533
 
            self.assert_(not repository.has_revision(rev.revision_id),
534
 
                'Revision {%s} present before applying bundle'
535
 
                % rev.revision_id)
 
577
            self.assertTrue(not repository.has_revision(rev.revision_id),
 
578
                            'Revision {%s} present before applying bundle'
 
579
                            % rev.revision_id)
536
580
        merge_bundle(info, to_tree, True, merge.Merge3Merger, False, False)
537
581
 
538
582
        for rev in info.real_revisions:
539
 
            self.assert_(repository.has_revision(rev.revision_id),
540
 
                'Missing revision {%s} after applying bundle'
541
 
                % rev.revision_id)
 
583
            self.assertTrue(repository.has_revision(rev.revision_id),
 
584
                            'Missing revision {%s} after applying bundle'
 
585
                            % rev.revision_id)
542
586
 
543
 
        self.assert_(to_tree.branch.repository.has_revision(info.target))
 
587
        self.assertTrue(to_tree.branch.repository.has_revision(info.target))
544
588
        # Do we also want to verify that all the texts have been added?
545
589
 
546
590
        self.assertEqual(original_parents + [info.target],
547
 
            to_tree.get_parent_ids())
 
591
                         to_tree.get_parent_ids())
548
592
 
549
593
        rev = info.real_revisions[-1]
550
594
        base_tree = self.b1.repository.revision_tree(rev.revision_id)
561
605
 
562
606
        for path, status, kind, fileid, entry in base_files:
563
607
            # Check that the meta information is the same
564
 
            self.assertEqual(base_tree.get_file_size(fileid),
565
 
                    to_tree.get_file_size(fileid))
566
 
            self.assertEqual(base_tree.get_file_sha1(fileid),
567
 
                    to_tree.get_file_sha1(fileid))
 
608
            self.assertEqual(base_tree.get_file_size(path, fileid),
 
609
                    to_tree.get_file_size(to_tree.id2path(fileid)))
 
610
            self.assertEqual(base_tree.get_file_sha1(path, fileid),
 
611
                    to_tree.get_file_sha1(to_tree.id2path(fileid)))
568
612
            # Check that the contents are the same
569
613
            # This is pretty expensive
570
614
            # self.assertEqual(base_tree.get_file(fileid).read(),
574
618
        self.tree1 = self.make_branch_and_tree('b1')
575
619
        self.b1 = self.tree1.branch
576
620
 
577
 
        self.build_tree_contents([('b1/one', 'one\n')])
578
 
        self.tree1.add('one', 'one-id')
579
 
        self.tree1.set_root_id('root-id')
580
 
        self.tree1.commit('add one', rev_id='a@cset-0-1')
 
621
        self.build_tree_contents([('b1/one', b'one\n')])
 
622
        self.tree1.add('one', b'one-id')
 
623
        self.tree1.set_root_id(b'root-id')
 
624
        self.tree1.commit('add one', rev_id=b'a@cset-0-1')
581
625
 
582
 
        bundle = self.get_valid_bundle('null:', 'a@cset-0-1')
 
626
        bundle = self.get_valid_bundle(b'null:', b'a@cset-0-1')
583
627
 
584
628
        # Make sure we can handle files with spaces, tabs, other
585
629
        # bogus characters
593
637
                , 'b1/sub/sub/'
594
638
                , 'b1/sub/sub/nonempty.txt'
595
639
                ])
596
 
        self.build_tree_contents([('b1/sub/sub/emptyfile.txt', ''),
597
 
                                  ('b1/dir/nolastnewline.txt', 'bloop')])
 
640
        self.build_tree_contents([('b1/sub/sub/emptyfile.txt', b''),
 
641
                                  ('b1/dir/nolastnewline.txt', b'bloop')])
598
642
        tt = TreeTransform(self.tree1)
599
 
        tt.new_file('executable', tt.root, '#!/bin/sh\n', 'exe-1', True)
 
643
        tt.new_file('executable', tt.root, [b'#!/bin/sh\n'], b'exe-1', True)
600
644
        tt.apply()
601
645
        # have to fix length of file-id so that we can predictably rewrite
602
646
        # a (length-prefixed) record containing it later.
603
 
        self.tree1.add('with space.txt', 'withspace-id')
 
647
        self.tree1.add('with space.txt', b'withspace-id')
604
648
        self.tree1.add([
605
649
                  'dir'
606
650
                , 'dir/filein subdir.c'
612
656
                , 'sub/sub/nonempty.txt'
613
657
                , 'sub/sub/emptyfile.txt'
614
658
                ])
615
 
        self.tree1.commit('add whitespace', rev_id='a@cset-0-2')
 
659
        self.tree1.commit('add whitespace', rev_id=b'a@cset-0-2')
616
660
 
617
 
        bundle = self.get_valid_bundle('a@cset-0-1', 'a@cset-0-2')
 
661
        bundle = self.get_valid_bundle(b'a@cset-0-1', b'a@cset-0-2')
618
662
 
619
663
        # Check a rollup bundle
620
 
        bundle = self.get_valid_bundle('null:', 'a@cset-0-2')
 
664
        bundle = self.get_valid_bundle(b'null:', b'a@cset-0-2')
621
665
 
622
666
        # Now delete entries
623
667
        self.tree1.remove(
626
670
                , 'sub/sub'
627
671
                ])
628
672
        tt = TreeTransform(self.tree1)
629
 
        trans_id = tt.trans_id_tree_file_id('exe-1')
 
673
        trans_id = tt.trans_id_tree_path('executable')
630
674
        tt.set_executability(False, trans_id)
631
675
        tt.apply()
632
 
        self.tree1.commit('removed', rev_id='a@cset-0-3')
 
676
        self.tree1.commit('removed', rev_id=b'a@cset-0-3')
633
677
 
634
 
        bundle = self.get_valid_bundle('a@cset-0-2', 'a@cset-0-3')
 
678
        bundle = self.get_valid_bundle(b'a@cset-0-2', b'a@cset-0-3')
635
679
        self.assertRaises((errors.TestamentMismatch,
636
680
            errors.VersionedFileInvalidChecksum,
637
681
            errors.BadBundle), self.get_invalid_bundle,
638
 
            'a@cset-0-2', 'a@cset-0-3')
 
682
            b'a@cset-0-2', b'a@cset-0-3')
639
683
        # Check a rollup bundle
640
 
        bundle = self.get_valid_bundle('null:', 'a@cset-0-3')
 
684
        bundle = self.get_valid_bundle(b'null:', b'a@cset-0-3')
641
685
 
642
686
        # Now move the directory
643
687
        self.tree1.rename_one('dir', 'sub/dir')
644
 
        self.tree1.commit('rename dir', rev_id='a@cset-0-4')
 
688
        self.tree1.commit('rename dir', rev_id=b'a@cset-0-4')
645
689
 
646
 
        bundle = self.get_valid_bundle('a@cset-0-3', 'a@cset-0-4')
 
690
        bundle = self.get_valid_bundle(b'a@cset-0-3', b'a@cset-0-4')
647
691
        # Check a rollup bundle
648
 
        bundle = self.get_valid_bundle('null:', 'a@cset-0-4')
 
692
        bundle = self.get_valid_bundle(b'null:', b'a@cset-0-4')
649
693
 
650
694
        # Modified files
651
 
        open('b1/sub/dir/WithCaps.txt', 'ab').write('\nAdding some text\n')
652
 
        open('b1/sub/dir/ pre space', 'ab').write(
653
 
             '\r\nAdding some\r\nDOS format lines\r\n')
654
 
        open('b1/sub/dir/nolastnewline.txt', 'ab').write('\n')
 
695
        with open('b1/sub/dir/WithCaps.txt', 'ab') as f: f.write(b'\nAdding some text\n')
 
696
        with open('b1/sub/dir/ pre space', 'ab') as f: f.write(
 
697
             b'\r\nAdding some\r\nDOS format lines\r\n')
 
698
        with open('b1/sub/dir/nolastnewline.txt', 'ab') as f: f.write(b'\n')
655
699
        self.tree1.rename_one('sub/dir/ pre space',
656
700
                              'sub/ start space')
657
 
        self.tree1.commit('Modified files', rev_id='a@cset-0-5')
658
 
        bundle = self.get_valid_bundle('a@cset-0-4', 'a@cset-0-5')
 
701
        self.tree1.commit('Modified files', rev_id=b'a@cset-0-5')
 
702
        bundle = self.get_valid_bundle(b'a@cset-0-4', b'a@cset-0-5')
659
703
 
660
704
        self.tree1.rename_one('sub/dir/WithCaps.txt', 'temp')
661
705
        self.tree1.rename_one('with space.txt', 'WithCaps.txt')
662
706
        self.tree1.rename_one('temp', 'with space.txt')
663
 
        self.tree1.commit(u'swap filenames', rev_id='a@cset-0-6',
 
707
        self.tree1.commit(u'swap filenames', rev_id=b'a@cset-0-6',
664
708
                          verbose=False)
665
 
        bundle = self.get_valid_bundle('a@cset-0-5', 'a@cset-0-6')
666
 
        other = self.get_checkout('a@cset-0-5')
 
709
        bundle = self.get_valid_bundle(b'a@cset-0-5', b'a@cset-0-6')
 
710
        other = self.get_checkout(b'a@cset-0-5')
667
711
        tree1_inv = get_inventory_text(self.tree1.branch.repository,
668
 
                                       'a@cset-0-5')
 
712
                                       b'a@cset-0-5')
669
713
        tree2_inv = get_inventory_text(other.branch.repository,
670
 
                                       'a@cset-0-5')
 
714
                                       b'a@cset-0-5')
671
715
        self.assertEqualDiff(tree1_inv, tree2_inv)
672
716
        other.rename_one('sub/dir/nolastnewline.txt', 'sub/nolastnewline.txt')
673
 
        other.commit('rename file', rev_id='a@cset-0-6b')
 
717
        other.commit('rename file', rev_id=b'a@cset-0-6b')
674
718
        self.tree1.merge_from_branch(other.branch)
675
 
        self.tree1.commit(u'Merge', rev_id='a@cset-0-7',
 
719
        self.tree1.commit(u'Merge', rev_id=b'a@cset-0-7',
676
720
                          verbose=False)
677
 
        bundle = self.get_valid_bundle('a@cset-0-6', 'a@cset-0-7')
 
721
        bundle = self.get_valid_bundle(b'a@cset-0-6', b'a@cset-0-7')
678
722
 
679
723
    def _test_symlink_bundle(self, link_name, link_target, new_link_target):
680
 
        link_id = 'link-1'
 
724
        link_id = b'link-1'
681
725
 
682
 
        self.requireFeature(tests.SymlinkFeature)
 
726
        self.requireFeature(features.SymlinkFeature)
683
727
        self.tree1 = self.make_branch_and_tree('b1')
684
728
        self.b1 = self.tree1.branch
685
729
 
686
730
        tt = TreeTransform(self.tree1)
687
731
        tt.new_symlink(link_name, tt.root, link_target, link_id)
688
732
        tt.apply()
689
 
        self.tree1.commit('add symlink', rev_id='l@cset-0-1')
690
 
        bundle = self.get_valid_bundle('null:', 'l@cset-0-1')
691
 
        if getattr(bundle ,'revision_tree', None) is not None:
 
733
        self.tree1.commit('add symlink', rev_id=b'l@cset-0-1')
 
734
        bundle = self.get_valid_bundle(b'null:', b'l@cset-0-1')
 
735
        if getattr(bundle, 'revision_tree', None) is not None:
692
736
            # Not all bundle formats supports revision_tree
693
 
            bund_tree = bundle.revision_tree(self.b1.repository, 'l@cset-0-1')
694
 
            self.assertEqual(link_target, bund_tree.get_symlink_target(link_id))
 
737
            bund_tree = bundle.revision_tree(self.b1.repository, b'l@cset-0-1')
 
738
            self.assertEqual(link_target, bund_tree.get_symlink_target(link_name))
695
739
 
696
740
        tt = TreeTransform(self.tree1)
697
 
        trans_id = tt.trans_id_tree_file_id(link_id)
 
741
        trans_id = tt.trans_id_tree_path(link_name)
698
742
        tt.adjust_path('link2', tt.root, trans_id)
699
743
        tt.delete_contents(trans_id)
700
744
        tt.create_symlink(new_link_target, trans_id)
701
745
        tt.apply()
702
 
        self.tree1.commit('rename and change symlink', rev_id='l@cset-0-2')
703
 
        bundle = self.get_valid_bundle('l@cset-0-1', 'l@cset-0-2')
704
 
        if getattr(bundle ,'revision_tree', None) is not None:
 
746
        self.tree1.commit('rename and change symlink', rev_id=b'l@cset-0-2')
 
747
        bundle = self.get_valid_bundle(b'l@cset-0-1', b'l@cset-0-2')
 
748
        if getattr(bundle, 'revision_tree', None) is not None:
705
749
            # Not all bundle formats supports revision_tree
706
 
            bund_tree = bundle.revision_tree(self.b1.repository, 'l@cset-0-2')
 
750
            bund_tree = bundle.revision_tree(self.b1.repository, b'l@cset-0-2')
707
751
            self.assertEqual(new_link_target,
708
 
                             bund_tree.get_symlink_target(link_id))
 
752
                             bund_tree.get_symlink_target('link2'))
709
753
 
710
754
        tt = TreeTransform(self.tree1)
711
 
        trans_id = tt.trans_id_tree_file_id(link_id)
 
755
        trans_id = tt.trans_id_tree_path('link2')
712
756
        tt.delete_contents(trans_id)
713
757
        tt.create_symlink('jupiter', trans_id)
714
758
        tt.apply()
715
 
        self.tree1.commit('just change symlink target', rev_id='l@cset-0-3')
716
 
        bundle = self.get_valid_bundle('l@cset-0-2', 'l@cset-0-3')
 
759
        self.tree1.commit('just change symlink target', rev_id=b'l@cset-0-3')
 
760
        bundle = self.get_valid_bundle(b'l@cset-0-2', b'l@cset-0-3')
717
761
 
718
762
        tt = TreeTransform(self.tree1)
719
 
        trans_id = tt.trans_id_tree_file_id(link_id)
 
763
        trans_id = tt.trans_id_tree_path('link2')
720
764
        tt.delete_contents(trans_id)
721
765
        tt.apply()
722
 
        self.tree1.commit('Delete symlink', rev_id='l@cset-0-4')
723
 
        bundle = self.get_valid_bundle('l@cset-0-3', 'l@cset-0-4')
 
766
        self.tree1.commit('Delete symlink', rev_id=b'l@cset-0-4')
 
767
        bundle = self.get_valid_bundle(b'l@cset-0-3', b'l@cset-0-4')
724
768
 
725
769
    def test_symlink_bundle(self):
726
770
        self._test_symlink_bundle('link', 'bar/foo', 'mars')
727
771
 
728
772
    def test_unicode_symlink_bundle(self):
729
 
        self.requireFeature(tests.UnicodeFilenameFeature)
 
773
        self.requireFeature(features.UnicodeFilenameFeature)
730
774
        self._test_symlink_bundle(u'\N{Euro Sign}link',
731
775
                                  u'bar/\N{Euro Sign}foo',
732
776
                                  u'mars\N{Euro Sign}')
737
781
        tt = TreeTransform(self.tree1)
738
782
 
739
783
        # Add
740
 
        tt.new_file('file', tt.root, '\x00\n\x00\r\x01\n\x02\r\xff', 'binary-1')
741
 
        tt.new_file('file2', tt.root, '\x01\n\x02\r\x03\n\x04\r\xff',
742
 
            'binary-2')
 
784
        tt.new_file('file', tt.root, [b'\x00\n\x00\r\x01\n\x02\r\xff'], b'binary-1')
 
785
        tt.new_file('file2', tt.root, [b'\x01\n\x02\r\x03\n\x04\r\xff'],
 
786
            b'binary-2')
743
787
        tt.apply()
744
 
        self.tree1.commit('add binary', rev_id='b@cset-0-1')
745
 
        self.get_valid_bundle('null:', 'b@cset-0-1')
 
788
        self.tree1.commit('add binary', rev_id=b'b@cset-0-1')
 
789
        self.get_valid_bundle(b'null:', b'b@cset-0-1')
746
790
 
747
791
        # Delete
748
792
        tt = TreeTransform(self.tree1)
749
 
        trans_id = tt.trans_id_tree_file_id('binary-1')
 
793
        trans_id = tt.trans_id_tree_path('file')
750
794
        tt.delete_contents(trans_id)
751
795
        tt.apply()
752
 
        self.tree1.commit('delete binary', rev_id='b@cset-0-2')
753
 
        self.get_valid_bundle('b@cset-0-1', 'b@cset-0-2')
 
796
        self.tree1.commit('delete binary', rev_id=b'b@cset-0-2')
 
797
        self.get_valid_bundle(b'b@cset-0-1', b'b@cset-0-2')
754
798
 
755
799
        # Rename & modify
756
800
        tt = TreeTransform(self.tree1)
757
 
        trans_id = tt.trans_id_tree_file_id('binary-2')
 
801
        trans_id = tt.trans_id_tree_path('file2')
758
802
        tt.adjust_path('file3', tt.root, trans_id)
759
803
        tt.delete_contents(trans_id)
760
 
        tt.create_file('file\rcontents\x00\n\x00', trans_id)
 
804
        tt.create_file([b'file\rcontents\x00\n\x00'], trans_id)
761
805
        tt.apply()
762
 
        self.tree1.commit('rename and modify binary', rev_id='b@cset-0-3')
763
 
        self.get_valid_bundle('b@cset-0-2', 'b@cset-0-3')
 
806
        self.tree1.commit('rename and modify binary', rev_id=b'b@cset-0-3')
 
807
        self.get_valid_bundle(b'b@cset-0-2', b'b@cset-0-3')
764
808
 
765
809
        # Modify
766
810
        tt = TreeTransform(self.tree1)
767
 
        trans_id = tt.trans_id_tree_file_id('binary-2')
 
811
        trans_id = tt.trans_id_tree_path('file3')
768
812
        tt.delete_contents(trans_id)
769
 
        tt.create_file('\x00file\rcontents', trans_id)
 
813
        tt.create_file([b'\x00file\rcontents'], trans_id)
770
814
        tt.apply()
771
 
        self.tree1.commit('just modify binary', rev_id='b@cset-0-4')
772
 
        self.get_valid_bundle('b@cset-0-3', 'b@cset-0-4')
 
815
        self.tree1.commit('just modify binary', rev_id=b'b@cset-0-4')
 
816
        self.get_valid_bundle(b'b@cset-0-3', b'b@cset-0-4')
773
817
 
774
818
        # Rollup
775
 
        self.get_valid_bundle('null:', 'b@cset-0-4')
 
819
        self.get_valid_bundle(b'null:', b'b@cset-0-4')
776
820
 
777
821
    def test_last_modified(self):
778
822
        self.tree1 = self.make_branch_and_tree('b1')
779
823
        self.b1 = self.tree1.branch
780
824
        tt = TreeTransform(self.tree1)
781
 
        tt.new_file('file', tt.root, 'file', 'file')
 
825
        tt.new_file('file', tt.root, [b'file'], b'file')
782
826
        tt.apply()
783
 
        self.tree1.commit('create file', rev_id='a@lmod-0-1')
 
827
        self.tree1.commit('create file', rev_id=b'a@lmod-0-1')
784
828
 
785
829
        tt = TreeTransform(self.tree1)
786
 
        trans_id = tt.trans_id_tree_file_id('file')
 
830
        trans_id = tt.trans_id_tree_path('file')
787
831
        tt.delete_contents(trans_id)
788
 
        tt.create_file('file2', trans_id)
 
832
        tt.create_file([b'file2'], trans_id)
789
833
        tt.apply()
790
 
        self.tree1.commit('modify text', rev_id='a@lmod-0-2a')
 
834
        self.tree1.commit('modify text', rev_id=b'a@lmod-0-2a')
791
835
 
792
 
        other = self.get_checkout('a@lmod-0-1')
 
836
        other = self.get_checkout(b'a@lmod-0-1')
793
837
        tt = TreeTransform(other)
794
 
        trans_id = tt.trans_id_tree_file_id('file')
 
838
        trans_id = tt.trans_id_tree_path('file2')
795
839
        tt.delete_contents(trans_id)
796
 
        tt.create_file('file2', trans_id)
 
840
        tt.create_file([b'file2'], trans_id)
797
841
        tt.apply()
798
 
        other.commit('modify text in another tree', rev_id='a@lmod-0-2b')
 
842
        other.commit('modify text in another tree', rev_id=b'a@lmod-0-2b')
799
843
        self.tree1.merge_from_branch(other.branch)
800
 
        self.tree1.commit(u'Merge', rev_id='a@lmod-0-3',
 
844
        self.tree1.commit(u'Merge', rev_id=b'a@lmod-0-3',
801
845
                          verbose=False)
802
 
        self.tree1.commit(u'Merge', rev_id='a@lmod-0-4')
803
 
        bundle = self.get_valid_bundle('a@lmod-0-2a', 'a@lmod-0-4')
 
846
        self.tree1.commit(u'Merge', rev_id=b'a@lmod-0-4')
 
847
        bundle = self.get_valid_bundle(b'a@lmod-0-2a', b'a@lmod-0-4')
804
848
 
805
849
    def test_hide_history(self):
806
850
        self.tree1 = self.make_branch_and_tree('b1')
807
851
        self.b1 = self.tree1.branch
808
852
 
809
 
        open('b1/one', 'wb').write('one\n')
 
853
        with open('b1/one', 'wb') as f: f.write(b'one\n')
810
854
        self.tree1.add('one')
811
 
        self.tree1.commit('add file', rev_id='a@cset-0-1')
812
 
        open('b1/one', 'wb').write('two\n')
813
 
        self.tree1.commit('modify', rev_id='a@cset-0-2')
814
 
        open('b1/one', 'wb').write('three\n')
815
 
        self.tree1.commit('modify', rev_id='a@cset-0-3')
816
 
        bundle_file = StringIO()
817
 
        rev_ids = write_bundle(self.tree1.branch.repository, 'a@cset-0-3',
818
 
                               'a@cset-0-1', bundle_file, format=self.format)
819
 
        self.assertNotContainsRe(bundle_file.getvalue(), '\btwo\b')
820
 
        self.assertContainsRe(self.get_raw(bundle_file), 'one')
821
 
        self.assertContainsRe(self.get_raw(bundle_file), 'three')
 
855
        self.tree1.commit('add file', rev_id=b'a@cset-0-1')
 
856
        with open('b1/one', 'wb') as f: f.write(b'two\n')
 
857
        self.tree1.commit('modify', rev_id=b'a@cset-0-2')
 
858
        with open('b1/one', 'wb') as f: f.write(b'three\n')
 
859
        self.tree1.commit('modify', rev_id=b'a@cset-0-3')
 
860
        bundle_file = BytesIO()
 
861
        rev_ids = write_bundle(self.tree1.branch.repository, b'a@cset-0-3',
 
862
                               b'a@cset-0-1', bundle_file, format=self.format)
 
863
        self.assertNotContainsRe(bundle_file.getvalue(), b'\btwo\b')
 
864
        self.assertContainsRe(self.get_raw(bundle_file), b'one')
 
865
        self.assertContainsRe(self.get_raw(bundle_file), b'three')
822
866
 
823
867
    def test_bundle_same_basis(self):
824
868
        """Ensure using the basis as the target doesn't cause an error"""
825
869
        self.tree1 = self.make_branch_and_tree('b1')
826
 
        self.tree1.commit('add file', rev_id='a@cset-0-1')
827
 
        bundle_file = StringIO()
828
 
        rev_ids = write_bundle(self.tree1.branch.repository, 'a@cset-0-1',
829
 
                               'a@cset-0-1', bundle_file)
 
870
        self.tree1.commit('add file', rev_id=b'a@cset-0-1')
 
871
        bundle_file = BytesIO()
 
872
        rev_ids = write_bundle(self.tree1.branch.repository, b'a@cset-0-1',
 
873
                               b'a@cset-0-1', bundle_file)
830
874
 
831
875
    @staticmethod
832
876
    def get_raw(bundle_file):
833
877
        return bundle_file.getvalue()
834
878
 
835
879
    def test_unicode_bundle(self):
836
 
        self.requireFeature(tests.UnicodeFilenameFeature)
 
880
        self.requireFeature(features.UnicodeFilenameFeature)
837
881
        # Handle international characters
838
882
        os.mkdir('b1')
839
883
        f = open(u'b1/with Dod\N{Euro Sign}', 'wb')
846
890
            u'William Dod\xe9\n').encode('utf-8'))
847
891
        f.close()
848
892
 
849
 
        self.tree1.add([u'with Dod\N{Euro Sign}'], ['withdod-id'])
 
893
        self.tree1.add([u'with Dod\N{Euro Sign}'], [b'withdod-id'])
850
894
        self.tree1.commit(u'i18n commit from William Dod\xe9',
851
 
                          rev_id='i18n-1', committer=u'William Dod\xe9')
 
895
                          rev_id=b'i18n-1', committer=u'William Dod\xe9')
852
896
 
853
897
        # Add
854
 
        bundle = self.get_valid_bundle('null:', 'i18n-1')
 
898
        bundle = self.get_valid_bundle(b'null:', b'i18n-1')
855
899
 
856
900
        # Modified
857
901
        f = open(u'b1/with Dod\N{Euro Sign}', 'wb')
858
902
        f.write(u'Modified \xb5\n'.encode('utf8'))
859
903
        f.close()
860
 
        self.tree1.commit(u'modified', rev_id='i18n-2')
 
904
        self.tree1.commit(u'modified', rev_id=b'i18n-2')
861
905
 
862
 
        bundle = self.get_valid_bundle('i18n-1', 'i18n-2')
 
906
        bundle = self.get_valid_bundle(b'i18n-1', b'i18n-2')
863
907
 
864
908
        # Renamed
865
909
        self.tree1.rename_one(u'with Dod\N{Euro Sign}', u'B\N{Euro Sign}gfors')
866
 
        self.tree1.commit(u'renamed, the new i18n man', rev_id='i18n-3',
 
910
        self.tree1.commit(u'renamed, the new i18n man', rev_id=b'i18n-3',
867
911
                          committer=u'Erik B\xe5gfors')
868
912
 
869
 
        bundle = self.get_valid_bundle('i18n-2', 'i18n-3')
 
913
        bundle = self.get_valid_bundle(b'i18n-2', b'i18n-3')
870
914
 
871
915
        # Removed
872
916
        self.tree1.remove([u'B\N{Euro Sign}gfors'])
873
 
        self.tree1.commit(u'removed', rev_id='i18n-4')
 
917
        self.tree1.commit(u'removed', rev_id=b'i18n-4')
874
918
 
875
 
        bundle = self.get_valid_bundle('i18n-3', 'i18n-4')
 
919
        bundle = self.get_valid_bundle(b'i18n-3', b'i18n-4')
876
920
 
877
921
        # Rollup
878
 
        bundle = self.get_valid_bundle('null:', 'i18n-4')
 
922
        bundle = self.get_valid_bundle(b'null:', b'i18n-4')
879
923
 
880
924
 
881
925
    def test_whitespace_bundle(self):
891
935
        #       once we actually support them
892
936
 
893
937
        # Added
894
 
        self.tree1.commit('funky whitespace', rev_id='white-1')
 
938
        self.tree1.commit('funky whitespace', rev_id=b'white-1')
895
939
 
896
 
        bundle = self.get_valid_bundle('null:', 'white-1')
 
940
        bundle = self.get_valid_bundle(b'null:', b'white-1')
897
941
 
898
942
        # Modified
899
 
        open('b1/trailing space ', 'ab').write('add some text\n')
900
 
        self.tree1.commit('add text', rev_id='white-2')
 
943
        with open('b1/trailing space ', 'ab') as f: f.write(b'add some text\n')
 
944
        self.tree1.commit('add text', rev_id=b'white-2')
901
945
 
902
 
        bundle = self.get_valid_bundle('white-1', 'white-2')
 
946
        bundle = self.get_valid_bundle(b'white-1', b'white-2')
903
947
 
904
948
        # Renamed
905
949
        self.tree1.rename_one('trailing space ', ' start and end space ')
906
 
        self.tree1.commit('rename', rev_id='white-3')
 
950
        self.tree1.commit('rename', rev_id=b'white-3')
907
951
 
908
 
        bundle = self.get_valid_bundle('white-2', 'white-3')
 
952
        bundle = self.get_valid_bundle(b'white-2', b'white-3')
909
953
 
910
954
        # Removed
911
955
        self.tree1.remove([' start and end space '])
912
 
        self.tree1.commit('removed', rev_id='white-4')
 
956
        self.tree1.commit('removed', rev_id=b'white-4')
913
957
 
914
 
        bundle = self.get_valid_bundle('white-3', 'white-4')
 
958
        bundle = self.get_valid_bundle(b'white-3', b'white-4')
915
959
 
916
960
        # Now test a complet roll-up
917
 
        bundle = self.get_valid_bundle('null:', 'white-4')
 
961
        bundle = self.get_valid_bundle(b'null:', b'white-4')
918
962
 
919
963
    def test_alt_timezone_bundle(self):
920
964
        self.tree1 = self.make_branch_and_memory_tree('b1')
927
971
        builder.finish_tree()
928
972
 
929
973
        # Asia/Colombo offset = 5 hours 30 minutes
930
 
        self.tree1.commit('non-hour offset timezone', rev_id='tz-1',
 
974
        self.tree1.commit('non-hour offset timezone', rev_id=b'tz-1',
931
975
                          timezone=19800, timestamp=1152544886.0)
932
976
 
933
 
        bundle = self.get_valid_bundle('null:', 'tz-1')
 
977
        bundle = self.get_valid_bundle(b'null:', b'tz-1')
934
978
 
935
979
        rev = bundle.revisions[0]
936
980
        self.assertEqual('Mon 2006-07-10 20:51:26.000000000 +0530', rev.date)
941
985
    def test_bundle_root_id(self):
942
986
        self.tree1 = self.make_branch_and_tree('b1')
943
987
        self.b1 = self.tree1.branch
944
 
        self.tree1.commit('message', rev_id='revid1')
945
 
        bundle = self.get_valid_bundle('null:', 'revid1')
946
 
        tree = self.get_bundle_tree(bundle, 'revid1')
947
 
        self.assertEqual('revid1', tree.inventory.root.revision)
 
988
        self.tree1.commit('message', rev_id=b'revid1')
 
989
        bundle = self.get_valid_bundle(b'null:', b'revid1')
 
990
        tree = self.get_bundle_tree(bundle, b'revid1')
 
991
        root_revision = tree.get_file_revision(u'', tree.get_root_id())
 
992
        self.assertEqual(b'revid1', root_revision)
948
993
 
949
994
    def test_install_revisions(self):
950
995
        self.tree1 = self.make_branch_and_tree('b1')
951
996
        self.b1 = self.tree1.branch
952
 
        self.tree1.commit('message', rev_id='rev2a')
953
 
        bundle = self.get_valid_bundle('null:', 'rev2a')
 
997
        self.tree1.commit('message', rev_id=b'rev2a')
 
998
        bundle = self.get_valid_bundle(b'null:', b'rev2a')
954
999
        branch2 = self.make_branch('b2')
955
 
        self.assertFalse(branch2.repository.has_revision('rev2a'))
 
1000
        self.assertFalse(branch2.repository.has_revision(b'rev2a'))
956
1001
        target_revision = bundle.install_revisions(branch2.repository)
957
 
        self.assertTrue(branch2.repository.has_revision('rev2a'))
958
 
        self.assertEqual('rev2a', target_revision)
 
1002
        self.assertTrue(branch2.repository.has_revision(b'rev2a'))
 
1003
        self.assertEqual(b'rev2a', target_revision)
959
1004
 
960
1005
    def test_bundle_empty_property(self):
961
1006
        """Test serializing revision properties with an empty value."""
962
1007
        tree = self.make_branch_and_memory_tree('tree')
963
1008
        tree.lock_write()
964
1009
        self.addCleanup(tree.unlock)
965
 
        tree.add([''], ['TREE_ROOT'])
966
 
        tree.commit('One', revprops={'one':'two', 'empty':''}, rev_id='rev1')
 
1010
        tree.add([''], [b'TREE_ROOT'])
 
1011
        tree.commit('One', revprops={u'one': 'two', u'empty': ''}, rev_id=b'rev1')
967
1012
        self.b1 = tree.branch
968
 
        bundle_sio, revision_ids = self.create_bundle_text('null:', 'rev1')
 
1013
        bundle_sio, revision_ids = self.create_bundle_text(b'null:', b'rev1')
969
1014
        bundle = read_bundle(bundle_sio)
970
1015
        revision_info = bundle.revisions[0]
971
 
        self.assertEqual('rev1', revision_info.revision_id)
 
1016
        self.assertEqual(b'rev1', revision_info.revision_id)
972
1017
        rev = revision_info.as_revision()
973
1018
        self.assertEqual({'branch-nick':'tree', 'empty':'', 'one':'two'},
974
1019
                         rev.properties)
979
1024
        tree.lock_write()
980
1025
        self.addCleanup(tree.unlock)
981
1026
 
982
 
        tree.add([''], ['TREE_ROOT'])
983
 
        tree.commit('One', rev_id='rev1',
984
 
                    revprops={'a':'4', 'b':'3', 'c':'2', 'd':'1'})
 
1027
        tree.add([''], [b'TREE_ROOT'])
 
1028
        tree.commit('One', rev_id=b'rev1',
 
1029
                    revprops={u'a':'4', u'b':'3', u'c':'2', u'd':'1'})
985
1030
        self.b1 = tree.branch
986
 
        bundle_sio, revision_ids = self.create_bundle_text('null:', 'rev1')
 
1031
        bundle_sio, revision_ids = self.create_bundle_text(b'null:', b'rev1')
987
1032
        bundle = read_bundle(bundle_sio)
988
1033
        revision_info = bundle.revisions[0]
989
 
        self.assertEqual('rev1', revision_info.revision_id)
 
1034
        self.assertEqual(b'rev1', revision_info.revision_id)
990
1035
        rev = revision_info.as_revision()
991
1036
        self.assertEqual({'branch-nick':'tree', 'a':'4', 'b':'3', 'c':'2',
992
1037
                          'd':'1'}, rev.properties)
997
1042
        tree.lock_write()
998
1043
        self.addCleanup(tree.unlock)
999
1044
 
1000
 
        tree.add([''], ['TREE_ROOT'])
 
1045
        tree.add([''], [b'TREE_ROOT'])
1001
1046
        # Revisions themselves do not require anything about revision property
1002
1047
        # keys, other than that they are a basestring, and do not contain
1003
1048
        # whitespace.
1004
1049
        # However, Testaments assert than they are str(), and thus should not
1005
1050
        # be Unicode.
1006
 
        tree.commit('One', rev_id='rev1',
1007
 
                    revprops={'omega':u'\u03a9', 'alpha':u'\u03b1'})
 
1051
        tree.commit('One', rev_id=b'rev1',
 
1052
                    revprops={u'omega':u'\u03a9', u'alpha':u'\u03b1'})
1008
1053
        self.b1 = tree.branch
1009
 
        bundle_sio, revision_ids = self.create_bundle_text('null:', 'rev1')
 
1054
        bundle_sio, revision_ids = self.create_bundle_text(b'null:', b'rev1')
1010
1055
        bundle = read_bundle(bundle_sio)
1011
1056
        revision_info = bundle.revisions[0]
1012
 
        self.assertEqual('rev1', revision_info.revision_id)
 
1057
        self.assertEqual(b'rev1', revision_info.revision_id)
1013
1058
        rev = revision_info.as_revision()
1014
1059
        self.assertEqual({'branch-nick':'tree', 'omega':u'\u03a9',
1015
1060
                          'alpha':u'\u03b1'}, rev.properties)
1017
1062
    def test_bundle_with_ghosts(self):
1018
1063
        tree = self.make_branch_and_tree('tree')
1019
1064
        self.b1 = tree.branch
1020
 
        self.build_tree_contents([('tree/file', 'content1')])
 
1065
        self.build_tree_contents([('tree/file', b'content1')])
1021
1066
        tree.add(['file'])
1022
1067
        tree.commit('rev1')
1023
 
        self.build_tree_contents([('tree/file', 'content2')])
1024
 
        tree.add_parent_tree_id('ghost')
1025
 
        tree.commit('rev2', rev_id='rev2')
1026
 
        bundle = self.get_valid_bundle('null:', 'rev2')
 
1068
        self.build_tree_contents([('tree/file', b'content2')])
 
1069
        tree.add_parent_tree_id(b'ghost')
 
1070
        tree.commit('rev2', rev_id=b'rev2')
 
1071
        bundle = self.get_valid_bundle(b'null:', b'rev2')
1027
1072
 
1028
1073
    def make_simple_tree(self, format=None):
1029
1074
        tree = self.make_branch_and_tree('b1', format=format)
1034
1079
 
1035
1080
    def test_across_serializers(self):
1036
1081
        tree = self.make_simple_tree('knit')
1037
 
        tree.commit('hello', rev_id='rev1')
1038
 
        tree.commit('hello', rev_id='rev2')
1039
 
        bundle = read_bundle(self.create_bundle_text('null:', 'rev2')[0])
 
1082
        tree.commit('hello', rev_id=b'rev1')
 
1083
        tree.commit('hello', rev_id=b'rev2')
 
1084
        bundle = read_bundle(self.create_bundle_text(b'null:', b'rev2')[0])
1040
1085
        repo = self.make_repository('repo', format='dirstate-with-subtree')
1041
1086
        bundle.install_revisions(repo)
1042
 
        inv_text = repo._get_inventory_xml('rev2')
1043
 
        self.assertNotContainsRe(inv_text, 'format="5"')
1044
 
        self.assertContainsRe(inv_text, 'format="7"')
 
1087
        inv_text = repo._get_inventory_xml(b'rev2')
 
1088
        self.assertNotContainsRe(inv_text, b'format="5"')
 
1089
        self.assertContainsRe(inv_text, b'format="7"')
1045
1090
 
1046
1091
    def make_repo_with_installed_revisions(self):
1047
1092
        tree = self.make_simple_tree('knit')
1048
 
        tree.commit('hello', rev_id='rev1')
1049
 
        tree.commit('hello', rev_id='rev2')
1050
 
        bundle = read_bundle(self.create_bundle_text('null:', 'rev2')[0])
 
1093
        tree.commit('hello', rev_id=b'rev1')
 
1094
        tree.commit('hello', rev_id=b'rev2')
 
1095
        bundle = read_bundle(self.create_bundle_text(b'null:', b'rev2')[0])
1051
1096
        repo = self.make_repository('repo', format='dirstate-with-subtree')
1052
1097
        bundle.install_revisions(repo)
1053
1098
        return repo
1054
1099
 
1055
1100
    def test_across_models(self):
1056
1101
        repo = self.make_repo_with_installed_revisions()
1057
 
        inv = repo.get_inventory('rev2')
1058
 
        self.assertEqual('rev2', inv.root.revision)
 
1102
        inv = repo.get_inventory(b'rev2')
 
1103
        self.assertEqual(b'rev2', inv.root.revision)
1059
1104
        root_id = inv.root.file_id
1060
1105
        repo.lock_read()
1061
1106
        self.addCleanup(repo.unlock)
1062
 
        self.assertEqual({(root_id, 'rev1'):(),
1063
 
            (root_id, 'rev2'):((root_id, 'rev1'),)},
1064
 
            repo.texts.get_parent_map([(root_id, 'rev1'), (root_id, 'rev2')]))
 
1107
        self.assertEqual({(root_id, b'rev1'):(),
 
1108
            (root_id, b'rev2'):((root_id, b'rev1'),)},
 
1109
            repo.texts.get_parent_map([(root_id, b'rev1'), (root_id, b'rev2')]))
1065
1110
 
1066
1111
    def test_inv_hash_across_serializers(self):
1067
1112
        repo = self.make_repo_with_installed_revisions()
1068
 
        recorded_inv_sha1 = repo.get_revision('rev2').inventory_sha1
1069
 
        xml = repo._get_inventory_xml('rev2')
 
1113
        recorded_inv_sha1 = repo.get_revision(b'rev2').inventory_sha1
 
1114
        xml = repo._get_inventory_xml(b'rev2')
1070
1115
        self.assertEqual(osutils.sha_string(xml), recorded_inv_sha1)
1071
1116
 
1072
1117
    def test_across_models_incompatible(self):
1073
1118
        tree = self.make_simple_tree('dirstate-with-subtree')
1074
 
        tree.commit('hello', rev_id='rev1')
1075
 
        tree.commit('hello', rev_id='rev2')
 
1119
        tree.commit('hello', rev_id=b'rev1')
 
1120
        tree.commit('hello', rev_id=b'rev2')
1076
1121
        try:
1077
 
            bundle = read_bundle(self.create_bundle_text('null:', 'rev1')[0])
 
1122
            bundle = read_bundle(self.create_bundle_text(b'null:', b'rev1')[0])
1078
1123
        except errors.IncompatibleBundleFormat:
1079
1124
            raise tests.TestSkipped("Format 0.8 doesn't work with knit3")
1080
1125
        repo = self.make_repository('repo', format='knit')
1081
1126
        bundle.install_revisions(repo)
1082
1127
 
1083
 
        bundle = read_bundle(self.create_bundle_text('null:', 'rev2')[0])
 
1128
        bundle = read_bundle(self.create_bundle_text(b'null:', b'rev2')[0])
1084
1129
        self.assertRaises(errors.IncompatibleRevision,
1085
1130
                          bundle.install_revisions, repo)
1086
1131
 
1087
1132
    def test_get_merge_request(self):
1088
1133
        tree = self.make_simple_tree()
1089
 
        tree.commit('hello', rev_id='rev1')
1090
 
        tree.commit('hello', rev_id='rev2')
1091
 
        bundle = read_bundle(self.create_bundle_text('null:', 'rev1')[0])
 
1134
        tree.commit('hello', rev_id=b'rev1')
 
1135
        tree.commit('hello', rev_id=b'rev2')
 
1136
        bundle = read_bundle(self.create_bundle_text(b'null:', b'rev1')[0])
1092
1137
        result = bundle.get_merge_request(tree.branch.repository)
1093
 
        self.assertEqual((None, 'rev1', 'inapplicable'), result)
 
1138
        self.assertEqual((None, b'rev1', 'inapplicable'), result)
1094
1139
 
1095
1140
    def test_with_subtree(self):
1096
1141
        tree = self.make_branch_and_tree('tree',
1099
1144
        subtree = self.make_branch_and_tree('tree/subtree',
1100
1145
                                            format='dirstate-with-subtree')
1101
1146
        tree.add('subtree')
1102
 
        tree.commit('hello', rev_id='rev1')
 
1147
        tree.commit('hello', rev_id=b'rev1')
1103
1148
        try:
1104
 
            bundle = read_bundle(self.create_bundle_text('null:', 'rev1')[0])
 
1149
            bundle = read_bundle(self.create_bundle_text(b'null:', b'rev1')[0])
1105
1150
        except errors.IncompatibleBundleFormat:
1106
1151
            raise tests.TestSkipped("Format 0.8 doesn't work with knit3")
1107
1152
        if isinstance(bundle, v09.BundleInfo09):
1116
1161
        self.tree1 = self.make_branch_and_tree('tree')
1117
1162
        self.b1 = self.tree1.branch
1118
1163
        try:
1119
 
            self.tree1.commit('Revision/id/with/slashes', rev_id='rev/id')
 
1164
            self.tree1.commit('Revision/id/with/slashes', rev_id=b'rev/id')
1120
1165
        except ValueError:
1121
1166
            raise tests.TestSkipped(
1122
1167
                "Repository doesn't support revision ids with slashes")
1123
 
        bundle = self.get_valid_bundle('null:', 'rev/id')
 
1168
        bundle = self.get_valid_bundle(b'null:', b'rev/id')
1124
1169
 
1125
1170
    def test_skip_file(self):
1126
1171
        """Make sure we don't accidentally write to the wrong versionedfile"""
1127
1172
        self.tree1 = self.make_branch_and_tree('tree')
1128
1173
        self.b1 = self.tree1.branch
1129
1174
        # rev1 is not present in bundle, done by fetch
1130
 
        self.build_tree_contents([('tree/file2', 'contents1')])
1131
 
        self.tree1.add('file2', 'file2-id')
1132
 
        self.tree1.commit('rev1', rev_id='reva')
1133
 
        self.build_tree_contents([('tree/file3', 'contents2')])
 
1175
        self.build_tree_contents([('tree/file2', b'contents1')])
 
1176
        self.tree1.add('file2', b'file2-id')
 
1177
        self.tree1.commit('rev1', rev_id=b'reva')
 
1178
        self.build_tree_contents([('tree/file3', b'contents2')])
1134
1179
        # rev2 is present in bundle, and done by fetch
1135
1180
        # having file1 in the bunle causes file1's versionedfile to be opened.
1136
 
        self.tree1.add('file3', 'file3-id')
1137
 
        self.tree1.commit('rev2')
 
1181
        self.tree1.add('file3', b'file3-id')
 
1182
        rev2 = self.tree1.commit('rev2')
1138
1183
        # Updating file2 should not cause an attempt to add to file1's vf
1139
 
        target = self.tree1.bzrdir.sprout('target').open_workingtree()
1140
 
        self.build_tree_contents([('tree/file2', 'contents3')])
1141
 
        self.tree1.commit('rev3', rev_id='rev3')
1142
 
        bundle = self.get_valid_bundle('reva', 'rev3')
 
1184
        target = self.tree1.controldir.sprout('target').open_workingtree()
 
1185
        self.build_tree_contents([('tree/file2', b'contents3')])
 
1186
        self.tree1.commit('rev3', rev_id=b'rev3')
 
1187
        bundle = self.get_valid_bundle(b'reva', b'rev3')
1143
1188
        if getattr(bundle, 'get_bundle_reader', None) is None:
1144
1189
            raise tests.TestSkipped('Bundle format cannot provide reader')
1145
 
        # be sure that file1 comes before file2
1146
 
        for b, m, k, r, f in bundle.get_bundle_reader().iter_records():
1147
 
            if f == 'file3-id':
1148
 
                break
1149
 
            self.assertNotEqual(f, 'file2-id')
 
1190
        file_ids = set(
 
1191
            (f, r) for b, m, k, r, f in bundle.get_bundle_reader().iter_records()
 
1192
            if f is not None)
 
1193
        self.assertEqual({(b'file2-id', b'rev3'), (b'file3-id', rev2)}, file_ids)
1150
1194
        bundle.install_revisions(target.branch.repository)
1151
1195
 
1152
1196
 
1159
1203
        tree = self.make_branch_and_memory_tree('tree')
1160
1204
        tree.lock_write()
1161
1205
        self.addCleanup(tree.unlock)
1162
 
        tree.add([''], ['TREE_ROOT'])
1163
 
        tree.commit('One', revprops={'one':'two', 'empty':''}, rev_id='rev1')
 
1206
        tree.add([''], [b'TREE_ROOT'])
 
1207
        tree.commit('One', revprops={u'one':'two', u'empty':''}, rev_id=b'rev1')
1164
1208
        self.b1 = tree.branch
1165
 
        bundle_sio, revision_ids = self.create_bundle_text('null:', 'rev1')
 
1209
        bundle_sio, revision_ids = self.create_bundle_text(b'null:', b'rev1')
1166
1210
        self.assertContainsRe(bundle_sio.getvalue(),
1167
 
                              '# properties:\n'
1168
 
                              '#   branch-nick: tree\n'
1169
 
                              '#   empty: \n'
1170
 
                              '#   one: two\n'
 
1211
                              b'# properties:\n'
 
1212
                              b'#   branch-nick: tree\n'
 
1213
                              b'#   empty: \n'
 
1214
                              b'#   one: two\n'
1171
1215
                             )
1172
1216
        bundle = read_bundle(bundle_sio)
1173
1217
        revision_info = bundle.revisions[0]
1174
 
        self.assertEqual('rev1', revision_info.revision_id)
 
1218
        self.assertEqual(b'rev1', revision_info.revision_id)
1175
1219
        rev = revision_info.as_revision()
1176
1220
        self.assertEqual({'branch-nick':'tree', 'empty':'', 'one':'two'},
1177
1221
                         rev.properties)
1178
1222
 
1179
1223
    def get_bundle_tree(self, bundle, revision_id):
1180
1224
        repository = self.make_repository('repo')
1181
 
        return bundle.revision_tree(repository, 'revid1')
 
1225
        return bundle.revision_tree(repository, b'revid1')
1182
1226
 
1183
1227
    def test_bundle_empty_property_alt(self):
1184
1228
        """Test serializing revision properties with an empty value.
1191
1235
        tree = self.make_branch_and_memory_tree('tree')
1192
1236
        tree.lock_write()
1193
1237
        self.addCleanup(tree.unlock)
1194
 
        tree.add([''], ['TREE_ROOT'])
1195
 
        tree.commit('One', revprops={'one':'two', 'empty':''}, rev_id='rev1')
 
1238
        tree.add([''], [b'TREE_ROOT'])
 
1239
        tree.commit('One', revprops={u'one':'two', u'empty':''}, rev_id=b'rev1')
1196
1240
        self.b1 = tree.branch
1197
 
        bundle_sio, revision_ids = self.create_bundle_text('null:', 'rev1')
 
1241
        bundle_sio, revision_ids = self.create_bundle_text(b'null:', b'rev1')
1198
1242
        txt = bundle_sio.getvalue()
1199
 
        loc = txt.find('#   empty: ') + len('#   empty:')
 
1243
        loc = txt.find(b'#   empty: ') + len(b'#   empty:')
1200
1244
        # Create a new bundle, which strips the trailing space after empty
1201
 
        bundle_sio = StringIO(txt[:loc] + txt[loc+1:])
 
1245
        bundle_sio = BytesIO(txt[:loc] + txt[loc+1:])
1202
1246
 
1203
1247
        self.assertContainsRe(bundle_sio.getvalue(),
1204
 
                              '# properties:\n'
1205
 
                              '#   branch-nick: tree\n'
1206
 
                              '#   empty:\n'
1207
 
                              '#   one: two\n'
 
1248
                              b'# properties:\n'
 
1249
                              b'#   branch-nick: tree\n'
 
1250
                              b'#   empty:\n'
 
1251
                              b'#   one: two\n'
1208
1252
                             )
1209
1253
        bundle = read_bundle(bundle_sio)
1210
1254
        revision_info = bundle.revisions[0]
1211
 
        self.assertEqual('rev1', revision_info.revision_id)
 
1255
        self.assertEqual(b'rev1', revision_info.revision_id)
1212
1256
        rev = revision_info.as_revision()
1213
1257
        self.assertEqual({'branch-nick':'tree', 'empty':'', 'one':'two'},
1214
1258
                         rev.properties)
1219
1263
        tree.lock_write()
1220
1264
        self.addCleanup(tree.unlock)
1221
1265
 
1222
 
        tree.add([''], ['TREE_ROOT'])
1223
 
        tree.commit('One', rev_id='rev1',
1224
 
                    revprops={'a':'4', 'b':'3', 'c':'2', 'd':'1'})
 
1266
        tree.add([''], [b'TREE_ROOT'])
 
1267
        tree.commit('One', rev_id=b'rev1',
 
1268
                    revprops={u'a':'4', u'b':'3', u'c':'2', u'd':'1'})
1225
1269
        self.b1 = tree.branch
1226
 
        bundle_sio, revision_ids = self.create_bundle_text('null:', 'rev1')
 
1270
        bundle_sio, revision_ids = self.create_bundle_text(b'null:', b'rev1')
1227
1271
        self.assertContainsRe(bundle_sio.getvalue(),
1228
 
                              '# properties:\n'
1229
 
                              '#   a: 4\n'
1230
 
                              '#   b: 3\n'
1231
 
                              '#   branch-nick: tree\n'
1232
 
                              '#   c: 2\n'
1233
 
                              '#   d: 1\n'
 
1272
                              b'# properties:\n'
 
1273
                              b'#   a: 4\n'
 
1274
                              b'#   b: 3\n'
 
1275
                              b'#   branch-nick: tree\n'
 
1276
                              b'#   c: 2\n'
 
1277
                              b'#   d: 1\n'
1234
1278
                             )
1235
1279
        bundle = read_bundle(bundle_sio)
1236
1280
        revision_info = bundle.revisions[0]
1237
 
        self.assertEqual('rev1', revision_info.revision_id)
 
1281
        self.assertEqual(b'rev1', revision_info.revision_id)
1238
1282
        rev = revision_info.as_revision()
1239
1283
        self.assertEqual({'branch-nick':'tree', 'a':'4', 'b':'3', 'c':'2',
1240
1284
                          'd':'1'}, rev.properties)
1245
1289
        tree.lock_write()
1246
1290
        self.addCleanup(tree.unlock)
1247
1291
 
1248
 
        tree.add([''], ['TREE_ROOT'])
 
1292
        tree.add([''], [b'TREE_ROOT'])
1249
1293
        # Revisions themselves do not require anything about revision property
1250
1294
        # keys, other than that they are a basestring, and do not contain
1251
1295
        # whitespace.
1252
1296
        # However, Testaments assert than they are str(), and thus should not
1253
1297
        # be Unicode.
1254
 
        tree.commit('One', rev_id='rev1',
1255
 
                    revprops={'omega':u'\u03a9', 'alpha':u'\u03b1'})
 
1298
        tree.commit('One', rev_id=b'rev1',
 
1299
                    revprops={u'omega':u'\u03a9', u'alpha':u'\u03b1'})
1256
1300
        self.b1 = tree.branch
1257
 
        bundle_sio, revision_ids = self.create_bundle_text('null:', 'rev1')
 
1301
        bundle_sio, revision_ids = self.create_bundle_text(b'null:', b'rev1')
1258
1302
        self.assertContainsRe(bundle_sio.getvalue(),
1259
 
                              '# properties:\n'
1260
 
                              '#   alpha: \xce\xb1\n'
1261
 
                              '#   branch-nick: tree\n'
1262
 
                              '#   omega: \xce\xa9\n'
 
1303
                              b'# properties:\n'
 
1304
                              b'#   alpha: \xce\xb1\n'
 
1305
                              b'#   branch-nick: tree\n'
 
1306
                              b'#   omega: \xce\xa9\n'
1263
1307
                             )
1264
1308
        bundle = read_bundle(bundle_sio)
1265
1309
        revision_info = bundle.revisions[0]
1266
 
        self.assertEqual('rev1', revision_info.revision_id)
 
1310
        self.assertEqual(b'rev1', revision_info.revision_id)
1267
1311
        rev = revision_info.as_revision()
1268
1312
        self.assertEqual({'branch-nick':'tree', 'omega':u'\u03a9',
1269
1313
                          'alpha':u'\u03b1'}, rev.properties)
1319
1363
            self.assertEqual(len(branch_rev.parent_ids),
1320
1364
                             len(bundle_rev.parent_ids))
1321
1365
        self.assertEqual(set(rev_ids),
1322
 
                         set([r.revision_id for r in bundle.real_revisions]))
 
1366
                         {r.revision_id for r in bundle.real_revisions})
1323
1367
        self.valid_apply_bundle(base_rev_id, bundle,
1324
1368
                                   checkout_dir=checkout_dir)
1325
1369
 
1331
1375
 
1332
1376
        :return: The in-memory bundle
1333
1377
        """
1334
 
        from bzrlib.bundle import serializer
 
1378
        from ..bundle import serializer
1335
1379
        bundle_txt, rev_ids = self.create_bundle_text(base_rev_id, rev_id)
1336
 
        new_text = self.get_raw(StringIO(''.join(bundle_txt)))
1337
 
        new_text = new_text.replace('<file file_id="exe-1"',
1338
 
                                    '<file executable="y" file_id="exe-1"')
1339
 
        new_text = new_text.replace('B260', 'B275')
1340
 
        bundle_txt = StringIO()
 
1380
        new_text = self.get_raw(BytesIO(b''.join(bundle_txt)))
 
1381
        new_text = new_text.replace(b'<file file_id="exe-1"',
 
1382
                                    b'<file executable="y" file_id="exe-1"')
 
1383
        new_text = new_text.replace(b'B260', b'B275')
 
1384
        bundle_txt = BytesIO()
1341
1385
        bundle_txt.write(serializer._get_bundle_header('4'))
1342
 
        bundle_txt.write('\n')
1343
 
        bundle_txt.write(new_text.encode('bz2'))
 
1386
        bundle_txt.write(b'\n')
 
1387
        bundle_txt.write(bz2.compress(new_text))
1344
1388
        bundle_txt.seek(0)
1345
1389
        bundle = read_bundle(bundle_txt)
1346
1390
        self.valid_apply_bundle(base_rev_id, bundle)
1347
1391
        return bundle
1348
1392
 
1349
1393
    def create_bundle_text(self, base_rev_id, rev_id):
1350
 
        bundle_txt = StringIO()
 
1394
        bundle_txt = BytesIO()
1351
1395
        rev_ids = write_bundle(self.b1.repository, rev_id, base_rev_id,
1352
1396
                               bundle_txt, format=self.format)
1353
1397
        bundle_txt.seek(0)
1354
1398
        self.assertEqual(bundle_txt.readline(),
1355
 
                         '# Bazaar revision bundle v%s\n' % self.format)
1356
 
        self.assertEqual(bundle_txt.readline(), '#\n')
 
1399
                         b'# Bazaar revision bundle v%s\n' % self.format.encode('ascii'))
 
1400
        self.assertEqual(bundle_txt.readline(), b'#\n')
1357
1401
        rev = self.b1.repository.get_revision(rev_id)
1358
1402
        bundle_txt.seek(0)
1359
1403
        return bundle_txt, rev_ids
1365
1409
 
1366
1410
    def test_creation(self):
1367
1411
        tree = self.make_branch_and_tree('tree')
1368
 
        self.build_tree_contents([('tree/file', 'contents1\nstatic\n')])
1369
 
        tree.add('file', 'fileid-2')
1370
 
        tree.commit('added file', rev_id='rev1')
1371
 
        self.build_tree_contents([('tree/file', 'contents2\nstatic\n')])
1372
 
        tree.commit('changed file', rev_id='rev2')
1373
 
        s = StringIO()
 
1412
        self.build_tree_contents([('tree/file', b'contents1\nstatic\n')])
 
1413
        tree.add('file', b'fileid-2')
 
1414
        tree.commit('added file', rev_id=b'rev1')
 
1415
        self.build_tree_contents([('tree/file', b'contents2\nstatic\n')])
 
1416
        tree.commit('changed file', rev_id=b'rev2')
 
1417
        s = BytesIO()
1374
1418
        serializer = BundleSerializerV4('1.0')
1375
 
        serializer.write(tree.branch.repository, ['rev1', 'rev2'], {}, s)
 
1419
        with tree.lock_read():
 
1420
            serializer.write_bundle(tree.branch.repository, b'rev2', b'null:', s)
1376
1421
        s.seek(0)
1377
1422
        tree2 = self.make_branch_and_tree('target')
1378
1423
        target_repo = tree2.branch.repository
1380
1425
        target_repo.lock_read()
1381
1426
        self.addCleanup(target_repo.unlock)
1382
1427
        # Turn the 'iterators_of_bytes' back into simple strings for comparison
1383
 
        repo_texts = dict((i, ''.join(content)) for i, content
 
1428
        repo_texts = dict((i, b''.join(content)) for i, content
1384
1429
                          in target_repo.iter_files_bytes(
1385
 
                                [('fileid-2', 'rev1', '1'),
1386
 
                                 ('fileid-2', 'rev2', '2')]))
1387
 
        self.assertEqual({'1':'contents1\nstatic\n',
1388
 
                          '2':'contents2\nstatic\n'},
 
1430
                                [(b'fileid-2', b'rev1', '1'),
 
1431
                                 (b'fileid-2', b'rev2', '2')]))
 
1432
        self.assertEqual({'1':b'contents1\nstatic\n',
 
1433
                          '2':b'contents2\nstatic\n'},
1389
1434
                         repo_texts)
1390
 
        rtree = target_repo.revision_tree('rev2')
 
1435
        rtree = target_repo.revision_tree(b'rev2')
1391
1436
        inventory_vf = target_repo.inventories
1392
1437
        # If the inventory store has a graph, it must match the revision graph.
1393
1438
        self.assertSubset(
1394
 
            [inventory_vf.get_parent_map([('rev2',)])[('rev2',)]],
1395
 
            [None, (('rev1',),)])
 
1439
            [inventory_vf.get_parent_map([(b'rev2',)])[(b'rev2',)]],
 
1440
            [None, ((b'rev1',),)])
1396
1441
        self.assertEqual('changed file',
1397
 
                         target_repo.get_revision('rev2').message)
 
1442
                         target_repo.get_revision(b'rev2').message)
1398
1443
 
1399
1444
    @staticmethod
1400
1445
    def get_raw(bundle_file):
1402
1447
        line = bundle_file.readline()
1403
1448
        line = bundle_file.readline()
1404
1449
        lines = bundle_file.readlines()
1405
 
        return ''.join(lines).decode('bz2')
 
1450
        return bz2.decompress(b''.join(lines))
1406
1451
 
1407
1452
    def test_copy_signatures(self):
1408
1453
        tree_a = self.make_branch_and_tree('tree_a')
1409
 
        import bzrlib.gpg
1410
 
        import bzrlib.commit as commit
1411
 
        oldstrategy = bzrlib.gpg.GPGStrategy
 
1454
        import breezy.gpg
 
1455
        import breezy.commit as commit
 
1456
        oldstrategy = breezy.gpg.GPGStrategy
1412
1457
        branch = tree_a.branch
1413
1458
        repo_a = branch.repository
1414
 
        tree_a.commit("base", allow_pointless=True, rev_id='A')
1415
 
        self.failIf(branch.repository.has_signature_for_revision_id('A'))
 
1459
        tree_a.commit("base", allow_pointless=True, rev_id=b'A')
 
1460
        self.assertFalse(branch.repository.has_signature_for_revision_id(b'A'))
1416
1461
        try:
1417
 
            from bzrlib.testament import Testament
 
1462
            from ..testament import Testament
1418
1463
            # monkey patch gpg signing mechanism
1419
 
            bzrlib.gpg.GPGStrategy = bzrlib.gpg.LoopbackGPGStrategy
1420
 
            new_config = test_commit.MustSignConfig(branch)
1421
 
            commit.Commit(config=new_config).commit(message="base",
 
1464
            breezy.gpg.GPGStrategy = breezy.gpg.LoopbackGPGStrategy
 
1465
            new_config = test_commit.MustSignConfig()
 
1466
            commit.Commit(config_stack=new_config).commit(message="base",
1422
1467
                                                    allow_pointless=True,
1423
 
                                                    rev_id='B',
 
1468
                                                    rev_id=b'B',
1424
1469
                                                    working_tree=tree_a)
1425
1470
            def sign(text):
1426
 
                return bzrlib.gpg.LoopbackGPGStrategy(None).sign(text)
1427
 
            self.assertTrue(repo_a.has_signature_for_revision_id('B'))
 
1471
                return breezy.gpg.LoopbackGPGStrategy(None).sign(text)
 
1472
            self.assertTrue(repo_a.has_signature_for_revision_id(b'B'))
1428
1473
        finally:
1429
 
            bzrlib.gpg.GPGStrategy = oldstrategy
 
1474
            breezy.gpg.GPGStrategy = oldstrategy
1430
1475
        tree_b = self.make_branch_and_tree('tree_b')
1431
1476
        repo_b = tree_b.branch.repository
1432
 
        s = StringIO()
 
1477
        s = BytesIO()
1433
1478
        serializer = BundleSerializerV4('4')
1434
 
        serializer.write(tree_a.branch.repository, ['A', 'B'], {}, s)
 
1479
        with tree_a.lock_read():
 
1480
            serializer.write_bundle(tree_a.branch.repository, b'B', b'null:', s)
1435
1481
        s.seek(0)
1436
1482
        install_bundle(repo_b, serializer.read(s))
1437
 
        self.assertTrue(repo_b.has_signature_for_revision_id('B'))
1438
 
        self.assertEqual(repo_b.get_signature_text('B'),
1439
 
                         repo_a.get_signature_text('B'))
 
1483
        self.assertTrue(repo_b.has_signature_for_revision_id(b'B'))
 
1484
        self.assertEqual(repo_b.get_signature_text(b'B'),
 
1485
                         repo_a.get_signature_text(b'B'))
1440
1486
        s.seek(0)
1441
1487
        # ensure repeat installs are harmless
1442
1488
        install_bundle(repo_b, serializer.read(s))
1443
1489
 
1444
1490
 
1445
 
class V4WeaveBundleTester(V4BundleTester):
1446
 
 
1447
 
    def bzrdir_format(self):
1448
 
        return 'metaweave'
1449
 
 
1450
 
 
1451
1491
class V4_2aBundleTester(V4BundleTester):
1452
1492
 
1453
1493
    def bzrdir_format(self):
1459
1499
 
1460
1500
        :return: The in-memory bundle
1461
1501
        """
1462
 
        from bzrlib.bundle import serializer
 
1502
        from ..bundle import serializer
1463
1503
        bundle_txt, rev_ids = self.create_bundle_text(base_rev_id, rev_id)
1464
 
        new_text = self.get_raw(StringIO(''.join(bundle_txt)))
 
1504
        new_text = self.get_raw(BytesIO(b''.join(bundle_txt)))
1465
1505
        # We are going to be replacing some text to set the executable bit on a
1466
1506
        # file. Make sure the text replacement actually works correctly.
1467
 
        self.assertContainsRe(new_text, '(?m)B244\n\ni 1\n<inventory')
1468
 
        new_text = new_text.replace('<file file_id="exe-1"',
1469
 
                                    '<file executable="y" file_id="exe-1"')
1470
 
        new_text = new_text.replace('B244', 'B259')
1471
 
        bundle_txt = StringIO()
 
1507
        self.assertContainsRe(new_text, b'(?m)B244\n\ni 1\n<inventory')
 
1508
        new_text = new_text.replace(b'<file file_id="exe-1"',
 
1509
                                    b'<file executable="y" file_id="exe-1"')
 
1510
        new_text = new_text.replace(b'B244', b'B259')
 
1511
        bundle_txt = BytesIO()
1472
1512
        bundle_txt.write(serializer._get_bundle_header('4'))
1473
 
        bundle_txt.write('\n')
1474
 
        bundle_txt.write(new_text.encode('bz2'))
 
1513
        bundle_txt.write(b'\n')
 
1514
        bundle_txt.write(bz2.compress(new_text))
1475
1515
        bundle_txt.seek(0)
1476
1516
        bundle = read_bundle(bundle_txt)
1477
1517
        self.valid_apply_bundle(base_rev_id, bundle)
1480
1520
    def make_merged_branch(self):
1481
1521
        builder = self.make_branch_builder('source')
1482
1522
        builder.start_series()
1483
 
        builder.build_snapshot('a@cset-0-1', None, [
1484
 
            ('add', ('', 'root-id', 'directory', None)),
1485
 
            ('add', ('file', 'file-id', 'file', 'original content\n')),
1486
 
            ])
1487
 
        builder.build_snapshot('a@cset-0-2a', ['a@cset-0-1'], [
1488
 
            ('modify', ('file-id', 'new-content\n')),
1489
 
            ])
1490
 
        builder.build_snapshot('a@cset-0-2b', ['a@cset-0-1'], [
1491
 
            ('add', ('other-file', 'file2-id', 'file', 'file2-content\n')),
1492
 
            ])
1493
 
        builder.build_snapshot('a@cset-0-3', ['a@cset-0-2a', 'a@cset-0-2b'], [
1494
 
            ('add', ('other-file', 'file2-id', 'file', 'file2-content\n')),
1495
 
            ])
 
1523
        builder.build_snapshot(None, [
 
1524
            ('add', ('', b'root-id', 'directory', None)),
 
1525
            ('add', ('file', b'file-id', 'file', b'original content\n')),
 
1526
            ], revision_id=b'a@cset-0-1')
 
1527
        builder.build_snapshot([b'a@cset-0-1'], [
 
1528
            ('modify', ('file', b'new-content\n')),
 
1529
            ], revision_id=b'a@cset-0-2a')
 
1530
        builder.build_snapshot([b'a@cset-0-1'], [
 
1531
            ('add', ('other-file', b'file2-id', 'file', b'file2-content\n')),
 
1532
            ], revision_id=b'a@cset-0-2b')
 
1533
        builder.build_snapshot([b'a@cset-0-2a', b'a@cset-0-2b'], [
 
1534
            ('add', ('other-file', b'file2-id', 'file', b'file2-content\n')),
 
1535
            ], revision_id=b'a@cset-0-3')
1496
1536
        builder.finish_series()
1497
1537
        self.b1 = builder.get_branch()
1498
1538
        self.b1.lock_read()
1501
1541
    def make_bundle_just_inventories(self, base_revision_id,
1502
1542
                                     target_revision_id,
1503
1543
                                     revision_ids):
1504
 
        sio = StringIO()
 
1544
        sio = BytesIO()
1505
1545
        writer = v4.BundleWriteOperation(base_revision_id, target_revision_id,
1506
1546
                                         self.b1.repository, sio)
1507
1547
        writer.bundle.begin()
1512
1552
 
1513
1553
    def test_single_inventory_multiple_parents_as_xml(self):
1514
1554
        self.make_merged_branch()
1515
 
        sio = self.make_bundle_just_inventories('a@cset-0-1', 'a@cset-0-3',
1516
 
                                                ['a@cset-0-3'])
 
1555
        sio = self.make_bundle_just_inventories(b'a@cset-0-1', b'a@cset-0-3',
 
1556
                                                [b'a@cset-0-3'])
1517
1557
        reader = v4.BundleReader(sio, stream_input=False)
1518
1558
        records = list(reader.iter_records())
1519
1559
        self.assertEqual(1, len(records))
1520
1560
        (bytes, metadata, repo_kind, revision_id,
1521
1561
         file_id) = records[0]
1522
1562
        self.assertIs(None, file_id)
1523
 
        self.assertEqual('a@cset-0-3', revision_id)
 
1563
        self.assertEqual(b'a@cset-0-3', revision_id)
1524
1564
        self.assertEqual('inventory', repo_kind)
1525
 
        self.assertEqual({'parents': ['a@cset-0-2a', 'a@cset-0-2b'],
1526
 
                          'sha1': '09c53b0c4de0895e11a2aacc34fef60a6e70865c',
1527
 
                          'storage_kind': 'mpdiff',
 
1565
        self.assertEqual({b'parents': [b'a@cset-0-2a', b'a@cset-0-2b'],
 
1566
                          b'sha1': b'09c53b0c4de0895e11a2aacc34fef60a6e70865c',
 
1567
                          b'storage_kind': b'mpdiff',
1528
1568
                         }, metadata)
1529
1569
        # We should have an mpdiff that takes some lines from both parents.
1530
1570
        self.assertEqualDiff(
1531
 
            'i 1\n'
1532
 
            '<inventory format="10" revision_id="a@cset-0-3">\n'
1533
 
            '\n'
1534
 
            'c 0 1 1 2\n'
1535
 
            'c 1 3 3 2\n', bytes)
 
1571
            b'i 1\n'
 
1572
            b'<inventory format="10" revision_id="a@cset-0-3">\n'
 
1573
            b'\n'
 
1574
            b'c 0 1 1 2\n'
 
1575
            b'c 1 3 3 2\n', bytes)
1536
1576
 
1537
1577
    def test_single_inv_no_parents_as_xml(self):
1538
1578
        self.make_merged_branch()
1539
 
        sio = self.make_bundle_just_inventories('null:', 'a@cset-0-1',
1540
 
                                                ['a@cset-0-1'])
 
1579
        sio = self.make_bundle_just_inventories(b'null:', b'a@cset-0-1',
 
1580
                                                [b'a@cset-0-1'])
1541
1581
        reader = v4.BundleReader(sio, stream_input=False)
1542
1582
        records = list(reader.iter_records())
1543
1583
        self.assertEqual(1, len(records))
1544
1584
        (bytes, metadata, repo_kind, revision_id,
1545
1585
         file_id) = records[0]
1546
1586
        self.assertIs(None, file_id)
1547
 
        self.assertEqual('a@cset-0-1', revision_id)
 
1587
        self.assertEqual(b'a@cset-0-1', revision_id)
1548
1588
        self.assertEqual('inventory', repo_kind)
1549
 
        self.assertEqual({'parents': [],
1550
 
                          'sha1': 'a13f42b142d544aac9b085c42595d304150e31a2',
1551
 
                          'storage_kind': 'mpdiff',
 
1589
        self.assertEqual({b'parents': [],
 
1590
                          b'sha1': b'a13f42b142d544aac9b085c42595d304150e31a2',
 
1591
                          b'storage_kind': b'mpdiff',
1552
1592
                         }, metadata)
1553
1593
        # We should have an mpdiff that takes some lines from both parents.
1554
1594
        self.assertEqualDiff(
1555
 
            'i 4\n'
1556
 
            '<inventory format="10" revision_id="a@cset-0-1">\n'
1557
 
            '<directory file_id="root-id" name=""'
1558
 
                ' revision="a@cset-0-1" />\n'
1559
 
            '<file file_id="file-id" name="file" parent_id="root-id"'
1560
 
                ' revision="a@cset-0-1"'
1561
 
                ' text_sha1="09c2f8647e14e49e922b955c194102070597c2d1"'
1562
 
                ' text_size="17" />\n'
1563
 
            '</inventory>\n'
1564
 
            '\n', bytes)
 
1595
            b'i 4\n'
 
1596
            b'<inventory format="10" revision_id="a@cset-0-1">\n'
 
1597
            b'<directory file_id="root-id" name=""'
 
1598
                b' revision="a@cset-0-1" />\n'
 
1599
            b'<file file_id="file-id" name="file" parent_id="root-id"'
 
1600
                b' revision="a@cset-0-1"'
 
1601
                b' text_sha1="09c2f8647e14e49e922b955c194102070597c2d1"'
 
1602
                b' text_size="17" />\n'
 
1603
            b'</inventory>\n'
 
1604
            b'\n', bytes)
1565
1605
 
1566
1606
    def test_multiple_inventories_as_xml(self):
1567
1607
        self.make_merged_branch()
1568
 
        sio = self.make_bundle_just_inventories('a@cset-0-1', 'a@cset-0-3',
1569
 
            ['a@cset-0-2a', 'a@cset-0-2b', 'a@cset-0-3'])
 
1608
        sio = self.make_bundle_just_inventories(b'a@cset-0-1', b'a@cset-0-3',
 
1609
            [b'a@cset-0-2a', b'a@cset-0-2b', b'a@cset-0-3'])
1570
1610
        reader = v4.BundleReader(sio, stream_input=False)
1571
1611
        records = list(reader.iter_records())
1572
1612
        self.assertEqual(3, len(records))
1573
1613
        revision_ids = [rev_id for b, m, k, rev_id, f in records]
1574
 
        self.assertEqual(['a@cset-0-2a', 'a@cset-0-2b', 'a@cset-0-3'],
 
1614
        self.assertEqual([b'a@cset-0-2a', b'a@cset-0-2b', b'a@cset-0-3'],
1575
1615
                         revision_ids)
1576
1616
        metadata_2a = records[0][1]
1577
 
        self.assertEqual({'parents': ['a@cset-0-1'],
1578
 
                          'sha1': '1e105886d62d510763e22885eec733b66f5f09bf',
1579
 
                          'storage_kind': 'mpdiff',
 
1617
        self.assertEqual({b'parents': [b'a@cset-0-1'],
 
1618
                          b'sha1': b'1e105886d62d510763e22885eec733b66f5f09bf',
 
1619
                          b'storage_kind': b'mpdiff',
1580
1620
                         }, metadata_2a)
1581
1621
        metadata_2b = records[1][1]
1582
 
        self.assertEqual({'parents': ['a@cset-0-1'],
1583
 
                          'sha1': 'f03f12574bdb5ed2204c28636c98a8547544ccd8',
1584
 
                          'storage_kind': 'mpdiff',
 
1622
        self.assertEqual({b'parents': [b'a@cset-0-1'],
 
1623
                          b'sha1': b'f03f12574bdb5ed2204c28636c98a8547544ccd8',
 
1624
                          b'storage_kind': b'mpdiff',
1585
1625
                         }, metadata_2b)
1586
1626
        metadata_3 = records[2][1]
1587
 
        self.assertEqual({'parents': ['a@cset-0-2a', 'a@cset-0-2b'],
1588
 
                          'sha1': '09c53b0c4de0895e11a2aacc34fef60a6e70865c',
1589
 
                          'storage_kind': 'mpdiff',
 
1627
        self.assertEqual({b'parents': [b'a@cset-0-2a', b'a@cset-0-2b'],
 
1628
                          b'sha1': b'09c53b0c4de0895e11a2aacc34fef60a6e70865c',
 
1629
                          b'storage_kind': b'mpdiff',
1590
1630
                         }, metadata_3)
1591
1631
        bytes_2a = records[0][0]
1592
1632
        self.assertEqualDiff(
1593
 
            'i 1\n'
1594
 
            '<inventory format="10" revision_id="a@cset-0-2a">\n'
1595
 
            '\n'
1596
 
            'c 0 1 1 1\n'
1597
 
            'i 1\n'
1598
 
            '<file file_id="file-id" name="file" parent_id="root-id"'
1599
 
                ' revision="a@cset-0-2a"'
1600
 
                ' text_sha1="50f545ff40e57b6924b1f3174b267ffc4576e9a9"'
1601
 
                ' text_size="12" />\n'
1602
 
            '\n'
1603
 
            'c 0 3 3 1\n', bytes_2a)
 
1633
            b'i 1\n'
 
1634
            b'<inventory format="10" revision_id="a@cset-0-2a">\n'
 
1635
            b'\n'
 
1636
            b'c 0 1 1 1\n'
 
1637
            b'i 1\n'
 
1638
            b'<file file_id="file-id" name="file" parent_id="root-id"'
 
1639
                b' revision="a@cset-0-2a"'
 
1640
                b' text_sha1="50f545ff40e57b6924b1f3174b267ffc4576e9a9"'
 
1641
                b' text_size="12" />\n'
 
1642
            b'\n'
 
1643
            b'c 0 3 3 1\n', bytes_2a)
1604
1644
        bytes_2b = records[1][0]
1605
1645
        self.assertEqualDiff(
1606
 
            'i 1\n'
1607
 
            '<inventory format="10" revision_id="a@cset-0-2b">\n'
1608
 
            '\n'
1609
 
            'c 0 1 1 2\n'
1610
 
            'i 1\n'
1611
 
            '<file file_id="file2-id" name="other-file" parent_id="root-id"'
1612
 
                ' revision="a@cset-0-2b"'
1613
 
                ' text_sha1="b46c0c8ea1e5ef8e46fc8894bfd4752a88ec939e"'
1614
 
                ' text_size="14" />\n'
1615
 
            '\n'
1616
 
            'c 0 3 4 1\n', bytes_2b)
 
1646
            b'i 1\n'
 
1647
            b'<inventory format="10" revision_id="a@cset-0-2b">\n'
 
1648
            b'\n'
 
1649
            b'c 0 1 1 2\n'
 
1650
            b'i 1\n'
 
1651
            b'<file file_id="file2-id" name="other-file" parent_id="root-id"'
 
1652
                b' revision="a@cset-0-2b"'
 
1653
                b' text_sha1="b46c0c8ea1e5ef8e46fc8894bfd4752a88ec939e"'
 
1654
                b' text_size="14" />\n'
 
1655
            b'\n'
 
1656
            b'c 0 3 4 1\n', bytes_2b)
1617
1657
        bytes_3 = records[2][0]
1618
1658
        self.assertEqualDiff(
1619
 
            'i 1\n'
1620
 
            '<inventory format="10" revision_id="a@cset-0-3">\n'
1621
 
            '\n'
1622
 
            'c 0 1 1 2\n'
1623
 
            'c 1 3 3 2\n', bytes_3)
 
1659
            b'i 1\n'
 
1660
            b'<inventory format="10" revision_id="a@cset-0-3">\n'
 
1661
            b'\n'
 
1662
            b'c 0 1 1 2\n'
 
1663
            b'c 1 3 3 2\n', bytes_3)
1624
1664
 
1625
1665
    def test_creating_bundle_preserves_chk_pages(self):
1626
1666
        self.make_merged_branch()
1627
 
        target = self.b1.bzrdir.sprout('target',
1628
 
                                       revision_id='a@cset-0-2a').open_branch()
1629
 
        bundle_txt, rev_ids = self.create_bundle_text('a@cset-0-2a',
1630
 
                                                      'a@cset-0-3')
1631
 
        self.assertEqual(['a@cset-0-2b', 'a@cset-0-3'], rev_ids)
 
1667
        target = self.b1.controldir.sprout('target',
 
1668
                                       revision_id=b'a@cset-0-2a').open_branch()
 
1669
        bundle_txt, rev_ids = self.create_bundle_text(b'a@cset-0-2a',
 
1670
                                                      b'a@cset-0-3')
 
1671
        self.assertEqual(set([b'a@cset-0-2b', b'a@cset-0-3']), set(rev_ids))
1632
1672
        bundle = read_bundle(bundle_txt)
1633
1673
        target.lock_write()
1634
1674
        self.addCleanup(target.unlock)
1635
1675
        install_bundle(target.repository, bundle)
1636
 
        inv1 = self.b1.repository.inventories.get_record_stream([
1637
 
            ('a@cset-0-3',)], 'unordered',
1638
 
            True).next().get_bytes_as('fulltext')
1639
 
        inv2 = target.repository.inventories.get_record_stream([
1640
 
            ('a@cset-0-3',)], 'unordered',
1641
 
            True).next().get_bytes_as('fulltext')
 
1676
        inv1 = next(self.b1.repository.inventories.get_record_stream([
 
1677
            (b'a@cset-0-3',)], 'unordered',
 
1678
            True)).get_bytes_as('fulltext')
 
1679
        inv2 = next(target.repository.inventories.get_record_stream([
 
1680
            (b'a@cset-0-3',)], 'unordered',
 
1681
            True)).get_bytes_as('fulltext')
1642
1682
        self.assertEqualDiff(inv1, inv2)
1643
1683
 
1644
1684
 
1649
1689
 
1650
1690
        self.build_tree(['b1/one'])
1651
1691
        wt.add('one')
1652
 
        wt.commit('add one', rev_id='a@cset-0-1')
 
1692
        wt.commit('add one', rev_id=b'a@cset-0-1')
1653
1693
        self.build_tree(['b1/two'])
1654
1694
        wt.add('two')
1655
 
        wt.commit('add two', rev_id='a@cset-0-2',
1656
 
                  revprops={'branch-nick':'test'})
 
1695
        wt.commit('add two', rev_id=b'a@cset-0-2',
 
1696
                  revprops={u'branch-nick':'test'})
1657
1697
 
1658
 
        bundle_txt = StringIO()
1659
 
        rev_ids = write_bundle(wt.branch.repository, 'a@cset-0-2',
1660
 
                               'a@cset-0-1', bundle_txt, self.format)
1661
 
        self.assertEqual(set(['a@cset-0-2']), set(rev_ids))
 
1698
        bundle_txt = BytesIO()
 
1699
        rev_ids = write_bundle(wt.branch.repository, b'a@cset-0-2',
 
1700
                               b'a@cset-0-1', bundle_txt, self.format)
 
1701
        self.assertEqual({b'a@cset-0-2'}, set(rev_ids))
1662
1702
        bundle_txt.seek(0, 0)
1663
1703
        return bundle_txt
1664
1704
 
1665
1705
    def check_valid(self, bundle):
1666
1706
        """Check that after whatever munging, the final object is valid."""
1667
 
        self.assertEqual(['a@cset-0-2'],
 
1707
        self.assertEqual([b'a@cset-0-2'],
1668
1708
            [r.revision_id for r in bundle.real_revisions])
1669
1709
 
1670
1710
    def test_extra_whitespace(self):
1674
1714
        # Adding one extra newline used to give us
1675
1715
        # TypeError: float() argument must be a string or a number
1676
1716
        bundle_txt.seek(0, 2)
1677
 
        bundle_txt.write('\n')
 
1717
        bundle_txt.write(b'\n')
1678
1718
        bundle_txt.seek(0)
1679
1719
 
1680
1720
        bundle = read_bundle(bundle_txt)
1687
1727
        # Adding two extra newlines used to give us
1688
1728
        # MalformedPatches: The first line of all patches should be ...
1689
1729
        bundle_txt.seek(0, 2)
1690
 
        bundle_txt.write('\n\n')
 
1730
        bundle_txt.write(b'\n\n')
1691
1731
        bundle_txt.seek(0)
1692
1732
 
1693
1733
        bundle = read_bundle(bundle_txt)
1707
1747
        # test is concerned with the exact case where the serializer
1708
1748
        # creates a blank line at the end, and fails if that
1709
1749
        # line is stripped
1710
 
        self.assertEqual('\n\n', raw[-2:])
1711
 
        bundle_txt = StringIO(raw[:-1])
 
1750
        self.assertEqual(b'\n\n', raw[-2:])
 
1751
        bundle_txt = BytesIO(raw[:-1])
1712
1752
 
1713
1753
        bundle = read_bundle(bundle_txt)
1714
1754
        self.check_valid(bundle)
1716
1756
    def test_opening_text(self):
1717
1757
        bundle_txt = self.build_test_bundle()
1718
1758
 
1719
 
        bundle_txt = StringIO("Some random\nemail comments\n"
1720
 
                              + bundle_txt.getvalue())
 
1759
        bundle_txt = BytesIO(
 
1760
            b"Some random\nemail comments\n" + bundle_txt.getvalue())
1721
1761
 
1722
1762
        bundle = read_bundle(bundle_txt)
1723
1763
        self.check_valid(bundle)
1725
1765
    def test_trailing_text(self):
1726
1766
        bundle_txt = self.build_test_bundle()
1727
1767
 
1728
 
        bundle_txt = StringIO(bundle_txt.getvalue() +
1729
 
                              "Some trailing\nrandom\ntext\n")
 
1768
        bundle_txt = BytesIO(
 
1769
            bundle_txt.getvalue() + b"Some trailing\nrandom\ntext\n")
1730
1770
 
1731
1771
        bundle = read_bundle(bundle_txt)
1732
1772
        self.check_valid(bundle)
1740
1780
class TestBundleWriterReader(tests.TestCase):
1741
1781
 
1742
1782
    def test_roundtrip_record(self):
1743
 
        fileobj = StringIO()
 
1783
        fileobj = BytesIO()
1744
1784
        writer = v4.BundleWriter(fileobj)
1745
1785
        writer.begin()
1746
 
        writer.add_info_record(foo='bar')
1747
 
        writer._add_record("Record body", {'parents': ['1', '3'],
1748
 
            'storage_kind':'fulltext'}, 'file', 'revid', 'fileid')
 
1786
        writer.add_info_record({b'foo': b'bar'})
 
1787
        writer._add_record(b"Record body", {b'parents': [b'1', b'3'],
 
1788
            b'storage_kind': b'fulltext'}, 'file', b'revid', b'fileid')
1749
1789
        writer.end()
1750
1790
        fileobj.seek(0)
1751
1791
        reader = v4.BundleReader(fileobj, stream_input=True)
1752
1792
        record_iter = reader.iter_records()
1753
 
        record = record_iter.next()
1754
 
        self.assertEqual((None, {'foo': 'bar', 'storage_kind': 'header'},
 
1793
        record = next(record_iter)
 
1794
        self.assertEqual((None, {b'foo': b'bar', b'storage_kind': b'header'},
1755
1795
            'info', None, None), record)
1756
 
        record = record_iter.next()
1757
 
        self.assertEqual(("Record body", {'storage_kind': 'fulltext',
1758
 
                          'parents': ['1', '3']}, 'file', 'revid', 'fileid'),
 
1796
        record = next(record_iter)
 
1797
        self.assertEqual((b"Record body", {b'storage_kind': b'fulltext',
 
1798
                          b'parents': [b'1', b'3']}, 'file', b'revid', b'fileid'),
1759
1799
                          record)
1760
1800
 
1761
1801
    def test_roundtrip_record_memory_hungry(self):
1762
 
        fileobj = StringIO()
 
1802
        fileobj = BytesIO()
1763
1803
        writer = v4.BundleWriter(fileobj)
1764
1804
        writer.begin()
1765
 
        writer.add_info_record(foo='bar')
1766
 
        writer._add_record("Record body", {'parents': ['1', '3'],
1767
 
            'storage_kind':'fulltext'}, 'file', 'revid', 'fileid')
 
1805
        writer.add_info_record({b'foo': b'bar'})
 
1806
        writer._add_record(b"Record body", {b'parents': [b'1', b'3'],
 
1807
            b'storage_kind': b'fulltext'}, 'file', b'revid', b'fileid')
1768
1808
        writer.end()
1769
1809
        fileobj.seek(0)
1770
1810
        reader = v4.BundleReader(fileobj, stream_input=False)
1771
1811
        record_iter = reader.iter_records()
1772
 
        record = record_iter.next()
1773
 
        self.assertEqual((None, {'foo': 'bar', 'storage_kind': 'header'},
 
1812
        record = next(record_iter)
 
1813
        self.assertEqual((None, {b'foo': b'bar', b'storage_kind': b'header'},
1774
1814
            'info', None, None), record)
1775
 
        record = record_iter.next()
1776
 
        self.assertEqual(("Record body", {'storage_kind': 'fulltext',
1777
 
                          'parents': ['1', '3']}, 'file', 'revid', 'fileid'),
 
1815
        record = next(record_iter)
 
1816
        self.assertEqual((b"Record body", {b'storage_kind': b'fulltext',
 
1817
                          b'parents': [b'1', b'3']}, 'file', b'revid', b'fileid'),
1778
1818
                          record)
1779
1819
 
1780
1820
    def test_encode_name(self):
1781
 
        self.assertEqual('revision/rev1',
1782
 
            v4.BundleWriter.encode_name('revision', 'rev1'))
1783
 
        self.assertEqual('file/rev//1/file-id-1',
1784
 
            v4.BundleWriter.encode_name('file', 'rev/1', 'file-id-1'))
1785
 
        self.assertEqual('info',
 
1821
        self.assertEqual(b'revision/rev1',
 
1822
            v4.BundleWriter.encode_name('revision', b'rev1'))
 
1823
        self.assertEqual(b'file/rev//1/file-id-1',
 
1824
            v4.BundleWriter.encode_name('file', b'rev/1', b'file-id-1'))
 
1825
        self.assertEqual(b'info',
1786
1826
            v4.BundleWriter.encode_name('info', None, None))
1787
1827
 
1788
1828
    def test_decode_name(self):
1789
 
        self.assertEqual(('revision', 'rev1', None),
1790
 
            v4.BundleReader.decode_name('revision/rev1'))
1791
 
        self.assertEqual(('file', 'rev/1', 'file-id-1'),
1792
 
            v4.BundleReader.decode_name('file/rev//1/file-id-1'))
 
1829
        self.assertEqual(('revision', b'rev1', None),
 
1830
            v4.BundleReader.decode_name(b'revision/rev1'))
 
1831
        self.assertEqual(('file', b'rev/1', b'file-id-1'),
 
1832
            v4.BundleReader.decode_name(b'file/rev//1/file-id-1'))
1793
1833
        self.assertEqual(('info', None, None),
1794
 
                         v4.BundleReader.decode_name('info'))
 
1834
                         v4.BundleReader.decode_name(b'info'))
1795
1835
 
1796
1836
    def test_too_many_names(self):
1797
 
        fileobj = StringIO()
 
1837
        fileobj = BytesIO()
1798
1838
        writer = v4.BundleWriter(fileobj)
1799
1839
        writer.begin()
1800
 
        writer.add_info_record(foo='bar')
1801
 
        writer._container.add_bytes_record('blah', ['two', 'names'])
 
1840
        writer.add_info_record({b'foo': b'bar'})
 
1841
        writer._container.add_bytes_record(b'blah', [(b'two', ), (b'names', )])
1802
1842
        writer.end()
1803
1843
        fileobj.seek(0)
1804
1844
        record_iter = v4.BundleReader(fileobj).iter_records()
1805
 
        record = record_iter.next()
1806
 
        self.assertEqual((None, {'foo': 'bar', 'storage_kind': 'header'},
 
1845
        record = next(record_iter)
 
1846
        self.assertEqual((None, {b'foo': b'bar', b'storage_kind': b'header'},
1807
1847
            'info', None, None), record)
1808
 
        self.assertRaises(errors.BadBundle, record_iter.next)
 
1848
        self.assertRaises(errors.BadBundle, next, record_iter)
1809
1849
 
1810
1850
 
1811
1851
class TestReadMergeableFromUrl(tests.TestCaseWithTransport):
1828
1868
    def test_infinite_redirects_are_not_a_bundle(self):
1829
1869
        """If a URL causes TooManyRedirections then NotABundle is raised.
1830
1870
        """
1831
 
        from bzrlib.tests.blackbox.test_push import RedirectingMemoryServer
 
1871
        from .blackbox.test_push import RedirectingMemoryServer
1832
1872
        server = RedirectingMemoryServer()
1833
1873
        self.start_server(server)
1834
1874
        url = server.get_url() + 'infinite-loop'
1839
1879
        bundle, then the ConnectionReset error should be propagated.
1840
1880
        """
1841
1881
        # Instantiate a server that will provoke a ConnectionReset
1842
 
        sock_server = _DisconnectingTCPServer()
 
1882
        sock_server = DisconnectingServer()
1843
1883
        self.start_server(sock_server)
1844
1884
        # We don't really care what the url is since the server will close the
1845
1885
        # connection without interpreting it
1847
1887
        self.assertRaises(errors.ConnectionReset, read_mergeable_from_url, url)
1848
1888
 
1849
1889
 
1850
 
class _DisconnectingTCPServer(object):
1851
 
    """A TCP server that immediately closes any connection made to it."""
1852
 
 
1853
 
    def start_server(self):
1854
 
        self.sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
1855
 
        self.sock.bind(('127.0.0.1', 0))
1856
 
        self.sock.listen(1)
1857
 
        self.port = self.sock.getsockname()[1]
1858
 
        self.thread = threading.Thread(
1859
 
            name='%s (port %d)' % (self.__class__.__name__, self.port),
1860
 
            target=self.accept_and_close)
1861
 
        self.thread.start()
1862
 
 
1863
 
    def accept_and_close(self):
1864
 
        conn, addr = self.sock.accept()
1865
 
        conn.shutdown(socket.SHUT_RDWR)
1866
 
        conn.close()
 
1890
class DisconnectingHandler(socketserver.BaseRequestHandler):
 
1891
    """A request handler that immediately closes any connection made to it."""
 
1892
 
 
1893
    def handle(self):
 
1894
        self.request.close()
 
1895
 
 
1896
 
 
1897
class DisconnectingServer(test_server.TestingTCPServerInAThread):
 
1898
 
 
1899
    def __init__(self):
 
1900
        super(DisconnectingServer, self).__init__(
 
1901
            ('127.0.0.1', 0),
 
1902
            test_server.TestingTCPServer,
 
1903
            DisconnectingHandler)
1867
1904
 
1868
1905
    def get_url(self):
1869
 
        return 'bzr://127.0.0.1:%d/' % (self.port,)
1870
 
 
1871
 
    def stop_server(self):
1872
 
        try:
1873
 
            # make sure the thread dies by connecting to the listening socket,
1874
 
            # just in case the test failed to do so.
1875
 
            conn = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
1876
 
            conn.connect(self.sock.getsockname())
1877
 
            conn.close()
1878
 
        except socket.error:
1879
 
            pass
1880
 
        self.sock.close()
1881
 
        self.thread.join()
 
1906
        """Return the url of the server"""
 
1907
        return "bzr://%s:%d/" % self.server.server_address