/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: 2019-02-15 17:41:17 UTC
  • mto: (7290.1.2 work)
  • mto: This revision was merged to the branch mainline in revision 7295.
  • Revision ID: jelmer@jelmer.uk-20190215174117-o9w1am2z88mg9g1q
Update references to home location.

~/.config/breezy rather than ~/.bazaar.

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