Commit | Line | Data |
---|---|---|
f6d6199c | 1 | #!/usr/bin/perl -w |
0300da75 MS |
2 | |
3 | BEGIN { | |
39234879 MS |
4 | if( $ENV{PERL_CORE} ) { |
5 | chdir 't' if -d 't'; | |
6 | unshift @INC, '../lib'; | |
7 | } | |
f6d6199c MS |
8 | else { |
9 | unshift @INC, 't/lib'; | |
10 | } | |
0300da75 | 11 | } |
39234879 | 12 | chdir 't'; |
0300da75 | 13 | |
f6d6199c MS |
14 | use strict; |
15 | ||
0300da75 | 16 | # these files help the test run |
2c91f887 | 17 | use Test::More tests => 41; |
0300da75 MS |
18 | use Cwd; |
19 | ||
20 | # these files are needed for the module itself | |
21 | use File::Spec; | |
22 | use File::Path; | |
57b1a898 MS |
23 | |
24 | # We're going to be chdir'ing and modules are sometimes loaded on the | |
25 | # fly in this test, so we need an absolute @INC. | |
26 | @INC = map { File::Spec->rel2abs($_) } @INC; | |
0300da75 MS |
27 | |
28 | # keep track of everything added so it can all be deleted | |
2530b651 | 29 | my %Files; |
0300da75 | 30 | sub add_file { |
479d2113 MS |
31 | my ($file, $data) = @_; |
32 | $data ||= 'foo'; | |
2530b651 | 33 | 1 while unlink $file; # or else we'll get multiple versions on VMS |
479d2113 MS |
34 | open( T, '>'.$file) or return; |
35 | print T $data; | |
2530b651 | 36 | ++$Files{$file}; |
57b1a898 | 37 | close T; |
0300da75 MS |
38 | } |
39 | ||
40 | sub read_manifest { | |
57b1a898 MS |
41 | open( M, 'MANIFEST' ) or return; |
42 | chomp( my @files = <M> ); | |
43 | close M; | |
0300da75 MS |
44 | return @files; |
45 | } | |
46 | ||
47 | sub catch_warning { | |
48 | my $warn; | |
49 | local $SIG{__WARN__} = sub { $warn .= $_[0] }; | |
50 | return join('', $_[0]->() ), $warn; | |
51 | } | |
52 | ||
53 | sub remove_dir { | |
54 | ok( rmdir( $_ ), "remove $_ directory" ) for @_; | |
55 | } | |
56 | ||
57 | # use module, import functions | |
f6d6199c MS |
58 | BEGIN { |
59 | use_ok( 'ExtUtils::Manifest', | |
60 | qw( mkmanifest manicheck filecheck fullcheck | |
479d2113 | 61 | maniread manicopy skipcheck maniadd) ); |
f6d6199c | 62 | } |
0300da75 MS |
63 | |
64 | my $cwd = Cwd::getcwd(); | |
65 | ||
66 | # Just in case any old files were lying around. | |
67 | rmtree('mantest'); | |
68 | ||
69 | ok( mkdir( 'mantest', 0777 ), 'make mantest directory' ); | |
70 | ok( chdir( 'mantest' ), 'chdir() to mantest' ); | |
71 | ok( add_file('foo'), 'add a temporary file' ); | |
72 | ||
73 | # there shouldn't be a MANIFEST there | |
74 | my ($res, $warn) = catch_warning( \&mkmanifest ); | |
f2e6bef3 | 75 | # Canonize the order. |
f6d6199c MS |
76 | $warn = join("", map { "$_|" } |
77 | sort { lc($a) cmp lc($b) } split /\r?\n/, $warn); | |
f2e6bef3 | 78 | is( $warn, "Added to MANIFEST: foo|Added to MANIFEST: MANIFEST|", |
f6d6199c | 79 | "mkmanifest() displayed its additions" ); |
0300da75 MS |
80 | |
81 | # and now you see it | |
82 | ok( -e 'MANIFEST', 'create MANIFEST file' ); | |
83 | ||
84 | my @list = read_manifest(); | |
85 | is( @list, 2, 'check files in MANIFEST' ); | |
86 | ok( ! ExtUtils::Manifest::filecheck(), 'no additional files in directory' ); | |
87 | ||
88 | # after adding bar, the MANIFEST is out of date | |
89 | ok( add_file( 'bar' ), 'add another file' ); | |
90 | ok( ! manicheck(), 'MANIFEST now out of sync' ); | |
91 | ||
92 | # it reports that bar has been added and throws a warning | |
93 | ($res, $warn) = catch_warning( \&filecheck ); | |
94 | ||
95 | like( $warn, qr/^Not in MANIFEST: bar/, 'warning that bar has been added' ); | |
96 | is( $res, 'bar', 'bar reported as new' ); | |
97 | ||
98 | # now quiet the warning that bar was added and test again | |
f6d6199c MS |
99 | ($res, $warn) = do { local $ExtUtils::Manifest::Quiet = 1; |
100 | catch_warning( \&skipcheck ) | |
101 | }; | |
102 | cmp_ok( $warn, 'eq', '', 'disabled warnings' ); | |
0300da75 | 103 | |
f6d6199c | 104 | # add a skip file with a rule to skip itself (and the nonexistent glob '*baz*') |
0300da75 MS |
105 | add_file( 'MANIFEST.SKIP', "baz\n.SKIP" ); |
106 | ||
107 | # this'll skip the new file | |
f6d6199c MS |
108 | ($res, $warn) = catch_warning( \&skipcheck ); |
109 | like( $warn, qr/^Skipping MANIFEST\.SKIP/i, 'got skipping warning' ); | |
0300da75 | 110 | |
45bc4d3a | 111 | my @skipped; |
0300da75 | 112 | catch_warning( sub { |
45bc4d3a | 113 | @skipped = skipcheck() |
0300da75 MS |
114 | }); |
115 | ||
45bc4d3a | 116 | is( join( ' ', @skipped ), 'MANIFEST.SKIP', 'listed skipped files' ); |
0300da75 | 117 | |
f6d6199c MS |
118 | { |
119 | local $ExtUtils::Manifest::Quiet = 1; | |
120 | is( join(' ', filecheck() ), 'bar', 'listing skipped with filecheck()' ); | |
121 | } | |
0300da75 MS |
122 | |
123 | # add a subdirectory and a file there that should be found | |
124 | ok( mkdir( 'moretest', 0777 ), 'created moretest directory' ); | |
f6d6199c MS |
125 | add_file( File::Spec->catfile('moretest', 'quux'), 'quux' ); |
126 | ok( exists( ExtUtils::Manifest::manifind()->{'moretest/quux'} ), | |
127 | "manifind found moretest/quux" ); | |
0300da75 MS |
128 | |
129 | # only MANIFEST and foo are in the manifest | |
2530b651 | 130 | $_ = 'foo'; |
0300da75 MS |
131 | my $files = maniread(); |
132 | is( keys %$files, 2, 'two files found' ); | |
f6d6199c MS |
133 | is( join(' ', sort { lc($a) cmp lc($b) } keys %$files), 'foo MANIFEST', |
134 | 'both files found' ); | |
2530b651 | 135 | is( $_, 'foo', q{maniread() doesn't clobber $_} ); |
0300da75 MS |
136 | |
137 | # poison the manifest, and add a comment that should be reported | |
138 | add_file( 'MANIFEST', 'none #none' ); | |
f6d6199c MS |
139 | is( ExtUtils::Manifest::maniread()->{none}, '#none', |
140 | 'maniread found comment' ); | |
0300da75 MS |
141 | |
142 | ok( mkdir( 'copy', 0777 ), 'made copy directory' ); | |
143 | ||
144 | $files = maniread(); | |
145 | eval { (undef, $warn) = catch_warning( sub { | |
57b1a898 | 146 | manicopy( $files, 'copy', 'cp' ) }) |
0300da75 | 147 | }; |
57b1a898 | 148 | like( $@, qr/^Can't read none: /, 'croaked about none' ); |
0300da75 MS |
149 | |
150 | # a newline comes through, so get rid of it | |
151 | chomp($warn); | |
152 | ||
153 | # the copy should have given one warning and one error | |
f6d6199c | 154 | like($warn, qr/^Skipping MANIFEST.SKIP/i, 'warned about MANIFEST.SKIP' ); |
0300da75 MS |
155 | |
156 | # tell ExtUtils::Manifest to use a different file | |
f6d6199c MS |
157 | { |
158 | local $ExtUtils::Manifest::MANIFEST = 'albatross'; | |
159 | ($res, $warn) = catch_warning( \&mkmanifest ); | |
160 | like( $warn, qr/Added to albatross: /, 'using a new manifest file' ); | |
161 | ||
162 | # add the new file to the list of files to be deleted | |
2530b651 | 163 | $Files{'albatross'}++; |
39234879 | 164 | } |
0300da75 | 165 | |
0300da75 | 166 | |
f6d6199c MS |
167 | # Make sure MANIFEST.SKIP is using complete relative paths |
168 | add_file( 'MANIFEST.SKIP' => "^moretest/q\n" ); | |
169 | ||
170 | # This'll skip moretest/quux | |
171 | ($res, $warn) = catch_warning( \&skipcheck ); | |
45bc4d3a JH |
172 | like( $warn, qr{^Skipping moretest/quux$}i, 'got skipping warning again' ); |
173 | ||
174 | ||
175 | # There was a bug where entries in MANIFEST would be blotted out | |
176 | # by MANIFEST.SKIP rules. | |
177 | add_file( 'MANIFEST.SKIP' => 'foo' ); | |
479d2113 | 178 | add_file( 'MANIFEST' => "foobar\n" ); |
45bc4d3a JH |
179 | add_file( 'foobar' => '123' ); |
180 | ($res, $warn) = catch_warning( \&manicheck ); | |
181 | is( $res, '', 'MANIFEST overrides MANIFEST.SKIP' ); | |
182 | is( $warn, undef, 'MANIFEST overrides MANIFEST.SKIP, no warnings' ); | |
f6d6199c | 183 | |
479d2113 MS |
184 | $files = maniread; |
185 | ok( !$files->{wibble}, 'MANIFEST in good state' ); | |
186 | maniadd({ wibble => undef }); | |
187 | maniadd({ yarrow => "hock" }); | |
188 | $files = maniread; | |
189 | is( $files->{wibble}, '', 'maniadd() with undef comment' ); | |
190 | is( $files->{yarrow}, 'hock',' with comment' ); | |
191 | is( $files->{foobar}, '', ' preserved old entries' ); | |
5ca25ae7 | 192 | |
2530b651 | 193 | add_file('MANIFEST' => 'Makefile.PL'); |
9d058bf8 | 194 | maniadd({ foo => 'bar' }); |
2530b651 MS |
195 | $files = maniread; |
196 | # VMS downcases the MANIFEST. We normalize it here to match. | |
197 | %$files = map { (lc $_ => $files->{$_}) } keys %$files; | |
198 | my %expect = ( 'makefile.pl' => '', | |
5ca25ae7 JH |
199 | 'foo' => 'bar' |
200 | ); | |
2530b651 | 201 | is_deeply( $files, \%expect, 'maniadd() vs MANIFEST without trailing newline'); |
0300da75 | 202 | |
5ca25ae7 JH |
203 | add_file('MANIFEST' => 'Makefile.PL'); |
204 | maniadd({ foo => 'bar' }); | |
205 | ||
2c91f887 JH |
206 | SKIP: { |
207 | chmod( 0400, 'MANIFEST' ); | |
208 | skip "Can't make MANIFEST read-only", 2 if -w 'MANIFEST'; | |
209 | ||
380d5532 MS |
210 | eval { |
211 | maniadd({ 'foo' => 'bar' }); | |
212 | }; | |
2c91f887 JH |
213 | is( $@, '', "maniadd() won't open MANIFEST if it doesn't need to" ); |
214 | ||
215 | eval { | |
216 | maniadd({ 'grrrwoof' => 'yippie' }); | |
217 | }; | |
30361541 | 218 | like( $@, qr/^\Qmaniadd() could not open MANIFEST:\E/, |
2c91f887 JH |
219 | "maniadd() dies if it can't open the MANIFEST" ); |
220 | ||
0aa703b2 | 221 | chmod( 0600, 'MANIFEST' ); |
2c91f887 JH |
222 | } |
223 | ||
224 | ||
0300da75 | 225 | END { |
2530b651 | 226 | is( unlink( keys %Files ), keys %Files, 'remove all added files' ); |
0300da75 MS |
227 | remove_dir( 'moretest', 'copy' ); |
228 | ||
229 | # now get rid of the parent directory | |
230 | ok( chdir( $cwd ), 'return to parent directory' ); | |
231 | remove_dir( 'mantest' ); | |
232 | } | |
349e1be1 | 233 |