test warning on directory creation

[p5sagit/local-lib.git] / t / stackable.t

diff --git a/t/stackable.t b/t/stackable.t
index 6029a3a..1922245 100644 (file)
--- a/t/stackable.t
+++ b/t/stackable.t
@@ -1,38 +1,34 @@
 use strict;
 use warnings;
 use Test::More;
-use File::Temp qw(tempdir);
-use Cwd;
+use File::Spec;
+use Config;
 
-plan tests => 19;
+plan tests => 24;
 
 use local::lib ();
 
-sub mk_temp_dir
-{
-    my $name_template = shift;
-
-    my $path = tempdir($name_template, DIR => Cwd::abs_path('t'), CLEANUP => 1);
-    local::lib->ensure_dir_structure_for($path);
-    # On Win32 the path where the distribution is built usually contains
-    # spaces. This is a problem for some parts of the CPAN toolchain, so
-    # local::lib uses the GetShortPathName trick do get an alternate
-    # representation of the path that doesn't constain spaces.
-    return ($^O eq 'MSWin32')
-         ? Win32::GetShortPathName($path)
-        : $path
-}
+use lib 't/lib'; use TempDir;
+
+delete $ENV{PERL_LOCAL_LIB_ROOT};
 
 my $dir1 = mk_temp_dir('test_local_lib-XXXXX');
 my $dir2 = mk_temp_dir('test_local_lib-XXXXX');
 
+my ($dir1_arch, $dir2_arch) = map { File::Spec->catfile($_, qw'lib perl5', $Config{archname}) } $dir1, $dir2;
+note $dir1_arch;
+note $dir2_arch;
+
 my $prev_active = () = local::lib->active_paths;
 
 local::lib->import($dir1);
 is +() = local::lib->active_paths, $prev_active + 1, 'one active path';
 like $ENV{PERL_LOCAL_LIB_ROOT}, qr/\Q$dir1/, 'added one dir in root';
 like $ENV{PERL5LIB}, qr/\Q$dir1/, 'added one dir in lib';
-like $ENV{PERL_MM_OPT}, qr/\Q$dir1/, 'first path is installation target';
+note $ENV{PERL5LIB};
+unlike $ENV{PERL5LIB}, qr/\Q$dir1_arch/, 'no arch in PERL5LIB';
+my $dir1_escape = local::lib::_mm_escape_path($dir1);
+like $ENV{PERL_MM_OPT}, qr/\Q$dir1_escape/, 'first path is installation target';
 
 local::lib->import($dir1);
 is +() = local::lib->active_paths, $prev_active + 1, 'still one active path after adding it twice';
@@ -41,19 +37,24 @@ local::lib->import($dir2);
 is +() = local::lib->active_paths, $prev_active + 2, 'two active paths';
 like $ENV{PERL_LOCAL_LIB_ROOT}, qr/\Q$dir2/, 'added another dir in root';
 like $ENV{PERL5LIB}, qr/\Q$dir2/, 'added another dir in lib';
+unlike $ENV{PERL5LIB}, qr/\Q$dir2_arch/, 'no arch in PERL5LIB';
 like $ENV{PERL_LOCAL_LIB_ROOT}, qr/\Q$dir1/, 'first dir is still in root';
 like $ENV{PERL5LIB}, qr/\Q$dir1/, 'first dir is still in lib';
-like $ENV{PERL_MM_OPT}, qr/\Q$dir2/, 'second path is installation target';
+unlike $ENV{PERL5LIB}, qr/\Q$dir1_arch/, 'no arch in PERL5LIB';
+my $dir2_escape = local::lib::_mm_escape_path($dir2);
+like $ENV{PERL_MM_OPT}, qr/\Q$dir2_escape/, 'second path is installation target';
 
 local::lib->import($dir1);
 my @active = local::lib->active_paths;
 is @active, $prev_active + 2, 'still two active dirs after re-adding first';
-is $active[-1], $dir1, 'first dir was re-added on top';
-like $ENV{PERL_MM_OPT}, qr/\Q$dir1/, 'first path is installation target again';
+is $active[0], $dir1, 'first dir was re-added on top';
+like $ENV{PERL_MM_OPT}, qr/\Q$dir1_escape/, 'first path is installation target again';
 
 local::lib->import('--deactivate', $dir2);
 unlike $ENV{PERL_LOCAL_LIB_ROOT}, qr/\Q$dir2/, 'second dir was removed from root';
 unlike $ENV{PERL5LIB}, qr/\Q$dir2/, 'second dir was removed from lib';
+unlike $ENV{PERL5LIB}, qr/\Q$dir2_arch/, 'no arch in PERL5LIB';
 like $ENV{PERL_LOCAL_LIB_ROOT}, qr/\Q$dir1/, q{first dir didn't go away from root};
 like $ENV{PERL5LIB}, qr/\Q$dir1/, q{first dir didn't go away from lib};
-like $ENV{PERL_MM_OPT}, qr/\Q$dir1/, 'first dir stays installation target';
+unlike $ENV{PERL5LIB}, qr/\Q$dir1_arch/, 'no arch in PERL5LIB';
+like $ENV{PERL_MM_OPT}, qr/\Q$dir1_escape/, 'first dir stays installation target';