r35448 - in /trunk/libtest-distmanifest-perl: CIRCULAR.SKIP Changes MANIFEST META.yml Makefile.PL README SIGNATURE debian/changelog debian/control debian/rules lib/Test/DistManifest.pm t/03core.t t/03warn-only.t

jawnsy-guest at users.alioth.debian.org jawnsy-guest at users.alioth.debian.org
Fri May 15 13:42:58 UTC 2009


Author: jawnsy-guest
Date: Fri May 15 13:42:53 2009
New Revision: 35448

URL: http://svn.debian.org/wsvn/pkg-perl/?sc=1&rev=35448
Log:
Uploaded new upstream release. Moved to new debhelper and removed the override* stuff. Updated dependency list

Added:
    trunk/libtest-distmanifest-perl/t/03warn-only.t
Modified:
    trunk/libtest-distmanifest-perl/CIRCULAR.SKIP
    trunk/libtest-distmanifest-perl/Changes
    trunk/libtest-distmanifest-perl/MANIFEST
    trunk/libtest-distmanifest-perl/META.yml
    trunk/libtest-distmanifest-perl/Makefile.PL
    trunk/libtest-distmanifest-perl/README
    trunk/libtest-distmanifest-perl/SIGNATURE
    trunk/libtest-distmanifest-perl/debian/changelog
    trunk/libtest-distmanifest-perl/debian/control
    trunk/libtest-distmanifest-perl/debian/rules
    trunk/libtest-distmanifest-perl/lib/Test/DistManifest.pm
    trunk/libtest-distmanifest-perl/t/03core.t

Modified: trunk/libtest-distmanifest-perl/CIRCULAR.SKIP
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/CIRCULAR.SKIP?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/CIRCULAR.SKIP (original)
+++ trunk/libtest-distmanifest-perl/CIRCULAR.SKIP Fri May 15 13:42:53 2009
@@ -13,6 +13,7 @@
 
 # Module::Build generated files and dirs.
 ^Build$
+^Build\.bat$
 ^_build/
 
 # Module::Install included files.
@@ -33,6 +34,7 @@
 \#$
 \b\.#
 \.bak$
+\.tmp$
 
 # Devel::Cover files.
 \bcover_db\b

Modified: trunk/libtest-distmanifest-perl/Changes
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/Changes?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/Changes (original)
+++ trunk/libtest-distmanifest-perl/Changes Fri May 15 13:42:53 2009
@@ -1,3 +1,42 @@
+2009-05-15  Jonathan Yu <frequency at cpan.org>
+
+	Oops, noted the version was bumped when it wasn't. Fixed now.
+
+2009-05-15  Jonathan Yu <frequency at cpan.org>
+
+	- Moved some small changes from MANIFEST.SKIP to CIRCULAR.SKIP
+	- Bumped version to 1.2.1
+	- This release is particularly important to get rid of Win32 test failures, where `Build.bat' is created instead of `Build'
+
+2009-05-14  Jonathan Yu <frequency at cpan.org>
+
+	These were some pretty important changes, bumped the version to 1.2.0 instead
+
+2009-05-14  Jonathan Yu <frequency at cpan.org>
+
+	Updated t/03warn-only.t test with new semantics (always fail if parsing fails)
+
+2009-05-14  Jonathan Yu <frequency at cpan.org>
+
+	- Note that circular dependencies and parsing errors will always be considered fatal, regardless of MANIFEST_WARN_ONLY
+	- Suppress refactoring requirement - even though manifest_ok is really complex, it doesn't make sense to exist in separate subs
+
+2009-05-14  Jonathan Yu <frequency at cpan.org>
+
+	Added t/03warn-only.t to MANIFEST
+
+2009-05-14  Jonathan Yu <frequency at cpan.org>
+
+	- Bumped version to 1.1.5
+	- Added new feature for nonfatal errors (see updated POD)
+	- Added tests for this new feature
+
+	There a problem still some bugs to be worked out.
+
+2009-05-12  Jonathan Yu <frequency at cpan.org>
+
+	- Cleaned up documentation (minor fixes)
+
 2009-04-26  Jonathan Yu <frequency at cpan.org>
 
 	- Updated to version 1.1.4, prepare for next release

Modified: trunk/libtest-distmanifest-perl/MANIFEST
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/MANIFEST?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/MANIFEST (original)
+++ trunk/libtest-distmanifest-perl/MANIFEST Fri May 15 13:42:53 2009
@@ -23,3 +23,4 @@
 t/01portability.t
 t/02manifest.t
 t/03core.t
+t/03warn-only.t

Modified: trunk/libtest-distmanifest-perl/META.yml
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/META.yml?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/META.yml (original)
+++ trunk/libtest-distmanifest-perl/META.yml Fri May 15 13:42:53 2009
@@ -1,11 +1,10 @@
 --- #YAML:1.0
 name: Test-DistManifest
-version: 1.001004
+version: 1.002001
 author:
   - 'Jonathan Yu <frequency at cpan.org>'
 abstract: |-
-  Tests that your MANIFEST matches the distribution as it
-  exists, excluding those in your MANIFEST.SKIP
+  Verify MANIFEST/MANIFEST.SKIP as an author test
 license: perl
 resources:
   Ratings: http://cpanratings.perl.org/d/Test-DistManifest
@@ -44,7 +43,7 @@
 provides:
   Test::DistManifest:
     file: lib/Test/DistManifest.pm
-    version: 1.001004
+    version: 1.002001
 generated_by: Module::Build version 0.3
 meta-spec:
   url: http://module-build.sourceforge.net/META-spec-v1.4.html

Modified: trunk/libtest-distmanifest-perl/Makefile.PL
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/Makefile.PL?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/Makefile.PL (original)
+++ trunk/libtest-distmanifest-perl/Makefile.PL Fri May 15 13:42:53 2009
@@ -1,7 +1,7 @@
 # Note: this file was auto-generated by Module::Build::Compat version 0.2808_01
 require 5.006;
 
-unless (eval "use Module::Build::Compat 0.02; 1") {
+unless (eval 'use Module::Build::Compat 0.02; 1') {
   print "This module requires Module::Build to install itself.\n";
 
   require ExtUtils::MakeMaker;
@@ -20,12 +20,12 @@
   my $cwd = Cwd::cwd();
   
   CPAN::Shell->install('Module::Build::Compat');
-  CPAN::Shell->expand("Module", "Module::Build::Compat")->uptodate
+  CPAN::Shell->expand('Module', 'Module::Build::Compat')->uptodate
     or die "Couldn't install Module::Build, giving up.\n";
 
   chdir $cwd or die "Cannot chdir() back to $cwd: $!";
 }
-eval "use Module::Build::Compat 0.02; 1" or die $@;
+eval 'use Module::Build::Compat 0.02; 1' or die $@;
 
 Module::Build::Compat->run_build_pl(args => \@ARGV);
 require Module::Build;

Modified: trunk/libtest-distmanifest-perl/README
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/README?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/README (original)
+++ trunk/libtest-distmanifest-perl/README Fri May 15 13:42:53 2009
@@ -1,8 +1,7 @@
 NAME
-    Test::DistManifest - Tests that your MANIFEST matches the distribution
-    as it exists, excluding those in your MANIFEST.SKIP
+    Test::DistManifest - Verify MANIFEST/MANIFEST.SKIP as an author test
 
-$Id: README 5755 2009-03-25 19:13:03Z FREQUENCY at cpan.org $
+$Id: README 7058 2009-05-12 22:54:02Z FREQUENCY at cpan.org $
 
 DESCRIPTION
     This module provides a simple method of testing that a MANIFEST matches

Modified: trunk/libtest-distmanifest-perl/SIGNATURE
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/SIGNATURE?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/SIGNATURE (original)
+++ trunk/libtest-distmanifest-perl/SIGNATURE Fri May 15 13:42:53 2009
@@ -15,17 +15,17 @@
 Hash: SHA1
 
 SHA1 374598505d2937f792550f19f5c8a668a1df5430 Build.PL
-SHA1 8190073af2323d80436cc0b79421cd19e095cbc9 CIRCULAR.SKIP
-SHA1 c60db0a32de8cb36d04398b2059d2019008c3c5e Changes
+SHA1 3427d2bae373d0b415568fe79b038f02e80b3e27 CIRCULAR.SKIP
+SHA1 54f8f765fe979dcca50123923d193edb730015a3 Changes
 SHA1 6bc8386fc92c74529ab0cfce62a91a369640e67e LICENSE
-SHA1 86ea176734ddbe1785af4978163e30f43c451e15 MANIFEST
+SHA1 012f60f398a7dec05451d97d83c4d93ebc11f258 MANIFEST
 SHA1 91e8519cd6ba58a9a1810ec16c6b5b30f94edc25 MANIFEST.EXTRA
 SHA1 c1ef3014c0bc3ed78eec43db3590cecd28f39a12 MANIFEST.SKIP
-SHA1 f671eba3c84a74f6d0ae5ea49420a13abb024904 META.yml
-SHA1 7cfa2f7bdb9c7494ceb0f3260077453c41deb412 Makefile.PL
-SHA1 e18b41ceeb4f9012fa12985fc7b23caee30e8967 README
+SHA1 d8cd5185ad3b6fb62aa0db0c7eed8449f218c741 META.yml
+SHA1 451655c0e9996854afe8fb2b4bc83edf218c38e9 Makefile.PL
+SHA1 98426bdb84a8ae5beb738a588754bcf85cfa18ed README
 SHA1 605a74373a17120acdd2497cd2f9a3fcbc81a730 examples/checkmanifest.t
-SHA1 f1cbdd9873cec1b2c7c4c084d117788de4a42d97 lib/Test/DistManifest.pm
+SHA1 bd4d88379df4145ec0c2820c7cb375670f483617 lib/Test/DistManifest.pm
 SHA1 b6f1c337969d535166dbd528ace297575b077aaf t/00signature.t
 SHA1 3f1dc7578eecb686008662e1b6400155111d3a82 t/01compile.t
 SHA1 9448154fc848b5e30726bfd0454e52cb65a57489 t/01critic.rc
@@ -37,11 +37,12 @@
 SHA1 fd570637144391b7cbd4a58e0305679c583d1a67 t/01pod.t
 SHA1 2364582da3cd9a6df6473f51e0388ed17ee5404e t/01portability.t
 SHA1 7d0f87938f966f5bad9f53097bd3fc162f64b864 t/02manifest.t
-SHA1 d5aab2d964d1b1f0e0d021053929e75c71dc3424 t/03core.t
+SHA1 1f7fc22b0620e7e56ce1ee13eb83edbea3be9dfa t/03core.t
+SHA1 ba50c1c242a08aef1bb94852451c4933b05fa64f t/03warn-only.t
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.9 (GNU/Linux)
 
-iEYEARECAAYFAkn1BUAACgkQYYN9T93mai/piwCfap24t4r0m2plUMukreAYOdaW
-W30AoJ7gylq/+3z+LQzv7RSCeN9zjsRl
-=erJG
+iEYEARECAAYFAkoM7MkACgkQYYN9T93mai/+7ACgkcwk1EOKW/OzVKxvmdbwyQnu
+MlQAn0znVVMPOZRh06jCLHuWcboBHFbz
+=HOvf
 -----END PGP SIGNATURE-----

Modified: trunk/libtest-distmanifest-perl/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/debian/changelog?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/debian/changelog (original)
+++ trunk/libtest-distmanifest-perl/debian/changelog Fri May 15 13:42:53 2009
@@ -1,3 +1,11 @@
+libtest-distmanifest-perl (1.2.1-1) UNRELEASED; urgency=low
+
+  * New upstream release
+  * Added non-fatal errors; that is, tests that warn but do not fail.
+    This is useful for Debian distributions.
+
+ -- Jonathan Yu <frequency at cpan.org>  Fri, 15 May 2009 09:02:17 -0400
+
 libtest-distmanifest-perl (1.1.4-1) unstable; urgency=low
 
   * Initial Release. (Closes: #519657)

Modified: trunk/libtest-distmanifest-perl/debian/control
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/debian/control?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/debian/control (original)
+++ trunk/libtest-distmanifest-perl/debian/control Fri May 15 13:42:53 2009
@@ -1,12 +1,13 @@
 Source: libtest-distmanifest-perl
 Section: perl
 Priority: optional
-Build-Depends: debhelper (>= 7.0.50)
+Build-Depends: debhelper (>= 7.2.10)
 Build-Depends-Indep: libmodule-manifest-perl (>= 0.07),
   libtest-perl-critic-perl (>= 1.01), libperl-critic-perl (>= 1.098),
   libtest-yaml-meta-perl (>= 0.11), libtest-pod-perl (>= 1.14),
   libtest-pod-coverage-perl (>= 1.04), libtest-nowarnings-perl (>= 0.084),
-  libtest-minimumversion-perl (>= 0.008), perl (>= 5.6.0-12)
+  libtest-minimumversion-perl (>= 0.008), libtest-signature-perl,
+  libtest-simple-perl (>= 0.86), perl (>= 5.6.0-12)
 Maintainer: Debian Perl Group <pkg-perl-maintainers at lists.alioth.debian.org>
 Uploaders: Jonathan Yu <frequency at cpan.org>
 Standards-Version: 3.8.1
@@ -16,7 +17,7 @@
 
 Package: libtest-distmanifest-perl
 Architecture: all
-Depends: ${misc:Depends}, ${perl:Depends},
+Depends: ${misc:Depends}, ${perl:Depends}, libtest-simple-perl (>= 0.86),
   libmodule-manifest-perl (>= 0.07)
 Description: Perl module for verifying CPAN MANIFEST files
  Test::DistManifest provides a simple method of testing that a MANIFEST

Modified: trunk/libtest-distmanifest-perl/debian/rules
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/debian/rules?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/debian/rules (original)
+++ trunk/libtest-distmanifest-perl/debian/rules Fri May 15 13:42:53 2009
@@ -4,9 +4,6 @@
 build-stamp:
 	TEST_AUTHOR=1 dh build
 	touch $@
-
-override_dh_auto_configure:
-	dh_auto_configure -- create_packlist=0
 
 clean:
 	dh $@

Modified: trunk/libtest-distmanifest-perl/lib/Test/DistManifest.pm
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/lib/Test/DistManifest.pm?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/lib/Test/DistManifest.pm (original)
+++ trunk/libtest-distmanifest-perl/lib/Test/DistManifest.pm Fri May 15 13:42:53 2009
@@ -1,7 +1,7 @@
 # Test::DistManifest
 #  Tests that your manifest matches the distribution as it exists.
 #
-# $Id: DistManifest.pm 6700 2009-04-27 01:05:33Z FREQUENCY at cpan.org $
+# $Id: DistManifest.pm 7095 2009-05-15 04:10:13Z FREQUENCY at cpan.org $
 #
 # Copyright (C) 2008-2009 by Jonathan Yu <frequency at cpan.org>
 #
@@ -16,16 +16,15 @@
 
 =head1 NAME
 
-Test::DistManifest - Tests that your MANIFEST matches the distribution as it
-exists, excluding those in your MANIFEST.SKIP
+Test::DistManifest - Verify MANIFEST/MANIFEST.SKIP as an author test
 
 =head1 VERSION
 
-Version 1.1.4 ($Id: DistManifest.pm 6700 2009-04-27 01:05:33Z FREQUENCY at cpan.org $)
+Version 1.2.1 ($Id: DistManifest.pm 7095 2009-05-15 04:10:13Z FREQUENCY at cpan.org $)
 
 =cut
 
-use version; our $VERSION = qv('1.1.4');
+use version; our $VERSION = qv('1.2.1');
 
 =head1 EXPORTS
 
@@ -149,9 +148,42 @@
   $ make dist # For ExtUtils::MakeMaker
   $ ./Build dist # For Module::Build
 
+=head2 Non-Fatal Errors
+
+By default, errors in the B<MANIFEST> or B<MANIFEST.SKIP> files are treated
+as fatal, which really is the purpose of using C<Test::DistManifest> as part
+of your author test suite.
+
+In some cases this is not desirable behaviour, such as with the Debian Perl
+Group, which runs all tests - including author tests - as part of its module
+packaging process. This wreaks havoc because Debian adds its control files
+in C<debian/> downstream, and that directory or its files are generally not
+in B<MANIFEST.SKIP>.
+
+By setting the environment variable B<MANIFEST_WARN_ONLY> to a true value,
+errors will be non-fatal - they show up as diagnostic messages only, but all
+tests pass from the perspective of C<Test::Harness>.
+
+This can be used in a test script as:
+
+  $ENV{MANIFEST_WARN_ONLY} = 1;
+
+or from other shell scripts as:
+
+  export MANIFEST_WARN_ONLY=1
+
+Note that parsing errors in each file (B<MANIFEST> and B<MANIFEST.SKIP>) and
+circular dependencies will always be considered fatal. The author is not aware
+of any other use cases where other behaviour would be useful.
+
 =cut
 
+# It's not the simplest subroutine, but it's still not complex enough nor
+# useful enough in its parts to really be refactored.
+## no critic(ProhibitExcessComplexity)
 sub manifest_ok {
+  my $warn_only = $ENV{MANIFEST_WARN_ONLY} || 0;
+
   my $manifile = shift || 'MANIFEST';
   my $skipfile = shift || 'MANIFEST.SKIP';
 
@@ -231,7 +263,8 @@
     }
     $test->diag($path);
   }
-  $test->ok($flag, 'All files are listed in MANIFEST or skipped');
+  $test->ok($warn_only || $flag, 'All files are listed in MANIFEST or ' .
+    'skipped');
 
   # Reset the flag and test $manifest->files now
   $flag = 1;
@@ -254,7 +287,8 @@
     }
     $test->diag($path);
   }
-  $test->ok($flag, 'All files listed in MANIFEST exist on disk');
+  $test->ok($warn_only || $flag, 'All files listed in MANIFEST exist ' .
+    'on disk');
 
   # Test for circular dependencies
   $flag = (scalar @circular == 0) ? 1 : 0;
@@ -398,8 +432,7 @@
 If you have a bug report or feature request, please file them on the CPAN
 Request Tracker at L<http://rt.cpan.org>. If you are able to submit your bug
 report in the form of failing unit tests, you are B<strongly> encouraged to do
-so. Regular bug reports are always accepted and appreciated via the CPAN bug
-tracker.
+so.
 
 =head1 SEE ALSO
 

Modified: trunk/libtest-distmanifest-perl/t/03core.t
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/t/03core.t?rev=35448&op=diff
==============================================================================
--- trunk/libtest-distmanifest-perl/t/03core.t (original)
+++ trunk/libtest-distmanifest-perl/t/03core.t Fri May 15 13:42:53 2009
@@ -3,7 +3,7 @@
 # t/03core.t
 #  Ensures the MANIFEST test output looks reasonable.
 #
-# $Id: 03core.t 5665 2009-03-16 16:19:40Z FREQUENCY at cpan.org $
+# $Id: 03core.t 7088 2009-05-15 02:51:39Z FREQUENCY at cpan.org $
 #
 # All rights to this test script are hereby disclaimed and its contents
 # released into the public domain by the author. Where this is not possible,
@@ -16,13 +16,18 @@
 use Test::DistManifest;
 use Test::NoWarnings; # 1 test
 
+# If MANIFEST_WARN_ONLY is set, unset it
+if (exists($ENV{MANIFEST_WARN_ONLY})) {
+  delete($ENV{MANIFEST_WARN_ONLY});
+}
+
 # This one is already tested as part of 02manifest.t
 #manifest_ok();
 
 manifest_ok('MANIFEST', 'MANIFEST.SKIP');
 
 # Run the test when MANIFEST is invalid but the MANIFEST.SKIP file is okay
-#  3 subtests
+#  1 test
 test_out('not ok 1 - Parse MANIFEST or equivalent');
 test_out('ok 2 - Parse MANIFEST.SKIP or equivalent');
 test_out('not ok 3 - All files are listed in MANIFEST or skipped');
@@ -37,7 +42,7 @@
 );
 
 # Run the test when MANIFEST is valid but the MANIFEST.SKIP file is missing
-#  3 subtests
+#  1 test
 test_out('ok 1 - Parse MANIFEST or equivalent');
 test_out('not ok 2 - Parse MANIFEST.SKIP or equivalent');
 test_out('not ok 3 - All files are listed in MANIFEST or skipped');
@@ -52,7 +57,7 @@
 );
 
 # Test what happens when MANIFEST contains some extra files
-#  1 subtest
+#  1 test
 test_out('ok 1 - Parse MANIFEST or equivalent');
 test_out('ok 2 - Parse MANIFEST.SKIP or equivalent');
 test_out('not ok 3 - All files are listed in MANIFEST or skipped');
@@ -67,7 +72,7 @@
 
 # Test what happens when we have some strange circular logic; that is,
 # when MANIFEST and MANIFEST.SKIP point to the same file!
-#  1 subtest
+#  1 test
 test_out('ok 1 - Parse MANIFEST or equivalent');
 test_out('ok 2 - Parse MANIFEST.SKIP or equivalent');
 test_out('ok 3 - All files are listed in MANIFEST or skipped');

Added: trunk/libtest-distmanifest-perl/t/03warn-only.t
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libtest-distmanifest-perl/t/03warn-only.t?rev=35448&op=file
==============================================================================
--- trunk/libtest-distmanifest-perl/t/03warn-only.t (added)
+++ trunk/libtest-distmanifest-perl/t/03warn-only.t Fri May 15 13:42:53 2009
@@ -1,0 +1,81 @@
+#!/usr/bin/perl -T
+
+# t/03warn-only.t
+#  Ensures the nonfatal mode works properly
+#
+# $Id: 03warn-only.t 7092 2009-05-15 03:32:37Z FREQUENCY at cpan.org $
+#
+# All rights to this test script are hereby disclaimed and its contents
+# released into the public domain by the author. Where this is not possible,
+# you may use this file under the same terms as Perl itself.
+
+use strict;
+use warnings;
+
+use Test::Builder::Tester tests => 5; # The sum of all subtests
+use Test::DistManifest;
+use Test::NoWarnings; # 1 test
+
+# Set MANIFEST_WARN_ONLY to true
+$ENV{MANIFEST_WARN_ONLY} = 1;
+
+# Run the same tests as 03core.t, but make sure they are all non-fatal now.
+
+# Run the test when MANIFEST is invalid but the MANIFEST.SKIP file is okay
+#  1 test
+test_out('not ok 1 - Parse MANIFEST or equivalent');
+test_out('ok 2 - Parse MANIFEST.SKIP or equivalent');
+test_out('ok 3 - All files are listed in MANIFEST or skipped');
+test_out('ok 4 - All files listed in MANIFEST exist on disk');
+test_diag('No such file or directory');
+test_fail(+1);
+test_out('ok 5 - No files are in both MANIFEST and MANIFEST.SKIP');
+manifest_ok('INVALID.FILE', 'MANIFEST.SKIP');
+test_test(
+  name        => 'Fails when MANIFEST cannot be parsed',
+  skip_err    => 1,
+);
+
+# Run the test when MANIFEST is valid but the MANIFEST.SKIP file is missing
+#  1 test
+test_out('ok 1 - Parse MANIFEST or equivalent');
+test_out('not ok 2 - Parse MANIFEST.SKIP or equivalent');
+test_out('ok 3 - All files are listed in MANIFEST or skipped');
+test_out('ok 4 - All files listed in MANIFEST exist on disk');
+test_diag('No such file or directory');
+test_out('ok 5 - No files are in both MANIFEST and MANIFEST.SKIP');
+test_fail(+1);
+manifest_ok('MANIFEST', 'INVALID.FILE');
+test_test(
+  name        => 'Fails when MANIFEST.SKIP cannot be parsed',
+  skip_err    => 1,
+);
+
+# Test what happens when MANIFEST contains some extra files
+#  1 test
+test_out('ok 1 - Parse MANIFEST or equivalent');
+test_out('ok 2 - Parse MANIFEST.SKIP or equivalent');
+test_out('ok 3 - All files are listed in MANIFEST or skipped');
+test_out('ok 4 - All files listed in MANIFEST exist on disk');
+test_out('ok 5 - No files are in both MANIFEST and MANIFEST.SKIP');
+test_fail(+1);
+manifest_ok('MANIFEST.EXTRA', 'MANIFEST.SKIP');
+test_test(
+  name        => 'Succeeds even when MANIFEST contains extra files',
+  skip_err    => 1,
+);
+
+# Test what happens when we have some strange circular logic; that is,
+# when MANIFEST and MANIFEST.SKIP point to the same file!
+#  1 test
+test_out('ok 1 - Parse MANIFEST or equivalent');
+test_out('ok 2 - Parse MANIFEST.SKIP or equivalent');
+test_out('ok 3 - All files are listed in MANIFEST or skipped');
+test_out('ok 4 - All files listed in MANIFEST exist on disk');
+test_out('not ok 5 - No files are in both MANIFEST and MANIFEST.SKIP');
+test_fail(+1);
+manifest_ok('MANIFEST', 'CIRCULAR.SKIP');
+test_test(
+  name        => 'Fails when files are in both MANIFEST and MANIFEST.SKIP',
+  skip_err    => 1,
+);




More information about the Pkg-perl-cvs-commits mailing list