Subject: | t/construct_tree.t skips some some test on missing LWP::UserAgent with message `URI::file or LWP::UserAgent not installed' |
If LWP::UserAgent is missing from system and URI::file is installed,
t/construct_tree.t skips some tests with confusing message:
ok 30 - found Gisle
ok 31 # skip URI::file or LWP::UserAgent not installed
ok 32 # skip URI::file or LWP::UserAgent not installed
ok 33 # skip URI::file or LWP::UserAgent not installed
ok 34 # skip URI::file or LWP::UserAgent not installed
ok 35 # skip URI::file or LWP::UserAgent not installed
ok 36 # skip URI::file or LWP::UserAgent not installed
ok 37 # skip URI::file or LWP::UserAgent not installed
ok 38 # skip URI::file or LWP::UserAgent not installed
ok 39 # skip URI::file or LWP::UserAgent not installed
ok 40 # skip URI::file or LWP::UserAgent not installed
ok 41 # skip URI::file or LWP::UserAgent not installed
ok 42 # skip URI::file or LWP::UserAgent not installed
ok 43 # skip URI::file or LWP::UserAgent not installed
ok 44 # skip URI::file or LWP::UserAgent not installed
ok 45 # skip URI::file or LWP::UserAgent not installed
ok 46 # skip URI::file or LWP::UserAgent not installed
ok 47 - new_from_content Scalar REF isa HTML::TreeBuilder
This is because the code does something different:
SKIP: {
eval {
# RECOMMEND PREREQ: URI::file
require URI::file;
require LWP::UserAgent;
1;
} or skip("URI::file or LWP::UserAgent not installed",
2 + 2 * tests_per_object);
The logical or is not implemented properly.