libwww-perl : missing test dependency

[ken@…]

Somewhere between versions 6.20 and 6.23 this developed a need for Test-RequiresInternet to run its testsuite. At least that is a trivial dependency, only needing core modules.

[ken@…]

I mistyped the version - I noticed this on 6.23.

Meanwhile, the book has moved on to 6.24, but that retains the dep:

  "TEST_REQUIRES" => {
    "ExtUtils::MakeMaker" => 0,
    "File::Spec" => 0,
    "FindBin" => 0,
    "Test::Fatal" => 0,
    "Test::More" => 0,
    "Test::RequiresInternet" => 0
  },
  "VERSION" => "6.24",

[bdubbs@…]

I don't understand what you want done. There are several modules above that we do not mention: ExtUtils::MakeMaker, File::Spec, FindBin, and Test::More" as well as Test::RequiresInternet

Do we really need to specify every module a parent uses for a test?

I personally do not see any issues with the tests as I use the 'cpan -i ' method of installing a module and it automatically pulls in needed modules whether for the module requested or for tests.

[ken@…]

If people want to pull in *all* the *possible* dependent modules using cpan -i, they can run a distro. But in that case we might as well tell everybody to just use cpan, and point them to the latest version (which is what it normally uses).

Of the modules you mentioned, ExtUtils::MakeMaker, File::Spec, FindBin and Test::More are core modules, which I documented in general/prog/perl-core-modules.txt.

My point is that for anybody whole builds modules individually, the tests are almost-always expected to pass. A run of 'make test' in a perl module which ends in failure implies a real problem, unlike for many packages where testsuites are often likely to fail.

Some of those failures are because of missing runtime dependencies, which are clearly needed. Others, like this example, are only important to stop the testsuite failing.

Third paragraph of the page -

Where a module is shown as '(for the testsuite)' it is required if you wish to run the test for the module which uses it. In these cases, only the required or recommended dependencies are listed - there might be other modules which allow more tests to be run, but omitting them will still allow the tests to PASS.

So, I think that Test::RequiresInternet should be added for libwww-perl in the same way that e.g. SUPER was added for Test::MockModule in the deps of Archive::Zip.

[bdubbs@…]

Oh, I don't object to adding it. I was saying that I didn't understand what was asked.

Do you want to do it? It sounds like it would be a quick update.

[ken@…]

Replying to bdubbs@…:

Oh, I don't object to adding it. I was saying that I didn't understand what was asked.

Do you want to do it? It sounds like it would be a quick update.

I still find editing that page to add new dependencies tends to be painful. At the moment I have no wish to touch it.

[bdubbs@…]

Fixed at revision 18514.

[bdubbs@…]

Milestone renamed

[bdubbs@…]

Milestone renamed