~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to doc/developers/testing.txt

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-03-25 00:02:51 UTC
  • mfrom: (5106.1.1 version-bump)
  • Revision ID: pqm@pqm.ubuntu.com-20100325000251-bwsv5c5d3l9x3lnn
(Jelmer) Bump API version for 2.2.0.

Show diffs side-by-side

added added

removed removed

Lines of Context:
6
6
The Importance of Testing
7
7
=========================
8
8
 
9
 
Reliability is a critical success factor for any version control system.
 
9
Reliability is a critical success factor for any Version Control System.
10
10
We want Bazaar to be highly reliable across multiple platforms while
11
11
evolving over time to meet the needs of its community.
12
12
 
183
183
 
184
184
.. _testrepository: https://launchpad.net/testrepository
185
185
 
186
 
 
187
 
Babune continuous integration
188
 
-----------------------------
189
 
 
190
 
We have a Hudson continuous-integration system that automatically runs 
191
 
tests across various platforms.  In the future we plan to add more 
192
 
combinations including testing plugins.  See 
193
 
<http://babune.ladeuil.net:24842/>.  (Babune = Bazaar Buildbot Network.)
194
 
 
195
 
 
196
 
Running tests in parallel
197
 
-------------------------
198
 
 
199
 
Bazaar can use subunit to spawn multiple test processes.  There is
200
 
slightly more chance you will hit ordering or timing-dependent bugs but
201
 
it's much faster::
202
 
 
203
 
  $ ./bzr selftest --parallel=fork
204
 
 
205
 
Note that you will need the Subunit library
206
 
<https://launchpad.net/subunit/> to use this, which is in
207
 
``python-subunit`` on Ubuntu.
208
 
 
209
 
 
210
 
Running tests from a ramdisk
211
 
----------------------------
212
 
 
213
 
The tests create and delete a lot of temporary files.  In some cases you
214
 
can make the test suite run much faster by running it on a ramdisk.  For
215
 
example::
216
 
 
217
 
  $ sudo mkdir /ram
218
 
  $ sudo mount -t tmpfs none /ram
219
 
  $ TMPDIR=/ram ./bzr selftest ...
220
 
 
221
 
You could also change ``/tmp`` in ``/etc/fstab`` to have type ``tmpfs``,
222
 
if you don't mind possibly losing other files in there when the machine
223
 
restarts.  Add this line (if there is none for ``/tmp`` already)::
224
 
 
225
 
  none           /tmp            tmpfs  defaults        0       0
226
 
 
227
 
With a 6-core machine and ``--parallel=fork`` using a tmpfs doubles the
228
 
test execution speed.
229
 
 
230
 
 
231
186
Writing Tests
232
187
=============
233
188
 
329
284
We make selective use of doctests__.  In general they should provide
330
285
*examples* within the API documentation which can incidentally be tested.  We
331
286
don't try to test every important case using doctests |--| regular Python
332
 
tests are generally a better solution.  That is, we just use doctests to make
333
 
our documentation testable, rather than as a way to make tests. Be aware that
334
 
doctests are not as well isolated as the unit tests, if you need more
335
 
isolation, you're likely want to write unit tests anyway if only to get a
336
 
better control of the test environment.
 
287
tests are generally a better solution.  That is, we just use doctests to
 
288
make our documentation testable, rather than as a way to make tests.
337
289
 
338
290
Most of these are in ``bzrlib/doc/api``.  More additions are welcome.
339
291
 
340
292
  __ http://docs.python.org/lib/module-doctest.html
341
293
 
342
 
There is an `assertDoctestExampleMatches` method in
343
 
`bzrlib.tests.TestCase` that allows you to match against doctest-style
344
 
string templates (including ``...`` to skip sections) from regular Python
345
 
tests.
346
 
 
347
294
 
348
295
Shell-like tests
349
296
----------------
350
297
 
351
 
``bzrlib/tests/script.py`` allows users to write tests in a syntax very
352
 
close to a shell session, using a restricted and limited set of commands
353
 
that should be enough to mimic most of the behaviours.
 
298
``bzrlib/tests/script.py`` allows users to write tests in a syntax very close to a shell session,
 
299
using a restricted and limited set of commands that should be enough to mimic
 
300
most of the behaviours.
354
301
 
355
302
A script is a set of commands, each command is composed of:
356
303
 
375
322
The execution stops as soon as an expected output or an expected error is not
376
323
matched.
377
324
 
378
 
If output occurs and no output is expected, the execution stops and the
379
 
test fails.  If unexpected output occurs on the standard error, then
380
 
execution stops and the test fails.
 
325
When no output is specified, any ouput from the command is accepted
 
326
and execution continue.
381
327
 
382
328
If an error occurs and no expected error is specified, the execution stops.
383
329
 
397
343
If you want the command to succeed for any output, just use::
398
344
 
399
345
  $ bzr add file
400
 
  ...
401
 
  2>...
402
 
 
403
 
or use the ``--quiet`` option::
404
 
 
405
 
  $ bzr add -q file
406
346
 
407
347
The following will stop with an error::
408
348
 
442
382
 
443
383
  $ cat file
444
384
 
445
 
You can run files containing shell-like scripts with::
446
 
 
447
 
  $ bzr test-script <script>
448
 
 
449
 
where ``<script>`` is the path to the file containing the shell-like script.
450
 
 
451
385
The actual use of ScriptRunner within a TestCase looks something like
452
386
this::
453
387
 
454
 
    from bzrlib.tests import script
455
 
 
456
 
    def test_unshelve_keep(self):
457
 
        # some setup here
458
 
        script.run_script(self, '''
459
 
            $ bzr add -q file
460
 
            $ bzr shelve -q --all -m Foo
461
 
            $ bzr shelve --list
462
 
            1: Foo
463
 
            $ bzr unshelve -q --keep
464
 
            $ bzr shelve --list
465
 
            1: Foo
466
 
            $ cat file
467
 
            contents of file
468
 
            ''')
469
 
 
470
 
You can also test commands that read user interaction::
471
 
 
472
 
    def test_confirm_action(self):
473
 
        """You can write tests that demonstrate user confirmation"""
474
 
        commands.builtin_command_registry.register(cmd_test_confirm)
475
 
        self.addCleanup(commands.builtin_command_registry.remove, 'test-confirm')
476
 
        self.run_script("""
477
 
            $ bzr test-confirm
478
 
            2>Really do it? [y/n]: 
479
 
            <yes
480
 
            yes
481
 
            """)
482
 
 
483
 
To avoid having to specify "-q" for all commands whose output is
484
 
irrelevant, the run_script() method may be passed the keyword argument
485
 
``null_output_matches_anything=True``.  For example::
486
 
 
487
 
    def test_ignoring_null_output(self):
488
 
        self.run_script("""
489
 
            $ bzr init
490
 
            $ bzr ci -m 'first revision' --unchanged
491
 
            $ bzr log --line
492
 
            1: ...
493
 
            """, null_output_matches_anything=True)
494
 
           
 
388
        def test_unshelve_keep(self):
 
389
                # some setup here
 
390
                sr = ScriptRunner()
 
391
                sr.run_script(self, '''
 
392
        $ bzr add file
 
393
        $ bzr shelve --all -m Foo
 
394
        $ bzr shelve --list
 
395
        1: Foo
 
396
        $ bzr unshelve --keep
 
397
        $ bzr shelve --list
 
398
        1: Foo
 
399
        $ cat file
 
400
        contents of file
 
401
        ''')
 
402
 
495
403
 
496
404
Import tariff tests
497
405
-------------------
512
420
  dependencies and only needed for particular cases.  If they're loaded in
513
421
  other cases then bzr may break for people who don't have those modules.
514
422
  
515
 
`test_import_tariff` allows us to check that removal of imports doesn't
 
423
`test_import_tarrif` allows us to check that removal of imports doesn't
516
424
regress.
517
425
 
518
426
This is done by running the command in a subprocess with
519
 
``PYTHON_VERBOSE=1``.  Starting a whole Python interpreter is pretty slow,
520
 
so we don't want exhaustive testing here, but just enough to guard against
521
 
distinct fixed problems.
 
427
``--profile-imports``.  Starting a whole Python interpreter is pretty
 
428
slow, so we don't want exhaustive testing here, but just enough to guard
 
429
against distinct fixed problems.
522
430
 
523
431
Assertions about precisely what is loaded tend to be brittle so we instead
524
432
make assertions that particular things aren't loaded.
550
458
Testing locking behaviour
551
459
-------------------------
552
460
 
553
 
In order to test the locking behaviour of commands, it is possible to install
554
 
a hook that is called when a write lock is: acquired, released or broken.
555
 
(Read locks also exist, they cannot be discovered in this way.)
556
 
 
557
 
A hook can be installed by calling bzrlib.lock.Lock.hooks.install_named_hook.
558
 
The three valid hooks are: `lock_acquired`, `lock_released` and `lock_broken`.
559
 
 
560
 
Example::
561
 
 
562
 
    locks_acquired = []
563
 
    locks_released = []
564
 
 
565
 
    lock.Lock.hooks.install_named_hook('lock_acquired',
566
 
        locks_acquired.append, None)
567
 
    lock.Lock.hooks.install_named_hook('lock_released',
568
 
        locks_released.append, None)
569
 
 
570
 
`locks_acquired` will now receive a LockResult instance for all locks acquired
571
 
since the time the hook is installed.
572
 
 
573
 
The last part of the `lock_url` allows you to identify the type of object that is locked.
574
 
 
575
 
- BzrDir: `/branch-lock`
576
 
- Working tree: `/checkout/lock`
577
 
- Branch: `/branch/lock`
578
 
- Repository: `/repository/lock`
579
 
 
580
 
To test if a lock is a write lock on a working tree, one can do the following::
581
 
 
582
 
    self.assertEndsWith(locks_acquired[0].lock_url, "/checkout/lock")
583
 
 
584
 
See bzrlib/tests/commands/test_revert.py for an example of how to use this for
585
 
testing locks.
586
 
 
 
461
You may want to write tests that particular objects are or aren't locked
 
462
during particular operations: see for example `bug 498409`__.  
 
463
 
 
464
 __ https://launchpad.net/bugs/498409
 
465
 
 
466
The `TestCase` base class registers hooks that record lock actions into 
 
467
``._lock_actions`` in this format::
 
468
 
 
469
  [
 
470
    ('acquired', LockResult(file:///tmp/testbzr-J2pcy2.tmp/.bzr/branch-lockc4au55ppz8wdym11z1aq)),
 
471
    ('released', LockResult(file:///tmp/testbzr-J2pcy2.tmp/.bzr/branch-lockc4au55ppz8wdym11z1aq)),
 
472
    ('acquired', LockResult(file:///tmp/testbzr-J2pcy2.tmp/.bzr/repository/lockyxb3rn4sw1oyx1jzkt45)),
 
473
    ('released', LockResult(file:///tmp/testbzr-J2pcy2.tmp/.bzr/repository/lockyxb3rn4sw1oyx1jzkt45)),
 
474
    ('acquired', LockResult(file:///tmp/testbzr-J2pcy2.tmp/.bzr/branch/lockh8c6t28rcjdkgxtndbje)),
 
475
    ('released', LockResult(file:///tmp/testbzr-J2pcy2.tmp/.bzr/branch/lockh8c6t28rcjdkgxtndbje)),
 
476
    ...
 
477
 
 
478
Alternatively you can register your own hooks to make custom assertions:
 
479
see `TestCase._check_locks` for an example.
587
480
 
588
481
Skipping tests
589
482
--------------
745
638
        _test_needs_features = [features.apport]
746
639
 
747
640
 
748
 
Testing translations
749
 
-----------------------
750
 
 
751
 
Translations are disabled by default in tests.  If you want to test
752
 
that code is translated you can use the ``ZzzTranslations`` class from
753
 
``test_i18n``::
754
 
 
755
 
    self.overrideAttr(i18n, '_translations', ZzzTranslations())
756
 
 
757
 
And check the output strings look like ``u"zz\xe5{{output}}"``.
758
 
 
759
 
To test the gettext setup and usage you override i18n.installed back
760
 
to self.i18nInstalled and _translations to None, see
761
 
test_i18n.TestInstall.
762
 
 
763
 
 
764
 
Testing deprecated code
765
 
-----------------------
766
 
 
767
 
When code is deprecated, it is still supported for some length of time,
768
 
usually until the next major version. The ``applyDeprecated`` helper
769
 
wraps calls to deprecated code to verify that it is correctly issuing the
770
 
deprecation warning, and also prevents the warnings from being printed
771
 
during test runs.
772
 
 
773
 
Typically patches that apply the ``@deprecated_function`` decorator should
774
 
update the accompanying tests to use the ``applyDeprecated`` wrapper.
775
 
 
776
 
``applyDeprecated`` is defined in ``bzrlib.tests.TestCase``. See the API
777
 
docs for more details.
778
 
 
779
 
 
780
641
Testing exceptions and errors
781
642
-----------------------------
782
643
 
855
716
whether a test should be added for that particular implementation,
856
717
or for all implementations of the interface.
857
718
 
 
719
The multiplication of tests for different implementations is normally
 
720
accomplished by overriding the ``load_tests`` function used to load tests
 
721
from a module.  This function typically loads all the tests, then applies
 
722
a TestProviderAdapter to them, which generates a longer suite containing
 
723
all the test variations.
 
724
 
858
725
See also `Per-implementation tests`_ (above).
859
726
 
860
727
 
861
 
Test scenarios and variations
862
 
-----------------------------
 
728
Test scenarios
 
729
--------------
863
730
 
864
731
Some utilities are provided for generating variations of tests.  This can
865
732
be used for per-implementation tests, or other cases where the same test
870
737
values to which the test should be applied.  The test suite should then
871
738
also provide a list of scenarios in which to run the tests.
872
739
 
873
 
A single *scenario* is defined by a `(name, parameter_dict)` tuple.  The
874
 
short string name is combined with the name of the test method to form the
875
 
test instance name.  The parameter dict is merged into the instance's
876
 
attributes.
877
 
 
878
 
For example::
879
 
 
880
 
    load_tests = load_tests_apply_scenarios
881
 
 
882
 
    class TestCheckout(TestCase):
883
 
 
884
 
        scenarios = multiply_scenarios(
885
 
            VaryByRepositoryFormat(), 
886
 
            VaryByTreeFormat(),
887
 
            )
888
 
 
889
 
The `load_tests` declaration or definition should be near the top of the
890
 
file so its effect can be seen.
 
740
Typically ``multiply_tests_from_modules`` should be called from the test
 
741
module's ``load_tests`` function.
891
742
 
892
743
 
893
744
Test support
906
757
 
907
758
TestCase
908
759
    A base TestCase that extends the Python standard library's
909
 
    TestCase in several ways.  TestCase is build on
910
 
    ``testtools.TestCase``, which gives it support for more assertion
911
 
    methods (e.g.  ``assertContainsRe``), ``addCleanup``, and other
912
 
    features (see its API docs for details).  It also has a ``setUp`` that
913
 
    makes sure that global state like registered hooks and loggers won't
914
 
    interfere with your test.  All tests should use this base class
915
 
    (whether directly or via a subclass).  Note that we are trying not to
916
 
    add more assertions at this point, and instead to build up a library
917
 
    of ``bzrlib.tests.matchers``.
 
760
    TestCase in several ways.  It adds more assertion methods (e.g.
 
761
    ``assertContainsRe``), ``addCleanup``, and other features (see its API
 
762
    docs for details).  It also has a ``setUp`` that makes sure that
 
763
    global state like registered hooks and loggers won't interfere with
 
764
    your test.  All tests should use this base class (whether directly or
 
765
    via a subclass).
918
766
 
919
767
TestCaseWithMemoryTransport
920
768
    Extends TestCase and adds methods like ``get_transport``,
992
840
 
993
841
Please see bzrlib.treebuilder for more details.
994
842
 
995
 
PreviewTree
996
 
~~~~~~~~~~~
997
 
 
998
 
PreviewTrees are based on TreeTransforms.  This means they can represent
999
 
virtually any state that a WorkingTree can have, including unversioned files.
1000
 
They can be used to test the output of anything that produces TreeTransforms,
1001
 
such as merge algorithms and revert.  They can also be used to test anything
1002
 
that takes arbitrary Trees as its input.
1003
 
 
1004
 
::
1005
 
 
1006
 
  # Get an empty tree to base the transform on.
1007
 
  b = self.make_branch('.')
1008
 
  empty_tree = b.repository.revision_tree(_mod_revision.NULL_REVISION)
1009
 
  tt = TransformPreview(empty_tree)
1010
 
  self.addCleanup(tt.finalize)
1011
 
  # Empty trees don't have a root, so add it first.
1012
 
  root = tt.new_directory('', ROOT_PARENT, 'tree-root')
1013
 
  # Set the contents of a file.
1014
 
  tt.new_file('new-file', root, 'contents', 'file-id')
1015
 
  preview = tt.get_preview_tree()
1016
 
  # Test the contents.
1017
 
  self.assertEqual('contents', preview.get_file_text('file-id'))
1018
 
 
1019
 
PreviewTrees can stack, with each tree falling back to the previous::
1020
 
 
1021
 
  tt2 = TransformPreview(preview)
1022
 
  self.addCleanup(tt2.finalize)
1023
 
  tt2.new_file('new-file2', tt2.root, 'contents2', 'file-id2')
1024
 
  preview2 = tt2.get_preview_tree()
1025
 
  self.assertEqual('contents', preview2.get_file_text('file-id'))
1026
 
  self.assertEqual('contents2', preview2.get_file_text('file-id2'))
1027
 
 
1028
 
 
1029
 
Temporarily changing state
1030
 
~~~~~~~~~~~~~~~~~~~~~~~~~~
1031
 
 
1032
 
If your test needs to temporarily mutate some global state, and you need
1033
 
it restored at the end, you can say for example::
1034
 
 
1035
 
    self.overrideAttr(osutils, '_cached_user_encoding', 'latin-1')
1036
 
 
1037
 
This should be used with discretion; sometimes it's better to make the
1038
 
underlying code more testable so that you don't need to rely on monkey
1039
 
patching.
1040
 
 
1041
 
 
1042
 
Observing calls to a function
1043
 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1044
 
 
1045
 
Sometimes it's useful to observe how a function is called, typically when
1046
 
calling it has side effects but the side effects are not easy to observe
1047
 
from a test case.  For instance the function may be expensive and we want
1048
 
to assert it is not called too many times, or it has effects on the
1049
 
machine that are safe to run during a test but not easy to measure.  In
1050
 
these cases, you can use `recordCalls` which will monkey-patch in a
1051
 
wrapper that records when the function is called.
1052
 
 
1053
 
 
1054
 
Temporarily changing environment variables
1055
 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1056
 
 
1057
 
If yout test needs to temporarily change some environment variable value
1058
 
(which generally means you want it restored at the end), you can use::
1059
 
 
1060
 
    self.overrideEnv('BZR_ENV_VAR', 'new_value')
1061
 
 
1062
 
If you want to remove a variable from the environment, you should use the
1063
 
special ``None`` value::
1064
 
 
1065
 
    self.overrideEnv('PATH', None)
1066
 
 
1067
 
If you add a new feature which depends on a new environment variable, make
1068
 
sure it behaves properly when this variable is not defined (if applicable) and
1069
 
if you need to enforce a specific default value, check the
1070
 
``TestCase._cleanEnvironment`` in ``bzrlib.tests.__init__.py`` which defines a
1071
 
proper set of values for all tests.
1072
 
 
1073
 
Cleaning up
1074
 
~~~~~~~~~~~
1075
 
 
1076
 
Our base ``TestCase`` class provides an ``addCleanup`` method, which
1077
 
should be used instead of ``tearDown``.  All the cleanups are run when the
1078
 
test finishes, regardless of whether it passes or fails.  If one cleanup
1079
 
fails, later cleanups are still run.
1080
 
 
1081
 
(The same facility is available outside of tests through
1082
 
``bzrlib.cleanup``.)
1083
 
 
1084
 
 
1085
 
Manual testing
1086
 
==============
1087
 
 
1088
 
Generally we prefer automated testing but sometimes a manual test is the
1089
 
right thing, especially for performance tests that want to measure elapsed
1090
 
time rather than effort.
1091
 
 
1092
 
Simulating slow networks
1093
 
------------------------
1094
 
 
1095
 
To get realistically slow network performance for manually measuring
1096
 
performance, we can simulate 500ms latency (thus 1000ms round trips)::
1097
 
 
1098
 
  $ sudo tc qdisc add dev lo root netem delay 500ms
1099
 
 
1100
 
Normal system behaviour is restored with ::
1101
 
 
1102
 
  $ sudo tc qdisc del dev lo root
1103
 
 
1104
 
A more precise version that only filters traffic to port 4155 is::
1105
 
 
1106
 
    tc qdisc add dev lo root handle 1: prio
1107
 
    tc qdisc add dev lo parent 1:3 handle 30: netem delay 500ms 
1108
 
    tc filter add dev lo protocol ip parent 1:0 prio 3 u32 match ip dport 4155 0xffff flowid 1:3 
1109
 
    tc filter add dev lo protocol ip parent 1:0 prio 3 u32 match ip sport 4155 0xffff flowid 1:3 
1110
 
 
1111
 
and to remove this::
1112
 
 
1113
 
    tc filter del dev lo protocol ip parent 1: pref 3 u32
1114
 
    tc qdisc del dev lo root handle 1:
1115
 
 
1116
 
You can use similar code to add additional delay to a real network
1117
 
interface, perhaps only when talking to a particular server or pointing at
1118
 
a VM.  For more information see <http://lartc.org/>.
1119
 
 
1120
843
 
1121
844
.. |--| unicode:: U+2014
1122
845
 
1123
846
..
1124
 
   vim: ft=rst tw=74 ai et sw=4
 
847
   vim: ft=rst tw=74 ai