← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_api.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_api.py
1
 
# Copyright (C) 2005 by Canonical Ltd
 
1
# Copyright (C) 2005 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
21
21
relevant test modules.
22
22
"""
23
23
 
24
 
import os
25
 
import sys
26
 
 
27
24
import bzrlib
 
25
import bzrlib.api
 
26
from bzrlib.errors import IncompatibleAPI
28
27
from bzrlib.tests import TestCase
29
28
 
30
29
class APITests(TestCase):
35
34
        self.assert_(isinstance(bzrlib.version_string, str))
36
35
        self.assert_(isinstance(bzrlib.version_info, tuple))
37
36
        self.assertEqual(len(bzrlib.version_info), 5)
 
37
 
 
38
 
 
39
class TrivialObject(object):
 
40
    """This class allows assignment to any attribute."""
 
41
 
 
42
 
 
43
class TestAPIVersioning(TestCase):
 
44
 
 
45
    def test_get_minimum_api_version_reads_api_minimum_version(self):
 
46
        an_object = TrivialObject()
 
47
        an_object.api_minimum_version = (0, 1, 2)
 
48
        self.assertEqual((0, 1, 2),
 
49
            bzrlib.api.get_minimum_api_version(an_object))
 
50
 
 
51
    def test_get_minimum_api_version_fallsback_to_bzr_minimum_version(self):
 
52
        an_object = TrivialObject()
 
53
        self.assertEqual(bzrlib.api_minimum_version,
 
54
            bzrlib.api.get_minimum_api_version(an_object))
 
55
 
 
56
    def test_get_current_api_version_reads_api_current_version(self):
 
57
        an_object = TrivialObject()
 
58
        an_object.api_current_version = (3, 2, 1)
 
59
        an_object.version_info = (1, 2, 3, "final", 0)
 
60
        self.assertEqual((3, 2, 1),
 
61
            bzrlib.api.get_current_api_version(an_object))
 
62
 
 
63
    def test_get_current_api_version_fallsback_to_version_info(self):
 
64
        an_object = TrivialObject()
 
65
        an_object.version_info = (1, 2, 3, "final", 0)
 
66
        self.assertEqual((1, 2, 3),
 
67
            bzrlib.api.get_current_api_version(an_object))
 
68
 
 
69
    def test_get_current_api_version_fallsback_to_bzrlib_version_info(self):
 
70
        an_object = TrivialObject()
 
71
        self.assertEqual(bzrlib.version_info[0:3],
 
72
            bzrlib.api.get_current_api_version(an_object))
 
73
 
 
74
    def test_require_api_wanted_is_minimum_is_ok(self):
 
75
        an_object = TrivialObject()
 
76
        an_object.api_minimum_version = (1, 2, 3)
 
77
        an_object.api_current_version = (4, 5, 6)
 
78
        bzrlib.api.require_api(an_object, (1, 2, 3))
 
79
 
 
80
    def test_require_api_wanted_is_current_is_ok(self):
 
81
        an_object = TrivialObject()
 
82
        an_object.api_minimum_version = (1, 2, 3)
 
83
        an_object.api_current_version = (4, 5, 6)
 
84
        bzrlib.api.require_api(an_object, (4, 5, 6))
 
85
 
 
86
    def test_require_api_wanted_is_above_minimum_is_ok(self):
 
87
        an_object = TrivialObject()
 
88
        an_object.api_minimum_version = (1, 2, 3)
 
89
        an_object.api_current_version = (4, 5, 6)
 
90
        bzrlib.api.require_api(an_object, (1, 2, 4))
 
91
 
 
92
    def test_require_api_wanted_is_below_current_is_ok(self):
 
93
        an_object = TrivialObject()
 
94
        an_object.api_minimum_version = (1, 2, 3)
 
95
        an_object.api_current_version = (4, 5, 6)
 
96
        bzrlib.api.require_api(an_object, (4, 5, 5))
 
97
 
 
98
    def test_require_api_wanted_is_below_minimum_raises(self):
 
99
        an_object = TrivialObject()
 
100
        an_object.api_minimum_version = (1, 2, 3)
 
101
        an_object.api_current_version = (4, 5, 6)
 
102
        err = self.assertRaises(IncompatibleAPI,
 
103
            bzrlib.api.require_api, an_object, (1, 2, 2))
 
104
        self.assertEqual(err.api, an_object)
 
105
        self.assertEqual(err.wanted, (1, 2, 2))
 
106
        self.assertEqual(err.minimum, (1, 2, 3))
 
107
        self.assertEqual(err.current, (4, 5, 6))
 
108
 
 
109
    def test_require_api_wanted_is_above_current_raises(self):
 
110
        an_object = TrivialObject()
 
111
        an_object.api_minimum_version = (1, 2, 3)
 
112
        an_object.api_current_version = (4, 5, 6)
 
113
        err = self.assertRaises(IncompatibleAPI,
 
114
            bzrlib.api.require_api, an_object, (4, 5, 7))
 
115
        self.assertEqual(err.api, an_object)
 
116
        self.assertEqual(err.wanted, (4, 5, 7))
 
117
        self.assertEqual(err.minimum, (1, 2, 3))
 
118
        self.assertEqual(err.current, (4, 5, 6))