X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F016_class_errors_and_edge_cases.t;h=36a1ac470cd81f27ba29dd297e74ba1a5afd78aa;hb=871e9eb5d05b8b9986b2de3f4095f65a31159c56;hp=20103d38bad7a11ae07fbb8a4059079e97e28f58;hpb=13b8971fa6cc4edf7d55a2e5482820d62ba38f16;p=gitmo%2FClass-MOP.git

diff --git a/t/016_class_errors_and_edge_cases.t b/t/016_class_errors_and_edge_cases.t
index 20103d3..36a1ac4 100644
--- a/t/016_class_errors_and_edge_cases.t
+++ b/t/016_class_errors_and_edge_cases.t
@@ -7,217 +7,217 @@ use Test::Fatal;
 use Class::MOP;
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->initialize();
-    }, '... initialize requires a name parameter';
+    }, undef, '... initialize requires a name parameter' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->initialize('');
-    }, '... initialize requires a name valid parameter';
+    }, undef, '... initialize requires a name valid parameter' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->initialize(bless {} => 'Foo');
-    }, '... initialize requires an unblessed parameter'
+    }, undef, '... initialize requires an unblessed parameter' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->_construct_class_instance();
-    }, '... _construct_class_instance requires an :package parameter';
+    }, undef, '... _construct_class_instance requires an :package parameter' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->_construct_class_instance(':package' => undef);
-    }, '... _construct_class_instance requires a defined :package parameter';
+    }, undef, '... _construct_class_instance requires a defined :package parameter' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->_construct_class_instance(':package' => '');
-    }, '... _construct_class_instance requires a valid :package parameter';
+    }, undef, '... _construct_class_instance requires a valid :package parameter' );
 }
 
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->create();
-    }, '... create requires an package_name parameter';
+    }, undef, '... create requires an package_name parameter' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->create(undef);
-    }, '... create requires a defined package_name parameter';
+    }, undef, '... create requires a defined package_name parameter' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->create('');
-    }, '... create requires a valid package_name parameter';
+    }, undef, '... create requires a valid package_name parameter' );
 
-    like exception {
+    like( exception {
         Class::MOP::Class->create('+++');
-    }, qr/^creation of \+\+\+ failed/, '... create requires a valid package_name parameter';
+    }, qr/^creation of \+\+\+ failed/, '... create requires a valid package_name parameter' );
 
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->clone_object(1);
-    }, '... can only clone instances';
+    }, undef, '... can only clone instances' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_method();
-    }, '... add_method dies as expected';
+    }, undef, '... add_method dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_method('');
-    }, '... add_method dies as expected';
+    }, undef, '... add_method dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_method('foo' => 'foo');
-    }, '... add_method dies as expected';
+    }, undef, '... add_method dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_method('foo' => []);
-    }, '... add_method dies as expected';
+    }, undef, '... add_method dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->has_method();
-    }, '... has_method dies as expected';
+    }, undef, '... has_method dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->has_method('');
-    }, '... has_method dies as expected';
+    }, undef, '... has_method dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->get_method();
-    }, '... get_method dies as expected';
+    }, undef, '... get_method dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->get_method('');
-    }, '... get_method dies as expected';
+    }, undef, '... get_method dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->remove_method();
-    }, '... remove_method dies as expected';
+    }, undef, '... remove_method dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->remove_method('');
-    }, '... remove_method dies as expected';
+    }, undef, '... remove_method dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->find_all_methods_by_name();
-    }, '... find_all_methods_by_name dies as expected';
+    }, undef, '... find_all_methods_by_name dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->find_all_methods_by_name('');
-    }, '... find_all_methods_by_name dies as expected';
+    }, undef, '... find_all_methods_by_name dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_attribute(bless {} => 'Foo');
-    }, '... add_attribute dies as expected';
+    }, undef, '... add_attribute dies as expected' );
 }
 
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->has_attribute();
-    }, '... has_attribute dies as expected';
+    }, undef, '... has_attribute dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->has_attribute('');
-    }, '... has_attribute dies as expected';
+    }, undef, '... has_attribute dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->get_attribute();
-    }, '... get_attribute dies as expected';
+    }, undef, '... get_attribute dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->get_attribute('');
-    }, '... get_attribute dies as expected';
+    }, undef, '... get_attribute dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->remove_attribute();
-    }, '... remove_attribute dies as expected';
+    }, undef, '... remove_attribute dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->remove_attribute('');
-    }, '... remove_attribute dies as expected';
+    }, undef, '... remove_attribute dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_package_symbol();
-    }, '... add_package_symbol dies as expected';
+    }, undef, '... add_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_package_symbol('');
-    }, '... add_package_symbol dies as expected';
+    }, undef, '... add_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_package_symbol('foo');
-    }, '... add_package_symbol dies as expected';
+    }, undef, '... add_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->add_package_symbol('&foo');
-    }, '... add_package_symbol dies as expected';
+    }, undef, '... add_package_symbol dies as expected' );
 
-#    like exception {
+#    throws_ok {
 #        Class::MOP::Class->meta->add_package_symbol('@-');
 #    } qr/^Could not create package variable \(\@\-\) because/,
 #      '... add_package_symbol dies as expected';
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->has_package_symbol();
-    }, '... has_package_symbol dies as expected';
+    }, undef, '... has_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->has_package_symbol('');
-    }, '... has_package_symbol dies as expected';
+    }, undef, '... has_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->has_package_symbol('foo');
-    }, '... has_package_symbol dies as expected';
+    }, undef, '... has_package_symbol dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->get_package_symbol();
-    }, '... get_package_symbol dies as expected';
+    }, undef, '... get_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->get_package_symbol('');
-    }, '... get_package_symbol dies as expected';
+    }, undef, '... get_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->get_package_symbol('foo');
-    }, '... get_package_symbol dies as expected';
+    }, undef, '... get_package_symbol dies as expected' );
 }
 
 {
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->remove_package_symbol();
-    }, '... remove_package_symbol dies as expected';
+    }, undef, '... remove_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->remove_package_symbol('');
-    }, '... remove_package_symbol dies as expected';
+    }, undef, '... remove_package_symbol dies as expected' );
 
-    ok exception {
+    isnt( exception {
         Class::MOP::Class->remove_package_symbol('foo');
-    }, '... remove_package_symbol dies as expected';
+    }, undef, '... remove_package_symbol dies as expected' );
 }
 
 done_testing;