[Catalyst-commits] r9427 - in Catalyst-Runtime/5.80/trunk: . t/aggregate t/lib/TestApp t/lib/TestApp/Controller/Action t/lib/TestApp/ControllerBase

[t0m at dev.catalyst.perl.org]

Author: t0m
Date: 2009-03-04 16:51:23 +0000 (Wed, 04 Mar 2009)
New Revision: 9427

Added:
   Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/
   Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/Middle.pm
   Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/OtherRoot.pm
   Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/RealMiddle.pm
   Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/Root.pm
Modified:
   Catalyst-Runtime/5.80/trunk/TODO
   Catalyst-Runtime/5.80/trunk/t/aggregate/live_component_controller_action_inheritance.t
   Catalyst-Runtime/5.80/trunk/t/lib/TestApp/Controller/Action/Inheritance.pm
Log:
>From #catalyst-dev before

Modified: Catalyst-Runtime/5.80/trunk/TODO
===================================================================
--- Catalyst-Runtime/5.80/trunk/TODO	2009-03-04 12:53:25 UTC (rev 9426)
+++ Catalyst-Runtime/5.80/trunk/TODO	2009-03-04 16:51:23 UTC (rev 9427)
@@ -4,6 +4,9 @@
    - Provide an extension so that components can have a Moose::Object
      constructor and an @ISA which makes c3 happy on both 5.7X and 5.8X.
 
+   - $self->config should warn as config should only ever be called as a
+     class method.
+
 Testing:
 
    - Run more smokes

Modified: Catalyst-Runtime/5.80/trunk/t/aggregate/live_component_controller_action_inheritance.t
===================================================================
--- Catalyst-Runtime/5.80/trunk/t/aggregate/live_component_controller_action_inheritance.t	2009-03-04 12:53:25 UTC (rev 9426)
+++ Catalyst-Runtime/5.80/trunk/t/aggregate/live_component_controller_action_inheritance.t	2009-03-04 16:51:23 UTC (rev 9427)
@@ -10,7 +10,7 @@
 
 BEGIN { $iters = $ENV{CAT_BENCH_ITERS} || 1; }
 
-use Test::More tests => 21*$iters;
+use Test::More tests => 26*$iters;
 use Catalyst::Test 'TestApp';
 
 if ( $ENV{CAT_BENCHMARK} ) {
@@ -116,4 +116,26 @@
             'Content is a serialized Catalyst::Request'
         );
     }
+
+    {
+        my @expected = qw[
+          TestApp::Controller::Action::Inheritance->begin
+          TestApp::Controller::Action::Inheritance->auto
+          TestApp::Controller::Action::Inheritance::Chained->chain_root
+          TestApp::Controller::Action::Inheritance::Chained->chain_first
+          TestApp::Controller::Action::Inheritance::Chained->chain_middle
+          TestApp::Controller::Action::Inheritance::Chained->chain_end
+          TestApp::Controller::Action::Inheritance->end
+        ];
+
+        my $expected = join( ", ", @expected );
+
+        ok( my $response = request('http://localhost/action/inheritance/chained/chain_end'),
+            'Request' );
+        ok( $response->is_success, 'Response Successful 2xx' );
+        is( $response->content_type, 'text/plain', 'Response Content-Type' );
+        is( $response->header('X-Catalyst-Action'), '/action/inheritance/chained/chain_end', 'Test Action' );
+        is( $response->header('X-Catalyst-Executed'),
+            $expected, 'Executed actions' );
+    }
 }

Modified: Catalyst-Runtime/5.80/trunk/t/lib/TestApp/Controller/Action/Inheritance.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/t/lib/TestApp/Controller/Action/Inheritance.pm	2009-03-04 12:53:25 UTC (rev 9426)
+++ Catalyst-Runtime/5.80/trunk/t/lib/TestApp/Controller/Action/Inheritance.pm	2009-03-04 16:51:23 UTC (rev 9427)
@@ -70,5 +70,13 @@
     my ( $self, $c ) = @_;
 }
 
+package TestApp::Controller::Action::Inheritance::Chained;
+
+use strict;
+use base qw/TestApp::ControllerBase::Middle TestApp::ControllerBase::RealMiddle/;
+
+sub chain_first : CaptureArgs(0) PathPart('') Chained('chain_root') {}
+
+
 1;
 

Added: Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/Middle.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/Middle.pm	                        (rev 0)
+++ Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/Middle.pm	2009-03-04 16:51:23 UTC (rev 9427)
@@ -0,0 +1,11 @@
+package TestApp::ControllerBase::Middle;
+
+use Moose;
+
+use namespace::clean -except => 'meta';
+
+BEGIN { extends qw/TestApp::ControllerBase::Root/; }
+
+__PACKAGE__->meta->make_immutable;
+
+1;

Added: Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/OtherRoot.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/OtherRoot.pm	                        (rev 0)
+++ Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/OtherRoot.pm	2009-03-04 16:51:23 UTC (rev 9427)
@@ -0,0 +1,13 @@
+package TestApp::ControllerBase::OtherRoot;
+
+use Moose;
+
+use namespace::clean -except => 'meta';
+
+BEGIN { extends qw/Catalyst::Controller/; }
+
+sub chain_middle : CaptureArgs(0) PathPart('') Chained('chain_first') {}
+
+__PACKAGE__->meta->make_immutable;
+
+1;

Added: Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/RealMiddle.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/RealMiddle.pm	                        (rev 0)
+++ Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/RealMiddle.pm	2009-03-04 16:51:23 UTC (rev 9427)
@@ -0,0 +1,11 @@
+package TestApp::ControllerBase::RealMiddle;
+
+use Moose;
+
+use namespace::clean -except => 'meta';
+
+BEGIN { extends qw/TestApp::ControllerBase::OtherRoot/; }
+
+__PACKAGE__->meta->make_immutable;
+
+1;

Added: Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/Root.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/Root.pm	                        (rev 0)
+++ Catalyst-Runtime/5.80/trunk/t/lib/TestApp/ControllerBase/Root.pm	2009-03-04 16:51:23 UTC (rev 9427)
@@ -0,0 +1,15 @@
+package TestApp::ControllerBase::Root;
+
+use Moose;
+
+use namespace::clean -except => 'meta';
+
+BEGIN { extends qw/Catalyst::Controller/; }
+
+sub chain_root : Chained('/') PathPrefix CaptureArgs(0) {}
+
+sub chain_end : Chained('chain_middle') Args(0) {}
+
+__PACKAGE__->meta->make_immutable;
+
+1;