patch to make prepare_parameters not be both a builder and a preparer

[catagits/Catalyst-Runtime.git] / t / lib / TestApp / Controller / Dump.pm

diff --git a/t/lib/TestApp/Controller/Dump.pm b/t/lib/TestApp/Controller/Dump.pm
index 8c78fc8..0864822 100644 (file)
--- a/t/lib/TestApp/Controller/Dump.pm
+++ b/t/lib/TestApp/Controller/Dump.pm
@@ -3,26 +3,44 @@ package TestApp::Controller::Dump;
 use strict;
 use base 'Catalyst::Controller';
 
-sub default : Action Private {
+sub default : Action {
     my ( $self, $c ) = @_;
     $c->forward('TestApp::View::Dump');
 }
 
 sub env : Action Relative {
     my ( $self, $c ) = @_;
-    $c->forward('TestApp::View::Dump', [\%ENV]);
+    $c->stash(env => $c->req->env);
+    $c->forward('TestApp::View::Dump::Env');
 }
 
-sub parameters : Action Relative {
+sub env_on_engine : Action Relative {
     my ( $self, $c ) = @_;
-    $c->forward('TestApp::View::Dump::Parameters');
+    $c->stash(env => $c->engine->env);
+    $c->forward('TestApp::View::Dump::Env');
 }
 
 sub request : Action Relative {
     my ( $self, $c ) = @_;
+    $c->req->params(undef); # Should be a no-op, and be ignored.
+                            # Back compat test for 5.7
     $c->forward('TestApp::View::Dump::Request');
 }
 
+sub prepare_parameters : Action Relative {
+    my ( $self, $c ) = @_;
+
+    die 'Must pass in parameters' unless keys %{$c->req->parameters};
+
+    $c->req->parameters( {} );
+    die 'parameters are not empty' if keys %{$c->req->parameters};
+
+    # Now reset and reload
+    $c->prepare_parameters;
+    die 'Parameters were not reset' unless keys %{$c->req->parameters};
+
+    $c->forward('TestApp::View::Dump::Request');
+}
 sub response : Action Relative {
     my ( $self, $c ) = @_;
     $c->forward('TestApp::View::Dump::Response');