← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_progress.py

  • Committer: Vincent Ladeuil
  • Date: 2008-01-29 15:16:31 UTC
  • mto: (3206.1.1 trunk)
  • mto: This revision was merged to the branch mainline in revision 3207.
  • Revision ID: v.ladeuil+lp@free.fr-20080129151631-vqjd13tb405mobx6
Fix two more leaking tmp dirs, by reworking TransformPreview lock handling.

* bzrlib/tests/test_transform.py:
(TestTransformMerge): Revert previous patch and cleanly call
preview.finalize now that we can.

* bzrlib/tests/test_merge.py:
(TestMerge.test_make_preview_transform): Catch TransformPreview
leak.

* bzrlib/builtins.py:
(cmd_merge._do_preview): Finalize the TransformPreview or the
limbodir will stay in /tmp.

* bzrlib/transform.py:
(TreeTransformBase.__init__): Create the _deletiondir since it's
reffered to by finalize.
(TreeTransformBase.finalize): Delete the dir only if _deletiondir
is set.
(TreeTransform.__init__): Use a temp var for deletiondir and set
the attribute after the base class __init__ has been called.
(TransformPreview.__init__): Read locks the tree since finalize
wants to unlock it (as suggested by Aaron).

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_progress.py
1
 
# Copyright (C) 2006 by Canonical Ltd
 
1
# Copyright (C) 2006, 2007 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
 
17
import os
17
18
from StringIO import StringIO
18
19
 
19
 
from bzrlib.progress import *
 
20
from bzrlib import errors
 
21
from bzrlib.progress import (
 
22
        DummyProgress,
 
23
        ChildProgress,
 
24
        TTYProgressBar,
 
25
        DotsProgressBar,
 
26
        ProgressBarStack,
 
27
        InstrumentedProgress,
 
28
        )
20
29
from bzrlib.tests import TestCase
21
30
 
 
31
 
22
32
class FakeStack:
23
33
    def __init__(self, top):
24
34
        self.__top = top
26
36
    def top(self):
27
37
        return self.__top
28
38
 
 
39
 
 
40
class _TTYStringIO(StringIO):
 
41
    """A helper class which makes a StringIO look like a terminal"""
 
42
 
 
43
    def isatty(self):
 
44
        return True
 
45
 
 
46
 
 
47
class _NonTTYStringIO(StringIO):
 
48
    """Helper that implements isatty() but returns False"""
 
49
 
 
50
    def isatty(self):
 
51
        return False
 
52
 
 
53
 
29
54
class TestProgress(TestCase):
30
55
    def setUp(self):
31
56
        q = DummyProgress()
90
115
                child.finished()
91
116
        finally:
92
117
            parent.finished()
 
118
 
 
119
    def test_throttling(self):
 
120
        pb = InstrumentedProgress(to_file=StringIO())
 
121
        # instantaneous updates should be squelched
 
122
        pb.update('me', 1, 1)
 
123
        self.assertTrue(pb.always_throttled)
 
124
        pb = InstrumentedProgress(to_file=StringIO())
 
125
        # It's like an instant sleep(1)!
 
126
        pb.start_time -= 1
 
127
        # Updates after a second should not be squelched
 
128
        pb.update('me', 1, 1)
 
129
        self.assertFalse(pb.always_throttled)
 
130
 
 
131
    def test_clear(self):
 
132
        sio = StringIO()
 
133
        pb = TTYProgressBar(to_file=sio, show_eta=False)
 
134
        pb.width = 20 # Just make it easier to test
 
135
        # This should not output anything
 
136
        pb.clear()
 
137
        # These two should not be displayed because
 
138
        # of throttling
 
139
        pb.update('foo', 1, 3)
 
140
        pb.update('bar', 2, 3)
 
141
        # So pb.clear() has nothing to do
 
142
        pb.clear()
 
143
 
 
144
        # Make sure the next update isn't throttled
 
145
        pb.start_time -= 1
 
146
        pb.update('baz', 3, 3)
 
147
        pb.clear()
 
148
 
 
149
        self.assertEqual('\r[=========] baz 3/3'
 
150
                         '\r                   \r',
 
151
                         sio.getvalue())
 
152
 
 
153
    def test_no_eta(self):
 
154
        # An old version of the progress bar would
 
155
        # store every update if show_eta was false
 
156
        # because the eta routine was where it was
 
157
        # cleaned out
 
158
        pb = InstrumentedProgress(to_file=StringIO(), show_eta=False)
 
159
        # Just make sure this first few are throttled
 
160
        pb.start_time += 5
 
161
 
 
162
        # These messages are throttled, and don't contribute
 
163
        for count in xrange(100):
 
164
            pb.update('x', count, 300)
 
165
        self.assertEqual(0, len(pb.last_updates))
 
166
 
 
167
        # Unthrottle by time
 
168
        pb.start_time -= 10
 
169
 
 
170
        # These happen too fast, so only one gets through
 
171
        for count in xrange(100):
 
172
            pb.update('x', count+100, 200)
 
173
        self.assertEqual(1, len(pb.last_updates))
 
174
 
 
175
        pb.MIN_PAUSE = 0.0
 
176
 
 
177
        # But all of these go through, don't let the
 
178
        # last_update list grow without bound
 
179
        for count in xrange(100):
 
180
            pb.update('x', count+100, 200)
 
181
 
 
182
        self.assertEqual(pb._max_last_updates, len(pb.last_updates))
 
183
 
 
184
 
 
185
class TestProgressTypes(TestCase):
 
186
    """Test that the right ProgressBar gets instantiated at the right time."""
 
187
 
 
188
    def get_nested(self, outf, term, env_progress=None):
 
189
        """Setup so that ProgressBar thinks we are in the supplied terminal."""
 
190
        orig_term = os.environ.get('TERM')
 
191
        orig_progress = os.environ.get('BZR_PROGRESS_BAR')
 
192
        os.environ['TERM'] = term
 
193
        if env_progress is not None:
 
194
            os.environ['BZR_PROGRESS_BAR'] = env_progress
 
195
        elif orig_progress is not None:
 
196
            del os.environ['BZR_PROGRESS_BAR']
 
197
 
 
198
        def reset():
 
199
            if orig_term is None:
 
200
                del os.environ['TERM']
 
201
            else:
 
202
                os.environ['TERM'] = orig_term
 
203
            # We may have never created BZR_PROGRESS_BAR
 
204
            # So we can't just delete like we can 'TERM' (which is always set)
 
205
            if orig_progress is None:
 
206
                if 'BZR_PROGRESS_BAR' in os.environ:
 
207
                    del os.environ['BZR_PROGRESS_BAR']
 
208
            else:
 
209
                os.environ['BZR_PROGRESS_BAR'] = orig_progress
 
210
 
 
211
        self.addCleanup(reset)
 
212
 
 
213
        stack = ProgressBarStack(to_file=outf)
 
214
        pb = stack.get_nested()
 
215
        pb.start_time -= 1 # Make sure it is ready to write
 
216
        pb.width = 20 # And it is of reasonable size
 
217
        return pb
 
218
 
 
219
    def test_tty_progress(self):
 
220
        # Make sure the ProgressBarStack thinks it is
 
221
        # writing out to a terminal, and thus uses a TTYProgressBar
 
222
        out = _TTYStringIO()
 
223
        pb = self.get_nested(out, 'xterm')
 
224
        self.assertIsInstance(pb, TTYProgressBar)
 
225
        try:
 
226
            pb.update('foo', 1, 2)
 
227
            pb.update('bar', 2, 2)
 
228
        finally:
 
229
            pb.finished()
 
230
 
 
231
        self.assertEqual('\r/ [====   ] foo 1/2'
 
232
                         '\r- [=======] bar 2/2'
 
233
                         '\r                   \r',
 
234
                         out.getvalue())
 
235
 
 
236
    def test_noninteractive_progress(self):
 
237
        out = _NonTTYStringIO()
 
238
        pb = self.get_nested(out, 'xterm')
 
239
        self.assertIsInstance(pb, DummyProgress)
 
240
        try:
 
241
            pb.update('foo', 1, 2)
 
242
            pb.update('bar', 2, 2)
 
243
        finally:
 
244
            pb.finished()
 
245
        self.assertEqual('', out.getvalue())
 
246
 
 
247
    def test_dots_progress(self):
 
248
        # make sure we get the right progress bar when not on a terminal
 
249
        out = _NonTTYStringIO()
 
250
        pb = self.get_nested(out, 'xterm', 'dots')
 
251
        self.assertIsInstance(pb, DotsProgressBar)
 
252
        try:
 
253
            pb.update('foo', 1, 2)
 
254
            pb.update('bar', 2, 2)
 
255
        finally:
 
256
            pb.finished()
 
257
        self.assertEqual('foo: .'
 
258
                         '\nbar: .'
 
259
                         '\n',
 
260
                         out.getvalue())
 
261
 
 
262
    def test_no_isatty_progress(self):
 
263
        # Make sure ProgressBarStack handles a plain StringIO()
 
264
        import cStringIO
 
265
        out = cStringIO.StringIO()
 
266
        pb = self.get_nested(out, 'xterm')
 
267
        pb.finished()
 
268
        self.assertIsInstance(pb, DummyProgress)
 
269
 
 
270
    def test_dumb_progress(self):
 
271
        # using a terminal that can't do cursor movement
 
272
        out = _TTYStringIO()
 
273
        pb = self.get_nested(out, 'dumb')
 
274
        pb.finished()
 
275
        self.assertIsInstance(pb, DummyProgress)
 
276
 
 
277
    def test_progress_env_tty(self):
 
278
        # The environ variable BZR_PROGRESS_BAR controls what type of
 
279
        # progress bar we will get, even if it wouldn't usually be that type
 
280
        import cStringIO
 
281
 
 
282
        # Usually, this would be a DotsProgressBar
 
283
        out = cStringIO.StringIO()
 
284
        pb = self.get_nested(out, 'dumb', 'tty')
 
285
        pb.finished()
 
286
        # Even though we are not a tty, the env_var will override
 
287
        self.assertIsInstance(pb, TTYProgressBar)
 
288
 
 
289
    def test_progress_env_none(self):
 
290
        # Even though we are in a valid tty, no progress
 
291
        out = _TTYStringIO()
 
292
        pb = self.get_nested(out, 'xterm', 'none')
 
293
        pb.finished()
 
294
        self.assertIsInstance(pb, DummyProgress)
 
295
 
 
296
    def test_progress_env_invalid(self):
 
297
        out = _TTYStringIO()
 
298
        self.assertRaises(errors.InvalidProgressBarType, self.get_nested,
 
299
            out, 'xterm', 'nonexistant')