X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2Funit_core_uri_for.t;h=4018b03d65d41c9853c2142752fc5db4620266f5;hb=68af38662275bb5259aa49d555205396d0a63fd3;hp=612d0b06db3a1df95a43223cd36cdc229faf9c41;hpb=57e74a1e0e82d0f06e56b58a50f84007af4c4798;p=catagits%2FCatalyst-Runtime.git diff --git a/t/unit_core_uri_for.t b/t/unit_core_uri_for.t index 612d0b0..4018b03 100644 --- a/t/unit_core_uri_for.t +++ b/t/unit_core_uri_for.t @@ -1,7 +1,7 @@ use strict; use warnings; -use Test::More tests => 13; +use Test::More tests => 18; use URI; use_ok('Catalyst'); @@ -58,6 +58,11 @@ is( 'http://127.0.0.1/foo/yada/quux?param1=%E2%98%A0', 'URI for undef action with query params in unicode' ); +is( + Catalyst::uri_for( $context, 'quux', { 'param:1' => "foo" } )->as_string, + 'http://127.0.0.1/foo/yada/quux?param%3A1=foo', + 'URI for undef action with query params in unicode' +); # test with object is( @@ -81,6 +86,12 @@ is( is( Catalyst::uri_for( $context, '/bar/baz' )->as_string, 'http://127.0.0.1/bar/baz', 'URI with no base or match' ); + + # test "0" as the path + is( Catalyst::uri_for( $context, qw/0 foo/ )->as_string, + 'http://127.0.0.1/0/foo', '0 as path is ok' + ); + } # test with undef -- no warnings should be thrown @@ -92,3 +103,27 @@ is( is( $warnings, 0, "no warnings emitted" ); } +# Test with parameters '/', 'foo', 'bar' - should not generate a // +is( Catalyst::uri_for( $context, qw| / foo bar | )->as_string, + 'http://127.0.0.1/foo/bar', 'uri is /foo/bar, not //foo/bar' +); + +TODO: { + local $TODO = 'RFCs are for people who, erm - fix this test..'; + # Test rfc3986 reserved characters. These characters should all be escaped + # according to the RFC, but it is a very big feature change so I've removed it + no warnings; # Yes, everything in qw is sane + is( + Catalyst::uri_for( $context, qw|! * ' ( ) ; : @ & = $ / ? % # [ ] ,|, )->as_string, + 'http://127.0.0.1/%21/%2A/%27/%2B/%29/%3B/%3A/%40/%26/%3D/%24/%2C/%2F/%3F/%25/%23/%5B/%5D', + 'rfc 3986 reserved characters' + ); +} + +# jshirley bug t0m fucked in r10097 +is( + Catalyst::uri_for( $context, qw|{1} {2}| )->as_string, + 'http://127.0.0.1/{1}/{2}', + 'not-escaping unreserved characters' +); +