~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_generate_ids.py

  • Committer: Eric Holmberg
  • Date: 2008-05-06 15:02:27 UTC
  • mto: This revision was merged to the branch mainline in revision 3449.
  • Revision ID: eholmberg@arrow.com-20080506150227-l3arq1yntdvnoxum
Fix for Bug #215426 in which bzr can cause a MemoryError in socket.recv while
downloading large packs over http.  This patch limits the request size for
socket.recv to avoid this problem.

Changes:
Added mock file object bzrlib.tests.file_utils.
Added new parameters to bzrlib.osutils.pumpfile.
Added unit tests for bzrlib.osutils.pumpfile.
Added usage of bzrlib.osutils.pumpfile to bzrlib.transport.http.response.

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2006 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
"""Tests for bzrlib/generate_ids.py"""
 
18
 
 
19
import re
 
20
 
 
21
from bzrlib import (
 
22
    generate_ids,
 
23
    tests,
 
24
    )
 
25
 
 
26
 
 
27
class TestFileIds(tests.TestCase):
 
28
    """Test functions which generate file ids"""
 
29
 
 
30
    def assertGenFileId(self, regex, filename):
 
31
        """gen_file_id should create a file id matching the regex.
 
32
 
 
33
        The file id should be ascii, and should be an 8-bit string
 
34
        """
 
35
        file_id = generate_ids.gen_file_id(filename)
 
36
        self.assertContainsRe(file_id, '^'+regex+'$')
 
37
        # It should be a utf8 file_id, not a unicode one
 
38
        self.assertIsInstance(file_id, str)
 
39
        # gen_file_id should always return ascii file ids.
 
40
        file_id.decode('ascii')
 
41
 
 
42
    def test_gen_file_id(self):
 
43
        gen_file_id = generate_ids.gen_file_id
 
44
 
 
45
        # We try to use the filename if possible
 
46
        self.assertStartsWith(gen_file_id('bar'), 'bar-')
 
47
 
 
48
        # but we squash capitalization, and remove non word characters
 
49
        self.assertStartsWith(gen_file_id('Mwoo oof\t m'), 'mwoooofm-')
 
50
 
 
51
        # We also remove leading '.' characters to prevent hidden file-ids
 
52
        self.assertStartsWith(gen_file_id('..gam.py'), 'gam.py-')
 
53
        self.assertStartsWith(gen_file_id('..Mwoo oof\t m'), 'mwoooofm-')
 
54
 
 
55
        # we remove unicode characters, and still don't end up with a 
 
56
        # hidden file id
 
57
        self.assertStartsWith(gen_file_id(u'\xe5\xb5.txt'), 'txt-')
 
58
 
 
59
        # Our current method of generating unique ids adds 33 characters
 
60
        # plus an serial number (log10(N) characters)
 
61
        # to the end of the filename. We now restrict the filename portion to
 
62
        # be <= 20 characters, so the maximum length should now be approx < 60
 
63
 
 
64
        # Test both case squashing and length restriction
 
65
        fid = gen_file_id('A'*50 + '.txt')
 
66
        self.assertStartsWith(fid, 'a'*20 + '-')
 
67
        self.failUnless(len(fid) < 60)
 
68
 
 
69
        # restricting length happens after the other actions, so
 
70
        # we preserve as much as possible
 
71
        fid = gen_file_id('\xe5\xb5..aBcd\tefGhijKLMnop\tqrstuvwxyz')
 
72
        self.assertStartsWith(fid, 'abcdefghijklmnopqrst-')
 
73
        self.failUnless(len(fid) < 60)
 
74
 
 
75
    def test_file_ids_are_ascii(self):
 
76
        tail = r'-\d{14}-[a-z0-9]{16}-\d+'
 
77
        self.assertGenFileId('foo' + tail, 'foo')
 
78
        self.assertGenFileId('foo' + tail, u'foo')
 
79
        self.assertGenFileId('bar' + tail, u'bar')
 
80
        self.assertGenFileId('br' + tail, u'b\xe5r')
 
81
 
 
82
    def test__next_id_suffix_sets_suffix(self):
 
83
        generate_ids._gen_file_id_suffix = None
 
84
        generate_ids._next_id_suffix()
 
85
        self.assertNotEqual(None, generate_ids._gen_file_id_suffix)
 
86
 
 
87
    def test__next_id_suffix_increments(self):
 
88
        generate_ids._gen_file_id_suffix = "foo-"
 
89
        generate_ids._gen_file_id_serial = 1
 
90
        try:
 
91
            self.assertEqual("foo-2", generate_ids._next_id_suffix())
 
92
            self.assertEqual("foo-3", generate_ids._next_id_suffix())
 
93
            self.assertEqual("foo-4", generate_ids._next_id_suffix())
 
94
            self.assertEqual("foo-5", generate_ids._next_id_suffix())
 
95
            self.assertEqual("foo-6", generate_ids._next_id_suffix())
 
96
            self.assertEqual("foo-7", generate_ids._next_id_suffix())
 
97
            self.assertEqual("foo-8", generate_ids._next_id_suffix())
 
98
            self.assertEqual("foo-9", generate_ids._next_id_suffix())
 
99
            self.assertEqual("foo-10", generate_ids._next_id_suffix())
 
100
        finally:
 
101
            # Reset so that all future ids generated in the test suite
 
102
            # don't end in 'foo-XXX'
 
103
            generate_ids._gen_file_id_suffix = None
 
104
            generate_ids._gen_file_id_serial = 0
 
105
 
 
106
    def test_gen_root_id(self):
 
107
        # Mostly just make sure gen_root_id() exists
 
108
        root_id = generate_ids.gen_root_id()
 
109
        self.assertStartsWith(root_id, 'tree_root-')
 
110
 
 
111
 
 
112
class TestGenRevisionId(tests.TestCase):
 
113
    """Test generating revision ids"""
 
114
 
 
115
    def assertGenRevisionId(self, regex, username, timestamp=None):
 
116
        """gen_revision_id should create a revision id matching the regex"""
 
117
        revision_id = generate_ids.gen_revision_id(username, timestamp)
 
118
        self.assertContainsRe(revision_id, '^'+regex+'$')
 
119
        # It should be a utf8 revision_id, not a unicode one
 
120
        self.assertIsInstance(revision_id, str)
 
121
        # gen_revision_id should always return ascii revision ids.
 
122
        revision_id.decode('ascii')
 
123
 
 
124
    def test_timestamp(self):
 
125
        """passing a timestamp should cause it to be used"""
 
126
        self.assertGenRevisionId(r'user@host-\d{14}-[a-z0-9]{16}', 'user@host')
 
127
        self.assertGenRevisionId('user@host-20061102205056-[a-z0-9]{16}',
 
128
                                 'user@host', 1162500656.688)
 
129
        self.assertGenRevisionId(r'user@host-20061102205024-[a-z0-9]{16}',
 
130
                                 'user@host', 1162500624.000)
 
131
 
 
132
    def test_gen_revision_id_email(self):
 
133
        """gen_revision_id uses email address if present"""
 
134
        regex = r'user\+joe_bar@foo-bar\.com-\d{14}-[a-z0-9]{16}'
 
135
        self.assertGenRevisionId(regex,'user+joe_bar@foo-bar.com')
 
136
        self.assertGenRevisionId(regex, '<user+joe_bar@foo-bar.com>')
 
137
        self.assertGenRevisionId(regex, 'Joe Bar <user+joe_bar@foo-bar.com>')
 
138
        self.assertGenRevisionId(regex, 'Joe Bar <user+Joe_Bar@Foo-Bar.com>')
 
139
        self.assertGenRevisionId(regex, u'Joe B\xe5r <user+Joe_Bar@Foo-Bar.com>')
 
140
 
 
141
    def test_gen_revision_id_user(self):
 
142
        """If there is no email, fall back to the whole username"""
 
143
        tail = r'-\d{14}-[a-z0-9]{16}'
 
144
        self.assertGenRevisionId('joe_bar' + tail, 'Joe Bar')
 
145
        self.assertGenRevisionId('joebar' + tail, 'joebar')
 
146
        self.assertGenRevisionId('joe_br' + tail, u'Joe B\xe5r')
 
147
        self.assertGenRevisionId(r'joe_br_user\+joe_bar_foo-bar.com' + tail,
 
148
                                 u'Joe B\xe5r <user+Joe_Bar_Foo-Bar.com>')
 
149
 
 
150
    def test_revision_ids_are_ascii(self):
 
151
        """gen_revision_id should always return an ascii revision id."""
 
152
        tail = r'-\d{14}-[a-z0-9]{16}'
 
153
        self.assertGenRevisionId('joe_bar' + tail, 'Joe Bar')
 
154
        self.assertGenRevisionId('joe_bar' + tail, u'Joe Bar')
 
155
        self.assertGenRevisionId('joe@foo' + tail, u'Joe Bar <joe@foo>')
 
156
        # We cheat a little with this one, because email-addresses shouldn't
 
157
        # contain non-ascii characters, but generate_ids should strip them
 
158
        # anyway.
 
159
        self.assertGenRevisionId('joe@f' + tail, u'Joe Bar <joe@f\xb6>')