/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_index.py

  • Committer: Jelmer Vernooij
  • Date: 2020-04-05 19:11:34 UTC
  • mto: (7490.7.16 work)
  • mto: This revision was merged to the branch mainline in revision 7501.
  • Revision ID: jelmer@jelmer.uk-20200405191134-0aebh8ikiwygxma5
Populate the .gitignore file.

Show diffs side-by-side

added added

removed removed

Lines of Context:
16
16
 
17
17
"""Tests for indices."""
18
18
 
19
 
from bzrlib import errors
20
 
from bzrlib.index import *
21
 
from bzrlib.tests import TestCaseWithMemoryTransport
22
 
from bzrlib.transport import get_transport
23
 
 
24
 
 
25
 
class TestGraphIndexBuilder(TestCaseWithMemoryTransport):
 
19
from .. import (
 
20
    errors,
 
21
    tests,
 
22
    transport,
 
23
    )
 
24
from ..sixish import int2byte
 
25
from ..bzr import (
 
26
    index as _mod_index,
 
27
    )
 
28
 
 
29
 
 
30
class ErrorTests(tests.TestCase):
 
31
 
 
32
    def test_bad_index_format_signature(self):
 
33
        error = _mod_index.BadIndexFormatSignature("foo", "bar")
 
34
        self.assertEqual("foo is not an index of type bar.",
 
35
                         str(error))
 
36
 
 
37
    def test_bad_index_data(self):
 
38
        error = _mod_index.BadIndexData("foo")
 
39
        self.assertEqual("Error in data for index foo.",
 
40
                         str(error))
 
41
 
 
42
    def test_bad_index_duplicate_key(self):
 
43
        error = _mod_index.BadIndexDuplicateKey("foo", "bar")
 
44
        self.assertEqual("The key 'foo' is already in index 'bar'.",
 
45
                         str(error))
 
46
 
 
47
    def test_bad_index_key(self):
 
48
        error = _mod_index.BadIndexKey("foo")
 
49
        self.assertEqual("The key 'foo' is not a valid key.",
 
50
                         str(error))
 
51
 
 
52
    def test_bad_index_options(self):
 
53
        error = _mod_index.BadIndexOptions("foo")
 
54
        self.assertEqual("Could not parse options for index foo.",
 
55
                         str(error))
 
56
 
 
57
    def test_bad_index_value(self):
 
58
        error = _mod_index.BadIndexValue("foo")
 
59
        self.assertEqual("The value 'foo' is not a valid value.",
 
60
                         str(error))
 
61
 
 
62
 
 
63
class TestGraphIndexBuilder(tests.TestCaseWithMemoryTransport):
26
64
 
27
65
    def test_build_index_empty(self):
28
 
        builder = GraphIndexBuilder()
 
66
        builder = _mod_index.GraphIndexBuilder()
29
67
        stream = builder.finish()
30
68
        contents = stream.read()
31
69
        self.assertEqual(
32
 
            "Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=0\n\n",
 
70
            b"Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=0\n\n",
33
71
            contents)
34
72
 
35
73
    def test_build_index_empty_two_element_keys(self):
36
 
        builder = GraphIndexBuilder(key_elements=2)
 
74
        builder = _mod_index.GraphIndexBuilder(key_elements=2)
37
75
        stream = builder.finish()
38
76
        contents = stream.read()
39
77
        self.assertEqual(
40
 
            "Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=2\nlen=0\n\n",
 
78
            b"Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=2\nlen=0\n\n",
41
79
            contents)
42
80
 
43
81
    def test_build_index_one_reference_list_empty(self):
44
 
        builder = GraphIndexBuilder(reference_lists=1)
 
82
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
45
83
        stream = builder.finish()
46
84
        contents = stream.read()
47
85
        self.assertEqual(
48
 
            "Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=0\n\n",
 
86
            b"Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=0\n\n",
49
87
            contents)
50
88
 
51
89
    def test_build_index_two_reference_list_empty(self):
52
 
        builder = GraphIndexBuilder(reference_lists=2)
 
90
        builder = _mod_index.GraphIndexBuilder(reference_lists=2)
53
91
        stream = builder.finish()
54
92
        contents = stream.read()
55
93
        self.assertEqual(
56
 
            "Bazaar Graph Index 1\nnode_ref_lists=2\nkey_elements=1\nlen=0\n\n",
 
94
            b"Bazaar Graph Index 1\nnode_ref_lists=2\nkey_elements=1\nlen=0\n\n",
57
95
            contents)
58
96
 
59
97
    def test_build_index_one_node_no_refs(self):
60
 
        builder = GraphIndexBuilder()
61
 
        builder.add_node(('akey', ), 'data')
 
98
        builder = _mod_index.GraphIndexBuilder()
 
99
        builder.add_node((b'akey', ), b'data')
62
100
        stream = builder.finish()
63
101
        contents = stream.read()
64
102
        self.assertEqual(
65
 
            "Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=1\n"
66
 
            "akey\x00\x00\x00data\n\n", contents)
 
103
            b"Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=1\n"
 
104
            b"akey\x00\x00\x00data\n\n", contents)
67
105
 
68
106
    def test_build_index_one_node_no_refs_accepts_empty_reflist(self):
69
 
        builder = GraphIndexBuilder()
70
 
        builder.add_node(('akey', ), 'data', ())
 
107
        builder = _mod_index.GraphIndexBuilder()
 
108
        builder.add_node((b'akey', ), b'data', ())
71
109
        stream = builder.finish()
72
110
        contents = stream.read()
73
111
        self.assertEqual(
74
 
            "Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=1\n"
75
 
            "akey\x00\x00\x00data\n\n", contents)
 
112
            b"Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=1\n"
 
113
            b"akey\x00\x00\x00data\n\n", contents)
76
114
 
77
115
    def test_build_index_one_node_2_element_keys(self):
78
116
        # multipart keys are separated by \x00 - because they are fixed length,
79
117
        # not variable this does not cause any issues, and seems clearer to the
80
118
        # author.
81
 
        builder = GraphIndexBuilder(key_elements=2)
82
 
        builder.add_node(('akey', 'secondpart'), 'data')
 
119
        builder = _mod_index.GraphIndexBuilder(key_elements=2)
 
120
        builder.add_node((b'akey', b'secondpart'), b'data')
83
121
        stream = builder.finish()
84
122
        contents = stream.read()
85
123
        self.assertEqual(
86
 
            "Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=2\nlen=1\n"
87
 
            "akey\x00secondpart\x00\x00\x00data\n\n", contents)
 
124
            b"Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=2\nlen=1\n"
 
125
            b"akey\x00secondpart\x00\x00\x00data\n\n", contents)
88
126
 
89
127
    def test_add_node_empty_value(self):
90
 
        builder = GraphIndexBuilder()
91
 
        builder.add_node(('akey', ), '')
 
128
        builder = _mod_index.GraphIndexBuilder()
 
129
        builder.add_node((b'akey', ), b'')
92
130
        stream = builder.finish()
93
131
        contents = stream.read()
94
132
        self.assertEqual(
95
 
            "Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=1\n"
96
 
            "akey\x00\x00\x00\n\n", contents)
 
133
            b"Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=1\n"
 
134
            b"akey\x00\x00\x00\n\n", contents)
97
135
 
98
136
    def test_build_index_nodes_sorted(self):
99
137
        # the highest sorted node comes first.
100
 
        builder = GraphIndexBuilder()
 
138
        builder = _mod_index.GraphIndexBuilder()
101
139
        # use three to have a good chance of glitching dictionary hash
102
140
        # lookups etc. Insert in randomish order that is not correct
103
141
        # and not the reverse of the correct order.
104
 
        builder.add_node(('2002', ), 'data')
105
 
        builder.add_node(('2000', ), 'data')
106
 
        builder.add_node(('2001', ), 'data')
 
142
        builder.add_node((b'2002', ), b'data')
 
143
        builder.add_node((b'2000', ), b'data')
 
144
        builder.add_node((b'2001', ), b'data')
107
145
        stream = builder.finish()
108
146
        contents = stream.read()
109
147
        self.assertEqual(
110
 
            "Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=3\n"
111
 
            "2000\x00\x00\x00data\n"
112
 
            "2001\x00\x00\x00data\n"
113
 
            "2002\x00\x00\x00data\n"
114
 
            "\n", contents)
 
148
            b"Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=1\nlen=3\n"
 
149
            b"2000\x00\x00\x00data\n"
 
150
            b"2001\x00\x00\x00data\n"
 
151
            b"2002\x00\x00\x00data\n"
 
152
            b"\n", contents)
115
153
 
116
154
    def test_build_index_2_element_key_nodes_sorted(self):
117
155
        # multiple element keys are sorted first-key, second-key.
118
 
        builder = GraphIndexBuilder(key_elements=2)
 
156
        builder = _mod_index.GraphIndexBuilder(key_elements=2)
119
157
        # use three values of each key element, to have a good chance of
120
158
        # glitching dictionary hash lookups etc. Insert in randomish order that
121
159
        # is not correct and not the reverse of the correct order.
122
 
        builder.add_node(('2002', '2002'), 'data')
123
 
        builder.add_node(('2002', '2000'), 'data')
124
 
        builder.add_node(('2002', '2001'), 'data')
125
 
        builder.add_node(('2000', '2002'), 'data')
126
 
        builder.add_node(('2000', '2000'), 'data')
127
 
        builder.add_node(('2000', '2001'), 'data')
128
 
        builder.add_node(('2001', '2002'), 'data')
129
 
        builder.add_node(('2001', '2000'), 'data')
130
 
        builder.add_node(('2001', '2001'), 'data')
 
160
        builder.add_node((b'2002', b'2002'), b'data')
 
161
        builder.add_node((b'2002', b'2000'), b'data')
 
162
        builder.add_node((b'2002', b'2001'), b'data')
 
163
        builder.add_node((b'2000', b'2002'), b'data')
 
164
        builder.add_node((b'2000', b'2000'), b'data')
 
165
        builder.add_node((b'2000', b'2001'), b'data')
 
166
        builder.add_node((b'2001', b'2002'), b'data')
 
167
        builder.add_node((b'2001', b'2000'), b'data')
 
168
        builder.add_node((b'2001', b'2001'), b'data')
131
169
        stream = builder.finish()
132
170
        contents = stream.read()
133
171
        self.assertEqual(
134
 
            "Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=2\nlen=9\n"
135
 
            "2000\x002000\x00\x00\x00data\n"
136
 
            "2000\x002001\x00\x00\x00data\n"
137
 
            "2000\x002002\x00\x00\x00data\n"
138
 
            "2001\x002000\x00\x00\x00data\n"
139
 
            "2001\x002001\x00\x00\x00data\n"
140
 
            "2001\x002002\x00\x00\x00data\n"
141
 
            "2002\x002000\x00\x00\x00data\n"
142
 
            "2002\x002001\x00\x00\x00data\n"
143
 
            "2002\x002002\x00\x00\x00data\n"
144
 
            "\n", contents)
 
172
            b"Bazaar Graph Index 1\nnode_ref_lists=0\nkey_elements=2\nlen=9\n"
 
173
            b"2000\x002000\x00\x00\x00data\n"
 
174
            b"2000\x002001\x00\x00\x00data\n"
 
175
            b"2000\x002002\x00\x00\x00data\n"
 
176
            b"2001\x002000\x00\x00\x00data\n"
 
177
            b"2001\x002001\x00\x00\x00data\n"
 
178
            b"2001\x002002\x00\x00\x00data\n"
 
179
            b"2002\x002000\x00\x00\x00data\n"
 
180
            b"2002\x002001\x00\x00\x00data\n"
 
181
            b"2002\x002002\x00\x00\x00data\n"
 
182
            b"\n", contents)
145
183
 
146
184
    def test_build_index_reference_lists_are_included_one(self):
147
 
        builder = GraphIndexBuilder(reference_lists=1)
148
 
        builder.add_node(('key', ), 'data', ([], ))
 
185
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
 
186
        builder.add_node((b'key', ), b'data', ([], ))
149
187
        stream = builder.finish()
150
188
        contents = stream.read()
151
189
        self.assertEqual(
152
 
            "Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=1\n"
153
 
            "key\x00\x00\x00data\n"
154
 
            "\n", contents)
 
190
            b"Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=1\n"
 
191
            b"key\x00\x00\x00data\n"
 
192
            b"\n", contents)
155
193
 
156
194
    def test_build_index_reference_lists_with_2_element_keys(self):
157
 
        builder = GraphIndexBuilder(reference_lists=1, key_elements=2)
158
 
        builder.add_node(('key', 'key2'), 'data', ([], ))
 
195
        builder = _mod_index.GraphIndexBuilder(
 
196
            reference_lists=1, key_elements=2)
 
197
        builder.add_node((b'key', b'key2'), b'data', ([], ))
159
198
        stream = builder.finish()
160
199
        contents = stream.read()
161
200
        self.assertEqual(
162
 
            "Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=2\nlen=1\n"
163
 
            "key\x00key2\x00\x00\x00data\n"
164
 
            "\n", contents)
 
201
            b"Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=2\nlen=1\n"
 
202
            b"key\x00key2\x00\x00\x00data\n"
 
203
            b"\n", contents)
165
204
 
166
205
    def test_build_index_reference_lists_are_included_two(self):
167
 
        builder = GraphIndexBuilder(reference_lists=2)
168
 
        builder.add_node(('key', ), 'data', ([], []))
 
206
        builder = _mod_index.GraphIndexBuilder(reference_lists=2)
 
207
        builder.add_node((b'key', ), b'data', ([], []))
169
208
        stream = builder.finish()
170
209
        contents = stream.read()
171
210
        self.assertEqual(
172
 
            "Bazaar Graph Index 1\nnode_ref_lists=2\nkey_elements=1\nlen=1\n"
173
 
            "key\x00\x00\t\x00data\n"
174
 
            "\n", contents)
 
211
            b"Bazaar Graph Index 1\nnode_ref_lists=2\nkey_elements=1\nlen=1\n"
 
212
            b"key\x00\x00\t\x00data\n"
 
213
            b"\n", contents)
175
214
 
176
215
    def test_clear_cache(self):
177
 
        builder = GraphIndexBuilder(reference_lists=2)
 
216
        builder = _mod_index.GraphIndexBuilder(reference_lists=2)
178
217
        # This is a no-op, but the api should exist
179
218
        builder.clear_cache()
180
219
 
181
220
    def test_node_references_are_byte_offsets(self):
182
 
        builder = GraphIndexBuilder(reference_lists=1)
183
 
        builder.add_node(('reference', ), 'data', ([], ))
184
 
        builder.add_node(('key', ), 'data', ([('reference', )], ))
 
221
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
 
222
        builder.add_node((b'reference', ), b'data', ([], ))
 
223
        builder.add_node((b'key', ), b'data', ([(b'reference', )], ))
185
224
        stream = builder.finish()
186
225
        contents = stream.read()
187
226
        self.assertEqual(
188
 
            "Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=2\n"
189
 
            "key\x00\x0072\x00data\n"
190
 
            "reference\x00\x00\x00data\n"
191
 
            "\n", contents)
 
227
            b"Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=2\n"
 
228
            b"key\x00\x0072\x00data\n"
 
229
            b"reference\x00\x00\x00data\n"
 
230
            b"\n", contents)
192
231
 
193
232
    def test_node_references_are_cr_delimited(self):
194
 
        builder = GraphIndexBuilder(reference_lists=1)
195
 
        builder.add_node(('reference', ), 'data', ([], ))
196
 
        builder.add_node(('reference2', ), 'data', ([], ))
197
 
        builder.add_node(('key', ), 'data', ([('reference', ), ('reference2', )], ))
 
233
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
 
234
        builder.add_node((b'reference', ), b'data', ([], ))
 
235
        builder.add_node((b'reference2', ), b'data', ([], ))
 
236
        builder.add_node((b'key', ), b'data',
 
237
                         ([(b'reference', ), (b'reference2', )], ))
198
238
        stream = builder.finish()
199
239
        contents = stream.read()
200
240
        self.assertEqual(
201
 
            "Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=3\n"
202
 
            "key\x00\x00077\r094\x00data\n"
203
 
            "reference\x00\x00\x00data\n"
204
 
            "reference2\x00\x00\x00data\n"
205
 
            "\n", contents)
 
241
            b"Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=3\n"
 
242
            b"key\x00\x00077\r094\x00data\n"
 
243
            b"reference\x00\x00\x00data\n"
 
244
            b"reference2\x00\x00\x00data\n"
 
245
            b"\n", contents)
206
246
 
207
247
    def test_multiple_reference_lists_are_tab_delimited(self):
208
 
        builder = GraphIndexBuilder(reference_lists=2)
209
 
        builder.add_node(('keference', ), 'data', ([], []))
210
 
        builder.add_node(('rey', ), 'data', ([('keference', )], [('keference', )]))
 
248
        builder = _mod_index.GraphIndexBuilder(reference_lists=2)
 
249
        builder.add_node((b'keference', ), b'data', ([], []))
 
250
        builder.add_node((b'rey', ), b'data',
 
251
                         ([(b'keference', )], [(b'keference', )]))
211
252
        stream = builder.finish()
212
253
        contents = stream.read()
213
254
        self.assertEqual(
214
 
            "Bazaar Graph Index 1\nnode_ref_lists=2\nkey_elements=1\nlen=2\n"
215
 
            "keference\x00\x00\t\x00data\n"
216
 
            "rey\x00\x0059\t59\x00data\n"
217
 
            "\n", contents)
 
255
            b"Bazaar Graph Index 1\nnode_ref_lists=2\nkey_elements=1\nlen=2\n"
 
256
            b"keference\x00\x00\t\x00data\n"
 
257
            b"rey\x00\x0059\t59\x00data\n"
 
258
            b"\n", contents)
218
259
 
219
260
    def test_add_node_referencing_missing_key_makes_absent(self):
220
 
        builder = GraphIndexBuilder(reference_lists=1)
221
 
        builder.add_node(('rey', ), 'data', ([('beference', ), ('aeference2', )], ))
 
261
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
 
262
        builder.add_node((b'rey', ), b'data',
 
263
                         ([(b'beference', ), (b'aeference2', )], ))
222
264
        stream = builder.finish()
223
265
        contents = stream.read()
224
266
        self.assertEqual(
225
 
            "Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=1\n"
226
 
            "aeference2\x00a\x00\x00\n"
227
 
            "beference\x00a\x00\x00\n"
228
 
            "rey\x00\x00074\r059\x00data\n"
229
 
            "\n", contents)
 
267
            b"Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=1\n"
 
268
            b"aeference2\x00a\x00\x00\n"
 
269
            b"beference\x00a\x00\x00\n"
 
270
            b"rey\x00\x00074\r059\x00data\n"
 
271
            b"\n", contents)
230
272
 
231
273
    def test_node_references_three_digits(self):
232
274
        # test the node digit expands as needed.
233
 
        builder = GraphIndexBuilder(reference_lists=1)
234
 
        references = [(str(val), ) for val in reversed(range(9))]
235
 
        builder.add_node(('2-key', ), '', (references, ))
 
275
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
 
276
        references = [((b"%d" % val), ) for val in range(8, -1, -1)]
 
277
        builder.add_node((b'2-key', ), b'', (references, ))
236
278
        stream = builder.finish()
237
279
        contents = stream.read()
238
280
        self.assertEqualDiff(
239
 
            "Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=1\n"
240
 
            "0\x00a\x00\x00\n"
241
 
            "1\x00a\x00\x00\n"
242
 
            "2\x00a\x00\x00\n"
243
 
            "2-key\x00\x00151\r145\r139\r133\r127\r121\r071\r065\r059\x00\n"
244
 
            "3\x00a\x00\x00\n"
245
 
            "4\x00a\x00\x00\n"
246
 
            "5\x00a\x00\x00\n"
247
 
            "6\x00a\x00\x00\n"
248
 
            "7\x00a\x00\x00\n"
249
 
            "8\x00a\x00\x00\n"
250
 
            "\n", contents)
 
281
            b"Bazaar Graph Index 1\nnode_ref_lists=1\nkey_elements=1\nlen=1\n"
 
282
            b"0\x00a\x00\x00\n"
 
283
            b"1\x00a\x00\x00\n"
 
284
            b"2\x00a\x00\x00\n"
 
285
            b"2-key\x00\x00151\r145\r139\r133\r127\r121\r071\r065\r059\x00\n"
 
286
            b"3\x00a\x00\x00\n"
 
287
            b"4\x00a\x00\x00\n"
 
288
            b"5\x00a\x00\x00\n"
 
289
            b"6\x00a\x00\x00\n"
 
290
            b"7\x00a\x00\x00\n"
 
291
            b"8\x00a\x00\x00\n"
 
292
            b"\n", contents)
251
293
 
252
294
    def test_absent_has_no_reference_overhead(self):
253
295
        # the offsets after an absent record should be correct when there are
254
296
        # >1 reference lists.
255
 
        builder = GraphIndexBuilder(reference_lists=2)
256
 
        builder.add_node(('parent', ), '', ([('aail', ), ('zther', )], []))
 
297
        builder = _mod_index.GraphIndexBuilder(reference_lists=2)
 
298
        builder.add_node((b'parent', ), b'', ([(b'aail', ), (b'zther', )], []))
257
299
        stream = builder.finish()
258
300
        contents = stream.read()
259
301
        self.assertEqual(
260
 
            "Bazaar Graph Index 1\nnode_ref_lists=2\nkey_elements=1\nlen=1\n"
261
 
            "aail\x00a\x00\x00\n"
262
 
            "parent\x00\x0059\r84\t\x00\n"
263
 
            "zther\x00a\x00\x00\n"
264
 
            "\n", contents)
 
302
            b"Bazaar Graph Index 1\nnode_ref_lists=2\nkey_elements=1\nlen=1\n"
 
303
            b"aail\x00a\x00\x00\n"
 
304
            b"parent\x00\x0059\r84\t\x00\n"
 
305
            b"zther\x00a\x00\x00\n"
 
306
            b"\n", contents)
265
307
 
266
308
    def test_add_node_bad_key(self):
267
 
        builder = GraphIndexBuilder()
268
 
        for bad_char in '\t\n\x0b\x0c\r\x00 ':
269
 
            self.assertRaises(errors.BadIndexKey, builder.add_node,
270
 
                ('a%skey' % bad_char, ), 'data')
271
 
        self.assertRaises(errors.BadIndexKey, builder.add_node,
272
 
                ('', ), 'data')
273
 
        self.assertRaises(errors.BadIndexKey, builder.add_node,
274
 
                'not-a-tuple', 'data')
 
309
        builder = _mod_index.GraphIndexBuilder()
 
310
        for bad_char in bytearray(b'\t\n\x0b\x0c\r\x00 '):
 
311
            self.assertRaises(_mod_index.BadIndexKey, builder.add_node,
 
312
                              (b'a%skey' % int2byte(bad_char), ), b'data')
 
313
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node,
 
314
                          (), b'data')
 
315
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node,
 
316
                          b'not-a-tuple', b'data')
275
317
        # not enough length
276
 
        self.assertRaises(errors.BadIndexKey, builder.add_node,
277
 
                (), 'data')
 
318
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node,
 
319
                          (), b'data')
278
320
        # too long
279
 
        self.assertRaises(errors.BadIndexKey, builder.add_node,
280
 
                ('primary', 'secondary'), 'data')
 
321
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node,
 
322
                          (b'primary', b'secondary'), b'data')
281
323
        # secondary key elements get checked too:
282
 
        builder = GraphIndexBuilder(key_elements=2)
283
 
        for bad_char in '\t\n\x0b\x0c\r\x00 ':
284
 
            self.assertRaises(errors.BadIndexKey, builder.add_node,
285
 
                ('prefix', 'a%skey' % bad_char), 'data')
 
324
        builder = _mod_index.GraphIndexBuilder(key_elements=2)
 
325
        for bad_char in bytearray(b'\t\n\x0b\x0c\r\x00 '):
 
326
            self.assertRaises(_mod_index.BadIndexKey, builder.add_node,
 
327
                              (b'prefix', b'a%skey' % int2byte(bad_char)), b'data')
286
328
 
287
329
    def test_add_node_bad_data(self):
288
 
        builder = GraphIndexBuilder()
289
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
290
 
            'data\naa')
291
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
292
 
            'data\x00aa')
 
330
        builder = _mod_index.GraphIndexBuilder()
 
331
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
332
                          b'data\naa')
 
333
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
334
                          b'data\x00aa')
293
335
 
294
336
    def test_add_node_bad_mismatched_ref_lists_length(self):
295
 
        builder = GraphIndexBuilder()
296
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
297
 
            'data aa', ([], ))
298
 
        builder = GraphIndexBuilder(reference_lists=1)
299
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
300
 
            'data aa')
301
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
302
 
            'data aa', (), )
303
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
304
 
            'data aa', ([], []))
305
 
        builder = GraphIndexBuilder(reference_lists=2)
306
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
307
 
            'data aa')
308
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
309
 
            'data aa', ([], ))
310
 
        self.assertRaises(errors.BadIndexValue, builder.add_node, ('akey', ),
311
 
            'data aa', ([], [], []))
 
337
        builder = _mod_index.GraphIndexBuilder()
 
338
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
339
                          b'data aa', ([], ))
 
340
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
 
341
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
342
                          b'data aa')
 
343
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
344
                          b'data aa', (), )
 
345
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
346
                          b'data aa', ([], []))
 
347
        builder = _mod_index.GraphIndexBuilder(reference_lists=2)
 
348
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
349
                          b'data aa')
 
350
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
351
                          b'data aa', ([], ))
 
352
        self.assertRaises(_mod_index.BadIndexValue, builder.add_node, (b'akey', ),
 
353
                          b'data aa', ([], [], []))
312
354
 
313
355
    def test_add_node_bad_key_in_reference_lists(self):
314
356
        # first list, first key - trivial
315
 
        builder = GraphIndexBuilder(reference_lists=1)
316
 
        self.assertRaises(errors.BadIndexKey, builder.add_node, ('akey', ),
317
 
            'data aa', ([('a key', )], ))
 
357
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
 
358
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node, (b'akey', ),
 
359
                          b'data aa', ([(b'a key', )], ))
318
360
        # references keys must be tuples too
319
 
        self.assertRaises(errors.BadIndexKey, builder.add_node, ('akey', ),
320
 
            'data aa', (['not-a-tuple'], ))
 
361
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node, (b'akey', ),
 
362
                          b'data aa', (['not-a-tuple'], ))
321
363
        # not enough length
322
 
        self.assertRaises(errors.BadIndexKey, builder.add_node, ('akey', ),
323
 
            'data aa', ([()], ))
 
364
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node, (b'akey', ),
 
365
                          b'data aa', ([()], ))
324
366
        # too long
325
 
        self.assertRaises(errors.BadIndexKey, builder.add_node, ('akey', ),
326
 
            'data aa', ([('primary', 'secondary')], ))
 
367
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node, (b'akey', ),
 
368
                          b'data aa', ([(b'primary', b'secondary')], ))
327
369
        # need to check more than the first key in the list
328
 
        self.assertRaises(errors.BadIndexKey, builder.add_node, ('akey', ),
329
 
            'data aa', ([('agoodkey', ), ('that is a bad key', )], ))
 
370
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node, (b'akey', ),
 
371
                          b'data aa', ([(b'agoodkey', ), (b'that is a bad key', )], ))
330
372
        # and if there is more than one list it should be getting checked
331
373
        # too
332
 
        builder = GraphIndexBuilder(reference_lists=2)
333
 
        self.assertRaises(errors.BadIndexKey, builder.add_node, ('akey', ),
334
 
            'data aa', ([], ['a bad key']))
 
374
        builder = _mod_index.GraphIndexBuilder(reference_lists=2)
 
375
        self.assertRaises(_mod_index.BadIndexKey, builder.add_node, (b'akey', ),
 
376
                          b'data aa', ([], ['a bad key']))
335
377
 
336
378
    def test_add_duplicate_key(self):
337
 
        builder = GraphIndexBuilder()
338
 
        builder.add_node(('key', ), 'data')
339
 
        self.assertRaises(errors.BadIndexDuplicateKey, builder.add_node, ('key', ),
340
 
            'data')
 
379
        builder = _mod_index.GraphIndexBuilder()
 
380
        builder.add_node((b'key', ), b'data')
 
381
        self.assertRaises(_mod_index.BadIndexDuplicateKey,
 
382
                          builder.add_node, (b'key', ), b'data')
341
383
 
342
384
    def test_add_duplicate_key_2_elements(self):
343
 
        builder = GraphIndexBuilder(key_elements=2)
344
 
        builder.add_node(('key', 'key'), 'data')
345
 
        self.assertRaises(errors.BadIndexDuplicateKey, builder.add_node,
346
 
            ('key', 'key'), 'data')
 
385
        builder = _mod_index.GraphIndexBuilder(key_elements=2)
 
386
        builder.add_node((b'key', b'key'), b'data')
 
387
        self.assertRaises(_mod_index.BadIndexDuplicateKey, builder.add_node,
 
388
                          (b'key', b'key'), b'data')
347
389
 
348
390
    def test_add_key_after_referencing_key(self):
349
 
        builder = GraphIndexBuilder(reference_lists=1)
350
 
        builder.add_node(('key', ), 'data', ([('reference', )], ))
351
 
        builder.add_node(('reference', ), 'data', ([],))
 
391
        builder = _mod_index.GraphIndexBuilder(reference_lists=1)
 
392
        builder.add_node((b'key', ), b'data', ([(b'reference', )], ))
 
393
        builder.add_node((b'reference', ), b'data', ([],))
352
394
 
353
395
    def test_add_key_after_referencing_key_2_elements(self):
354
 
        builder = GraphIndexBuilder(reference_lists=1, key_elements=2)
355
 
        builder.add_node(('k', 'ey'), 'data', ([('reference', 'tokey')], ))
356
 
        builder.add_node(('reference', 'tokey'), 'data', ([],))
 
396
        builder = _mod_index.GraphIndexBuilder(
 
397
            reference_lists=1, key_elements=2)
 
398
        builder.add_node((b'k', b'ey'), b'data',
 
399
                         ([(b'reference', b'tokey')], ))
 
400
        builder.add_node((b'reference', b'tokey'), b'data', ([],))
357
401
 
358
402
    def test_set_optimize(self):
359
 
        builder = GraphIndexBuilder(reference_lists=1, key_elements=2)
 
403
        builder = _mod_index.GraphIndexBuilder(
 
404
            reference_lists=1, key_elements=2)
360
405
        builder.set_optimize(for_size=True)
361
406
        self.assertTrue(builder._optimize_for_size)
362
407
        builder.set_optimize(for_size=False)
363
408
        self.assertFalse(builder._optimize_for_size)
364
409
 
365
410
 
366
 
class TestGraphIndex(TestCaseWithMemoryTransport):
 
411
class TestGraphIndex(tests.TestCaseWithMemoryTransport):
367
412
 
368
413
    def make_key(self, number):
369
 
        return (str(number) + 'X'*100,)
 
414
        return ((b'%d' % number) + b'X' * 100,)
370
415
 
371
416
    def make_value(self, number):
372
 
            return str(number) + 'Y'*100
 
417
        return (b'%d' % number) + b'Y' * 100
373
418
 
374
419
    def make_nodes(self, count=64):
375
420
        # generate a big enough index that we only read some of it on a typical
376
421
        # bisection lookup.
377
422
        nodes = []
378
423
        for counter in range(count):
379
 
            nodes.append((self.make_key(counter), self.make_value(counter), ()))
 
424
            nodes.append(
 
425
                (self.make_key(counter), self.make_value(counter), ()))
380
426
        return nodes
381
427
 
382
428
    def make_index(self, ref_lists=0, key_elements=1, nodes=[]):
383
 
        builder = GraphIndexBuilder(ref_lists, key_elements=key_elements)
 
429
        builder = _mod_index.GraphIndexBuilder(
 
430
            ref_lists, key_elements=key_elements)
384
431
        for key, value, references in nodes:
385
432
            builder.add_node(key, value, references)
386
433
        stream = builder.finish()
387
 
        trans = get_transport('trace+' + self.get_url())
 
434
        trans = transport.get_transport_from_url('trace+' + self.get_url())
388
435
        size = trans.put_file('index', stream)
389
 
        return GraphIndex(trans, 'index', size)
 
436
        return _mod_index.GraphIndex(trans, 'index', size)
390
437
 
391
438
    def make_index_with_offset(self, ref_lists=0, key_elements=1, nodes=[],
392
439
                               offset=0):
393
 
        builder = GraphIndexBuilder(ref_lists, key_elements=key_elements)
 
440
        builder = _mod_index.GraphIndexBuilder(
 
441
            ref_lists, key_elements=key_elements)
394
442
        for key, value, references in nodes:
395
443
            builder.add_node(key, value, references)
396
444
        content = builder.finish().read()
397
445
        size = len(content)
398
446
        trans = self.get_transport()
399
 
        trans.put_bytes('index', (' '*offset) + content)
400
 
        return GraphIndex(trans, 'index', size, offset=offset)
 
447
        trans.put_bytes('index', (b' ' * offset) + content)
 
448
        return _mod_index.GraphIndex(trans, 'index', size, offset=offset)
401
449
 
402
450
    def test_clear_cache(self):
403
451
        index = self.make_index()
407
455
 
408
456
    def test_open_bad_index_no_error(self):
409
457
        trans = self.get_transport()
410
 
        trans.put_bytes('name', "not an index\n")
411
 
        index = GraphIndex(trans, 'name', 13)
 
458
        trans.put_bytes('name', b"not an index\n")
 
459
        idx = _mod_index.GraphIndex(trans, 'name', 13)
412
460
 
413
461
    def test_with_offset(self):
414
462
        nodes = self.make_nodes(200)
415
 
        index = self.make_index_with_offset(offset=1234567, nodes=nodes)
416
 
        self.assertEqual(200, index.key_count())
 
463
        idx = self.make_index_with_offset(offset=1234567, nodes=nodes)
 
464
        self.assertEqual(200, idx.key_count())
417
465
 
418
466
    def test_buffer_all_with_offset(self):
419
467
        nodes = self.make_nodes(200)
420
 
        index = self.make_index_with_offset(offset=1234567, nodes=nodes)
421
 
        index._buffer_all()
422
 
        self.assertEqual(200, index.key_count())
 
468
        idx = self.make_index_with_offset(offset=1234567, nodes=nodes)
 
469
        idx._buffer_all()
 
470
        self.assertEqual(200, idx.key_count())
423
471
 
424
472
    def test_side_effect_buffering_with_offset(self):
425
473
        nodes = self.make_nodes(20)
426
474
        index = self.make_index_with_offset(offset=1234567, nodes=nodes)
427
 
        index._transport.recommended_page_size = lambda:64*1024
 
475
        index._transport.recommended_page_size = lambda: 64 * 1024
428
476
        subset_nodes = [nodes[0][0], nodes[10][0], nodes[19][0]]
429
477
        entries = [n[1] for n in index.iter_entries(subset_nodes)]
430
478
        self.assertEqual(sorted(subset_nodes), sorted(entries))
456
504
        # do a _lookup_keys_via_location call for the middle of the file, which
457
505
        # is what bisection uses.
458
506
        result = index._lookup_keys_via_location(
459
 
            [(index._size // 2, ('missing', ))])
 
507
            [(index._size // 2, (b'missing', ))])
460
508
        # this should have asked for a readv request, with adjust_for_latency,
461
509
        # and two regions: the header, and half-way into the file.
462
510
        self.assertEqual([
465
513
            index._transport._activity)
466
514
        # and the result should be that the key cannot be present, because this
467
515
        # is a trivial index.
468
 
        self.assertEqual([((index._size // 2, ('missing', )), False)],
469
 
            result)
 
516
        self.assertEqual([((index._size // 2, (b'missing', )), False)],
 
517
                         result)
470
518
        # And this should have caused the file to be fully buffered
471
519
        self.assertIsNot(None, index._nodes)
472
520
        self.assertEqual([], index._parsed_byte_map)
486
534
        # is what bisection uses.
487
535
        start_lookup = index._size // 2
488
536
        result = index._lookup_keys_via_location(
489
 
            [(start_lookup, ('40missing', ))])
 
537
            [(start_lookup, (b'40missing', ))])
490
538
        # this should have asked for a readv request, with adjust_for_latency,
491
539
        # and two regions: the header, and half-way into the file.
492
540
        self.assertEqual([
496
544
            index._transport._activity)
497
545
        # and the result should be that the key cannot be present, because this
498
546
        # is a trivial index.
499
 
        self.assertEqual([((start_lookup, ('40missing', )), False)],
500
 
            result)
 
547
        self.assertEqual([((start_lookup, (b'40missing', )), False)],
 
548
                         result)
501
549
        # And this should not have caused the file to be fully buffered
502
550
        self.assertIs(None, index._nodes)
503
551
        # And the regions of the file that have been parsed should be in the
504
552
        # parsed_byte_map and the parsed_key_map
505
553
        self.assertEqual([(0, 4008), (5046, 8996)], index._parsed_byte_map)
506
 
        self.assertEqual([(None, self.make_key(26)),
 
554
        self.assertEqual([((), self.make_key(26)),
507
555
                          (self.make_key(31), self.make_key(48))],
508
556
                         index._parsed_key_map)
509
557
 
510
558
    def test_parsing_non_adjacent_data_trims(self):
511
559
        index = self.make_index(nodes=self.make_nodes(64))
512
560
        result = index._lookup_keys_via_location(
513
 
            [(index._size // 2, ('40', ))])
 
561
            [(index._size // 2, (b'40', ))])
514
562
        # and the result should be that the key cannot be present, because key is
515
563
        # in the middle of the observed data from a 4K read - the smallest transport
516
564
        # will do today with this api.
517
 
        self.assertEqual([((index._size // 2, ('40', )), False)],
518
 
            result)
 
565
        self.assertEqual([((index._size // 2, (b'40', )), False)],
 
566
                         result)
519
567
        # and we should have a parse map that includes the header and the
520
568
        # region that was parsed after trimming.
521
569
        self.assertEqual([(0, 4008), (5046, 8996)], index._parsed_byte_map)
522
 
        self.assertEqual([(None, self.make_key(26)),
 
570
        self.assertEqual([((), self.make_key(26)),
523
571
                          (self.make_key(31), self.make_key(48))],
524
 
            index._parsed_key_map)
 
572
                         index._parsed_key_map)
525
573
 
526
574
    def test_parsing_data_handles_parsed_contained_regions(self):
527
575
        # the following patten creates a parsed region that is wholly within a
539
587
        # locations of both keys.
540
588
        index = self.make_index(nodes=self.make_nodes(128))
541
589
        result = index._lookup_keys_via_location(
542
 
            [(index._size // 2, ('40', ))])
 
590
            [(index._size // 2, (b'40', ))])
543
591
        # and we should have a parse map that includes the header and the
544
592
        # region that was parsed after trimming.
545
593
        self.assertEqual([(0, 4045), (11759, 15707)], index._parsed_byte_map)
546
 
        self.assertEqual([(None, self.make_key(116)),
 
594
        self.assertEqual([((), self.make_key(116)),
547
595
                          (self.make_key(35), self.make_key(51))],
548
 
            index._parsed_key_map)
 
596
                         index._parsed_key_map)
549
597
        # now ask for two keys, right before and after the parsed region
550
598
        result = index._lookup_keys_via_location(
551
599
            [(11450, self.make_key(34)), (15707, self.make_key(52))])
563
611
        # bisection lookup.
564
612
        index = self.make_index(nodes=self.make_nodes(64))
565
613
        # lookup the keys in the middle of the file
566
 
        result =index._lookup_keys_via_location(
567
 
            [(index._size // 2, ('40', ))])
 
614
        result = index._lookup_keys_via_location(
 
615
            [(index._size // 2, (b'40', ))])
568
616
        # check the parse map, this determines the test validity
569
617
        self.assertEqual([(0, 4008), (5046, 8996)], index._parsed_byte_map)
570
 
        self.assertEqual([(None, self.make_key(26)),
 
618
        self.assertEqual([((), self.make_key(26)),
571
619
                          (self.make_key(31), self.make_key(48))],
572
 
            index._parsed_key_map)
 
620
                         index._parsed_key_map)
573
621
        # reset the transport log
574
622
        del index._transport._activity[:]
575
623
        # now looking up a key in the portion of the file already parsed should
577
625
        # be in the index) - even when the byte location we ask for is outside
578
626
        # the parsed region
579
627
        result = index._lookup_keys_via_location(
580
 
            [(4000, ('40', ))])
581
 
        self.assertEqual([((4000, ('40', )), False)],
582
 
            result)
 
628
            [(4000, (b'40', ))])
 
629
        self.assertEqual([((4000, (b'40', )), False)],
 
630
                         result)
583
631
        self.assertEqual([], index._transport._activity)
584
632
 
585
633
    def test_lookup_present_key_answers_without_io_when_map_permits(self):
587
635
        # bisection lookup.
588
636
        index = self.make_index(nodes=self.make_nodes(64))
589
637
        # lookup the keys in the middle of the file
590
 
        result =index._lookup_keys_via_location(
591
 
            [(index._size // 2, ('40', ))])
 
638
        result = index._lookup_keys_via_location(
 
639
            [(index._size // 2, (b'40', ))])
592
640
        # check the parse map, this determines the test validity
593
641
        self.assertEqual([(0, 4008), (5046, 8996)], index._parsed_byte_map)
594
 
        self.assertEqual([(None, self.make_key(26)),
 
642
        self.assertEqual([((), self.make_key(26)),
595
643
                          (self.make_key(31), self.make_key(48))],
596
 
            index._parsed_key_map)
 
644
                         index._parsed_key_map)
597
645
        # reset the transport log
598
646
        del index._transport._activity[:]
599
647
        # now looking up a key in the portion of the file already parsed should
614
662
        index = self.make_index(nodes=self.make_nodes(64))
615
663
        # ask for the key in the middle, but a key that is located in the
616
664
        # unparsed region before the middle.
617
 
        result =index._lookup_keys_via_location(
618
 
            [(index._size // 2, ('30', ))])
 
665
        result = index._lookup_keys_via_location(
 
666
            [(index._size // 2, (b'30', ))])
619
667
        # check the parse map, this determines the test validity
620
668
        self.assertEqual([(0, 4008), (5046, 8996)], index._parsed_byte_map)
621
 
        self.assertEqual([(None, self.make_key(26)),
 
669
        self.assertEqual([((), self.make_key(26)),
622
670
                          (self.make_key(31), self.make_key(48))],
623
 
            index._parsed_key_map)
624
 
        self.assertEqual([((index._size // 2, ('30', )), -1)],
625
 
            result)
 
671
                         index._parsed_key_map)
 
672
        self.assertEqual([((index._size // 2, (b'30', )), -1)],
 
673
                         result)
626
674
 
627
675
    def test_lookup_key_above_probed_area(self):
628
676
        # generate a big enough index that we only read some of it on a typical
630
678
        index = self.make_index(nodes=self.make_nodes(64))
631
679
        # ask for the key in the middle, but a key that is located in the
632
680
        # unparsed region after the middle.
633
 
        result =index._lookup_keys_via_location(
634
 
            [(index._size // 2, ('50', ))])
 
681
        result = index._lookup_keys_via_location(
 
682
            [(index._size // 2, (b'50', ))])
635
683
        # check the parse map, this determines the test validity
636
684
        self.assertEqual([(0, 4008), (5046, 8996)], index._parsed_byte_map)
637
 
        self.assertEqual([(None, self.make_key(26)),
 
685
        self.assertEqual([((), self.make_key(26)),
638
686
                          (self.make_key(31), self.make_key(48))],
639
 
            index._parsed_key_map)
640
 
        self.assertEqual([((index._size // 2, ('50', )), +1)],
641
 
            result)
 
687
                         index._parsed_key_map)
 
688
        self.assertEqual([((index._size // 2, (b'50', )), +1)],
 
689
                         result)
642
690
 
643
691
    def test_lookup_key_resolves_references(self):
644
692
        # generate a big enough index that we only read some of it on a typical
646
694
        nodes = []
647
695
        for counter in range(99):
648
696
            nodes.append((self.make_key(counter), self.make_value(counter),
649
 
                ((self.make_key(counter + 20),),)  ))
 
697
                          ((self.make_key(counter + 20),),)))
650
698
        index = self.make_index(ref_lists=1, nodes=nodes)
651
699
        # lookup a key in the middle that does not exist, so that when we can
652
700
        # check that the referred-to-keys are not accessed automatically.
653
701
        index_size = index._size
654
702
        index_center = index_size // 2
655
703
        result = index._lookup_keys_via_location(
656
 
            [(index_center, ('40', ))])
 
704
            [(index_center, (b'40', ))])
657
705
        # check the parse map - only the start and middle should have been
658
706
        # parsed.
659
707
        self.assertEqual([(0, 4027), (10198, 14028)], index._parsed_byte_map)
660
 
        self.assertEqual([(None, self.make_key(17)),
 
708
        self.assertEqual([((), self.make_key(17)),
661
709
                          (self.make_key(44), self.make_key(5))],
662
 
            index._parsed_key_map)
 
710
                         index._parsed_key_map)
663
711
        # and check the transport activity likewise.
664
712
        self.assertEqual(
665
713
            [('readv', 'index', [(index_center, 800), (0, 200)], True,
666
 
                                  index_size)],
 
714
              index_size)],
667
715
            index._transport._activity)
668
716
        # reset the transport log for testing the reference lookup
669
717
        del index._transport._activity[:]
676
724
               ((self.make_key(65),),)))],
677
725
            result)
678
726
        self.assertEqual([('readv', 'index', [(15093, 800)], True, index_size)],
679
 
            index._transport._activity)
 
727
                         index._transport._activity)
680
728
 
681
729
    def test_lookup_key_can_buffer_all(self):
682
730
        nodes = []
683
731
        for counter in range(64):
684
732
            nodes.append((self.make_key(counter), self.make_value(counter),
685
 
                ((self.make_key(counter + 20),),)  ))
 
733
                          ((self.make_key(counter + 20),),)))
686
734
        index = self.make_index(ref_lists=1, nodes=nodes)
687
735
        # lookup a key in the middle that does not exist, so that when we can
688
736
        # check that the referred-to-keys are not accessed automatically.
689
737
        index_size = index._size
690
738
        index_center = index_size // 2
691
 
        result = index._lookup_keys_via_location([(index_center, ('40', ))])
 
739
        result = index._lookup_keys_via_location([(index_center, (b'40', ))])
692
740
        # check the parse map - only the start and middle should have been
693
741
        # parsed.
694
742
        self.assertEqual([(0, 3890), (6444, 10274)], index._parsed_byte_map)
695
 
        self.assertEqual([(None, self.make_key(25)),
 
743
        self.assertEqual([((), self.make_key(25)),
696
744
                          (self.make_key(37), self.make_key(52))],
697
 
            index._parsed_key_map)
 
745
                         index._parsed_key_map)
698
746
        # and check the transport activity likewise.
699
747
        self.assertEqual(
700
748
            [('readv', 'index', [(index_center, 800), (0, 200)], True,
701
 
                                  index_size)],
 
749
              index_size)],
702
750
            index._transport._activity)
703
751
        # reset the transport log for testing the reference lookup
704
752
        del index._transport._activity[:]
719
767
        self.assertEqual([], list(index.iter_all_entries()))
720
768
 
721
769
    def test_iter_all_entries_simple(self):
722
 
        index = self.make_index(nodes=[(('name', ), 'data', ())])
723
 
        self.assertEqual([(index, ('name', ), 'data')],
724
 
            list(index.iter_all_entries()))
 
770
        index = self.make_index(nodes=[((b'name', ), b'data', ())])
 
771
        self.assertEqual([(index, (b'name', ), b'data')],
 
772
                         list(index.iter_all_entries()))
725
773
 
726
774
    def test_iter_all_entries_simple_2_elements(self):
727
775
        index = self.make_index(key_elements=2,
728
 
            nodes=[(('name', 'surname'), 'data', ())])
729
 
        self.assertEqual([(index, ('name', 'surname'), 'data')],
730
 
            list(index.iter_all_entries()))
 
776
                                nodes=[((b'name', b'surname'), b'data', ())])
 
777
        self.assertEqual([(index, (b'name', b'surname'), b'data')],
 
778
                         list(index.iter_all_entries()))
731
779
 
732
780
    def test_iter_all_entries_references_resolved(self):
733
781
        index = self.make_index(1, nodes=[
734
 
            (('name', ), 'data', ([('ref', )], )),
735
 
            (('ref', ), 'refdata', ([], ))])
736
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),)),
737
 
            (index, ('ref', ), 'refdata', ((), ))]),
738
 
            set(index.iter_all_entries()))
 
782
            ((b'name', ), b'data', ([(b'ref', )], )),
 
783
            ((b'ref', ), b'refdata', ([], ))])
 
784
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),)),
 
785
                          (index, (b'ref', ), b'refdata', ((), ))},
 
786
                         set(index.iter_all_entries()))
739
787
 
740
788
    def test_iter_entries_buffers_once(self):
741
789
        index = self.make_index(nodes=self.make_nodes(2))
742
790
        # reset the transport log
743
791
        del index._transport._activity[:]
744
 
        self.assertEqual(set([(index, self.make_key(1), self.make_value(1))]),
 
792
        self.assertEqual({(index, self.make_key(1), self.make_value(1))},
745
793
                         set(index.iter_entries([self.make_key(1)])))
746
794
        # We should have requested reading the header bytes
747
795
        # But not needed any more than that because it would have triggered a
781
829
 
782
830
    def test_iter_entries_references_resolved(self):
783
831
        index = self.make_index(1, nodes=[
784
 
            (('name', ), 'data', ([('ref', ), ('ref', )], )),
785
 
            (('ref', ), 'refdata', ([], ))])
786
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),('ref',)),)),
787
 
            (index, ('ref', ), 'refdata', ((), ))]),
788
 
            set(index.iter_entries([('name',), ('ref',)])))
 
832
            ((b'name', ), b'data', ([(b'ref', ), (b'ref', )], )),
 
833
            ((b'ref', ), b'refdata', ([], ))])
 
834
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',), (b'ref',)),)),
 
835
                          (index, (b'ref', ), b'refdata', ((), ))},
 
836
                         set(index.iter_entries([(b'name',), (b'ref',)])))
789
837
 
790
838
    def test_iter_entries_references_2_refs_resolved(self):
791
839
        index = self.make_index(2, nodes=[
792
 
            (('name', ), 'data', ([('ref', )], [('ref', )])),
793
 
            (('ref', ), 'refdata', ([], []))])
794
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),), (('ref',),))),
795
 
            (index, ('ref', ), 'refdata', ((), ()))]),
796
 
            set(index.iter_entries([('name',), ('ref',)])))
 
840
            ((b'name', ), b'data', ([(b'ref', )], [(b'ref', )])),
 
841
            ((b'ref', ), b'refdata', ([], []))])
 
842
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),), ((b'ref',),))),
 
843
                          (index, (b'ref', ), b'refdata', ((), ()))},
 
844
                         set(index.iter_entries([(b'name',), (b'ref',)])))
797
845
 
798
846
    def test_iteration_absent_skipped(self):
799
847
        index = self.make_index(1, nodes=[
800
 
            (('name', ), 'data', ([('ref', )], ))])
801
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),))]),
802
 
            set(index.iter_all_entries()))
803
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),))]),
804
 
            set(index.iter_entries([('name', )])))
805
 
        self.assertEqual([], list(index.iter_entries([('ref', )])))
 
848
            ((b'name', ), b'data', ([(b'ref', )], ))])
 
849
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),))},
 
850
                         set(index.iter_all_entries()))
 
851
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),))},
 
852
                         set(index.iter_entries([(b'name', )])))
 
853
        self.assertEqual([], list(index.iter_entries([(b'ref', )])))
806
854
 
807
855
    def test_iteration_absent_skipped_2_element_keys(self):
808
856
        index = self.make_index(1, key_elements=2, nodes=[
809
 
            (('name', 'fin'), 'data', ([('ref', 'erence')], ))])
810
 
        self.assertEqual(set([(index, ('name', 'fin'), 'data', ((('ref', 'erence'),),))]),
811
 
            set(index.iter_all_entries()))
812
 
        self.assertEqual(set([(index, ('name', 'fin'), 'data', ((('ref', 'erence'),),))]),
813
 
            set(index.iter_entries([('name', 'fin')])))
814
 
        self.assertEqual([], list(index.iter_entries([('ref', 'erence')])))
 
857
            ((b'name', b'fin'), b'data', ([(b'ref', b'erence')], ))])
 
858
        self.assertEqual([(index, (b'name', b'fin'), b'data', (((b'ref', b'erence'),),))],
 
859
                         list(index.iter_all_entries()))
 
860
        self.assertEqual([(index, (b'name', b'fin'), b'data', (((b'ref', b'erence'),),))],
 
861
                         list(index.iter_entries([(b'name', b'fin')])))
 
862
        self.assertEqual([], list(index.iter_entries([(b'ref', b'erence')])))
815
863
 
816
864
    def test_iter_all_keys(self):
817
865
        index = self.make_index(1, nodes=[
818
 
            (('name', ), 'data', ([('ref', )], )),
819
 
            (('ref', ), 'refdata', ([], ))])
820
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),)),
821
 
            (index, ('ref', ), 'refdata', ((), ))]),
822
 
            set(index.iter_entries([('name', ), ('ref', )])))
 
866
            ((b'name', ), b'data', ([(b'ref', )], )),
 
867
            ((b'ref', ), b'refdata', ([], ))])
 
868
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),)),
 
869
                          (index, (b'ref', ), b'refdata', ((), ))},
 
870
                         set(index.iter_entries([(b'name', ), (b'ref', )])))
823
871
 
824
872
    def test_iter_nothing_empty(self):
825
873
        index = self.make_index()
827
875
 
828
876
    def test_iter_missing_entry_empty(self):
829
877
        index = self.make_index()
830
 
        self.assertEqual([], list(index.iter_entries([('a', )])))
 
878
        self.assertEqual([], list(index.iter_entries([(b'a', )])))
831
879
 
832
880
    def test_iter_missing_entry_empty_no_size(self):
833
 
        index = self.make_index()
834
 
        index = GraphIndex(index._transport, 'index', None)
835
 
        self.assertEqual([], list(index.iter_entries([('a', )])))
 
881
        idx = self.make_index()
 
882
        idx = _mod_index.GraphIndex(idx._transport, 'index', None)
 
883
        self.assertEqual([], list(idx.iter_entries([(b'a', )])))
836
884
 
837
885
    def test_iter_key_prefix_1_element_key_None(self):
838
886
        index = self.make_index()
839
 
        self.assertRaises(errors.BadIndexKey, list,
840
 
            index.iter_entries_prefix([(None, )]))
 
887
        self.assertRaises(_mod_index.BadIndexKey, list,
 
888
                          index.iter_entries_prefix([(None, )]))
841
889
 
842
890
    def test_iter_key_prefix_wrong_length(self):
843
891
        index = self.make_index()
844
 
        self.assertRaises(errors.BadIndexKey, list,
845
 
            index.iter_entries_prefix([('foo', None)]))
 
892
        self.assertRaises(_mod_index.BadIndexKey, list,
 
893
                          index.iter_entries_prefix([(b'foo', None)]))
846
894
        index = self.make_index(key_elements=2)
847
 
        self.assertRaises(errors.BadIndexKey, list,
848
 
            index.iter_entries_prefix([('foo', )]))
849
 
        self.assertRaises(errors.BadIndexKey, list,
850
 
            index.iter_entries_prefix([('foo', None, None)]))
 
895
        self.assertRaises(_mod_index.BadIndexKey, list,
 
896
                          index.iter_entries_prefix([(b'foo', )]))
 
897
        self.assertRaises(_mod_index.BadIndexKey, list,
 
898
                          index.iter_entries_prefix([(b'foo', None, None)]))
851
899
 
852
900
    def test_iter_key_prefix_1_key_element_no_refs(self):
853
 
        index = self.make_index( nodes=[
854
 
            (('name', ), 'data', ()),
855
 
            (('ref', ), 'refdata', ())])
856
 
        self.assertEqual(set([(index, ('name', ), 'data'),
857
 
            (index, ('ref', ), 'refdata')]),
858
 
            set(index.iter_entries_prefix([('name', ), ('ref', )])))
 
901
        index = self.make_index(nodes=[
 
902
            ((b'name', ), b'data', ()),
 
903
            ((b'ref', ), b'refdata', ())])
 
904
        self.assertEqual({(index, (b'name', ), b'data'),
 
905
                          (index, (b'ref', ), b'refdata')},
 
906
                         set(index.iter_entries_prefix([(b'name', ), (b'ref', )])))
859
907
 
860
908
    def test_iter_key_prefix_1_key_element_refs(self):
861
909
        index = self.make_index(1, nodes=[
862
 
            (('name', ), 'data', ([('ref', )], )),
863
 
            (('ref', ), 'refdata', ([], ))])
864
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),)),
865
 
            (index, ('ref', ), 'refdata', ((), ))]),
866
 
            set(index.iter_entries_prefix([('name', ), ('ref', )])))
 
910
            ((b'name', ), b'data', ([(b'ref', )], )),
 
911
            ((b'ref', ), b'refdata', ([], ))])
 
912
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),)),
 
913
                          (index, (b'ref', ), b'refdata', ((), ))},
 
914
                         set(index.iter_entries_prefix([(b'name', ), (b'ref', )])))
867
915
 
868
916
    def test_iter_key_prefix_2_key_element_no_refs(self):
869
917
        index = self.make_index(key_elements=2, nodes=[
870
 
            (('name', 'fin1'), 'data', ()),
871
 
            (('name', 'fin2'), 'beta', ()),
872
 
            (('ref', 'erence'), 'refdata', ())])
873
 
        self.assertEqual(set([(index, ('name', 'fin1'), 'data'),
874
 
            (index, ('ref', 'erence'), 'refdata')]),
875
 
            set(index.iter_entries_prefix([('name', 'fin1'), ('ref', 'erence')])))
876
 
        self.assertEqual(set([(index, ('name', 'fin1'), 'data'),
877
 
            (index, ('name', 'fin2'), 'beta')]),
878
 
            set(index.iter_entries_prefix([('name', None)])))
 
918
            ((b'name', b'fin1'), b'data', ()),
 
919
            ((b'name', b'fin2'), b'beta', ()),
 
920
            ((b'ref', b'erence'), b'refdata', ())])
 
921
        self.assertEqual({(index, (b'name', b'fin1'), b'data'),
 
922
                          (index, (b'ref', b'erence'), b'refdata')},
 
923
                         set(index.iter_entries_prefix([(b'name', b'fin1'), (b'ref', b'erence')])))
 
924
        self.assertEqual({(index, (b'name', b'fin1'), b'data'),
 
925
                          (index, (b'name', b'fin2'), b'beta')},
 
926
                         set(index.iter_entries_prefix([(b'name', None)])))
879
927
 
880
928
    def test_iter_key_prefix_2_key_element_refs(self):
881
929
        index = self.make_index(1, key_elements=2, nodes=[
882
 
            (('name', 'fin1'), 'data', ([('ref', 'erence')], )),
883
 
            (('name', 'fin2'), 'beta', ([], )),
884
 
            (('ref', 'erence'), 'refdata', ([], ))])
885
 
        self.assertEqual(set([(index, ('name', 'fin1'), 'data', ((('ref', 'erence'),),)),
886
 
            (index, ('ref', 'erence'), 'refdata', ((), ))]),
887
 
            set(index.iter_entries_prefix([('name', 'fin1'), ('ref', 'erence')])))
888
 
        self.assertEqual(set([(index, ('name', 'fin1'), 'data', ((('ref', 'erence'),),)),
889
 
            (index, ('name', 'fin2'), 'beta', ((), ))]),
890
 
            set(index.iter_entries_prefix([('name', None)])))
 
930
            ((b'name', b'fin1'), b'data', ([(b'ref', b'erence')], )),
 
931
            ((b'name', b'fin2'), b'beta', ([], )),
 
932
            ((b'ref', b'erence'), b'refdata', ([], ))])
 
933
        self.assertEqual({(index, (b'name', b'fin1'), b'data', (((b'ref', b'erence'),),)),
 
934
                          (index, (b'ref', b'erence'), b'refdata', ((), ))},
 
935
                         set(index.iter_entries_prefix([(b'name', b'fin1'), (b'ref', b'erence')])))
 
936
        self.assertEqual({(index, (b'name', b'fin1'), b'data', (((b'ref', b'erence'),),)),
 
937
                          (index, (b'name', b'fin2'), b'beta', ((), ))},
 
938
                         set(index.iter_entries_prefix([(b'name', None)])))
891
939
 
892
940
    def test_key_count_empty(self):
893
941
        index = self.make_index()
894
942
        self.assertEqual(0, index.key_count())
895
943
 
896
944
    def test_key_count_one(self):
897
 
        index = self.make_index(nodes=[(('name', ), '', ())])
 
945
        index = self.make_index(nodes=[((b'name', ), b'', ())])
898
946
        self.assertEqual(1, index.key_count())
899
947
 
900
948
    def test_key_count_two(self):
901
949
        index = self.make_index(nodes=[
902
 
            (('name', ), '', ()), (('foo', ), '', ())])
 
950
            ((b'name', ), b'', ()), ((b'foo', ), b'', ())])
903
951
        self.assertEqual(2, index.key_count())
904
952
 
905
953
    def test_read_and_parse_tracks_real_read_value(self):
917
965
 
918
966
    def test_read_and_parse_triggers_buffer_all(self):
919
967
        index = self.make_index(key_elements=2, nodes=[
920
 
            (('name', 'fin1'), 'data', ()),
921
 
            (('name', 'fin2'), 'beta', ()),
922
 
            (('ref', 'erence'), 'refdata', ())])
 
968
            ((b'name', b'fin1'), b'data', ()),
 
969
            ((b'name', b'fin2'), b'beta', ()),
 
970
            ((b'ref', b'erence'), b'refdata', ())])
923
971
        self.assertTrue(index._size > 0)
924
972
        self.assertIs(None, index._nodes)
925
973
        index._read_and_parse([(0, index._size)])
927
975
 
928
976
    def test_validate_bad_index_errors(self):
929
977
        trans = self.get_transport()
930
 
        trans.put_bytes('name', "not an index\n")
931
 
        index = GraphIndex(trans, 'name', 13)
932
 
        self.assertRaises(errors.BadIndexFormatSignature, index.validate)
 
978
        trans.put_bytes('name', b"not an index\n")
 
979
        idx = _mod_index.GraphIndex(trans, 'name', 13)
 
980
        self.assertRaises(_mod_index.BadIndexFormatSignature, idx.validate)
933
981
 
934
982
    def test_validate_bad_node_refs(self):
935
 
        index = self.make_index(2)
 
983
        idx = self.make_index(2)
936
984
        trans = self.get_transport()
937
985
        content = trans.get_bytes('index')
938
986
        # change the options line to end with a rather than a parseable number
939
 
        new_content = content[:-2] + 'a\n\n'
 
987
        new_content = content[:-2] + b'a\n\n'
940
988
        trans.put_bytes('index', new_content)
941
 
        self.assertRaises(errors.BadIndexOptions, index.validate)
 
989
        self.assertRaises(_mod_index.BadIndexOptions, idx.validate)
942
990
 
943
991
    def test_validate_missing_end_line_empty(self):
944
992
        index = self.make_index(2)
946
994
        content = trans.get_bytes('index')
947
995
        # truncate the last byte
948
996
        trans.put_bytes('index', content[:-1])
949
 
        self.assertRaises(errors.BadIndexData, index.validate)
 
997
        self.assertRaises(_mod_index.BadIndexData, index.validate)
950
998
 
951
999
    def test_validate_missing_end_line_nonempty(self):
952
 
        index = self.make_index(2, nodes=[(('key', ), '', ([], []))])
 
1000
        index = self.make_index(2, nodes=[((b'key', ), b'', ([], []))])
953
1001
        trans = self.get_transport()
954
1002
        content = trans.get_bytes('index')
955
1003
        # truncate the last byte
956
1004
        trans.put_bytes('index', content[:-1])
957
 
        self.assertRaises(errors.BadIndexData, index.validate)
 
1005
        self.assertRaises(_mod_index.BadIndexData, index.validate)
958
1006
 
959
1007
    def test_validate_empty(self):
960
1008
        index = self.make_index()
961
1009
        index.validate()
962
1010
 
963
1011
    def test_validate_no_refs_content(self):
964
 
        index = self.make_index(nodes=[(('key', ), 'value', ())])
 
1012
        index = self.make_index(nodes=[((b'key', ), b'value', ())])
965
1013
        index.validate()
966
1014
 
967
1015
    # XXX: external_references tests are duplicated in test_btree_index.  We
972
1020
 
973
1021
    def test_external_references_no_results(self):
974
1022
        index = self.make_index(ref_lists=1, nodes=[
975
 
            (('key',), 'value', ([],))])
 
1023
            ((b'key',), b'value', ([],))])
976
1024
        self.assertEqual(set(), index.external_references(0))
977
1025
 
978
1026
    def test_external_references_missing_ref(self):
979
 
        missing_key = ('missing',)
 
1027
        missing_key = (b'missing',)
980
1028
        index = self.make_index(ref_lists=1, nodes=[
981
 
            (('key',), 'value', ([missing_key],))])
982
 
        self.assertEqual(set([missing_key]), index.external_references(0))
 
1029
            ((b'key',), b'value', ([missing_key],))])
 
1030
        self.assertEqual({missing_key}, index.external_references(0))
983
1031
 
984
1032
    def test_external_references_multiple_ref_lists(self):
985
 
        missing_key = ('missing',)
 
1033
        missing_key = (b'missing',)
986
1034
        index = self.make_index(ref_lists=2, nodes=[
987
 
            (('key',), 'value', ([], [missing_key]))])
 
1035
            ((b'key',), b'value', ([], [missing_key]))])
988
1036
        self.assertEqual(set([]), index.external_references(0))
989
 
        self.assertEqual(set([missing_key]), index.external_references(1))
 
1037
        self.assertEqual({missing_key}, index.external_references(1))
990
1038
 
991
1039
    def test_external_references_two_records(self):
992
1040
        index = self.make_index(ref_lists=1, nodes=[
993
 
            (('key-1',), 'value', ([('key-2',)],)),
994
 
            (('key-2',), 'value', ([],)),
 
1041
            ((b'key-1',), b'value', ([(b'key-2',)],)),
 
1042
            ((b'key-2',), b'value', ([],)),
995
1043
            ])
996
1044
        self.assertEqual(set([]), index.external_references(0))
997
1045
 
998
1046
    def test__find_ancestors(self):
999
 
        key1 = ('key-1',)
1000
 
        key2 = ('key-2',)
 
1047
        key1 = (b'key-1',)
 
1048
        key2 = (b'key-2',)
1001
1049
        index = self.make_index(ref_lists=1, key_elements=1, nodes=[
1002
 
            (key1, 'value', ([key2],)),
1003
 
            (key2, 'value', ([],)),
 
1050
            (key1, b'value', ([key2],)),
 
1051
            (key2, b'value', ([],)),
1004
1052
            ])
1005
1053
        parent_map = {}
1006
1054
        missing_keys = set()
1007
 
        search_keys = index._find_ancestors([key1], 0, parent_map, missing_keys)
 
1055
        search_keys = index._find_ancestors(
 
1056
            [key1], 0, parent_map, missing_keys)
1008
1057
        self.assertEqual({key1: (key2,)}, parent_map)
1009
1058
        self.assertEqual(set(), missing_keys)
1010
 
        self.assertEqual(set([key2]), search_keys)
 
1059
        self.assertEqual({key2}, search_keys)
1011
1060
        search_keys = index._find_ancestors(search_keys, 0, parent_map,
1012
1061
                                            missing_keys)
1013
1062
        self.assertEqual({key1: (key2,), key2: ()}, parent_map)
1015
1064
        self.assertEqual(set(), search_keys)
1016
1065
 
1017
1066
    def test__find_ancestors_w_missing(self):
1018
 
        key1 = ('key-1',)
1019
 
        key2 = ('key-2',)
1020
 
        key3 = ('key-3',)
 
1067
        key1 = (b'key-1',)
 
1068
        key2 = (b'key-2',)
 
1069
        key3 = (b'key-3',)
1021
1070
        index = self.make_index(ref_lists=1, key_elements=1, nodes=[
1022
 
            (key1, 'value', ([key2],)),
1023
 
            (key2, 'value', ([],)),
 
1071
            (key1, b'value', ([key2],)),
 
1072
            (key2, b'value', ([],)),
1024
1073
            ])
1025
1074
        parent_map = {}
1026
1075
        missing_keys = set()
1027
1076
        search_keys = index._find_ancestors([key2, key3], 0, parent_map,
1028
1077
                                            missing_keys)
1029
1078
        self.assertEqual({key2: ()}, parent_map)
1030
 
        self.assertEqual(set([key3]), missing_keys)
 
1079
        self.assertEqual({key3}, missing_keys)
1031
1080
        self.assertEqual(set(), search_keys)
1032
1081
 
1033
1082
    def test__find_ancestors_dont_search_known(self):
1034
 
        key1 = ('key-1',)
1035
 
        key2 = ('key-2',)
1036
 
        key3 = ('key-3',)
 
1083
        key1 = (b'key-1',)
 
1084
        key2 = (b'key-2',)
 
1085
        key3 = (b'key-3',)
1037
1086
        index = self.make_index(ref_lists=1, key_elements=1, nodes=[
1038
 
            (key1, 'value', ([key2],)),
1039
 
            (key2, 'value', ([key3],)),
1040
 
            (key3, 'value', ([],)),
 
1087
            (key1, b'value', ([key2],)),
 
1088
            (key2, b'value', ([key3],)),
 
1089
            (key3, b'value', ([],)),
1041
1090
            ])
1042
1091
        # We already know about key2, so we won't try to search for key3
1043
1092
        parent_map = {key2: (key3,)}
1049
1098
        self.assertEqual(set(), search_keys)
1050
1099
 
1051
1100
    def test_supports_unlimited_cache(self):
1052
 
        builder = GraphIndexBuilder(0, key_elements=1)
 
1101
        builder = _mod_index.GraphIndexBuilder(0, key_elements=1)
1053
1102
        stream = builder.finish()
1054
 
        trans = get_transport(self.get_url())
 
1103
        trans = self.get_transport()
1055
1104
        size = trans.put_file('index', stream)
1056
1105
        # It doesn't matter what unlimited_cache does here, just that it can be
1057
1106
        # passed
1058
 
        index = GraphIndex(trans, 'index', size, unlimited_cache=True)
1059
 
 
1060
 
 
1061
 
class TestCombinedGraphIndex(TestCaseWithMemoryTransport):
 
1107
        idx = _mod_index.GraphIndex(trans, 'index', size, unlimited_cache=True)
 
1108
 
 
1109
 
 
1110
class TestCombinedGraphIndex(tests.TestCaseWithMemoryTransport):
1062
1111
 
1063
1112
    def make_index(self, name, ref_lists=0, key_elements=1, nodes=[]):
1064
 
        builder = GraphIndexBuilder(ref_lists, key_elements=key_elements)
 
1113
        builder = _mod_index.GraphIndexBuilder(
 
1114
            ref_lists, key_elements=key_elements)
1065
1115
        for key, value, references in nodes:
1066
1116
            builder.add_node(key, value, references)
1067
1117
        stream = builder.finish()
1068
1118
        trans = self.get_transport()
1069
1119
        size = trans.put_file(name, stream)
1070
 
        return GraphIndex(trans, name, size)
 
1120
        return _mod_index.GraphIndex(trans, name, size)
1071
1121
 
1072
1122
    def make_combined_index_with_missing(self, missing=['1', '2']):
1073
1123
        """Create a CombinedGraphIndex which will have missing indexes.
1079
1129
        :param missing: The underlying indexes to delete
1080
1130
        :return: (CombinedGraphIndex, reload_counter)
1081
1131
        """
1082
 
        index1 = self.make_index('1', nodes=[(('1',), '', ())])
1083
 
        index2 = self.make_index('2', nodes=[(('2',), '', ())])
1084
 
        index3 = self.make_index('3', nodes=[
1085
 
            (('1',), '', ()),
1086
 
            (('2',), '', ())])
 
1132
        idx1 = self.make_index('1', nodes=[((b'1',), b'', ())])
 
1133
        idx2 = self.make_index('2', nodes=[((b'2',), b'', ())])
 
1134
        idx3 = self.make_index('3', nodes=[
 
1135
            ((b'1',), b'', ()),
 
1136
            ((b'2',), b'', ())])
1087
1137
 
1088
1138
        # total_reloads, num_changed, num_unchanged
1089
1139
        reload_counter = [0, 0, 0]
 
1140
 
1090
1141
        def reload():
1091
1142
            reload_counter[0] += 1
1092
 
            new_indices = [index3]
1093
 
            if index._indices == new_indices:
 
1143
            new_indices = [idx3]
 
1144
            if idx._indices == new_indices:
1094
1145
                reload_counter[2] += 1
1095
1146
                return False
1096
1147
            reload_counter[1] += 1
1097
 
            index._indices[:] = new_indices
 
1148
            idx._indices[:] = new_indices
1098
1149
            return True
1099
 
        index = CombinedGraphIndex([index1, index2], reload_func=reload)
 
1150
        idx = _mod_index.CombinedGraphIndex([idx1, idx2], reload_func=reload)
1100
1151
        trans = self.get_transport()
1101
1152
        for fname in missing:
1102
1153
            trans.delete(fname)
1103
 
        return index, reload_counter
 
1154
        return idx, reload_counter
1104
1155
 
1105
1156
    def test_open_missing_index_no_error(self):
1106
1157
        trans = self.get_transport()
1107
 
        index1 = GraphIndex(trans, 'missing', 100)
1108
 
        index = CombinedGraphIndex([index1])
 
1158
        idx1 = _mod_index.GraphIndex(trans, 'missing', 100)
 
1159
        idx = _mod_index.CombinedGraphIndex([idx1])
1109
1160
 
1110
1161
    def test_add_index(self):
1111
 
        index = CombinedGraphIndex([])
1112
 
        index1 = self.make_index('name', 0, nodes=[(('key', ), '', ())])
1113
 
        index.insert_index(0, index1)
1114
 
        self.assertEqual([(index1, ('key', ), '')], list(index.iter_all_entries()))
 
1162
        idx = _mod_index.CombinedGraphIndex([])
 
1163
        idx1 = self.make_index('name', 0, nodes=[((b'key', ), b'', ())])
 
1164
        idx.insert_index(0, idx1)
 
1165
        self.assertEqual([(idx1, (b'key', ), b'')],
 
1166
                         list(idx.iter_all_entries()))
1115
1167
 
1116
1168
    def test_clear_cache(self):
1117
1169
        log = []
1128
1180
                log.append(self._index)
1129
1181
                return self._index.clear_cache()
1130
1182
 
1131
 
        index = CombinedGraphIndex([])
1132
 
        index1 = self.make_index('name', 0, nodes=[(('key', ), '', ())])
1133
 
        index.insert_index(0, ClearCacheProxy(index1))
1134
 
        index2 = self.make_index('name', 0, nodes=[(('key', ), '', ())])
1135
 
        index.insert_index(1, ClearCacheProxy(index2))
 
1183
        idx = _mod_index.CombinedGraphIndex([])
 
1184
        idx1 = self.make_index('name', 0, nodes=[((b'key', ), b'', ())])
 
1185
        idx.insert_index(0, ClearCacheProxy(idx1))
 
1186
        idx2 = self.make_index('name', 0, nodes=[((b'key', ), b'', ())])
 
1187
        idx.insert_index(1, ClearCacheProxy(idx2))
1136
1188
        # CombinedGraphIndex should call 'clear_cache()' on all children
1137
 
        index.clear_cache()
1138
 
        self.assertEqual(sorted([index1, index2]), sorted(log))
 
1189
        idx.clear_cache()
 
1190
        self.assertEqual(sorted([idx1, idx2]), sorted(log))
1139
1191
 
1140
1192
    def test_iter_all_entries_empty(self):
1141
 
        index = CombinedGraphIndex([])
1142
 
        self.assertEqual([], list(index.iter_all_entries()))
 
1193
        idx = _mod_index.CombinedGraphIndex([])
 
1194
        self.assertEqual([], list(idx.iter_all_entries()))
1143
1195
 
1144
1196
    def test_iter_all_entries_children_empty(self):
1145
 
        index1 = self.make_index('name')
1146
 
        index = CombinedGraphIndex([index1])
1147
 
        self.assertEqual([], list(index.iter_all_entries()))
 
1197
        idx1 = self.make_index('name')
 
1198
        idx = _mod_index.CombinedGraphIndex([idx1])
 
1199
        self.assertEqual([], list(idx.iter_all_entries()))
1148
1200
 
1149
1201
    def test_iter_all_entries_simple(self):
1150
 
        index1 = self.make_index('name', nodes=[(('name', ), 'data', ())])
1151
 
        index = CombinedGraphIndex([index1])
1152
 
        self.assertEqual([(index1, ('name', ), 'data')],
1153
 
            list(index.iter_all_entries()))
 
1202
        idx1 = self.make_index('name', nodes=[((b'name', ), b'data', ())])
 
1203
        idx = _mod_index.CombinedGraphIndex([idx1])
 
1204
        self.assertEqual([(idx1, (b'name', ), b'data')],
 
1205
                         list(idx.iter_all_entries()))
1154
1206
 
1155
1207
    def test_iter_all_entries_two_indices(self):
1156
 
        index1 = self.make_index('name1', nodes=[(('name', ), 'data', ())])
1157
 
        index2 = self.make_index('name2', nodes=[(('2', ), '', ())])
1158
 
        index = CombinedGraphIndex([index1, index2])
1159
 
        self.assertEqual([(index1, ('name', ), 'data'),
1160
 
            (index2, ('2', ), '')],
1161
 
            list(index.iter_all_entries()))
 
1208
        idx1 = self.make_index('name1', nodes=[((b'name', ), b'data', ())])
 
1209
        idx2 = self.make_index('name2', nodes=[((b'2', ), b'', ())])
 
1210
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1211
        self.assertEqual([(idx1, (b'name', ), b'data'),
 
1212
                          (idx2, (b'2', ), b'')],
 
1213
                         list(idx.iter_all_entries()))
1162
1214
 
1163
1215
    def test_iter_entries_two_indices_dup_key(self):
1164
 
        index1 = self.make_index('name1', nodes=[(('name', ), 'data', ())])
1165
 
        index2 = self.make_index('name2', nodes=[(('name', ), 'data', ())])
1166
 
        index = CombinedGraphIndex([index1, index2])
1167
 
        self.assertEqual([(index1, ('name', ), 'data')],
1168
 
            list(index.iter_entries([('name', )])))
 
1216
        idx1 = self.make_index('name1', nodes=[((b'name', ), b'data', ())])
 
1217
        idx2 = self.make_index('name2', nodes=[((b'name', ), b'data', ())])
 
1218
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1219
        self.assertEqual([(idx1, (b'name', ), b'data')],
 
1220
                         list(idx.iter_entries([(b'name', )])))
1169
1221
 
1170
1222
    def test_iter_all_entries_two_indices_dup_key(self):
1171
 
        index1 = self.make_index('name1', nodes=[(('name', ), 'data', ())])
1172
 
        index2 = self.make_index('name2', nodes=[(('name', ), 'data', ())])
1173
 
        index = CombinedGraphIndex([index1, index2])
1174
 
        self.assertEqual([(index1, ('name', ), 'data')],
1175
 
            list(index.iter_all_entries()))
 
1223
        idx1 = self.make_index('name1', nodes=[((b'name', ), b'data', ())])
 
1224
        idx2 = self.make_index('name2', nodes=[((b'name', ), b'data', ())])
 
1225
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1226
        self.assertEqual([(idx1, (b'name', ), b'data')],
 
1227
                         list(idx.iter_all_entries()))
1176
1228
 
1177
1229
    def test_iter_key_prefix_2_key_element_refs(self):
1178
 
        index1 = self.make_index('1', 1, key_elements=2, nodes=[
1179
 
            (('name', 'fin1'), 'data', ([('ref', 'erence')], ))])
1180
 
        index2 = self.make_index('2', 1, key_elements=2, nodes=[
1181
 
            (('name', 'fin2'), 'beta', ([], )),
1182
 
            (('ref', 'erence'), 'refdata', ([], ))])
1183
 
        index = CombinedGraphIndex([index1, index2])
1184
 
        self.assertEqual(set([(index1, ('name', 'fin1'), 'data', ((('ref', 'erence'),),)),
1185
 
            (index2, ('ref', 'erence'), 'refdata', ((), ))]),
1186
 
            set(index.iter_entries_prefix([('name', 'fin1'), ('ref', 'erence')])))
1187
 
        self.assertEqual(set([(index1, ('name', 'fin1'), 'data', ((('ref', 'erence'),),)),
1188
 
            (index2, ('name', 'fin2'), 'beta', ((), ))]),
1189
 
            set(index.iter_entries_prefix([('name', None)])))
 
1230
        idx1 = self.make_index('1', 1, key_elements=2, nodes=[
 
1231
            ((b'name', b'fin1'), b'data', ([(b'ref', b'erence')], ))])
 
1232
        idx2 = self.make_index('2', 1, key_elements=2, nodes=[
 
1233
            ((b'name', b'fin2'), b'beta', ([], )),
 
1234
            ((b'ref', b'erence'), b'refdata', ([], ))])
 
1235
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1236
        self.assertEqual({(idx1, (b'name', b'fin1'), b'data',
 
1237
                           (((b'ref', b'erence'),),)),
 
1238
                          (idx2, (b'ref', b'erence'), b'refdata', ((), ))},
 
1239
                         set(idx.iter_entries_prefix([(b'name', b'fin1'),
 
1240
                                                      (b'ref', b'erence')])))
 
1241
        self.assertEqual({(idx1, (b'name', b'fin1'), b'data',
 
1242
                           (((b'ref', b'erence'),),)),
 
1243
                          (idx2, (b'name', b'fin2'), b'beta', ((), ))},
 
1244
                         set(idx.iter_entries_prefix([(b'name', None)])))
1190
1245
 
1191
1246
    def test_iter_nothing_empty(self):
1192
 
        index = CombinedGraphIndex([])
1193
 
        self.assertEqual([], list(index.iter_entries([])))
 
1247
        idx = _mod_index.CombinedGraphIndex([])
 
1248
        self.assertEqual([], list(idx.iter_entries([])))
1194
1249
 
1195
1250
    def test_iter_nothing_children_empty(self):
1196
 
        index1 = self.make_index('name')
1197
 
        index = CombinedGraphIndex([index1])
1198
 
        self.assertEqual([], list(index.iter_entries([])))
 
1251
        idx1 = self.make_index('name')
 
1252
        idx = _mod_index.CombinedGraphIndex([idx1])
 
1253
        self.assertEqual([], list(idx.iter_entries([])))
1199
1254
 
1200
1255
    def test_iter_all_keys(self):
1201
 
        index1 = self.make_index('1', 1, nodes=[
1202
 
            (('name', ), 'data', ([('ref', )], ))])
1203
 
        index2 = self.make_index('2', 1, nodes=[
1204
 
            (('ref', ), 'refdata', ((), ))])
1205
 
        index = CombinedGraphIndex([index1, index2])
1206
 
        self.assertEqual(set([(index1, ('name', ), 'data', ((('ref', ), ), )),
1207
 
            (index2, ('ref', ), 'refdata', ((), ))]),
1208
 
            set(index.iter_entries([('name', ), ('ref', )])))
 
1256
        idx1 = self.make_index('1', 1, nodes=[((b'name', ), b'data',
 
1257
                                               ([(b'ref', )], ))])
 
1258
        idx2 = self.make_index(
 
1259
            '2', 1, nodes=[((b'ref', ), b'refdata', ((), ))])
 
1260
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1261
        self.assertEqual({(idx1, (b'name', ), b'data', (((b'ref', ), ), )),
 
1262
                          (idx2, (b'ref', ), b'refdata', ((), ))},
 
1263
                         set(idx.iter_entries([(b'name', ), (b'ref', )])))
1209
1264
 
1210
1265
    def test_iter_all_keys_dup_entry(self):
1211
 
        index1 = self.make_index('1', 1, nodes=[
1212
 
            (('name', ), 'data', ([('ref', )], )),
1213
 
            (('ref', ), 'refdata', ([], ))])
1214
 
        index2 = self.make_index('2', 1, nodes=[
1215
 
            (('ref', ), 'refdata', ([], ))])
1216
 
        index = CombinedGraphIndex([index1, index2])
1217
 
        self.assertEqual(set([(index1, ('name', ), 'data', ((('ref',),),)),
1218
 
            (index1, ('ref', ), 'refdata', ((), ))]),
1219
 
            set(index.iter_entries([('name', ), ('ref', )])))
 
1266
        idx1 = self.make_index('1', 1, nodes=[((b'name', ), b'data',
 
1267
                                               ([(b'ref', )], )),
 
1268
                                              ((b'ref', ), b'refdata', ([], ))])
 
1269
        idx2 = self.make_index(
 
1270
            '2', 1, nodes=[((b'ref', ), b'refdata', ([], ))])
 
1271
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1272
        self.assertEqual({(idx1, (b'name', ), b'data', (((b'ref',),),)),
 
1273
                          (idx1, (b'ref', ), b'refdata', ((), ))},
 
1274
                         set(idx.iter_entries([(b'name', ), (b'ref', )])))
1220
1275
 
1221
1276
    def test_iter_missing_entry_empty(self):
1222
 
        index = CombinedGraphIndex([])
1223
 
        self.assertEqual([], list(index.iter_entries([('a', )])))
 
1277
        idx = _mod_index.CombinedGraphIndex([])
 
1278
        self.assertEqual([], list(idx.iter_entries([('a', )])))
1224
1279
 
1225
1280
    def test_iter_missing_entry_one_index(self):
1226
 
        index1 = self.make_index('1')
1227
 
        index = CombinedGraphIndex([index1])
1228
 
        self.assertEqual([], list(index.iter_entries([('a', )])))
 
1281
        idx1 = self.make_index('1')
 
1282
        idx = _mod_index.CombinedGraphIndex([idx1])
 
1283
        self.assertEqual([], list(idx.iter_entries([(b'a', )])))
1229
1284
 
1230
1285
    def test_iter_missing_entry_two_index(self):
1231
 
        index1 = self.make_index('1')
1232
 
        index2 = self.make_index('2')
1233
 
        index = CombinedGraphIndex([index1, index2])
1234
 
        self.assertEqual([], list(index.iter_entries([('a', )])))
 
1286
        idx1 = self.make_index('1')
 
1287
        idx2 = self.make_index('2')
 
1288
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1289
        self.assertEqual([], list(idx.iter_entries([('a', )])))
1235
1290
 
1236
1291
    def test_iter_entry_present_one_index_only(self):
1237
 
        index1 = self.make_index('1', nodes=[(('key', ), '', ())])
1238
 
        index2 = self.make_index('2', nodes=[])
1239
 
        index = CombinedGraphIndex([index1, index2])
1240
 
        self.assertEqual([(index1, ('key', ), '')],
1241
 
            list(index.iter_entries([('key', )])))
 
1292
        idx1 = self.make_index('1', nodes=[((b'key', ), b'', ())])
 
1293
        idx2 = self.make_index('2', nodes=[])
 
1294
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1295
        self.assertEqual([(idx1, (b'key', ), b'')],
 
1296
                         list(idx.iter_entries([(b'key', )])))
1242
1297
        # and in the other direction
1243
 
        index = CombinedGraphIndex([index2, index1])
1244
 
        self.assertEqual([(index1, ('key', ), '')],
1245
 
            list(index.iter_entries([('key', )])))
 
1298
        idx = _mod_index.CombinedGraphIndex([idx2, idx1])
 
1299
        self.assertEqual([(idx1, (b'key', ), b'')],
 
1300
                         list(idx.iter_entries([(b'key', )])))
1246
1301
 
1247
1302
    def test_key_count_empty(self):
1248
 
        index1 = self.make_index('1', nodes=[])
1249
 
        index2 = self.make_index('2', nodes=[])
1250
 
        index = CombinedGraphIndex([index1, index2])
1251
 
        self.assertEqual(0, index.key_count())
 
1303
        idx1 = self.make_index('1', nodes=[])
 
1304
        idx2 = self.make_index('2', nodes=[])
 
1305
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1306
        self.assertEqual(0, idx.key_count())
1252
1307
 
1253
1308
    def test_key_count_sums_index_keys(self):
1254
 
        index1 = self.make_index('1', nodes=[
1255
 
            (('1',), '', ()),
1256
 
            (('2',), '', ())])
1257
 
        index2 = self.make_index('2', nodes=[(('1',), '', ())])
1258
 
        index = CombinedGraphIndex([index1, index2])
1259
 
        self.assertEqual(3, index.key_count())
 
1309
        idx1 = self.make_index('1', nodes=[
 
1310
            ((b'1',), b'', ()),
 
1311
            ((b'2',), b'', ())])
 
1312
        idx2 = self.make_index('2', nodes=[((b'1',), b'', ())])
 
1313
        idx = _mod_index.CombinedGraphIndex([idx1, idx2])
 
1314
        self.assertEqual(3, idx.key_count())
1260
1315
 
1261
1316
    def test_validate_bad_child_index_errors(self):
1262
1317
        trans = self.get_transport()
1263
 
        trans.put_bytes('name', "not an index\n")
1264
 
        index1 = GraphIndex(trans, 'name', 13)
1265
 
        index = CombinedGraphIndex([index1])
1266
 
        self.assertRaises(errors.BadIndexFormatSignature, index.validate)
 
1318
        trans.put_bytes('name', b"not an index\n")
 
1319
        idx1 = _mod_index.GraphIndex(trans, 'name', 13)
 
1320
        idx = _mod_index.CombinedGraphIndex([idx1])
 
1321
        self.assertRaises(_mod_index.BadIndexFormatSignature, idx.validate)
1267
1322
 
1268
1323
    def test_validate_empty(self):
1269
 
        index = CombinedGraphIndex([])
1270
 
        index.validate()
 
1324
        idx = _mod_index.CombinedGraphIndex([])
 
1325
        idx.validate()
1271
1326
 
1272
1327
    def test_key_count_reloads(self):
1273
 
        index, reload_counter = self.make_combined_index_with_missing()
1274
 
        self.assertEqual(2, index.key_count())
 
1328
        idx, reload_counter = self.make_combined_index_with_missing()
 
1329
        self.assertEqual(2, idx.key_count())
1275
1330
        self.assertEqual([1, 1, 0], reload_counter)
1276
1331
 
1277
1332
    def test_key_count_no_reload(self):
1278
 
        index, reload_counter = self.make_combined_index_with_missing()
1279
 
        index._reload_func = None
 
1333
        idx, reload_counter = self.make_combined_index_with_missing()
 
1334
        idx._reload_func = None
1280
1335
        # Without a _reload_func we just raise the exception
1281
 
        self.assertRaises(errors.NoSuchFile, index.key_count)
 
1336
        self.assertRaises(errors.NoSuchFile, idx.key_count)
1282
1337
 
1283
1338
    def test_key_count_reloads_and_fails(self):
1284
1339
        # We have deleted all underlying indexes, so we will try to reload, but
1285
1340
        # still fail. This is mostly to test we don't get stuck in an infinite
1286
1341
        # loop trying to reload
1287
 
        index, reload_counter = self.make_combined_index_with_missing(
1288
 
                                    ['1', '2', '3'])
1289
 
        self.assertRaises(errors.NoSuchFile, index.key_count)
 
1342
        idx, reload_counter = self.make_combined_index_with_missing(
 
1343
            ['1', '2', '3'])
 
1344
        self.assertRaises(errors.NoSuchFile, idx.key_count)
1290
1345
        self.assertEqual([2, 1, 1], reload_counter)
1291
1346
 
1292
1347
    def test_iter_entries_reloads(self):
1293
1348
        index, reload_counter = self.make_combined_index_with_missing()
1294
 
        result = list(index.iter_entries([('1',), ('2',), ('3',)]))
 
1349
        result = list(index.iter_entries([(b'1',), (b'2',), (b'3',)]))
1295
1350
        index3 = index._indices[0]
1296
 
        self.assertEqual([(index3, ('1',), ''), (index3, ('2',), '')],
1297
 
                         result)
 
1351
        self.assertEqual({(index3, (b'1',), b''), (index3, (b'2',), b'')},
 
1352
                         set(result))
1298
1353
        self.assertEqual([1, 1, 0], reload_counter)
1299
1354
 
1300
1355
    def test_iter_entries_reloads_midway(self):
1302
1357
        # through
1303
1358
        index, reload_counter = self.make_combined_index_with_missing(['2'])
1304
1359
        index1, index2 = index._indices
1305
 
        result = list(index.iter_entries([('1',), ('2',), ('3',)]))
 
1360
        result = list(index.iter_entries([(b'1',), (b'2',), (b'3',)]))
1306
1361
        index3 = index._indices[0]
1307
 
        # We had already yielded '1', so we just go on to the next, we should
1308
 
        # not yield '1' twice.
1309
 
        self.assertEqual([(index1, ('1',), ''), (index3, ('2',), '')],
 
1362
        # We had already yielded b'1', so we just go on to the next, we should
 
1363
        # not yield b'1' twice.
 
1364
        self.assertEqual([(index1, (b'1',), b''), (index3, (b'2',), b'')],
1310
1365
                         result)
1311
1366
        self.assertEqual([1, 1, 0], reload_counter)
1312
1367
 
1318
1373
 
1319
1374
    def test_iter_entries_reloads_and_fails(self):
1320
1375
        index, reload_counter = self.make_combined_index_with_missing(
1321
 
                                    ['1', '2', '3'])
 
1376
            ['1', '2', '3'])
1322
1377
        self.assertListRaises(errors.NoSuchFile, index.iter_entries, [('3',)])
1323
1378
        self.assertEqual([2, 1, 1], reload_counter)
1324
1379
 
1326
1381
        index, reload_counter = self.make_combined_index_with_missing()
1327
1382
        result = list(index.iter_all_entries())
1328
1383
        index3 = index._indices[0]
1329
 
        self.assertEqual([(index3, ('1',), ''), (index3, ('2',), '')],
1330
 
                         result)
 
1384
        self.assertEqual({(index3, (b'1',), b''), (index3, (b'2',), b'')},
 
1385
                         set(result))
1331
1386
        self.assertEqual([1, 1, 0], reload_counter)
1332
1387
 
1333
1388
    def test_iter_all_entries_reloads_midway(self):
1337
1392
        index3 = index._indices[0]
1338
1393
        # We had already yielded '1', so we just go on to the next, we should
1339
1394
        # not yield '1' twice.
1340
 
        self.assertEqual([(index1, ('1',), ''), (index3, ('2',), '')],
 
1395
        self.assertEqual([(index1, (b'1',), b''), (index3, (b'2',), b'')],
1341
1396
                         result)
1342
1397
        self.assertEqual([1, 1, 0], reload_counter)
1343
1398
 
1348
1403
 
1349
1404
    def test_iter_all_entries_reloads_and_fails(self):
1350
1405
        index, reload_counter = self.make_combined_index_with_missing(
1351
 
                                    ['1', '2', '3'])
 
1406
            ['1', '2', '3'])
1352
1407
        self.assertListRaises(errors.NoSuchFile, index.iter_all_entries)
1353
1408
 
1354
1409
    def test_iter_entries_prefix_reloads(self):
1355
1410
        index, reload_counter = self.make_combined_index_with_missing()
1356
 
        result = list(index.iter_entries_prefix([('1',)]))
 
1411
        result = list(index.iter_entries_prefix([(b'1',)]))
1357
1412
        index3 = index._indices[0]
1358
 
        self.assertEqual([(index3, ('1',), '')], result)
 
1413
        self.assertEqual([(index3, (b'1',), b'')], result)
1359
1414
        self.assertEqual([1, 1, 0], reload_counter)
1360
1415
 
1361
1416
    def test_iter_entries_prefix_reloads_midway(self):
1362
1417
        index, reload_counter = self.make_combined_index_with_missing(['2'])
1363
1418
        index1, index2 = index._indices
1364
 
        result = list(index.iter_entries_prefix([('1',)]))
 
1419
        result = list(index.iter_entries_prefix([(b'1',)]))
1365
1420
        index3 = index._indices[0]
1366
 
        # We had already yielded '1', so we just go on to the next, we should
1367
 
        # not yield '1' twice.
1368
 
        self.assertEqual([(index1, ('1',), '')], result)
 
1421
        # We had already yielded b'1', so we just go on to the next, we should
 
1422
        # not yield b'1' twice.
 
1423
        self.assertEqual([(index1, (b'1',), b'')], result)
1369
1424
        self.assertEqual([1, 1, 0], reload_counter)
1370
1425
 
1371
1426
    def test_iter_entries_prefix_no_reload(self):
1372
1427
        index, reload_counter = self.make_combined_index_with_missing()
1373
1428
        index._reload_func = None
1374
1429
        self.assertListRaises(errors.NoSuchFile, index.iter_entries_prefix,
1375
 
                                                 [('1',)])
 
1430
                              [(b'1',)])
1376
1431
 
1377
1432
    def test_iter_entries_prefix_reloads_and_fails(self):
1378
1433
        index, reload_counter = self.make_combined_index_with_missing(
1379
 
                                    ['1', '2', '3'])
 
1434
            ['1', '2', '3'])
1380
1435
        self.assertListRaises(errors.NoSuchFile, index.iter_entries_prefix,
1381
 
                                                 [('1',)])
1382
 
 
 
1436
                              [(b'1',)])
1383
1437
 
1384
1438
    def make_index_with_simple_nodes(self, name, num_nodes=1):
1385
1439
        """Make an index named after 'name', with keys named after 'name' too.
1387
1441
        Nodes will have a value of '' and no references.
1388
1442
        """
1389
1443
        nodes = [
1390
 
            (('index-%s-key-%s' % (name, n),), '', ())
1391
 
            for n in range(1, num_nodes+1)]
 
1444
            ((('index-%s-key-%s' % (name, n)).encode('ascii'),), b'', ())
 
1445
            for n in range(1, num_nodes + 1)]
1392
1446
        return self.make_index('index-%s' % name, 0, nodes=nodes)
1393
1447
 
1394
1448
    def test_reorder_after_iter_entries(self):
1395
 
        # Four indices: [key1] in index1, [key2,key3] in index2, [] in index3,
1396
 
        # [key4] in index4.
1397
 
        index = CombinedGraphIndex([])
1398
 
        index.insert_index(0, self.make_index_with_simple_nodes('1'), '1')
1399
 
        index.insert_index(1, self.make_index_with_simple_nodes('2'), '2')
1400
 
        index.insert_index(2, self.make_index_with_simple_nodes('3'), '3')
1401
 
        index.insert_index(3, self.make_index_with_simple_nodes('4'), '4')
1402
 
        index1, index2, index3, index4 = index._indices
1403
 
        # Query a key from index4 and index2.
1404
 
        self.assertLength(2, list(index.iter_entries(
1405
 
            [('index-4-key-1',), ('index-2-key-1',)])))
1406
 
        # Now index2 and index4 should be moved to the front (and index1 should
1407
 
        # still be before index3).
1408
 
        self.assertEqual([index2, index4, index1, index3], index._indices)
1409
 
        self.assertEqual(['2', '4', '1', '3'], index._index_names)
 
1449
        # Four indices: [key1] in idx1, [key2,key3] in idx2, [] in idx3,
 
1450
        # [key4] in idx4.
 
1451
        idx = _mod_index.CombinedGraphIndex([])
 
1452
        idx.insert_index(0, self.make_index_with_simple_nodes('1'), b'1')
 
1453
        idx.insert_index(1, self.make_index_with_simple_nodes('2'), b'2')
 
1454
        idx.insert_index(2, self.make_index_with_simple_nodes('3'), b'3')
 
1455
        idx.insert_index(3, self.make_index_with_simple_nodes('4'), b'4')
 
1456
        idx1, idx2, idx3, idx4 = idx._indices
 
1457
        # Query a key from idx4 and idx2.
 
1458
        self.assertLength(2, list(idx.iter_entries(
 
1459
            [(b'index-4-key-1',), (b'index-2-key-1',)])))
 
1460
        # Now idx2 and idx4 should be moved to the front (and idx1 should
 
1461
        # still be before idx3).
 
1462
        self.assertEqual([idx2, idx4, idx1, idx3], idx._indices)
 
1463
        self.assertEqual([b'2', b'4', b'1', b'3'], idx._index_names)
1410
1464
 
1411
1465
    def test_reorder_propagates_to_siblings(self):
1412
1466
        # Two CombinedGraphIndex objects, with the same number of indicies with
1413
1467
        # matching names.
1414
 
        cgi1 = CombinedGraphIndex([])
1415
 
        cgi2 = CombinedGraphIndex([])
 
1468
        cgi1 = _mod_index.CombinedGraphIndex([])
 
1469
        cgi2 = _mod_index.CombinedGraphIndex([])
1416
1470
        cgi1.insert_index(0, self.make_index_with_simple_nodes('1-1'), 'one')
1417
1471
        cgi1.insert_index(1, self.make_index_with_simple_nodes('1-2'), 'two')
1418
1472
        cgi2.insert_index(0, self.make_index_with_simple_nodes('2-1'), 'one')
1420
1474
        index2_1, index2_2 = cgi2._indices
1421
1475
        cgi1.set_sibling_indices([cgi2])
1422
1476
        # Trigger a reordering in cgi1.  cgi2 will be reordered as well.
1423
 
        list(cgi1.iter_entries([('index-1-2-key-1',)]))
 
1477
        list(cgi1.iter_entries([(b'index-1-2-key-1',)]))
1424
1478
        self.assertEqual([index2_2, index2_1], cgi2._indices)
1425
1479
        self.assertEqual(['two', 'one'], cgi2._index_names)
1426
1480
 
1427
1481
    def test_validate_reloads(self):
1428
 
        index, reload_counter = self.make_combined_index_with_missing()
1429
 
        index.validate()
 
1482
        idx, reload_counter = self.make_combined_index_with_missing()
 
1483
        idx.validate()
1430
1484
        self.assertEqual([1, 1, 0], reload_counter)
1431
1485
 
1432
1486
    def test_validate_reloads_midway(self):
1433
 
        index, reload_counter = self.make_combined_index_with_missing(['2'])
1434
 
        index.validate()
 
1487
        idx, reload_counter = self.make_combined_index_with_missing(['2'])
 
1488
        idx.validate()
1435
1489
 
1436
1490
    def test_validate_no_reload(self):
1437
 
        index, reload_counter = self.make_combined_index_with_missing()
1438
 
        index._reload_func = None
1439
 
        self.assertRaises(errors.NoSuchFile, index.validate)
 
1491
        idx, reload_counter = self.make_combined_index_with_missing()
 
1492
        idx._reload_func = None
 
1493
        self.assertRaises(errors.NoSuchFile, idx.validate)
1440
1494
 
1441
1495
    def test_validate_reloads_and_fails(self):
1442
 
        index, reload_counter = self.make_combined_index_with_missing(
1443
 
                                    ['1', '2', '3'])
1444
 
        self.assertRaises(errors.NoSuchFile, index.validate)
 
1496
        idx, reload_counter = self.make_combined_index_with_missing(
 
1497
            ['1', '2', '3'])
 
1498
        self.assertRaises(errors.NoSuchFile, idx.validate)
1445
1499
 
1446
1500
    def test_find_ancestors_across_indexes(self):
1447
 
        key1 = ('key-1',)
1448
 
        key2 = ('key-2',)
1449
 
        key3 = ('key-3',)
1450
 
        key4 = ('key-4',)
 
1501
        key1 = (b'key-1',)
 
1502
        key2 = (b'key-2',)
 
1503
        key3 = (b'key-3',)
 
1504
        key4 = (b'key-4',)
1451
1505
        index1 = self.make_index('12', ref_lists=1, nodes=[
1452
 
            (key1, 'value', ([],)),
1453
 
            (key2, 'value', ([key1],)),
 
1506
            (key1, b'value', ([],)),
 
1507
            (key2, b'value', ([key1],)),
1454
1508
            ])
1455
1509
        index2 = self.make_index('34', ref_lists=1, nodes=[
1456
 
            (key3, 'value', ([key2],)),
1457
 
            (key4, 'value', ([key3],)),
 
1510
            (key3, b'value', ([key2],)),
 
1511
            (key4, b'value', ([key3],)),
1458
1512
            ])
1459
 
        c_index = CombinedGraphIndex([index1, index2])
 
1513
        c_index = _mod_index.CombinedGraphIndex([index1, index2])
1460
1514
        parent_map, missing_keys = c_index.find_ancestry([key1], 0)
1461
1515
        self.assertEqual({key1: ()}, parent_map)
1462
1516
        self.assertEqual(set(), missing_keys)
1468
1522
        self.assertEqual(set(), missing_keys)
1469
1523
 
1470
1524
    def test_find_ancestors_missing_keys(self):
1471
 
        key1 = ('key-1',)
1472
 
        key2 = ('key-2',)
1473
 
        key3 = ('key-3',)
1474
 
        key4 = ('key-4',)
 
1525
        key1 = (b'key-1',)
 
1526
        key2 = (b'key-2',)
 
1527
        key3 = (b'key-3',)
 
1528
        key4 = (b'key-4',)
1475
1529
        index1 = self.make_index('12', ref_lists=1, nodes=[
1476
 
            (key1, 'value', ([],)),
1477
 
            (key2, 'value', ([key1],)),
 
1530
            (key1, b'value', ([],)),
 
1531
            (key2, b'value', ([key1],)),
1478
1532
            ])
1479
1533
        index2 = self.make_index('34', ref_lists=1, nodes=[
1480
 
            (key3, 'value', ([key2],)),
 
1534
            (key3, b'value', ([key2],)),
1481
1535
            ])
1482
 
        c_index = CombinedGraphIndex([index1, index2])
 
1536
        c_index = _mod_index.CombinedGraphIndex([index1, index2])
1483
1537
        # Searching for a key which is actually not present at all should
1484
1538
        # eventually converge
1485
1539
        parent_map, missing_keys = c_index.find_ancestry([key4], 0)
1486
1540
        self.assertEqual({}, parent_map)
1487
 
        self.assertEqual(set([key4]), missing_keys)
 
1541
        self.assertEqual({key4}, missing_keys)
1488
1542
 
1489
1543
    def test_find_ancestors_no_indexes(self):
1490
 
        c_index = CombinedGraphIndex([])
1491
 
        key1 = ('key-1',)
 
1544
        c_index = _mod_index.CombinedGraphIndex([])
 
1545
        key1 = (b'key-1',)
1492
1546
        parent_map, missing_keys = c_index.find_ancestry([key1], 0)
1493
1547
        self.assertEqual({}, parent_map)
1494
 
        self.assertEqual(set([key1]), missing_keys)
 
1548
        self.assertEqual({key1}, missing_keys)
1495
1549
 
1496
1550
    def test_find_ancestors_ghost_parent(self):
1497
 
        key1 = ('key-1',)
1498
 
        key2 = ('key-2',)
1499
 
        key3 = ('key-3',)
1500
 
        key4 = ('key-4',)
 
1551
        key1 = (b'key-1',)
 
1552
        key2 = (b'key-2',)
 
1553
        key3 = (b'key-3',)
 
1554
        key4 = (b'key-4',)
1501
1555
        index1 = self.make_index('12', ref_lists=1, nodes=[
1502
 
            (key1, 'value', ([],)),
1503
 
            (key2, 'value', ([key1],)),
 
1556
            (key1, b'value', ([],)),
 
1557
            (key2, b'value', ([key1],)),
1504
1558
            ])
1505
1559
        index2 = self.make_index('34', ref_lists=1, nodes=[
1506
 
            (key4, 'value', ([key2, key3],)),
 
1560
            (key4, b'value', ([key2, key3],)),
1507
1561
            ])
1508
 
        c_index = CombinedGraphIndex([index1, index2])
 
1562
        c_index = _mod_index.CombinedGraphIndex([index1, index2])
1509
1563
        # Searching for a key which is actually not present at all should
1510
1564
        # eventually converge
1511
1565
        parent_map, missing_keys = c_index.find_ancestry([key4], 0)
1512
1566
        self.assertEqual({key4: (key2, key3), key2: (key1,), key1: ()},
1513
1567
                         parent_map)
1514
 
        self.assertEqual(set([key3]), missing_keys)
 
1568
        self.assertEqual({key3}, missing_keys)
1515
1569
 
1516
1570
    def test__find_ancestors_empty_index(self):
1517
 
        index = self.make_index('test', ref_lists=1, key_elements=1, nodes=[])
 
1571
        idx = self.make_index('test', ref_lists=1, key_elements=1, nodes=[])
1518
1572
        parent_map = {}
1519
1573
        missing_keys = set()
1520
 
        search_keys = index._find_ancestors([('one',), ('two',)], 0, parent_map,
1521
 
                                            missing_keys)
 
1574
        search_keys = idx._find_ancestors([(b'one',), (b'two',)], 0, parent_map,
 
1575
                                          missing_keys)
1522
1576
        self.assertEqual(set(), search_keys)
1523
1577
        self.assertEqual({}, parent_map)
1524
 
        self.assertEqual(set([('one',), ('two',)]), missing_keys)
1525
 
 
1526
 
 
1527
 
class TestInMemoryGraphIndex(TestCaseWithMemoryTransport):
 
1578
        self.assertEqual({(b'one',), (b'two',)}, missing_keys)
 
1579
 
 
1580
 
 
1581
class TestInMemoryGraphIndex(tests.TestCaseWithMemoryTransport):
1528
1582
 
1529
1583
    def make_index(self, ref_lists=0, key_elements=1, nodes=[]):
1530
 
        result = InMemoryGraphIndex(ref_lists, key_elements=key_elements)
 
1584
        result = _mod_index.InMemoryGraphIndex(
 
1585
            ref_lists, key_elements=key_elements)
1531
1586
        result.add_nodes(nodes)
1532
1587
        return result
1533
1588
 
1534
1589
    def test_add_nodes_no_refs(self):
1535
1590
        index = self.make_index(0)
1536
 
        index.add_nodes([(('name', ), 'data')])
1537
 
        index.add_nodes([(('name2', ), ''), (('name3', ), '')])
1538
 
        self.assertEqual(set([
1539
 
            (index, ('name', ), 'data'),
1540
 
            (index, ('name2', ), ''),
1541
 
            (index, ('name3', ), ''),
1542
 
            ]), set(index.iter_all_entries()))
 
1591
        index.add_nodes([((b'name', ), b'data')])
 
1592
        index.add_nodes([((b'name2', ), b''), ((b'name3', ), b'')])
 
1593
        self.assertEqual({
 
1594
            (index, (b'name', ), b'data'),
 
1595
            (index, (b'name2', ), b''),
 
1596
            (index, (b'name3', ), b''),
 
1597
            }, set(index.iter_all_entries()))
1543
1598
 
1544
1599
    def test_add_nodes(self):
1545
1600
        index = self.make_index(1)
1546
 
        index.add_nodes([(('name', ), 'data', ([],))])
1547
 
        index.add_nodes([(('name2', ), '', ([],)), (('name3', ), '', ([('r', )],))])
1548
 
        self.assertEqual(set([
1549
 
            (index, ('name', ), 'data', ((),)),
1550
 
            (index, ('name2', ), '', ((),)),
1551
 
            (index, ('name3', ), '', ((('r', ), ), )),
1552
 
            ]), set(index.iter_all_entries()))
 
1601
        index.add_nodes([((b'name', ), b'data', ([],))])
 
1602
        index.add_nodes([((b'name2', ), b'', ([],)),
 
1603
                         ((b'name3', ), b'', ([(b'r', )],))])
 
1604
        self.assertEqual({
 
1605
            (index, (b'name', ), b'data', ((),)),
 
1606
            (index, (b'name2', ), b'', ((),)),
 
1607
            (index, (b'name3', ), b'', (((b'r', ), ), )),
 
1608
            }, set(index.iter_all_entries()))
1553
1609
 
1554
1610
    def test_iter_all_entries_empty(self):
1555
1611
        index = self.make_index()
1556
1612
        self.assertEqual([], list(index.iter_all_entries()))
1557
1613
 
1558
1614
    def test_iter_all_entries_simple(self):
1559
 
        index = self.make_index(nodes=[(('name', ), 'data')])
1560
 
        self.assertEqual([(index, ('name', ), 'data')],
1561
 
            list(index.iter_all_entries()))
 
1615
        index = self.make_index(nodes=[((b'name', ), b'data')])
 
1616
        self.assertEqual([(index, (b'name', ), b'data')],
 
1617
                         list(index.iter_all_entries()))
1562
1618
 
1563
1619
    def test_iter_all_entries_references(self):
1564
1620
        index = self.make_index(1, nodes=[
1565
 
            (('name', ), 'data', ([('ref', )], )),
1566
 
            (('ref', ), 'refdata', ([], ))])
1567
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref', ),),)),
1568
 
            (index, ('ref', ), 'refdata', ((), ))]),
1569
 
            set(index.iter_all_entries()))
 
1621
            ((b'name', ), b'data', ([(b'ref', )], )),
 
1622
            ((b'ref', ), b'refdata', ([], ))])
 
1623
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref', ),),)),
 
1624
                          (index, (b'ref', ), b'refdata', ((), ))},
 
1625
                         set(index.iter_all_entries()))
1570
1626
 
1571
1627
    def test_iteration_absent_skipped(self):
1572
1628
        index = self.make_index(1, nodes=[
1573
 
            (('name', ), 'data', ([('ref', )], ))])
1574
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),))]),
1575
 
            set(index.iter_all_entries()))
1576
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),))]),
1577
 
            set(index.iter_entries([('name', )])))
1578
 
        self.assertEqual([], list(index.iter_entries([('ref', )])))
 
1629
            ((b'name', ), b'data', ([(b'ref', )], ))])
 
1630
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),))},
 
1631
                         set(index.iter_all_entries()))
 
1632
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),))},
 
1633
                         set(index.iter_entries([(b'name', )])))
 
1634
        self.assertEqual([], list(index.iter_entries([(b'ref', )])))
1579
1635
 
1580
1636
    def test_iter_all_keys(self):
1581
1637
        index = self.make_index(1, nodes=[
1582
 
            (('name', ), 'data', ([('ref', )], )),
1583
 
            (('ref', ), 'refdata', ([], ))])
1584
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),)),
1585
 
            (index, ('ref', ), 'refdata', ((), ))]),
1586
 
            set(index.iter_entries([('name', ), ('ref', )])))
 
1638
            ((b'name', ), b'data', ([(b'ref', )], )),
 
1639
            ((b'ref', ), b'refdata', ([], ))])
 
1640
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),)),
 
1641
                          (index, (b'ref', ), b'refdata', ((), ))},
 
1642
                         set(index.iter_entries([(b'name', ), (b'ref', )])))
1587
1643
 
1588
1644
    def test_iter_key_prefix_1_key_element_no_refs(self):
1589
 
        index = self.make_index( nodes=[
1590
 
            (('name', ), 'data'),
1591
 
            (('ref', ), 'refdata')])
1592
 
        self.assertEqual(set([(index, ('name', ), 'data'),
1593
 
            (index, ('ref', ), 'refdata')]),
1594
 
            set(index.iter_entries_prefix([('name', ), ('ref', )])))
 
1645
        index = self.make_index(nodes=[
 
1646
            ((b'name', ), b'data'),
 
1647
            ((b'ref', ), b'refdata')])
 
1648
        self.assertEqual({(index, (b'name', ), b'data'),
 
1649
                          (index, (b'ref', ), b'refdata')},
 
1650
                         set(index.iter_entries_prefix([(b'name', ), (b'ref', )])))
1595
1651
 
1596
1652
    def test_iter_key_prefix_1_key_element_refs(self):
1597
1653
        index = self.make_index(1, nodes=[
1598
 
            (('name', ), 'data', ([('ref', )], )),
1599
 
            (('ref', ), 'refdata', ([], ))])
1600
 
        self.assertEqual(set([(index, ('name', ), 'data', ((('ref',),),)),
1601
 
            (index, ('ref', ), 'refdata', ((), ))]),
1602
 
            set(index.iter_entries_prefix([('name', ), ('ref', )])))
 
1654
            ((b'name', ), b'data', ([(b'ref', )], )),
 
1655
            ((b'ref', ), b'refdata', ([], ))])
 
1656
        self.assertEqual({(index, (b'name', ), b'data', (((b'ref',),),)),
 
1657
                          (index, (b'ref', ), b'refdata', ((), ))},
 
1658
                         set(index.iter_entries_prefix([(b'name', ), (b'ref', )])))
1603
1659
 
1604
1660
    def test_iter_key_prefix_2_key_element_no_refs(self):
1605
1661
        index = self.make_index(key_elements=2, nodes=[
1606
 
            (('name', 'fin1'), 'data'),
1607
 
            (('name', 'fin2'), 'beta'),
1608
 
            (('ref', 'erence'), 'refdata')])
1609
 
        self.assertEqual(set([(index, ('name', 'fin1'), 'data'),
1610
 
            (index, ('ref', 'erence'), 'refdata')]),
1611
 
            set(index.iter_entries_prefix([('name', 'fin1'), ('ref', 'erence')])))
1612
 
        self.assertEqual(set([(index, ('name', 'fin1'), 'data'),
1613
 
            (index, ('name', 'fin2'), 'beta')]),
1614
 
            set(index.iter_entries_prefix([('name', None)])))
 
1662
            ((b'name', b'fin1'), b'data'),
 
1663
            ((b'name', b'fin2'), b'beta'),
 
1664
            ((b'ref', b'erence'), b'refdata')])
 
1665
        self.assertEqual({(index, (b'name', b'fin1'), b'data'),
 
1666
                          (index, (b'ref', b'erence'), b'refdata')},
 
1667
                         set(index.iter_entries_prefix([(b'name', b'fin1'), (b'ref', b'erence')])))
 
1668
        self.assertEqual({(index, (b'name', b'fin1'), b'data'),
 
1669
                          (index, (b'name', b'fin2'), b'beta')},
 
1670
                         set(index.iter_entries_prefix([(b'name', None)])))
1615
1671
 
1616
1672
    def test_iter_key_prefix_2_key_element_refs(self):
1617
1673
        index = self.make_index(1, key_elements=2, nodes=[
1618
 
            (('name', 'fin1'), 'data', ([('ref', 'erence')], )),
1619
 
            (('name', 'fin2'), 'beta', ([], )),
1620
 
            (('ref', 'erence'), 'refdata', ([], ))])
1621
 
        self.assertEqual(set([(index, ('name', 'fin1'), 'data', ((('ref', 'erence'),),)),
1622
 
            (index, ('ref', 'erence'), 'refdata', ((), ))]),
1623
 
            set(index.iter_entries_prefix([('name', 'fin1'), ('ref', 'erence')])))
1624
 
        self.assertEqual(set([(index, ('name', 'fin1'), 'data', ((('ref', 'erence'),),)),
1625
 
            (index, ('name', 'fin2'), 'beta', ((), ))]),
1626
 
            set(index.iter_entries_prefix([('name', None)])))
 
1674
            ((b'name', b'fin1'), b'data', ([(b'ref', b'erence')], )),
 
1675
            ((b'name', b'fin2'), b'beta', ([], )),
 
1676
            ((b'ref', b'erence'), b'refdata', ([], ))])
 
1677
        self.assertEqual({(index, (b'name', b'fin1'), b'data', (((b'ref', b'erence'),),)),
 
1678
                          (index, (b'ref', b'erence'), b'refdata', ((), ))},
 
1679
                         set(index.iter_entries_prefix([(b'name', b'fin1'), (b'ref', b'erence')])))
 
1680
        self.assertEqual({(index, (b'name', b'fin1'), b'data', (((b'ref', b'erence'),),)),
 
1681
                          (index, (b'name', b'fin2'), b'beta', ((), ))},
 
1682
                         set(index.iter_entries_prefix([(b'name', None)])))
1627
1683
 
1628
1684
    def test_iter_nothing_empty(self):
1629
1685
        index = self.make_index()
1631
1687
 
1632
1688
    def test_iter_missing_entry_empty(self):
1633
1689
        index = self.make_index()
1634
 
        self.assertEqual([], list(index.iter_entries(['a'])))
 
1690
        self.assertEqual([], list(index.iter_entries([b'a'])))
1635
1691
 
1636
1692
    def test_key_count_empty(self):
1637
1693
        index = self.make_index()
1638
1694
        self.assertEqual(0, index.key_count())
1639
1695
 
1640
1696
    def test_key_count_one(self):
1641
 
        index = self.make_index(nodes=[(('name', ), '')])
 
1697
        index = self.make_index(nodes=[((b'name', ), b'')])
1642
1698
        self.assertEqual(1, index.key_count())
1643
1699
 
1644
1700
    def test_key_count_two(self):
1645
 
        index = self.make_index(nodes=[(('name', ), ''), (('foo', ), '')])
 
1701
        index = self.make_index(nodes=[((b'name', ), b''), ((b'foo', ), b'')])
1646
1702
        self.assertEqual(2, index.key_count())
1647
1703
 
1648
1704
    def test_validate_empty(self):
1650
1706
        index.validate()
1651
1707
 
1652
1708
    def test_validate_no_refs_content(self):
1653
 
        index = self.make_index(nodes=[(('key', ), 'value')])
 
1709
        index = self.make_index(nodes=[((b'key', ), b'value')])
1654
1710
        index.validate()
1655
1711
 
1656
1712
 
1657
 
class TestGraphIndexPrefixAdapter(TestCaseWithMemoryTransport):
 
1713
class TestGraphIndexPrefixAdapter(tests.TestCaseWithMemoryTransport):
1658
1714
 
1659
 
    def make_index(self, ref_lists=1, key_elements=2, nodes=[], add_callback=False):
1660
 
        result = InMemoryGraphIndex(ref_lists, key_elements=key_elements)
 
1715
    def make_index(self, ref_lists=1, key_elements=2, nodes=[],
 
1716
                   add_callback=False):
 
1717
        result = _mod_index.InMemoryGraphIndex(
 
1718
            ref_lists, key_elements=key_elements)
1661
1719
        result.add_nodes(nodes)
1662
1720
        if add_callback:
1663
1721
            add_nodes_callback = result.add_nodes
1664
1722
        else:
1665
1723
            add_nodes_callback = None
1666
 
        adapter = GraphIndexPrefixAdapter(result, ('prefix', ), key_elements - 1,
 
1724
        adapter = _mod_index.GraphIndexPrefixAdapter(
 
1725
            result, (b'prefix', ), key_elements - 1,
1667
1726
            add_nodes_callback=add_nodes_callback)
1668
1727
        return result, adapter
1669
1728
 
1670
1729
    def test_add_node(self):
1671
1730
        index, adapter = self.make_index(add_callback=True)
1672
 
        adapter.add_node(('key',), 'value', ((('ref',),),))
1673
 
        self.assertEqual(set([(index, ('prefix', 'key'), 'value', ((('prefix', 'ref'),),))]),
1674
 
            set(index.iter_all_entries()))
 
1731
        adapter.add_node((b'key',), b'value', (((b'ref',),),))
 
1732
        self.assertEqual({(index, (b'prefix', b'key'), b'value',
 
1733
                           (((b'prefix', b'ref'),),))},
 
1734
                         set(index.iter_all_entries()))
1675
1735
 
1676
1736
    def test_add_nodes(self):
1677
1737
        index, adapter = self.make_index(add_callback=True)
1678
1738
        adapter.add_nodes((
1679
 
            (('key',), 'value', ((('ref',),),)),
1680
 
            (('key2',), 'value2', ((),)),
 
1739
            ((b'key',), b'value', (((b'ref',),),)),
 
1740
            ((b'key2',), b'value2', ((),)),
1681
1741
            ))
1682
 
        self.assertEqual(set([
1683
 
            (index, ('prefix', 'key2'), 'value2', ((),)),
1684
 
            (index, ('prefix', 'key'), 'value', ((('prefix', 'ref'),),))
1685
 
            ]),
 
1742
        self.assertEqual({
 
1743
            (index, (b'prefix', b'key2'), b'value2', ((),)),
 
1744
            (index, (b'prefix', b'key'), b'value', (((b'prefix', b'ref'),),))
 
1745
            },
1686
1746
            set(index.iter_all_entries()))
1687
1747
 
1688
1748
    def test_construct(self):
1689
 
        index = InMemoryGraphIndex()
1690
 
        adapter = GraphIndexPrefixAdapter(index, ('prefix', ), 1)
 
1749
        idx = _mod_index.InMemoryGraphIndex()
 
1750
        adapter = _mod_index.GraphIndexPrefixAdapter(idx, (b'prefix', ), 1)
1691
1751
 
1692
1752
    def test_construct_with_callback(self):
1693
 
        index = InMemoryGraphIndex()
1694
 
        adapter = GraphIndexPrefixAdapter(index, ('prefix', ), 1, index.add_nodes)
 
1753
        idx = _mod_index.InMemoryGraphIndex()
 
1754
        adapter = _mod_index.GraphIndexPrefixAdapter(idx, (b'prefix', ), 1,
 
1755
                                                     idx.add_nodes)
1695
1756
 
1696
1757
    def test_iter_all_entries_cross_prefix_map_errors(self):
1697
1758
        index, adapter = self.make_index(nodes=[
1698
 
            (('prefix', 'key1'), 'data1', ((('prefixaltered', 'key2'),),))])
1699
 
        self.assertRaises(errors.BadIndexData, list, adapter.iter_all_entries())
 
1759
            ((b'prefix', b'key1'), b'data1', (((b'prefixaltered', b'key2'),),))])
 
1760
        self.assertRaises(_mod_index.BadIndexData, list,
 
1761
                          adapter.iter_all_entries())
1700
1762
 
1701
1763
    def test_iter_all_entries(self):
1702
1764
        index, adapter = self.make_index(nodes=[
1703
 
            (('notprefix', 'key1'), 'data', ((), )),
1704
 
            (('prefix', 'key1'), 'data1', ((), )),
1705
 
            (('prefix', 'key2'), 'data2', ((('prefix', 'key1'),),))])
1706
 
        self.assertEqual(set([(index, ('key1', ), 'data1', ((),)),
1707
 
            (index, ('key2', ), 'data2', ((('key1',),),))]),
1708
 
            set(adapter.iter_all_entries()))
 
1765
            ((b'notprefix', b'key1'), b'data', ((), )),
 
1766
            ((b'prefix', b'key1'), b'data1', ((), )),
 
1767
            ((b'prefix', b'key2'), b'data2', (((b'prefix', b'key1'),),))])
 
1768
        self.assertEqual({(index, (b'key1', ), b'data1', ((),)),
 
1769
                          (index, (b'key2', ), b'data2', (((b'key1',),),))},
 
1770
                         set(adapter.iter_all_entries()))
1709
1771
 
1710
1772
    def test_iter_entries(self):
1711
1773
        index, adapter = self.make_index(nodes=[
1712
 
            (('notprefix', 'key1'), 'data', ((), )),
1713
 
            (('prefix', 'key1'), 'data1', ((), )),
1714
 
            (('prefix', 'key2'), 'data2', ((('prefix', 'key1'),),))])
 
1774
            ((b'notprefix', b'key1'), b'data', ((), )),
 
1775
            ((b'prefix', b'key1'), b'data1', ((), )),
 
1776
            ((b'prefix', b'key2'), b'data2', (((b'prefix', b'key1'),),))])
1715
1777
        # ask for many - get all
1716
 
        self.assertEqual(set([(index, ('key1', ), 'data1', ((),)),
1717
 
            (index, ('key2', ), 'data2', ((('key1', ),),))]),
1718
 
            set(adapter.iter_entries([('key1', ), ('key2', )])))
 
1778
        self.assertEqual({(index, (b'key1', ), b'data1', ((),)),
 
1779
                          (index, (b'key2', ), b'data2', (((b'key1', ),),))},
 
1780
                         set(adapter.iter_entries([(b'key1', ), (b'key2', )])))
1719
1781
        # ask for one, get one
1720
 
        self.assertEqual(set([(index, ('key1', ), 'data1', ((),))]),
1721
 
            set(adapter.iter_entries([('key1', )])))
 
1782
        self.assertEqual({(index, (b'key1', ), b'data1', ((),))},
 
1783
                         set(adapter.iter_entries([(b'key1', )])))
1722
1784
        # ask for missing, get none
1723
1785
        self.assertEqual(set(),
1724
 
            set(adapter.iter_entries([('key3', )])))
 
1786
                         set(adapter.iter_entries([(b'key3', )])))
1725
1787
 
1726
1788
    def test_iter_entries_prefix(self):
1727
1789
        index, adapter = self.make_index(key_elements=3, nodes=[
1728
 
            (('notprefix', 'foo', 'key1'), 'data', ((), )),
1729
 
            (('prefix', 'prefix2', 'key1'), 'data1', ((), )),
1730
 
            (('prefix', 'prefix2', 'key2'), 'data2', ((('prefix', 'prefix2', 'key1'),),))])
 
1790
            ((b'notprefix', b'foo', b'key1'), b'data', ((), )),
 
1791
            ((b'prefix', b'prefix2', b'key1'), b'data1', ((), )),
 
1792
            ((b'prefix', b'prefix2', b'key2'), b'data2', (((b'prefix', b'prefix2', b'key1'),),))])
1731
1793
        # ask for a prefix, get the results for just that prefix, adjusted.
1732
 
        self.assertEqual(set([(index, ('prefix2', 'key1', ), 'data1', ((),)),
1733
 
            (index, ('prefix2', 'key2', ), 'data2', ((('prefix2', 'key1', ),),))]),
1734
 
            set(adapter.iter_entries_prefix([('prefix2', None)])))
 
1794
        self.assertEqual({(index, (b'prefix2', b'key1', ), b'data1', ((),)),
 
1795
                          (index, (b'prefix2', b'key2', ), b'data2', (((b'prefix2', b'key1', ),),))},
 
1796
                         set(adapter.iter_entries_prefix([(b'prefix2', None)])))
1735
1797
 
1736
1798
    def test_key_count_no_matching_keys(self):
1737
1799
        index, adapter = self.make_index(nodes=[
1738
 
            (('notprefix', 'key1'), 'data', ((), ))])
 
1800
            ((b'notprefix', b'key1'), b'data', ((), ))])
1739
1801
        self.assertEqual(0, adapter.key_count())
1740
1802
 
1741
1803
    def test_key_count_some_keys(self):
1742
1804
        index, adapter = self.make_index(nodes=[
1743
 
            (('notprefix', 'key1'), 'data', ((), )),
1744
 
            (('prefix', 'key1'), 'data1', ((), )),
1745
 
            (('prefix', 'key2'), 'data2', ((('prefix', 'key1'),),))])
 
1805
            ((b'notprefix', b'key1'), b'data', ((), )),
 
1806
            ((b'prefix', b'key1'), b'data1', ((), )),
 
1807
            ((b'prefix', b'key2'), b'data2', (((b'prefix', b'key1'),),))])
1746
1808
        self.assertEqual(2, adapter.key_count())
1747
1809
 
1748
1810
    def test_validate(self):
1749
1811
        index, adapter = self.make_index()
1750
1812
        calls = []
 
1813
 
1751
1814
        def validate():
1752
1815
            calls.append('called')
1753
1816
        index.validate = validate