From: Graham Knop <haarg@haarg.org>
Date: Mon, 27 Feb 2017 10:10:27 +0000 (+0100)
Subject: check for load failure rather than empty result
X-Git-Tag: v0.29~4
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=0354f86ab5460df39f404093fd338f09b36aa54a;p=p5sagit%2FConfig-Any.git

check for load failure rather than empty result
---

diff --git a/t/53-perl.t b/t/53-perl.t
index 1a749a1..cd61a1f 100644
--- a/t/53-perl.t
+++ b/t/53-perl.t
@@ -1,7 +1,7 @@
 use strict;
 use warnings;
 
-use Test::More tests => 7;
+use Test::More tests => 9;
 use Config::Any;
 use Config::Any::Perl;
 
@@ -19,8 +19,13 @@ use Config::Any::Perl;
 # test invalid config
 {
     my $file = 't/invalid/conf.pl';
-    my $config = eval { Config::Any::Perl->load( $file ) };
+    my $config;
+    my $loaded = eval {
+        $config = Config::Any::Perl->load( $file );
+        1;
+    };
 
+    ok( !$loaded, 'config load failed' );
     ok( !$config, 'config load failed' );
     ok( $@,       "error thrown ($@)" );
 }
@@ -28,8 +33,14 @@ use Config::Any::Perl;
 # parse error generated on invalid config
 {
     my $file = 't/invalid/conf.pl';
-    my $config = eval { Config::Any->load_files( { files => [$file], use_ext => 1} ) };
-
+    my $config;
+    my $loaded = eval {
+        $config = Config::Any::Perl->load( $file );
+        Config::Any->load_files( { files => [$file], use_ext => 1} );
+        1;
+    };
+
+    ok( !$loaded, 'config load failed' );
     ok( !$config, 'config load failed' );
     ok( $@,       "error thrown ($@)" );
 }