/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 bzrlib/tests/scenarios.py

  • Committer: John Arbash Meinel
  • Date: 2011-04-20 14:27:19 UTC
  • mto: This revision was merged to the branch mainline in revision 5837.
  • Revision ID: john@arbash-meinel.com-20110420142719-advs1k5vztqzbrgv
https://launchpad.net/bugs/767177
Fix bug #767177. Be more agressive with file.close() calls.

Our test suite gets a number of thread leaks and failures because it happens to get async
SFTPFile.close() calls. (if an SFTPFile closes due to __del__ it is done as an async request,
while if you call SFTPFile.close() it is done as a synchronous request.)
We have a couple other cases, probably. Namely SFTPTransport.get() also does an async
prefetch of the content, so if you don't .read() you'll also leak threads that think they
are doing work that you want.

The biggest change here, though, is using a try/finally in a generator, which is not 
python2.4 compatible.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/scenarios.py
 
1
# Copyright (C) 2010 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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
 
 
18
"""Generate multiple variations in different scenarios.
 
19
 
 
20
For a class whose tests should be repeated in varying scenarios, set a
 
21
`scenarios` member to a list of scenarios where it should be repeated.
 
22
 
 
23
This is similar to the interface provided by
 
24
<http://launchpad.net/testscenarios/>.
 
25
"""
 
26
 
 
27
 
 
28
from bzrlib.tests import (
 
29
    iter_suite_tests,
 
30
    multiply_scenarios,
 
31
    multiply_tests,
 
32
    )
 
33
 
 
34
 
 
35
def load_tests_apply_scenarios(standard_tests, module, loader):
 
36
    """Multiply tests depending on their 'scenarios' attribute.
 
37
 
 
38
    This can be assigned to 'load_tests' in any test module to make this
 
39
    automatically work across tests in the module.
 
40
    """
 
41
    result = loader.suiteClass()
 
42
    multiply_tests_by_their_scenarios(standard_tests, result)
 
43
    return result
 
44
 
 
45
 
 
46
def multiply_tests_by_their_scenarios(some_tests, into_suite):
 
47
    """Multiply the tests in the given suite by their declared scenarios.
 
48
 
 
49
    Each test must have a 'scenarios' attribute which is a list of 
 
50
    (name, params) pairs.
 
51
 
 
52
    :param some_tests: TestSuite or Test.
 
53
    :param into_suite: A TestSuite into which the resulting tests will be
 
54
        inserted.
 
55
    """
 
56
    for test in iter_suite_tests(some_tests):
 
57
        scenarios = getattr(test, 'scenarios', None)
 
58
        if scenarios is None:
 
59
            into_suite.addTest(test)
 
60
        else:
 
61
            multiply_tests(test, test.scenarios, into_suite)