[Bps-public-commit] r16737 - in sd/branches/init-and-clone: .

[sartak at bestpractical.com]

Author: sartak
Date: Fri Nov  7 12:40:16 2008
New Revision: 16737

Modified:
   sd/branches/init-and-clone/   (props changed)
   sd/branches/init-and-clone/t/05-config-file-loading.t

Log:
 r75161 at onn:  sartak | 2008-11-07 12:39:29 -0500
 Delete instead of setting to undef to avoid undef warnings


Modified: sd/branches/init-and-clone/t/05-config-file-loading.t
==============================================================================
--- sd/branches/init-and-clone/t/05-config-file-loading.t	(original)
+++ sd/branches/init-and-clone/t/05-config-file-loading.t	Fri Nov  7 12:40:16 2008
@@ -15,7 +15,7 @@
     $ENV{'PROPHET_REPO'} = $ENV{'SD_REPO'} = $ENV{'HOME'} = File::Temp::tempdir( CLEANUP => 0 ) . '/_svb';
     diag "export SD_REPO=".$ENV{'PROPHET_REPO'} ."\n";
     diag "export HOME=".$ENV{'PROPHET_REPO'} ."\n";
-    $ENV{'PROPHET_APP_CONFIG'} = undef; # clear this because Prophet::Test sets it
+    delete $ENV{'PROPHET_APP_CONFIG'}; # clear this because Prophet::Test sets it
 }
 
 # Tests the config file order of preference laid out in App::SD::Config
@@ -34,7 +34,10 @@
     [ qr/(\d+) YATTA new/]
 );
 
-$ENV{'SD_CONFIG'} = $ENV{'PROPHET_APP_CONFIG'} = undef;  # override App::SD::Test
+# override App::SD::Test
+delete $ENV{'SD_CONFIG'};
+delete $ENV{'PROPHET_APP_CONFIG'};
+
 ok( ! $ENV{'SD_CONFIG'}, "SD_CONFIG env var has been cleared" );
 ok( ! $ENV{'PROPHET_APP_CONFIG'}, "PROPHET_APP_CONFIG env var has been cleared" );