Moose now warns when you try to load it from the main package. Added a
Dave Rolsky [Tue, 5 Aug 2008 03:04:22 +0000 (03:04 +0000)]
test for this (which requires Test::Output to run).

The tests had many "use_ok('Moose')" lines that started warning. The
use_ok tests weren't really testing anything, since the tests need the
modules they were trying to load in order to run. You actually get a
better error message by just using the module normally (if said module
fails to compile).

112 files changed:
Changes
lib/Moose.pm
t/000_load.t
t/000_recipes/basics/001_point.t
t/000_recipes/basics/002_bank_account.t
t/000_recipes/basics/003_binary_tree.t
t/000_recipes/basics/004_company.t
t/000_recipes/basics/005_coercion.t
t/000_recipes/basics/006_augment_inner.t
t/000_recipes/meta/002_meta_attribute.t
t/000_recipes/meta/003_attribute_trait.t
t/000_recipes/roles/001_roles.t
t/010_basics/001_basic_class_setup.t
t/010_basics/002_require_superclasses.t
t/010_basics/003_super_and_override.t
t/010_basics/004_inner_and_augment.t
t/010_basics/005_override_augment_inner_super.t
t/010_basics/006_override_and_foreign_classes.t
t/010_basics/008_wrapped_method_cxt_propagation.t
t/010_basics/009_import_unimport.t
t/010_basics/010_method_modifier_with_regexp.t
t/010_basics/011_moose_respects_type_constraints.t
t/010_basics/012_rebless.t
t/010_basics/013_create.t
t/010_basics/014_create_anon.t
t/010_basics/016_load_into_main.t [new file with mode: 0644]
t/020_attributes/001_attribute_reader_generation.t
t/020_attributes/002_attribute_writer_generation.t
t/020_attributes/003_attribute_accessor_generation.t
t/020_attributes/004_attribute_triggers.t
t/020_attributes/005_attribute_does.t
t/020_attributes/006_attribute_required.t
t/020_attributes/007_attribute_custom_metaclass.t
t/020_attributes/008_attribute_type_unions.t
t/020_attributes/009_attribute_inherited_slot_specs.t
t/020_attributes/010_attribute_delegation.t
t/020_attributes/012_misc_attribute_tests.t
t/020_attributes/013_attr_dereference_test.t
t/020_attributes/014_misc_attribute_coerce_lazy.t
t/020_attributes/015_attribute_traits.t
t/020_attributes/016_attribute_traits_registered.t
t/020_attributes/017_attribute_traits_n_meta.t
t/020_attributes/018_no_init_arg.t
t/020_attributes/019_attribute_lazy_initializer.t
t/020_attributes/020_trigger_and_coerce.t
t/020_attributes/021_method_generation_rules.t
t/020_attributes/022_legal_options_for_inheritance.t
t/030_roles/001_meta_role.t
t/030_roles/004_role_composition_errors.t
t/030_roles/005_role_conflict_detection.t
t/030_roles/006_role_exclusion.t
t/030_roles/007_roles_and_req_method_edge_cases.t
t/030_roles/008_role_conflict_edge_cases.t
t/030_roles/009_more_role_edge_cases.t
t/030_roles/010_run_time_role_composition.t
t/030_roles/011_overriding.t
t/030_roles/012_method_exclusion_in_composition.t
t/030_roles/013_method_aliasing_in_composition.t
t/030_roles/014_more_alias_and_exclude.t
t/030_roles/015_runtime_roles_and_attrs.t
t/030_roles/016_runtime_roles_and_nonmoose.t
t/030_roles/017_extending_role_attrs.t
t/030_roles/018_runtime_roles_w_params.t
t/030_roles/020_role_composite.t
t/030_roles/021_role_composite_exclusion.t
t/030_roles/022_role_composition_req_methods.t
t/030_roles/023_role_composition_attributes.t
t/030_roles/024_role_composition_methods.t
t/030_roles/025_role_composition_override.t
t/030_roles/026_role_composition_method_mods.t
t/030_roles/030_role_parameterized.t
t/040_type_constraints/006_util_type_reloading.t
t/040_type_constraints/007_util_more_type_coercion.t
t/040_type_constraints/009_union_types_and_coercions.t
t/040_type_constraints/021_maybe_type_constraint.t
t/040_type_constraints/023_types_and_undef.t
t/050_metaclasses/001_custom_attr_meta_with_roles.t
t/050_metaclasses/002_custom_attr_meta_as_role.t
t/050_metaclasses/003_moose_w_metaclass.t
t/050_metaclasses/004_moose_for_meta.t
t/050_metaclasses/010_extending_and_embedding.t
t/050_metaclasses/011_init_meta.t
t/060_compat/002_moose_respects_base.t
t/060_compat/003_foreign_inheritence.t
t/100_bugs/001_subtype_quote_bug.t
t/100_bugs/002_subtype_conflict_bug.t
t/100_bugs/004_subclass_use_base_bug.t
t/100_bugs/005_inline_reader_bug.t
t/100_bugs/007_reader_precedence_bug.t
t/100_bugs/008_new_w_undef.t
t/100_bugs/009_augment_recursion_bug.t
t/100_bugs/010_immutable_n_default_x2.t
t/100_bugs/011_DEMOLISH_eats_exceptions.t
t/100_bugs/012_DEMOLISH_eats_mini.t
t/100_bugs/016_inheriting_from_roles.t
t/100_bugs/017_type_constraint_messages.t
t/100_bugs/018_immutable_metaclass_does_role.t
t/200_examples/001_example.t
t/200_examples/002_example_Moose_POOP.t
t/200_examples/003_example.t
t/200_examples/004_example_w_DCS.t
t/200_examples/005_example_w_TestDeep.t
t/200_examples/008_record_set_iterator.t
t/300_immutable/001_immutable_moose.t
t/300_immutable/002_apply_roles_to_immutable.t
t/300_immutable/003_immutable_meta_class.t
t/300_immutable/004_inlined_constructors_n_types.t
t/300_immutable/005_multiple_demolish_inline.t
t/300_immutable/006_immutable_nonmoose_subclass.t
t/300_immutable/007_immutable_trigger_from_constructor.t
t/300_immutable/008_immutable_constructor_error.t
t/600_todo_tests/005_moose_and_threads.t

diff --git a/Changes b/Changes
index 175298b..d28929a 100644 (file)
--- a/Changes
+++ b/Changes
@@ -6,6 +6,11 @@ Revision history for Perl extension Moose
       - Add a remove_keywords function so if you extend Moose
         you don't have to cargo cult Moose's unimport (Sartak)
 
+    * Moose
+      - Moose now warns when you try to have it export its sugar
+        functions into the "main" package. Previously it silently did
+        nothing. (Dave Rolsky)
+
 0.55 Sun August 3, 2008
     * Moose::Meta::Attribute
       - breaking down the way 'handles' methods are
index 9551c07..21ef374 100644 (file)
@@ -167,7 +167,10 @@ use Moose::Util ();
         warnings->import;
 
         # we should never export to main
-        return if $CALLER eq 'main';
+        if ($CALLER eq 'main') {
+            warn qq{Moose does not export its sugar to the 'main' package.\n};
+            return;
+        }
 
         init_meta( $CALLER, 'Moose::Object' );
 
index 0c59d05..697464c 100644 (file)
@@ -5,6 +5,10 @@ use warnings;
 
 use Test::More tests => 1;
 
-BEGIN {
-    use_ok('Moose');           
-}
+package Foo;
+
+# Moose will issue a warning if we try to load it from the main
+# package.
+::use_ok('Moose');
+
+
index 771a795..46dbf9c 100644 (file)
@@ -3,13 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 58;
+use Test::More tests => 57;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
-
 {
        package Point;  
        use Moose;
index 4e0b571..36c677b 100644 (file)
@@ -3,13 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 24;
+use Test::More tests => 23;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
-
 {
     package BankAccount;
     use Moose;
index cc7afbd..0f85f95 100644 (file)
@@ -3,15 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 34;
+use Test::More tests => 33;
 use Test::Exception;
 
 use Scalar::Util 'isweak';
 
-BEGIN {
-    use_ok('Moose');           
-}
-
 {
     package BinaryTree;
     use Moose;
index 44bfa13..cc53a71 100644 (file)
@@ -8,15 +8,13 @@ use Test::More;
 BEGIN {
     eval "use Regexp::Common; use Locale::US;";
     plan skip_all => "Regexp::Common & Locale::US required for this test" if $@;        
-    plan tests => 66;    
+    plan tests => 65;    
 }
 
 use Test::Exception;
 use Scalar::Util 'isweak';
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Address;
index 7ea0371..773742b 100644 (file)
@@ -8,15 +8,11 @@ use Test::More;
 BEGIN {
     eval "use HTTP::Headers; use Params::Coerce; use URI;";
     plan skip_all => "HTTP::Headers & Params::Coerce & URI required for this test" if $@;        
-    plan tests => 18;    
+    plan tests => 17;    
 }
 
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
-
 {
        package Request;
        use Moose;
index 53d2b75..04f2f1d 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 3;
+use Test::More tests => 2;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 ## Augment/Inner
 
index 7e5abcf..efe675c 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 2;
+use Test::More tests => 1;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 ## meta-attribute example
 {
index 223fb65..7b8157f 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 3;
+use Test::More tests => 2;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 ## attribute trait example
 {
index fb6f834..6f89b83 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 64;
+use Test::More tests => 63;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 ## Roles
 
index 7647693..0520394 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 23;
+use Test::More tests => 22;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index ced023f..28b52e0 100644 (file)
@@ -5,11 +5,9 @@ use warnings;
 
 use lib 't/lib', 'lib';
 
-use Test::More tests => 6;
+use Test::More tests => 5;
+
 
-BEGIN {
-    use_ok('Moose');           
-}
 
 {
     package Bar;
index e04eea5..54d5c2d 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 17;
+use Test::More tests => 16;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index f129ef2..15f4248 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 17;
+use Test::More tests => 16;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index 6732c60..2d4a330 100644 (file)
@@ -3,11 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 6;
+use Test::More tests => 5;
+
 
-BEGIN {
-    use_ok('Moose');           
-}
 
 {
     package Foo;
index 812002e..26fbb44 100644 (file)
@@ -3,11 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 16;
+use Test::More tests => 15;
+
 
-BEGIN {
-    use_ok('Moose');           
-}
 
 =pod
 
index 8e97669..226cf54 100644 (file)
@@ -3,11 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 8;
+use Test::More tests => 7;
+
 
-BEGIN {
-    use_ok('Moose');
-}
 
 {
     package TouchyBase;
index 9bc2c76..7e58c7c 100644 (file)
@@ -3,11 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 47;
+use Test::More tests => 46;
+
 
-BEGIN {
-    use_ok('Moose');           
-}
 
 my @moose_exports = qw(
     extends with 
index c57030b..41f55cd 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 10;
+use Test::More tests => 9;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
 
index ead0d1a..a62f147 100644 (file)
@@ -3,13 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 9;
+use Test::More tests => 7;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-    use_ok('Moose::Util::TypeConstraints');               
-}
+use Moose::Util::TypeConstraints;
 
 =pod
 
index fcc5dd6..c8ec23e 100644 (file)
@@ -3,14 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 13;
+use Test::More tests => 11;
 use Test::Exception;
 use Scalar::Util 'blessed';
 
-BEGIN {
-    use_ok('Moose');
-    use_ok("Moose::Util::TypeConstraints");
-}
+use Moose::Util::TypeConstraints;
 
 subtype 'Positive'
      => as 'Num'
index 8fbafe7..e83972d 100644 (file)
@@ -3,15 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 10;
+use Test::More tests => 7;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose::Meta::Class');
-    use_ok('Moose');
-    use_ok('Moose::Role');
-}
-
 { 
     package Class;
     use Moose;
index d989455..d324b4b 100644 (file)
@@ -3,13 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 11;
+use Test::More tests => 8;
 
-BEGIN {
-    use_ok('Moose::Meta::Class');
-    use_ok('Moose');
-    use_ok('Moose::Role');
-}
+use Moose::Meta::Class;
 
 { 
     package Class;
diff --git a/t/010_basics/016_load_into_main.t b/t/010_basics/016_load_into_main.t
new file mode 100644 (file)
index 0000000..c8cc9fd
--- /dev/null
@@ -0,0 +1,16 @@
+#!/usr/bin/perl
+
+use strict;
+use warnings;
+
+use Test::More;
+
+BEGIN {
+    eval "use Test::Output;";
+    plan skip_all => "Test::Output is required for this test" if $@;
+    plan tests => 1;
+}
+
+stderr_is( sub { package main; eval 'use Moose' },
+           "Moose does not export its sugar to the 'main' package.\n",
+           'Moose warns when loaded from the main package' );
index 7ad1447..d27dbfc 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 14;
+use Test::More tests => 13;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index e803775..1b25f3c 100644 (file)
@@ -3,14 +3,12 @@
 use strict;
 use warnings;
 
-use Test::More tests => 30;
+use Test::More tests => 29;
 use Test::Exception;
 
 use Scalar::Util 'isweak';
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index d8b3638..0355c2b 100644 (file)
@@ -3,14 +3,12 @@
 use strict;
 use warnings;
 
-use Test::More tests => 58;
+use Test::More tests => 57;
 use Test::Exception;
 
 use Scalar::Util 'isweak';
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index b5cf34e..0695dc3 100644 (file)
@@ -5,12 +5,10 @@ use warnings;
 
 use Scalar::Util 'isweak';
 
-use Test::More tests => 26;
+use Test::More tests => 25;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index 8161537..d05a808 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 10;
+use Test::More tests => 9;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo::Role;
index 4f65021..0975765 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 16;
+use Test::More tests => 15;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package Foo;
index a7b8bdb..eb74b0c 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 17;
+use Test::More tests => 16;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {    
     package Foo::Meta::Attribute;
index 05b94eb..95ce5f9 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 19;
+use Test::More tests => 18;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index 7f3b859..7d84bfc 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 83;
+use Test::More tests => 82;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');  
-}
+
 
 {
     package Thing;
index 2e6e585..91bcf17 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 85;
+use Test::More tests => 84;
 use Test::Exception;
 
-BEGIN {  
-    use_ok('Moose');               
-}
+
 
 # -------------------------------------------------------------------
 # HASH handles
index 24e3f70..a8f4bc0 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 44;
+use Test::More tests => 43;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     {
index 903134b..11b75d7 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 12;
+use Test::More tests => 11;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package Customer;
index 37e354d..4fe4ee4 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 3;
+use Test::More tests => 2;
 use Test::Exception;
 
-BEGIN {
-       use_ok('Moose');
-}
+
 
 {
     package HTTPHeader;
index 0cb44fd..189d212 100644 (file)
@@ -3,13 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 13;
+use Test::More tests => 12;
 use Test::Exception;
 use Test::Moose;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Attribute::Trait;
index 01b9536..91dc88a 100644 (file)
@@ -3,13 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 24;
+use Test::More tests => 23;
 use Test::Exception;
 use Test::Moose;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Attribute::Trait;
index 6930dc1..c4dc206 100644 (file)
@@ -3,13 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 8;
+use Test::More tests => 7;
 use Test::Exception;
 use Test::Moose;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Meta::Attribute::DefaultReadOnly;
index ae714ec..f5ebf87 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 5;
+use Test::More tests => 4;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index 2c6f9ac..6735822 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 24;
+use Test::More tests => 23;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Foo;
index 712cacf..5d102ba 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 12;
+use Test::More tests => 11;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {   
     package Fake::DateTime;
index db96f52..d0fbe0b 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 18;
+use Test::More tests => 17;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 =pod
 
index a391465..e30cc97 100644 (file)
@@ -2,11 +2,9 @@
 
 use strict;
 use warnings;
-use Test::More tests => 3;
+use Test::More tests => 2;
+
 
-BEGIN {
-    use_ok('Moose');
-}
 
 {
     package Bar::Meta::Attribute;
index c13b336..f8b377b 100644 (file)
@@ -3,13 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 29;
+use Test::More tests => 27;
 use Test::Exception;
 
-BEGIN {  
-    use_ok('Moose'); 
-    use_ok('Moose::Meta::Role');               
-}
+use Moose::Meta::Role;
 
 {
     package FooRole;
index a96244c..0325dc4 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 11;
+use Test::More tests => 10;
 use Test::Exception;
 
-BEGIN {  
-    use_ok('Moose');               
-}
+
 
 {
     package Foo::Role;
index 0c4f9a8..09a664e 100644 (file)
@@ -3,14 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 89; # it's really 126 with kolibre's tests;
+use Test::More tests => 87; # it's really 124 with kolibre's tests;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Role');    
-}
-
 =pod
 
 Mutually recursive roles.
index 2f99164..578ba82 100644 (file)
@@ -3,14 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 24;
+use Test::More tests => 22;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Role');    
-}
-
 =pod
 
 The idea and examples for this feature are taken
index fa63489..9017f85 100644 (file)
@@ -3,7 +3,7 @@
 use strict;
 use warnings;
 
-use Test::More tests => 17;
+use Test::More tests => 15;
 use Test::Exception;
 
 =pod
@@ -16,15 +16,6 @@ are actually related to class construction order and
 not any real functionality.
 - SL
 
-=cut
-
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Role');    
-}
-
-=pod
-
 Role which requires a method implemented 
 in another role as an override (it does 
 not remove the requirement)
index 9d31847..12e4873 100644 (file)
@@ -3,14 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 34;
+use Test::More tests => 32;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Role');    
-}
-
 =pod
 
 Check for repeated inheritence causing 
index 41f7a35..0fec31c 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 75;
+use Test::More tests => 74;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     # NOTE:
index d4a5da9..1a86a18 100644 (file)
@@ -3,13 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 28;
+use Test::More tests => 27;
 
 use Scalar::Util qw(blessed);
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 =pod
 
index 1ebc3eb..1ee53e1 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 40;
+use Test::More tests => 39;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 { 
     # test no conflicts here
index 30f77f4..b3128b3 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 20;
+use Test::More tests => 19;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Role;
index 0405e13..4eeeed6 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 36;
+use Test::More tests => 35;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Role;
index 3fce48d..346c636 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 10;
+use Test::More tests => 9;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package Foo;
index 3cf2fb6..a2b3842 100644 (file)
@@ -3,13 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 12;
+use Test::More tests => 11;
 use Test::Exception;
 use Scalar::Util 'blessed';
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 
 {
index 056aef7..ecf1af0 100644 (file)
@@ -3,13 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 8;
+use Test::More tests => 7;
 use Test::Exception;
 use Scalar::Util 'blessed';
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 
 {
index 2cb8c11..95572a5 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 28;
+use Test::More tests => 27;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 =pod
 
index 44941ee..d3eddae 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 22;
+use Test::More tests => 21;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package Foo;
index ff70579..506968e 100644 (file)
@@ -3,14 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 17;
+use Test::More tests => 14;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role::Application::RoleSummation');
-    use_ok('Moose::Meta::Role::Composite');
-}
+use Moose::Meta::Role::Application::RoleSummation;
+use Moose::Meta::Role::Composite;
 
 {
     package Role::Foo;
index 4a1cad6..aa618f5 100644 (file)
@@ -3,14 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 15;
+use Test::More tests => 12;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role::Application::RoleSummation');    
-    use_ok('Moose::Meta::Role::Composite');
-}
+use Moose::Meta::Role::Application::RoleSummation;
+use Moose::Meta::Role::Composite;
 
 {
     package Role::Foo;
index 0d04cbc..be56016 100644 (file)
@@ -3,14 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 19;
+use Test::More tests => 16;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role::Application::RoleSummation');    
-    use_ok('Moose::Meta::Role::Composite');
-}
+use Moose::Meta::Role::Application::RoleSummation;
+use Moose::Meta::Role::Composite;
 
 {
     package Role::Foo;
index bc6da7e..b0e7ad2 100644 (file)
@@ -3,14 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 10;
+use Test::More tests => 7;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role::Application::RoleSummation');    
-    use_ok('Moose::Meta::Role::Composite');
-}
+use Moose::Meta::Role::Application::RoleSummation;
+use Moose::Meta::Role::Composite;
 
 {
     package Role::Foo;
index a77a4dc..36c3bff 100644 (file)
@@ -3,14 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 22;
+use Test::More tests => 19;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role::Application::RoleSummation');    
-    use_ok('Moose::Meta::Role::Composite');
-}
+use Moose::Meta::Role::Application::RoleSummation;
+use Moose::Meta::Role::Composite;
 
 {
     package Role::Foo;
index 9f38de3..33b61cd 100644 (file)
@@ -3,14 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 11;
+use Test::More tests => 8;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role::Application::RoleSummation');    
-    use_ok('Moose::Meta::Role::Composite');
-}
+use Moose::Meta::Role::Application::RoleSummation;
+use Moose::Meta::Role::Composite;
 
 {
     package Role::Foo;
index ce94eda..d83ee4e 100644 (file)
@@ -3,14 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 10;
+use Test::More tests => 7;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role::Application::RoleSummation');    
-    use_ok('Moose::Meta::Role::Composite');
-}
+use Moose::Meta::Role::Application::RoleSummation;
+use Moose::Meta::Role::Composite;
 
 {
     package Role::Foo;
index 75fbdc5..997520b 100644 (file)
@@ -3,14 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 1;
+use Test::More skip_all => 'The feature this test exercises is not yet written';
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
-
-=pod
 
 {
     package Scalar;
@@ -40,6 +35,3 @@ is_deeply(
     
     sub eq { shift == shift }
 }
-
-=cut
-
index e0a253f..4cde153 100644 (file)
@@ -5,12 +5,10 @@ use warnings;
 
 use lib 't/lib', 'lib';
 
-use Test::More tests => 5;
+use Test::More tests => 4;
 use Test::Exception;
 
-BEGIN {
-       use_ok('Moose');
-}
+
 
 $SIG{__WARN__} = sub { 0 };
 
index 6480222..2bcafc1 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 26;
+use Test::More tests => 25;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package HTTPHeader;
index a7d33d9..69254f5 100644 (file)
@@ -9,12 +9,10 @@ use Test::Exception;
 BEGIN {
     eval "use IO::String; use IO::File;";
     plan skip_all => "IO::String and IO::File are required for this test" if $@;        
-    plan tests => 29;    
+    plan tests => 28;    
 }
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
     package Email::Moose;
index c4f62d7..06528d3 100644 (file)
@@ -3,13 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 19;
+use Test::More tests => 17;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Util::TypeConstraints');
-}
+use Moose::Util::TypeConstraints;
 
 my $type = Moose::Util::TypeConstraints::find_or_parse_type_constraint('Maybe[Int]');
 isa_ok($type, 'Moose::Meta::TypeConstraint');
index f5eccd1..f20ee2e 100644 (file)
@@ -3,13 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 55;
+use Test::More tests => 54;
 use Test::Exception;
 
-BEGIN
-{
-    use_ok('Moose');
-}
+
 
 {
     package Foo;
index e22cf9b..ba18c69 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 4;
+use Test::More tests => 3;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Custom::Meta::Attr;
index e979ea9..6c0fbfe 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 3;
+use Test::More tests => 2;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-};
+;
 
 lives_ok {
     package MooseX::Attribute::Test;
index dce160b..bc6db3e 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 5;
+use Test::More tests => 4;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 =pod
 
index dc966cd..d355dbc 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 17;
+use Test::More tests => 16;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 =pod
 
index 3fce149..9771a73 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 8;
+use Test::More tests => 7;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 BEGIN {
     package MyFramework::Base;
index 0f83849..96290dd 100644 (file)
@@ -3,11 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 5;
+use Test::More tests => 4;
+
+use Moose ();
 
-BEGIN {
-    use_ok('Moose');
-}
 
 { package Foo; }
 
index c967c22..e146561 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 7;
+use Test::More tests => 6;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 =pod
 
index 8fcb324..323b97e 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 7;
+use Test::More tests => 6;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 {
        package Elk;
index f01d168..a97e2d1 100644 (file)
@@ -28,4 +28,5 @@ be well from now on.
 
 { package Object::Test; }
 
-use_ok('Moose');      
+package Foo;
+::use_ok('Moose');      
index 095213c..1dabe6c 100644 (file)
@@ -5,11 +5,9 @@ use warnings;
 
 use lib 't/lib', 'lib';
 
-use Test::More tests => 3;
+use Test::More tests => 2;
+
 
-BEGIN {
-    use_ok('Moose');           
-}
 
 use_ok('MyMooseA');
 use_ok('MyMooseB');
\ No newline at end of file
index 6052f83..689cc21 100644 (file)
@@ -3,11 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 3;
+use Test::More tests => 2;
+
 
-BEGIN {
-    use_ok('Moose');
-}
 
 =pod
 
index 7321764..2e67750 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 2;
+use Test::More tests => 1;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 =pod
 
index b4c18cc..c36721f 100644 (file)
@@ -2,7 +2,6 @@
 
 use strict;
 use warnings;
-use Moose;
 
 use Test::More tests => 3;
 
index 58d7074..4001dc3 100644 (file)
@@ -2,7 +2,6 @@
 
 use strict;
 use warnings;
-use Moose;
 
 use Test::More tests => 1;
 use Test::Exception;
index df985e7..3e41104 100644 (file)
@@ -3,11 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 4;
+use Test::More tests => 3;
+
 
-BEGIN {
-    use_ok('Moose');
-}
 
 {
     package Foo;
index e5d3b19..5cc2fb0 100644 (file)
@@ -3,11 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 3;
+use Test::More tests => 2;
+
 
-BEGIN {
-    use_ok('Moose');
-}
 
 {
     package Foo;
index 10262f6..7574db7 100644 (file)
@@ -4,13 +4,10 @@ use strict;
 use warnings;
 use FindBin;
 
-use Test::More tests => 146;
+use Test::More tests => 144;
 use Test::Exception;
 
-BEGIN {
-     use_ok('Moose');
-     use_ok('Moose::Util::TypeConstraints');
-}
+use Moose::Util::TypeConstraints;
 
 subtype 'FilePath'
     => as 'Str'
index 130e2c0..5773a60 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 5;
+use Test::More tests => 4;
 use Test::Exception;
 
-BEGIN {
-     use_ok('Moose');
-}
+
 
 {
     package Foo;
index 4d423be..2fa0357 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 2;
+use Test::More tests => 1;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Role;
index d425787..6740e81 100644 (file)
@@ -6,9 +6,7 @@ use warnings;
 use Test::More no_plan => 1;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 # RT #37569
 
index a245079..be4f016 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 37;
+use Test::More tests => 36;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 BEGIN {
     package MyRole;
index 33f9058..473da12 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 21;
+use Test::More tests => 20;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 ## Roles
 
index 0298123..9e5d472 100644 (file)
@@ -10,7 +10,7 @@ BEGIN {
     plan skip_all => "DBM::Deep 1.0003 (or greater) is required for this test" if $@;              
     eval "use DateTime::Format::MySQL;";
     plan skip_all => "DateTime::Format::MySQL is required for this test" if $@;            
-    plan tests => 89;    
+    plan tests => 88;    
 }
 
 use Test::Exception;
@@ -24,9 +24,7 @@ END {
     unlink('newswriter.db') if -e 'newswriter.db';
 }
 
-BEGIN {
-    use_ok('Moose');           
-}
+
 
 =pod
 
index 3c3e463..9e1c530 100644 (file)
@@ -3,14 +3,9 @@
 use strict;
 use warnings;
 
-use Test::More tests => 32;
+use Test::More tests => 30;
 use Test::Exception;
 
-BEGIN {  
-    use_ok('Moose');
-    use_ok('Moose::Role');
-}
-
 sub U {
     my $f = shift;
     sub { $f->($f, @_) };
index e8d1d25..3dd0e6a 100644 (file)
@@ -17,16 +17,11 @@ Pretty well if I do say so myself :)
 BEGIN {
     eval "use Declare::Constraints::Simple;";
     plan skip_all => "Declare::Constraints::Simple is required for this test" if $@;        
-    plan tests => 11;    
+    plan tests => 9;    
 }
 
 use Test::Exception;
 
-BEGIN {  
-    use_ok('Moose');
-    use_ok('Moose::Util::TypeConstraints');    
-}
-
 {
     package Foo;
     use Moose;
index 53daaa8..cd33fa9 100644 (file)
@@ -18,16 +18,11 @@ but it is not completely horrid either.
 BEGIN {
     eval "use Test::Deep;";
     plan skip_all => "Test::Deep is required for this test" if $@;        
-    plan tests => 7;    
+    plan tests => 5;    
 }
 
 use Test::Exception;
 
-BEGIN {  
-    use_ok('Moose');
-    use_ok('Moose::Util::TypeConstraints');    
-}
-
 {
     package Foo;
     use Moose;
index 990f2a8..00eac6b 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 9;
+use Test::More tests => 8;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package Record;
index 026d516..9d4e089 100644 (file)
@@ -3,13 +3,11 @@
 use strict;
 use warnings;
 
-use Test::More tests => 17;
+use Test::More tests => 15;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role');
-}
+use Moose::Meta::Role;
+
 
 {
   package FooRole;
index c1ba765..e373311 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 5;
+use Test::More tests => 4;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Role;
index a4214e9..d8b1c75 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 2;
+use Test::More tests => 1;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package My::Meta;
index 7830752..7e21b27 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 4;
+use Test::More tests => 3;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 =pod
 
index 4f543ce..53b31a0 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 3;
+use Test::More tests => 2;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package Foo;
index c04a44b..c3787e4 100644 (file)
@@ -3,15 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 10;
+use Test::More tests => 8;
 use Test::Exception;
 use Scalar::Util 'blessed';
 
-BEGIN {
-    use_ok('Moose');
-    use_ok('Moose::Meta::Role');
-}
-
 =pod
 
 This test it kind of odd, it tests 
index 6a42b73..cab557f 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 4;
+use Test::More tests => 3;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 {
     package AClass;
index 1c235f4..62d6d3c 100644 (file)
@@ -3,12 +3,10 @@
 use strict;
 use warnings;
 
-use Test::More tests => 3;
+use Test::More tests => 2;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 =pod
 
index 4c77671..112e7bd 100644 (file)
@@ -6,9 +6,7 @@ use warnings;
 use Test::More no_plan => 1;
 use Test::Exception;
 
-BEGIN {
-    use_ok('Moose');
-}
+
 
 =pod