<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">Le sam. 24 nov. 2018 à 12:44, Thomas Petazzoni <<a href="mailto:thomas.petazzoni@bootlin.com">thomas.petazzoni@bootlin.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
On Sat, 24 Nov 2018 11:56:30 +0100, François Perrad wrote:<br>
<br>
> > Sorry if I haven't followed the previous discussion, but why do we have<br>
> > this docstring ? Is it just for information ?<br>
> >  <br>
> Yes, this is for information.<br>
<br>
OK.<br>
<br>
> By this way, we know that we don't need to write another test for HTTP-Date<br>
> or IO-HTML.<br>
<br>
This is not really true. Indeed, if you test HTTP-Date with plenty of<br>
other packages installed, how are you sure that HTTP-Date has all the<br>
necessary runtime dependencies it needs ?<br>
<br>
Only a test with only HTTP-Date installed and its dependencies will<br>
allow to verify this.<br>
<br></blockquote><div><br></div><div>I want also to limit the number of test, see cover letter.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> > > +    def test_run(self):<br>
> > > +        self.login()<br>
> > > +        self.module_test("LWP")<br>
> > > +        self.module_test("LWP::UserAgent")<br>
> > > +        self.module_test("LWP::Authen::Basic")<br>
> > > +        self.module_test("LWP::Authen::Digest")<br>
> > > +        self.module_test("HTTP::Message")<br>
> > > +        self.module_test("HTTP::Daemon")<br>
> > > +        self.module_test("WWW::RobotRules")  <br>
> ><br>
> > Why are you testing specifically those modules ? What is the rule to<br>
> > decide what modules are tested here ?<br>
> ><br>
> >  <br>
> libwww-perl is a special package/distribution from the beginning of the WWW<br>
> history<br>
> (started in 1995, see <a href="https://metacpan.org/changes/distribution/libwww-perl" rel="noreferrer" target="_blank">https://metacpan.org/changes/distribution/libwww-perl</a><br>
> ).<br>
> In fact, libwww-perl collection is a set of Perl modules.<br>
> LWP (Lib World-Wide Web Perl) is just a name space for some of them, and<br>
> doesn't match with the distribution name libwww.<br>
> So, we need load most of them.<br>
<br>
OK, so all those modules are the ones part of libwww-perl, i.e we are<br>
not testing modules installed by other packages, correct ?<br>
<br></blockquote><div><br></div><div>Correct.</div><div><br></div><div>François<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks,<br>
<br>
Thomas<br>
-- <br>
Thomas Petazzoni, CTO, Bootlin<br>
Embedded Linux and Kernel engineering<br>
<a href="https://bootlin.com" rel="noreferrer" target="_blank">https://bootlin.com</a><br>
</blockquote></div></div>