Skip to content

Commit fcc8e00

Browse files
committed
improve explanations for skip funcs per timotimo; remove spurious space at EOL
1 parent 11a1b16 commit fcc8e00

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

doc/Language/testing.pod

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -435,14 +435,14 @@ else {
435435
}
436436
=end code
437437
438-
Note that the C<skip> function and the tests to be skipped must be in
439-
two separate parts of an C<if/else> construct.
438+
Note that if you mark a test as skipped, you must also prevent that
439+
test from running.
440440
441441
=item X<skip-rest($reason?)|skip-rest>
442442
443443
Skip the remaining tests. If the remainder of the tests in the test file
444444
would all fail due to some condition, use this function to skip them,
445-
providing an optional C<$reason> as to why.
445+
providing an optional C<$reason> as to why.
446446
447447
=begin code
448448
unless $location ~~ "Wimbledon Common" {
@@ -458,8 +458,10 @@ ok womble();
458458
Note that C<skip-rest> requires a C<plan> to be set, otherwise the
459459
C<skip-rest> call will throw an error. If no plan was set, you can use
460460
C<done> to indicate the end of the test run (though of course the test
461-
summary will not count skipped tests). Note also the C<exit> is required
462-
to prevent the tests from executing.
461+
summary will not count skipped tests). Note also the C<exit> is
462+
required in the example to prevent the skipped tests from running.
463+
Alternatively, you could enclose the remaining tests in an C<else>
464+
branch.
463465
464466
=head1 Manual control
465467

0 commit comments

Comments
 (0)