/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 doc/developers/testing.txt

  • Committer: Andrew Bennetts
  • Date: 2010-02-12 04:33:05 UTC
  • mfrom: (5031 +trunk)
  • mto: This revision was merged to the branch mainline in revision 5032.
  • Revision ID: andrew.bennetts@canonical.com-20100212043305-ujdbsdoviql2t7i3
MergeĀ lp:bzr

Show diffs side-by-side

added added

removed removed

Lines of Context:
39
39
Running the Test Suite
40
40
======================
41
41
 
 
42
As of Bazaar 2.1, you must have the testtools_ library installed to run
 
43
the bzr test suite.
 
44
 
 
45
.. _testtools: https://launchpad.net/testtools/
 
46
 
 
47
To test all of Bazaar, just run::
 
48
 
 
49
  bzr selftest 
 
50
 
 
51
With ``--verbose`` bzr will print the name of every test as it is run.
 
52
 
 
53
This should always pass, whether run from a source tree or an installed
 
54
copy of Bazaar.  Please investigate and/or report any failures.
 
55
 
 
56
 
 
57
Running particular tests
 
58
------------------------
 
59
 
42
60
Currently, bzr selftest is used to invoke tests.
43
61
You can provide a pattern argument to run a subset. For example,
44
62
to run just the blackbox tests, run::
86
104
--load-list. The later is rarely used but allows to run a subset of a list of
87
105
failing tests for example.
88
106
 
 
107
Disabling plugins
 
108
-----------------
 
109
 
 
110
To test only the bzr core, ignoring any plugins you may have installed,
 
111
use::
 
112
 
 
113
  ./bzr --no-plugins selftest 
 
114
 
 
115
Disabling crash reporting
 
116
-------------------------
 
117
 
 
118
By default Bazaar uses apport_ to report program crashes.  In developing
 
119
Bazaar it's normal and expected to have it crash from time to time, at
 
120
least because a test failed if for no other reason.
 
121
 
 
122
Therefore you should probably add ``debug_flags = no_apport`` to your
 
123
``bazaar.conf`` file (in ``~/.bazaar/`` on Unix), so that failures just
 
124
print a traceback rather than writing a crash file.
 
125
 
 
126
.. _apport: https://launchpad.net/apport/
 
127
 
89
128
 
90
129
Test suite debug flags
91
130
----------------------
97
136
  This can provide useful logging to help debug test failures when used
98
137
  with e.g. ``bzr -Dhpss selftest -E=allow_debug``
99
138
 
 
139
  Note that this will probably cause some tests to fail, because they
 
140
  don't expect to run with any debug flags on.
 
141
 
 
142
 
 
143
Using subunit
 
144
-------------
 
145
 
 
146
Bazaar can optionally produce output in the machine-readable subunit_
 
147
format, so that test output can be post-processed by various tools.
 
148
 
 
149
.. _subunit: https://launchpad.net/subunit/
 
150
 
 
151
 
100
152
 
101
153
Writing Tests
102
154
=============
103
155
 
 
156
Normally you should add or update a test for all bug fixes or new features
 
157
in Bazaar.
 
158
 
 
159
 
104
160
Where should I put a new test?
105
161
------------------------------
106
162
 
154
210
    cmd_object.run() method directly. This is a lot faster than
155
211
    subprocesses and generates the same logging output as running it in a
156
212
    subprocess (which invoking the method directly does not).
157
 
 
 
213
 
158
214
 3. Only test the one command in a single test script. Use the bzrlib
159
215
    library when setting up tests and when evaluating the side-effects of
160
216
    the command. We do this so that the library api has continual pressure
173
229
 
174
230
Per-implementation tests are tests that are defined once and then run
175
231
against multiple implementations of an interface.  For example,
176
 
``test_transport_implementations.py`` defines tests that all Transport
177
 
implementations (local filesystem, HTTP, and so on) must pass.
178
 
 
179
 
They are found in ``bzrlib/tests/*_implementations/test_*.py``,
180
 
``bzrlib/tests/per_*/*.py``, and
181
 
``bzrlib/tests/test_*_implementations.py``.
 
232
``per_transport.py`` defines tests that all Transport implementations
 
233
(local filesystem, HTTP, and so on) must pass. They are found in
 
234
``bzrlib/tests/per_*/*.py``, and ``bzrlib/tests/per_*.py``.
182
235
 
183
236
These are really a sub-category of unit tests, but an important one.
184
237
 
 
238
Along the same lines are tests for extension modules. We generally have
 
239
both a pure-python and a compiled implementation for each module. As such,
 
240
we want to run the same tests against both implementations. These can
 
241
generally be found in ``bzrlib/tests/*__*.py`` since extension modules are
 
242
usually prefixed with an underscore. Since there are only two
 
243
implementations, we have a helper function
 
244
``bzrlib.tests.permute_for_extension``, which can simplify the
 
245
``load_tests`` implementation.
 
246
 
185
247
 
186
248
Doctests
187
249
~~~~~~~~
198
260
 
199
261
 
200
262
Shell-like tests
201
 
~~~~~~~~~~~~~~~~
 
263
----------------
202
264
 
203
265
``bzrlib/tests/script.py`` allows users to write tests in a syntax very close to a shell session,
204
266
using a restricted and limited set of commands that should be enough to mimic
225
287
the line.
226
288
 
227
289
The execution stops as soon as an expected output or an expected error is not
228
 
matched. 
 
290
matched.
229
291
 
230
292
When no output is specified, any ouput from the command is accepted
231
 
and execution continue. 
 
293
and execution continue.
232
294
 
233
295
If an error occurs and no expected error is specified, the execution stops.
234
296
 
287
349
 
288
350
  $ cat file
289
351
 
290
 
 
291
 
 
292
 
.. Effort tests
293
 
.. ~~~~~~~~~~~~
294
 
 
 
352
The actual use of ScriptRunner within a TestCase looks something like
 
353
this::
 
354
 
 
355
        def test_unshelve_keep(self):
 
356
                # some setup here
 
357
                sr = ScriptRunner()
 
358
                sr.run_script(self, '''
 
359
        $ bzr add file
 
360
        $ bzr shelve --all -m Foo
 
361
        $ bzr shelve --list
 
362
        1: Foo
 
363
        $ bzr unshelve --keep
 
364
        $ bzr shelve --list
 
365
        1: Foo
 
366
        $ cat file
 
367
        contents of file
 
368
        ''')
 
369
 
 
370
 
 
371
Import tariff tests
 
372
-------------------
 
373
 
 
374
`bzrlib.tests.test_import_tariff` has some tests that measure how many
 
375
Python modules are loaded to run some representative commands.
 
376
 
 
377
We want to avoid loading code unnecessarily, for reasons including:
 
378
 
 
379
* Python modules are interpreted when they're loaded, either to define
 
380
  classes or modules or perhaps to initialize some structures.
 
381
 
 
382
* With a cold cache we may incur blocking real disk IO for each module.
 
383
 
 
384
* Some modules depend on many others.
 
385
 
 
386
* Some optional modules such as `testtools` are meant to be soft
 
387
  dependencies and only needed for particular cases.  If they're loaded in
 
388
  other cases then bzr may break for people who don't have those modules.
 
389
  
 
390
`test_import_tarrif` allows us to check that removal of imports doesn't
 
391
regress.
 
392
 
 
393
This is done by running the command in a subprocess with
 
394
``--profile-imports``.  Starting a whole Python interpreter is pretty
 
395
slow, so we don't want exhaustive testing here, but just enough to guard
 
396
against distinct fixed problems.
 
397
 
 
398
Assertions about precisely what is loaded tend to be brittle so we instead
 
399
make assertions that particular things aren't loaded.
 
400
 
 
401
Unless selftest is run with ``--no-plugins``, modules will be loaded in
 
402
the usual way and checks made on what they cause to be loaded.  This is
 
403
probably worth checking into, because many bzr users have at least some
 
404
plugins installed (and they're included in binary installers).
 
405
 
 
406
In theory, plugins might have a good reason to load almost anything:
 
407
someone might write a plugin that opens a network connection or pops up a
 
408
gui window every time you run 'bzr status'.  However, it's more likely
 
409
that the code to do these things is just being loaded accidentally.  We
 
410
might eventually need to have a way to make exceptions for particular
 
411
plugins.
 
412
 
 
413
Some things to check:
 
414
 
 
415
* non-GUI commands shouldn't load GUI libraries
 
416
 
 
417
* operations on bzr native formats sholudn't load foreign branch libraries
 
418
 
 
419
* network code shouldn't be loaded for purely local operations
 
420
 
 
421
* particularly expensive Python built-in modules shouldn't be loaded
 
422
  unless there is a good reason
295
423
 
296
424
 
297
425
Skipping tests
343
471
        The test exists but is known to fail, for example this might be
344
472
        appropriate to raise if you've committed a test for a bug but not
345
473
        the fix for it, or if something works on Unix but not on Windows.
346
 
        
 
474
 
347
475
        Raising this allows you to distinguish these failures from the
348
476
        ones that are not expected to fail.  If the test would fail
349
477
        because of something we don't expect or intend to fix,
353
481
        KnownFailure should be used with care as we don't want a
354
482
        proliferation of quietly broken tests.
355
483
 
 
484
 
 
485
 
356
486
We plan to support three modes for running the test suite to control the
357
487
interpretation of these results.  Strict mode is for use in situations
358
488
like merges to the mainline and releases where we want to make sure that
369
499
UnavailableFeature      fail    pass    pass
370
500
KnownFailure            fail    pass    pass
371
501
======================= ======= ======= ========
372
 
     
 
502
 
373
503
 
374
504
Test feature dependencies
375
505
-------------------------
398
528
 
399
529
    self.requireFeature(StraceFeature)
400
530
 
401
 
Features already defined in bzrlib.tests include:
402
 
 
403
 
 - SymlinkFeature,
404
 
 - HardlinkFeature,
405
 
 - OsFifoFeature,
406
 
 - UnicodeFilenameFeature,
407
 
 - FTPServerFeature, and
 
531
The old naming style for features is CamelCase, but because they're
 
532
actually instances not classses they're now given instance-style names
 
533
like ``apport``.
 
534
 
 
535
Features already defined in ``bzrlib.tests`` and ``bzrlib.tests.features``
 
536
include:
 
537
 
 
538
 - apport
 
539
 - paramiko
 
540
 - SymlinkFeature
 
541
 - HardlinkFeature
 
542
 - OsFifoFeature
 
543
 - UnicodeFilenameFeature
 
544
 - FTPServerFeature
408
545
 - CaseInsensitiveFilesystemFeature.
409
546
 
410
547
 
416
553
``_probe`` and ``feature_name`` methods.  For example::
417
554
 
418
555
    class _SymlinkFeature(Feature):
419
 
    
 
556
 
420
557
        def _probe(self):
421
558
            return osutils.has_symlinks()
422
 
    
 
559
 
423
560
        def feature_name(self):
424
561
            return 'symlinks'
425
 
    
 
562
 
426
563
    SymlinkFeature = _SymlinkFeature()
427
564
 
 
565
A helper for handling running tests based on whether a python
 
566
module is available. This can handle 3rd-party dependencies (is
 
567
``paramiko`` available?) as well as stdlib (``termios``) or
 
568
extension modules (``bzrlib._groupcompress_pyx``). You create a
 
569
new feature instance with::
 
570
 
 
571
    # in bzrlib/tests/features.py
 
572
    apport = tests.ModuleAvailableFeature('apport')
 
573
 
 
574
 
 
575
    # then in bzrlib/tests/test_apport.py
 
576
    class TestApportReporting(TestCaseInTempDir):
 
577
 
 
578
        _test_needs_features = [features.apport]
 
579
 
428
580
 
429
581
Testing exceptions and errors
430
582
-----------------------------