use strict;
use warnings;
-
+use FindBin qw/$Bin/;
+use lib "$FindBin::Bin/../lib";
use Test::More;
use URI;
-use_ok('Catalyst');
+use_ok('TestApp');
my $request = Catalyst::Request->new( {
+ _log => Catalyst::Log->new,
base => URI->new('http://127.0.0.1/foo')
} );
-
-my $context = Catalyst->new( {
+my $dispatcher = TestApp->dispatcher;
+my $context = TestApp->new( {
request => $request,
namespace => 'yada',
} );
'Plus is not encoded'
);
+is(
+ Catalyst::uri_for( $context, '/bar#fragment', { param1 => 'value1' } )->as_string,
+ 'http://127.0.0.1/foo/bar?param1=value1#fragment',
+ 'URI for path with fragment and query params 1'
+);
+
+is(
+ Catalyst::uri_for( $context, '0#fragment', { param1 => 'value1' } )->as_string,
+ 'http://127.0.0.1/foo/yada/0?param1=value1#fragment',
+ 'URI for path 0 with fragment and query params 1'
+);
+
+is(
+ Catalyst::uri_for( $context, '/bar#fragment^%$', { param1 => 'value1' } )->as_string,
+ 'http://127.0.0.1/foo/bar?param1=value1#fragment^%$',
+ 'URI for path with fragment and query params 3'
+);
+
+is(
+ Catalyst::uri_for( $context, '/foo#bar/baz', { param1 => 'value1' } )->as_string,
+ 'http://127.0.0.1/foo/foo?param1=value1#bar/baz',
+ 'URI for path with fragment and query params 3'
+);
+
# test with utf-8
is(
Catalyst::uri_for( $context, 'quux', { param1 => "\x{2620}" } )->as_string,
Catalyst::uri_for( $context, 'quux', { param1 => $request->base } )->as_string,
'http://127.0.0.1/foo/yada/quux?param1=http%3A%2F%2F127.0.0.1%2Ffoo',
'URI for undef action with query param as object'
-);
+ );
+
+# test with empty arg
+{
+ my @warnings;
+ local $SIG{__WARN__} = sub { push @warnings, @_ };
+ is(
+ Catalyst::uri_for( $context )->as_string,
+ 'http://127.0.0.1/foo/yada',
+ 'URI with no action'
+ );
+
+ is(
+ Catalyst::uri_for( $context, 0 )->as_string,
+ 'http://127.0.0.1/foo/yada/0',
+ 'URI with 0 path'
+ );
+
+ is_deeply(\@warnings, [], "No warnings with no path argument");
+}
$request->base( URI->new('http://localhost:3000/') );
$request->match( 'orderentry/contract' );
"uri_for() doesn't mess up query parameter hash in the caller");
}
-# 5.80018 is only encoding the first of the / in the arg. See line 1271.
+
+{
+ my $path_action = $dispatcher->get_action_by_path(
+ '/action/path/six'
+ );
+
+ # 5.80018 is only encoding the first of the / in the arg.
+ is(
+ Catalyst::uri_for( $context, $path_action, 'foo/bar/baz' )->as_string,
+ 'http://127.0.0.1/action/path/six/foo%2Fbar%2Fbaz',
+ 'Escape all forward slashes in args as %2F'
+ );
+}
+
+{
+ my $index_not_private = $dispatcher->get_action_by_path(
+ '/action/chained/argsorder/index'
+ );
+
+ is(
+ Catalyst::uri_for( $context, $index_not_private )->as_string,
+ 'http://127.0.0.1/argsorder',
+ 'Return non-DispatchType::Index path for index action with args'
+ );
+}
+
+{
+ package MyStringThing;
+
+ use overload '""' => sub { $_[0]->{string} }, fallback => 1;
+}
+
is(
- Catalyst::uri_for( $context, 'controller/action', 'foo/bar/baz' )->as_string,
- 'http://127.0.0.1/controller/action/foo%2Fbar%2Fbaz',
- 'Escape both forward slashes in the arg as %2F'
+ Catalyst::uri_for( $context, bless( { string => 'test' }, 'MyStringThing' ) ),
+ 'http://127.0.0.1/test',
+ 'overloaded object handled correctly'
);
done_testing;
-