Test::Requires must come before Test::NoWarnings

[gitmo/MooseX-Getopt.git] / t / 112_configfile_constructor_arg.t

diff --git a/t/112_configfile_constructor_arg.t b/t/112_configfile_constructor_arg.t
index 7b55a83..68455b9 100644 (file)
--- a/t/112_configfile_constructor_arg.t
+++ b/t/112_configfile_constructor_arg.t
@@ -1,12 +1,18 @@
 use strict;
 use warnings;
 
-use Test::Requires
-    'MooseX::SimpleConfig'; # skip all if not installed
-
 # respect the configfile value passed into the constructor.
 
-use Test::More tests => 2;
+use Test::Requires 'MooseX::SimpleConfig';  # skip all if not installed
+use Test::More tests => 3;
+use Test::NoWarnings 1.04 ':early';
+use Path::Class;    # exports file, dir
+
+# avoid warning if all we have installed is YAML or YAML::Syck - the user will
+# see this eventually when he actually uses MooseX::SimpleConfig in his own
+# code
+use Config::Any::YAML;
+$Config::Any::YAML::NO_YAML_XS_WARNING = 1;
 
 {
     package Foo;
@@ -19,11 +25,8 @@ use Test::More tests => 2;
     );
 }
 
-TODO:
 {
-    local $TODO = "doh, this doesn't work!";
-
-    my $configfile = 't/112_configfile_constructor_arg.yml';
+    my $configfile = file(qw(t 112_configfile_constructor_arg.yml))->stringify;
 
     my $obj = Foo->new_with_options(configfile => $configfile);