X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2Faccessors.t;h=41130710e1462ba5c3716846da91b204e526e9be;hb=6c6bc8c2ab1a427010501b54b2959c9e2ba23515;hp=cf219b0fc6f7cfc5e78d4ef7d0441691d10baed1;hpb=de1673798e9795ee442cae7581215bfda4af3607;p=p5sagit%2FClass-Accessor-Grouped.git

diff --git a/t/accessors.t b/t/accessors.t
index cf219b0..4113071 100644
--- a/t/accessors.t
+++ b/t/accessors.t
@@ -1,4 +1,4 @@
-use Test::More tests => 117;
+use Test::More tests => 136;
 use strict;
 use warnings;
 use lib 't/lib';
@@ -10,103 +10,86 @@ use B qw/svref_2object/;
 # things
 my $use_xs;
 BEGIN {
-    $Class::Accessor::Grouped::USE_XS = 0
-        unless defined $Class::Accessor::Grouped::USE_XS;
-    $ENV{CAG_USE_XS} = 1;
-    $use_xs = $Class::Accessor::Grouped::USE_XS;
+  $Class::Accessor::Grouped::USE_XS = 0
+    unless defined $Class::Accessor::Grouped::USE_XS;
+  $ENV{CAG_USE_XS} = 1;
+  $use_xs = $Class::Accessor::Grouped::USE_XS;
 };
 
 use AccessorGroupsSubclass;
-
-{
-    my $obj = AccessorGroupsSubclass->new;
-    my $class = ref $obj;
-    my $name = 'multiple1';
-    my $alias = "_${name}_accessor";
-
-    my $warned = 0;
-    local $SIG{__WARN__} = sub {
-        if  (shift =~ /DESTROY/i) {
-            $warned++;
-        };
-    };
-
-    no warnings qw/once/;
-    local *AccessorGroupsSubclass::DESTROY = sub {};
-
-    $class->mk_group_accessors('warnings', 'DESTROY');
-    ok($warned);
-};
-
 my $obj = AccessorGroupsSubclass->new;
 
 my $test_accessors = {
-    singlefield => {
-        is_xs => $use_xs,
-        has_extra => 1,
-    },
-    runtime_around => {
-        # even though this accessor is simple it will *not* be XSified
-        # due to the runtime 'around'
-        is_xs => 0,
-        has_extra => 1,
-    },
-    multiple1 => {
-    },
-    multiple2 => {
-    },
-    lr1name => {
-        custom_field => 'lr1;field',
-    },
-    lr2name => {
-        custom_field => "lr2'field",
-    },
+  singlefield => {
+    is_xs => $use_xs,
+    has_extra => 1,
+  },
+  runtime_around => {
+    # even though this accessor is simple it will *not* be XSified
+    # due to the runtime 'around'
+    is_xs => 0,
+    has_extra => 1,
+  },
+  multiple1 => {
+  },
+  multiple2 => {
+  },
+  lr1name => {
+    custom_field => 'lr1;field',
+  },
+  lr2name => {
+    custom_field => "lr2'field",
+  },
+  fieldname_torture => {
+    custom_field => join ('', map { chr($_) } (0..255) ),
+    is_xs => $use_xs,
+  },
 };
 
 for my $name (sort keys %$test_accessors) {
-    my $alias = "_${name}_accessor";
-    my $field = $test_accessors->{$name}{custom_field} || $name;
-    my $extra = $test_accessors->{$name}{has_extra};
-
-    can_ok($obj, $name, $alias);
-    ok(!$obj->can($field))
-      if $field ne $name;
-
-    for my $meth ($name, $alias) {
-        my $cv = svref_2object( $obj->can($meth) );
-        is($cv->GV->NAME, $meth, "$meth accessor is named");
-        is($cv->GV->STASH->NAME, 'AccessorGroups', "$meth class correct");
-    }
-
-    is($obj->$name, undef);
-    is($obj->$alias, undef);
-
-    # get/set via name
-    is($obj->$name('a'), 'a');
-    is($obj->$name, 'a');
-    is($obj->{$field}, $extra ? 'a Extra tackled on' : 'a');
-
-    # alias gets same as name
-    is($obj->$alias, 'a');
-
-    # get/set via alias
-    is($obj->$alias('b'), 'b');
-    is($obj->$alias, 'b');
-    is($obj->{$field}, $extra ? 'b Extra tackled on' : 'b');
-
-    # alias gets same as name
-    is($obj->$name, 'b');
-
-    for my $meth ($name, $alias) {
-        my $cv = svref_2object( $obj->can($meth) );
-        is($cv->GV->NAME, $meth, "$meth accessor is named after operations");
-        is(
-          $cv->GV->STASH->NAME,
-          # XS lazyinstalls install into each caller, not into the original parent
-          $test_accessors->{$name}{is_xs} ? 'AccessorGroupsSubclass' :'AccessorGroups',
-          "$meth class correct after operations",
-        );
-    }
+  my $alias = "_${name}_accessor";
+  my $field = $test_accessors->{$name}{custom_field} || $name;
+  my $extra = $test_accessors->{$name}{has_extra};
+
+  can_ok($obj, $name, $alias);
+  ok(!$obj->can($field))
+    if $field ne $name;
+
+  for my $meth ($name, $alias) {
+    my $cv = svref_2object( $obj->can($meth) );
+    is($cv->GV->NAME, $meth, "$meth accessor is named");
+    is($cv->GV->STASH->NAME, 'AccessorGroups', "$meth class correct");
+  }
+
+  is($obj->$name, undef);
+  is($obj->$alias, undef);
+
+  # get/set via name
+  is($obj->$name('a'), 'a');
+  is($obj->$name, 'a');
+  is($obj->{$field}, $extra ? 'a Extra tackled on' : 'a');
+
+  # alias gets same as name
+  is($obj->$alias, 'a');
+
+  # get/set via alias
+  is($obj->$alias('b'), 'b');
+  is($obj->$alias, 'b');
+  is($obj->{$field}, $extra ? 'b Extra tackled on' : 'b');
+
+  # alias gets same as name
+  is($obj->$name, 'b');
+
+  for my $meth ($name, $alias) {
+    my $cv = svref_2object( $obj->can($meth) );
+    is($cv->GV->NAME, $meth, "$meth accessor is named after operations");
+    is(
+      $cv->GV->STASH->NAME,
+      # XS lazyinstalls install into each caller, not into the original parent
+      $test_accessors->{$name}{is_xs} ? 'AccessorGroupsSubclass' :'AccessorGroups',
+      "$meth class correct after operations",
+    );
+  }
 };
 
 # important