X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=blobdiff_plain;f=t%2Faggregate%2Funit_core_uri_for_action.t;h=f7cd481eb6ace4bb81de8fd3365ba74eabea7cce;hp=19a10af90fba039d90485bb9965f0b19fdfebbcd;hb=0cff119a540a3c8961750bd499f4c470771ac342;hpb=2e9f1ec69ede01cd175742085b41c7f26c76be90 diff --git a/t/aggregate/unit_core_uri_for_action.t b/t/aggregate/unit_core_uri_for_action.t index 19a10af..f7cd481 100644 --- a/t/aggregate/unit_core_uri_for_action.t +++ b/t/aggregate/unit_core_uri_for_action.t @@ -8,8 +8,6 @@ use lib "$FindBin::Bin/../lib"; use Test::More; -plan tests => 30; - use_ok('TestApp'); my $dispatcher = TestApp->dispatcher; @@ -54,6 +52,21 @@ is($dispatcher->uri_for_action($regex_action, [ 'foo', 123 ]), "/action/regexp/foo/123", "Regex action interpolates captures correctly"); +my $regex_action_bs = $dispatcher->get_action_by_path( + '/action/regexp/one_backslashes' + ); + +ok(!defined($dispatcher->uri_for_action($regex_action_bs)), + "Regex action without captures returns undef"); + +ok(!defined($dispatcher->uri_for_action($regex_action_bs, [ 1, 2, 3 ])), + "Regex action with too many captures returns undef"); + +is($dispatcher->uri_for_action($regex_action_bs, [ 'foo', 123 ]), + "/action/regexp/foo/123.html", + "Regex action interpolates captures correctly"); + + # # Index Action # @@ -97,6 +110,10 @@ my $context = TestApp->new( { namespace => 'yada', } ); +is($context->uri_for($context->controller('Action')), + "http://127.0.0.1/foo/yada/action/", + "uri_for a controller"); + is($context->uri_for($path_action), "http://127.0.0.1/foo/action/relative/relative", "uri_for correct for path action"); @@ -124,10 +141,18 @@ is($context->uri_for($chained_action, [ 1 ], 2, { q => 1 }), 'http://127.0.0.1/foo/chained/foo2/1/2/end2/3/4?x=5', 'uri_for_action correct for chained with multiple captures and args' ); + is( $context->uri_for_action( '/action/chained/endpoint2', [1,2,3,4], { x => 5 } ), + 'http://127.0.0.1/foo/chained/foo2/1/2/end2/3/4?x=5', + 'uri_for_action correct for chained with multiple captures and args combined' ); + is( $context->uri_for_action( '/action/chained/three_end', [1,2,3], (4,5,6) ), 'http://127.0.0.1/foo/chained/one/1/two/2/3/three/4/5/6', 'uri_for_action correct for chained with multiple capturing actions' ); + is( $context->uri_for_action( '/action/chained/three_end', [1,2,3,4,5,6] ), + 'http://127.0.0.1/foo/chained/one/1/two/2/3/three/4/5/6', + 'uri_for_action correct for chained with multiple capturing actions and args combined' ); + my $action_needs_two = '/action/chained/endpoint2'; ok( ! defined( $context->uri_for_action($action_needs_two, [1], (2,3)) ), @@ -136,7 +161,11 @@ is($context->uri_for($chained_action, [ 1 ], 2, { q => 1 }), is( $context->uri_for_action($action_needs_two, [1,2], (2,3)), 'http://127.0.0.1/foo/chained/foo2/1/2/end2/2/3', 'uri_for_action returns correct uri for correct captures' ); - + + is( $context->uri_for_action($action_needs_two, [1,2,2,3]), + 'http://127.0.0.1/foo/chained/foo2/1/2/end2/2/3', + 'uri_for_action returns correct uri for correct captures and args combined' ); + ok( ! defined( $context->uri_for_action($action_needs_two, [1,2,3], (2,3)) ), 'uri_for_action returns undef for too many captures' ); @@ -144,30 +173,49 @@ is($context->uri_for($chained_action, [ 1 ], 2, { q => 1 }), 'http://127.0.0.1/foo/chained/foo2/1/2/end2/3', 'uri_for_action returns uri with lesser args than specified on action' ); + is( $context->uri_for_action($action_needs_two, [1,2,3]), + 'http://127.0.0.1/foo/chained/foo2/1/2/end2/3', + 'uri_for_action returns uri with lesser args than specified on action with captures combined' ); + is( $context->uri_for_action($action_needs_two, [1,2], (3,4,5)), 'http://127.0.0.1/foo/chained/foo2/1/2/end2/3/4/5', 'uri_for_action returns uri with more args than specified on action' ); + is( $context->uri_for_action($action_needs_two, [1,2,3,4,5]), + 'http://127.0.0.1/foo/chained/foo2/1/2/end2/3/4/5', + 'uri_for_action returns uri with more args than specified on action with captures combined' ); + is( $context->uri_for_action($action_needs_two, [1,''], (3,4)), 'http://127.0.0.1/foo/chained/foo2/1//end2/3/4', 'uri_for_action returns uri with empty capture on undef capture' ); + is( $context->uri_for_action($action_needs_two, [1,'',3,4]), + 'http://127.0.0.1/foo/chained/foo2/1//end2/3/4', + 'uri_for_action returns uri with empty capture on undef capture and args combined' ); + is( $context->uri_for_action($action_needs_two, [1,2], ('',3)), 'http://127.0.0.1/foo/chained/foo2/1/2/end2//3', 'uri_for_action returns uri with empty arg on undef argument' ); + is( $context->uri_for_action($action_needs_two, [1,2,'',3]), + 'http://127.0.0.1/foo/chained/foo2/1/2/end2//3', + 'uri_for_action returns uri with empty arg on undef argument and args combined' ); + is( $context->uri_for_action($action_needs_two, [1,2], (3,'')), 'http://127.0.0.1/foo/chained/foo2/1/2/end2/3/', 'uri_for_action returns uri with empty arg on undef last argument' ); - is( $context->uri_for_action($action_needs_two, [ 'foo' , 'bar/baz' ], (3,'')), - 'http://127.0.0.1/foo/chained/foo2/foo/bar%2Fbaz/end2/3/', - 'uri_for_action returns uri with empty arg on undef last argument' ); + is( $context->uri_for_action($action_needs_two, [1,2,3,'']), + 'http://127.0.0.1/foo/chained/foo2/1/2/end2/3/', + 'uri_for_action returns uri with empty arg on undef last argument with captures combined' ); my $complex_chained = '/action/chained/empty_chain_f'; is( $context->uri_for_action( $complex_chained, [23], (13), {q => 3} ), 'http://127.0.0.1/foo/chained/empty/23/13?q=3', 'uri_for_action returns correct uri for chain with many empty path parts' ); + is( $context->uri_for_action( $complex_chained, [23,13], {q => 3} ), + 'http://127.0.0.1/foo/chained/empty/23/13?q=3', + 'uri_for_action returns correct uri for chain with many empty path parts with captures and args combined' ); eval { $context->uri_for_action( '/does/not/exist' ) }; like $@, qr{^Can't find action for path '/does/not/exist'}, @@ -175,3 +223,5 @@ is($context->uri_for($chained_action, [ 1 ], 2, { q => 1 }), } +done_testing; +