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

  • Committer: Vincent Ladeuil
  • Date: 2009-05-08 13:39:32 UTC
  • mto: (4343.1.1 integration)
  • mto: This revision was merged to the branch mainline in revision 4344.
  • Revision ID: v.ladeuil+lp@free.fr-20090508133932-4fbx4z8fvraeyjdy
Fixed as per John's and Markus reviews.

* bzrlib/tests/blackbox/test_push.py:
(TestPush.test_push_with_revisionspec): Fix wrong run_bzr_error
call. Additionnally the error regexp was wrong !

* bzrlib/tests/blackbox/test_ls.py:
(TestLS.test_ls_path, TestLS.test_kinds): Fix wrong run_bzr_error
calls.

* bzrlib/tests/__init__.py:
(TestCase.run_bzr): Add a check to catch errors in error_regexes
usage where people use a a string that it then iterated char by
char for matching (which is most certainly not the intent of the
test writer).

* bzrlib/tests/blackbox/test_log.py:
Fix run_bzr_error calls. Fix some white spaces issues (some
expected strings were using TABs and were wrongly caught by
test_coding_style(), but fixing the strings wasn't really a
problem either.

Show diffs side-by-side

added added

removed removed

Lines of Context:
92
92
        self.stats.save(f)
93
93
        data1 = cPickle.load(open(f))
94
94
        self.assertEqual(type(data1), bzrlib.lsprof.Stats)
95
 
 
96
 
 
97
 
class TestBzrProfiler(tests.TestCase):
98
 
 
99
 
    _test_needs_features = [LSProfFeature]
100
 
 
101
 
    def test_start_call_stuff_stop(self):
102
 
        profiler = bzrlib.lsprof.BzrProfiler()
103
 
        profiler.start()
104
 
        try:
105
 
            def a_function():
106
 
                pass
107
 
            a_function()
108
 
        finally:
109
 
            stats = profiler.stop()
110
 
        stats.freeze()
111
 
        lines = [str(data) for data in stats.data]
112
 
        lines = [line for line in lines if 'a_function' in line]
113
 
        self.assertLength(1, lines)