Skip Menu |

This queue is for tickets about the Test-Skip-UnlessExistsExecutable CPAN distribution.

Report information
The Basics
Id: 55965
Status: resolved
Priority: 0/
Queue: Test-Skip-UnlessExistsExecutable

People
Owner: Nobody in particular
Requestors: NKH [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)



Subject: sub names and arguments
The function names have not been changed to reflect the functionality I'd like to suggest skip_all_unless_exists to skip all the test in the file and skip_unless_exists to skip a single block of tests. Also, when loading a module, multiple arguments can be given but only a single argument can be given to the sub interface. most test subs take a text argument that is displayed when the test is run, or in your case skipped. That is not the case in your module Suggestion 1: skip_all_test_unless_exists [ '...', '...'], 'no xxx needed for test TTT' ; Suggestion 2: skip_test_unless_exists \@pre_requisits { many tests in this block }, 'missing pre-requisits to run the many tests' ; # it would be nice to be able to display which pre-requisits are missing in the message ok(1) ; # should run whether the above many tests were run or not
above names would be even more correct if I had suggested correct names! skip_all_tests_unless_exists_executable skip_tests_unless_exists_executable
Thanks. I changed "the api name from test_exists_executable" to "skip_all_unless_exists".