Commit | Line | Data |
0300da75 |
1 | #!./perl |
2 | |
3 | BEGIN { |
4 | chdir 't' if -d 't'; |
5 | unshift @INC, '../lib'; |
6 | } |
7 | |
8 | # these files help the test run |
9 | use Test::More tests => 31; |
10 | use Cwd; |
11 | |
12 | # these files are needed for the module itself |
13 | use File::Spec; |
14 | use File::Path; |
15 | use Carp::Heavy; |
16 | |
17 | # keep track of everything added so it can all be deleted |
18 | my @files; |
19 | sub add_file { |
20 | my ($file, $data) = @_; |
21 | $data ||= 'foo'; |
020b036f |
22 | open( my $T, '>', $file) or return; |
0300da75 |
23 | print $T $data; |
24 | push @files, $file; |
25 | } |
26 | |
27 | sub read_manifest { |
28 | open( my $M, 'MANIFEST' ) or return; |
29 | chomp( my @files = <$M> ); |
30 | return @files; |
31 | } |
32 | |
33 | sub catch_warning { |
34 | my $warn; |
35 | local $SIG{__WARN__} = sub { $warn .= $_[0] }; |
36 | return join('', $_[0]->() ), $warn; |
37 | } |
38 | |
39 | sub remove_dir { |
40 | ok( rmdir( $_ ), "remove $_ directory" ) for @_; |
41 | } |
42 | |
43 | # use module, import functions |
44 | use_ok( 'ExtUtils::Manifest', |
45 | qw( mkmanifest manicheck filecheck fullcheck maniread manicopy) ); |
46 | |
47 | my $cwd = Cwd::getcwd(); |
48 | |
49 | # Just in case any old files were lying around. |
50 | rmtree('mantest'); |
51 | |
52 | ok( mkdir( 'mantest', 0777 ), 'make mantest directory' ); |
53 | ok( chdir( 'mantest' ), 'chdir() to mantest' ); |
54 | ok( add_file('foo'), 'add a temporary file' ); |
55 | |
56 | # there shouldn't be a MANIFEST there |
57 | my ($res, $warn) = catch_warning( \&mkmanifest ); |
58 | is( $warn, <<ADDING, "mkmanifest() displayed it's additions" ); |
59 | Added to MANIFEST: MANIFEST |
60 | Added to MANIFEST: foo |
61 | ADDING |
62 | |
63 | # and now you see it |
64 | ok( -e 'MANIFEST', 'create MANIFEST file' ); |
65 | |
66 | my @list = read_manifest(); |
67 | is( @list, 2, 'check files in MANIFEST' ); |
68 | ok( ! ExtUtils::Manifest::filecheck(), 'no additional files in directory' ); |
69 | |
70 | # after adding bar, the MANIFEST is out of date |
71 | ok( add_file( 'bar' ), 'add another file' ); |
72 | ok( ! manicheck(), 'MANIFEST now out of sync' ); |
73 | |
74 | # it reports that bar has been added and throws a warning |
75 | ($res, $warn) = catch_warning( \&filecheck ); |
76 | |
77 | like( $warn, qr/^Not in MANIFEST: bar/, 'warning that bar has been added' ); |
78 | is( $res, 'bar', 'bar reported as new' ); |
79 | |
80 | # now quiet the warning that bar was added and test again |
81 | use vars qw($ExtUtils::Manifest::Quiet); |
82 | $ExtUtils::Manifest::Quiet = 1; |
83 | ($res, $warn) = catch_warning( \&ExtUtils::Manifest::skipcheck ); |
84 | is( $warn, '', 'disabled warnings' ); |
85 | |
86 | # add a skip file with a rule to skip itself |
87 | add_file( 'MANIFEST.SKIP', "baz\n.SKIP" ); |
88 | |
89 | # this'll skip the new file |
90 | ($res, $warn) = catch_warning( \&ExtUtils::Manifest::skipcheck ); |
91 | like( $warn, qr/^Skipping MANIFEST\.SKIP/, 'got skipping warning' ); |
92 | |
93 | # I'm not sure why this should be... shouldn't $missing be the only one? |
94 | my ($found, $missing ); |
95 | catch_warning( sub { |
96 | ( $found, $missing ) = ExtUtils::Manifest::skipcheck() |
97 | }); |
98 | |
99 | # nothing new should be found, bar should be skipped |
100 | is( @$found, 0, 'no output here' ); |
101 | is( join( ' ', @$missing ), 'bar', 'listed skipped files' ); |
102 | |
103 | is( join(' ', filecheck() ), 'bar', 'listing skipped with filecheck()' ); |
104 | |
105 | # add a subdirectory and a file there that should be found |
106 | ok( mkdir( 'moretest', 0777 ), 'created moretest directory' ); |
107 | my $quux = File::Spec->catfile( 'moretest', 'quux' ); |
8dee7ff3 |
108 | $quux =~ s#\\#/#g; |
0300da75 |
109 | add_file( $quux, 'quux' ); |
110 | ok( exists( ExtUtils::Manifest::manifind()->{$quux} ), "manifind found $quux" ); |
111 | |
112 | # only MANIFEST and foo are in the manifest |
113 | my $files = maniread(); |
114 | is( keys %$files, 2, 'two files found' ); |
115 | is( join(' ', sort keys %$files), 'MANIFEST foo', 'both files found' ); |
116 | |
117 | # poison the manifest, and add a comment that should be reported |
118 | add_file( 'MANIFEST', 'none #none' ); |
119 | is( ExtUtils::Manifest::maniread()->{none}, '#none', 'maniread found comment' ); |
120 | |
121 | ok( mkdir( 'copy', 0777 ), 'made copy directory' ); |
122 | |
123 | $files = maniread(); |
124 | eval { (undef, $warn) = catch_warning( sub { |
125 | manicopy( $files, 'copy', 'cp' ) }) |
126 | }; |
127 | |
128 | # a newline comes through, so get rid of it |
129 | chomp($warn); |
130 | |
131 | # the copy should have given one warning and one error |
132 | is($warn, 'Skipping MANIFEST.SKIP', 'warned about MANIFEST.SKIP' ); |
86e8f854 |
133 | like( $@, qr/^Can't read none: /, |
0300da75 |
134 | 'carped about none' ); |
135 | |
136 | # tell ExtUtils::Manifest to use a different file |
137 | use vars qw($ExtUtils::Manifest::MANIFEST); |
138 | $ExtUtils::Manifest::MANIFEST = 'albatross'; |
139 | |
140 | ($res, $warn) = catch_warning( \&mkmanifest ); |
141 | like( $warn, qr/Added to albatross: /, 'using a new manifest file' ); |
142 | |
143 | # add the new file to the list of files to be deleted |
144 | push @files, 'albatross'; |
145 | |
146 | END { |
147 | # the arrays are evaluated in scalar context |
148 | is( unlink( @files ), @files, 'remove all added files' ); |
149 | remove_dir( 'moretest', 'copy' ); |
150 | |
151 | # now get rid of the parent directory |
152 | ok( chdir( $cwd ), 'return to parent directory' ); |
153 | remove_dir( 'mantest' ); |
154 | } |