Hi,

knock out 1 of the todo/bugs_apr_ext :)
FYI: APR::NetLib does not exist.

See attached.

HTH

--
END
------------------------------------------------------
Philip M. Gollucci ([EMAIL PROTECTED]) 301.254.5198
Consultant / http://p6m7g8.net/Resume/resume.shtml
Senior Developer / Liquidity Services, Inc.
        http://www.liquidityservicesinc.com
Index: t/apr-ext/xsloader.t
===================================================================
--- t/apr-ext/xsloader.t        (revision 0)
+++ t/apr-ext/xsloader.t        (revision 0)
@@ -0,0 +1,11 @@
+#!perl -T
+
+use strict;
+use warnings FATAL => 'all';
+use Apache::Test;
+
+use TestAPRlib::xsloader;
+
+plan tests => TestAPRlib::xsloader::num_of_tests();
+
+TestAPRlib::xsloader::test();
Index: t/apr-ext/os.t
===================================================================
--- t/apr-ext/os.t      (revision 0)
+++ t/apr-ext/os.t      (revision 0)
@@ -0,0 +1,11 @@
+#!perl -T
+
+use strict;
+use warnings FATAL => 'all';
+use Apache::Test;
+
+use TestAPRlib::os;
+
+plan tests => TestAPRlib::os::num_of_tests();
+
+TestAPRlib::os::test();
Index: t/apr-ext/const.t
===================================================================
--- t/apr-ext/const.t   (revision 0)
+++ t/apr-ext/const.t   (revision 0)
@@ -0,0 +1,11 @@
+#!perl -T
+
+use strict;
+use warnings FATAL => 'all';
+use Apache::Test;
+
+use TestAPRlib::const;
+
+plan tests => TestAPRlib::const::num_of_tests();
+
+TestAPRlib::const::test();
Index: t/apr-ext/socket.t
===================================================================
--- t/apr-ext/socket.t  (revision 0)
+++ t/apr-ext/socket.t  (revision 0)
@@ -0,0 +1,11 @@
+#!perl -T
+
+use strict;
+use warnings FATAL => 'all';
+use Apache::Test;
+
+use TestAPRlib::socket;
+
+plan tests => TestAPRlib::socket::num_of_tests();
+
+TestAPRlib::socket::test();
Index: t/apr-ext/error.t
===================================================================
--- t/apr-ext/error.t   (revision 0)
+++ t/apr-ext/error.t   (revision 0)
@@ -0,0 +1,11 @@
+#!perl -T
+
+use strict;
+use warnings FATAL => 'all';
+use Apache::Test;
+
+use TestAPRlib::error;
+
+plan tests => TestAPRlib::error::num_of_tests();
+
+TestAPRlib::error::test();
Index: t/apr-ext/sockaddr.t
===================================================================
--- t/apr-ext/sockaddr.t        (revision 0)
+++ t/apr-ext/sockaddr.t        (revision 0)
@@ -0,0 +1,11 @@
+#!perl -T
+
+use strict;
+use warnings FATAL => 'all';
+use Apache::Test;
+
+use TestAPRlib::sockaddr;
+
+plan tests => TestAPRlib::sockaddr::num_of_tests();
+
+TestAPRlib::sockaddr::test();
Index: t/apr-ext/brigade.t
===================================================================
--- t/apr-ext/brigade.t (revision 0)
+++ t/apr-ext/brigade.t (revision 0)
@@ -0,0 +1,11 @@
+#!perl -T
+
+use strict;
+use warnings FATAL => 'all';
+use Apache::Test;
+
+use TestAPRlib::brigade;
+
+plan tests => TestAPRlib::brigade::num_of_tests();
+
+TestAPRlib::brigade::test();
Index: t/lib/TestAPRlib/xsloader.pm
===================================================================
--- t/lib/TestAPRlib/xsloader.pm        (revision 0)
+++ t/lib/TestAPRlib/xsloader.pm        (revision 0)
@@ -0,0 +1,19 @@
+package TestAPRlib::xsloader;
+
+# testing APR::XSLoader API
+
+use strict;
+use warnings FATAL => 'all';
+
+use Apache::Test;
+use Apache::TestUtil;
+
+sub num_of_tests {
+    return 1;
+}
+
+sub test {
+       ok require APR::XSLoader;
+}
+
+1;
Index: t/lib/TestAPRlib/os.pm
===================================================================
--- t/lib/TestAPRlib/os.pm      (revision 0)
+++ t/lib/TestAPRlib/os.pm      (revision 0)
@@ -0,0 +1,19 @@
+package TestAPRlib::os;
+
+# testing APR::OS API
+
+use strict;
+use warnings FATAL => 'all';
+
+use Apache::Test;
+use Apache::TestUtil;
+
+sub num_of_tests {
+    return 1;
+}
+
+sub test {
+       ok require APR::OS;
+}
+
+1;
Index: t/lib/TestAPRlib/const.pm
===================================================================
--- t/lib/TestAPRlib/const.pm   (revision 0)
+++ t/lib/TestAPRlib/const.pm   (revision 0)
@@ -0,0 +1,19 @@
+package TestAPRlib::const;
+
+# testing APR::Const API
+
+use strict;
+use warnings FATAL => 'all';
+
+use Apache::Test;
+use Apache::TestUtil;
+
+sub num_of_tests {
+    return 1;
+}
+
+sub test {
+       ok require APR::Const;
+}
+
+1;
Index: t/lib/TestAPRlib/socket.pm
===================================================================
--- t/lib/TestAPRlib/socket.pm  (revision 0)
+++ t/lib/TestAPRlib/socket.pm  (revision 0)
@@ -0,0 +1,19 @@
+package TestAPRlib::socket;
+
+# testing APR::Socket API
+
+use strict;
+use warnings FATAL => 'all';
+
+use Apache::Test;
+use Apache::TestUtil;
+
+sub num_of_tests {
+    return 1;
+}
+
+sub test {
+       ok require APR::Socket;
+}
+
+1;
Index: t/lib/TestAPRlib/error.pm
===================================================================
--- t/lib/TestAPRlib/error.pm   (revision 0)
+++ t/lib/TestAPRlib/error.pm   (revision 0)
@@ -0,0 +1,19 @@
+package TestAPRlib::error;
+
+# testing APR::Error API
+
+use strict;
+use warnings FATAL => 'all';
+
+use Apache::Test;
+use Apache::TestUtil;
+
+sub num_of_tests {
+    return 1;
+}
+
+sub test {
+       ok require APR::Error;
+}
+
+1;
Index: t/lib/TestAPRlib/sockaddr.pm
===================================================================
--- t/lib/TestAPRlib/sockaddr.pm        (revision 0)
+++ t/lib/TestAPRlib/sockaddr.pm        (revision 0)
@@ -0,0 +1,19 @@
+package TestAPRlib::sockaddr;
+
+# testing APR::SockAddr API
+
+use strict;
+use warnings FATAL => 'all';
+
+use Apache::Test;
+use Apache::TestUtil;
+
+sub num_of_tests {
+    return 1;
+}
+
+sub test {
+       ok require APR::SockAddr;
+}
+
+1;
Index: t/lib/TestAPRlib/brigade.pm
===================================================================
--- t/lib/TestAPRlib/brigade.pm (revision 0)
+++ t/lib/TestAPRlib/brigade.pm (revision 0)
@@ -0,0 +1,19 @@
+package TestAPRlib::brigade;
+
+# testing APR::Brigade API
+
+use strict;
+use warnings FATAL => 'all';
+
+use Apache::Test;
+use Apache::TestUtil;
+
+sub num_of_tests {
+    return 1;
+}
+
+sub test {
+       ok require APR::Brigade;
+}
+
+1;
Index: todo/bugs_apr_ext
===================================================================
--- todo/bugs_apr_ext   (revision 164158)
+++ todo/bugs_apr_ext   (working copy)
@@ -21,27 +21,3 @@
   modperl_interp_unselect. At the moment I've worked around it by
   defining an empty function in APR.xs. But a cleaner solution is
   desirable.
-
-* modules that we need to add tests for in apr-ext:
-  APR::Date
-  APR::Const
-  APR::URI
-  APR::Base64
-  APR::String
-  APR::Error
-  APR::XSLoader
-  APR::PerlIO
-  APR::OS
-  APR::Socket
-  APR::Brigade
-  APR::Finfo
-  APR::Util
-  APR::Bucket
-  APR::NetLib
-  APR::ThreadMutex
-  APR::SockAddr
-
-  I think that it's enough to just load them, since most of them are
-  already thoroughly tested under mod_perl. Then if we find some
-  running-outside-mod_perl specific bugs we can write specific tests
-  for those.
Index: Changes
===================================================================
--- Changes     (revision 164158)
+++ Changes     (working copy)
@@ -12,6 +12,9 @@
 
 =item 1.999_23-dev
 
+flush out the missing APR:: tests for apr-ext
+[Philip M. Gollucci <[EMAIL PROTECTED]>]
+
 restore the DESTDIR support partially nuked by the apache2 rename
 branch [Torsten F�rtsch <torsten.foertsch gmx.net>]
 

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to