William A. Rowe, Jr. wrote:

From: "Stas Bekman" <[EMAIL PROTECTED]>
Sent: Wednesday, October 17, 2001 11:14 PM



William A. Rowe, Jr. wrote:


From: "Stas Bekman" <[EMAIL PROTECTED]>
Sent: Wednesday, October 17, 2001 10:40 PM


exec ./t/TEST protocol/eliza
protocol/eliza....skipped: missing Chatbot::Eliza1
All tests successful, 1 test skipped.

The extension of Test::plan is already somewhat tricky, so in order to preserve the trick we have to make sure that our extension always returns a zero or one value. Since CODE and ARRAY refs were already taken I've used the available HASH ref. Now have module returns a reason for its failure, the CODE and ARRAY refs do or can do the same, here is how it works (this is incomplete, just to see if you like it):


Without groking the code, definately +1 to allow Foo::try syntax, since we
on non-unix platforms already have problems with Foo/try vs Foo\try ;)

Hmm, I guess you are talking about somethins else. What I try to do now is to print the reason for the skipped test.


That is, in and of itself, a great thing!

:)



Can you please expand on the problem you are talking about? And how do you want it to be handled?


Right now, we pass the argument to t/TEST somesuite/item.

If we adopted the Perl syntax of somesuite::item, the parsers wouldn't have
to deal with differences between somesuite/item.t and somesuite\item.t which would make documenting the 'invoking' feature much simpler for all
platforms.


Right now, we generally worry about the 'unixish' family, and OS2, Win32
and Netware all accept foo/bar even though their native tounge is foo\bar.

If we ever support far stranger platforms [and you seem ready with all of
the file::spec code in the tree ;-] then this will become more difficult.


Apache::Test is built on top of Test::Harness which is used by 'make test' of Perl itself on all platforms Perl is running on. And I think Perl is running on about the same platforms as Apache, if not more (I'm not sure). Therefore I think what's good for Test::Harness is good for Apache::Test. And if something is not working the fix should be done in Test::Harness.

The only nuance is that Apache::Test tests for the test file existance if you explicitly specify these, but since the path is preserved I don't think why there should be any problem at all. So if you call t/TEST with foo\\bar, Perl will test for -e foo\\bar, we just need to make sure to use File::Spec to correctly prepend the t/ prefix. but this is an easy fix. Is that the only problem with the test names?

Your suggestion to adopt foo::bar seems to be confusing, since many times I'm using file completion and do ./t/TEST t/foo/bar.t using the real path, with your proposal you cannot do this anymore, not talking about the fact that some tests are just grouped in the same dir, but have nothing to do with dir:: prefix. ... I don't know, I suggest to solve the problem if we have any and continue using the real paths. What do you think?



_____________________________________________________________________
Stas Bekman             JAm_pH      --   Just Another mod_perl Hacker
http://stason.org/      mod_perl Guide   http://perl.apache.org/guide
mailto:[EMAIL PROTECTED]  http://ticketmaster.com http://apacheweek.com
http://singlesheaven.com http://perl.apache.org http://perlmonth.com/



Reply via email to