X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2Fdispatch_parser.t;h=f8e7e06cb6a0976f26b6e8ab687b1ca395bda0ef;hb=eb9e0e25ce6999be81b80424ccb16d04966debed;hp=7c5c0ebcbace77c94c689dc9fdb6407e6d11b6e8;hpb=b0420ad6326a86fe33cdbecfb58bba08bec52b74;p=catagits%2FWeb-Simple.git diff --git a/t/dispatch_parser.t b/t/dispatch_parser.t index 7c5c0eb..f8e7e06 100644 --- a/t/dispatch_parser.t +++ b/t/dispatch_parser.t @@ -162,3 +162,127 @@ is_deeply( [], '(GET+/foo)|POST does not match PUT /foo' ); + +{ + local $@; + ok( + !eval { $dp->parse_dispatch_specification('/foo+(GET'); 1 }, + 'Death with missing closing )' + ); + my $err = q{ + /foo+(GET + ^ + }; + (s/^\n//s,s/\n $//s,s/^ //mg) for $err; + like( + $@, + qr{\Q$err\E}, + "Error $@ matches\n${err}\n" + ); +} + +my $not = $dp->parse_dispatch_specification('!.html+.*'); + +is_deeply( + [ $not->({ PATH_INFO => '/foo.xml' }) ], + [ { PATH_INFO => '/foo' }, 'xml' ], + '!.html+.* matches /foo.xml' +); + +is_deeply( + [ $not->({ PATH_INFO => '/foo.html' }) ], + [], + '!.html+.* does not match /foo.html' +); + +is_deeply( + [ $not->({ PATH_INFO => '/foo' }) ], + [], + '!.html+.* does not match /foo' +); + +my $sub = $dp->parse_dispatch_specification('/foo/*/...'); + +is_deeply( + [ $sub->({ PATH_INFO => '/foo/1/bar' }) ], + [ { PATH_INFO => '/bar' }, 1 ], + '/foo/*/... matches /foo/1/bar and strips to /bar' +); + +is_deeply( + [ $sub->({ PATH_INFO => '/foo/1/' }) ], + [ { PATH_INFO => '/' }, 1 ], + '/foo/*/... matches /foo/1/bar and strips to /' +); + +is_deeply( + [ $sub->({ PATH_INFO => '/foo/1' }) ], + [], + '/foo/*/... does not match /foo/1 (no trailing /)' +); + +my $q = 'foo=FOO&bar=BAR1&baz=one+two&quux=QUUX1&quux=QUUX2' + .'&bar=BAR2&quux=QUUX3&evil=%2F'; + +my %all_single = ( + foo => 'FOO', + bar => 'BAR2', + baz => 'one two', + quux => 'QUUX3', + evil => '/', +); + +my %all_multi = ( + foo => [ 'FOO' ], + bar => [ qw(BAR1 BAR2) ], + baz => [ 'one two' ], + quux => [ qw(QUUX1 QUUX2 QUUX3) ], + evil => [ '/' ], +); + +foreach my $lose ('?foo=','?:foo=','?@foo=','?:@foo=') { + my $foo = $dp->parse_dispatch_specification($lose); + + is_deeply( + [ $foo->({ QUERY_STRING => '' }) ], + [], + "${lose} fails with no query" + ); + + is_deeply( + [ $foo->({ QUERY_STRING => 'bar=baz' }) ], + [], + "${lose} fails with query missing foo key" + ); +} + +foreach my $win ( + [ '?foo=' => 'FOO' ], + [ '?:foo=' => { foo => 'FOO' } ], + [ '?spoo~' => undef ], + [ '?:spoo~' => {} ], + [ '?@spoo~' => [] ], + [ '?:@spoo~' => { spoo => [] } ], + [ '?bar=' => 'BAR2' ], + [ '?:bar=' => { bar => 'BAR2' } ], + [ '?@bar=' => [ qw(BAR1 BAR2) ] ], + [ '?:@bar=' => { bar => [ qw(BAR1 BAR2) ] } ], + [ '?foo=&@bar=' => 'FOO', [ qw(BAR1 BAR2) ] ], + [ '?foo=&:@bar=' => 'FOO', { bar => [ qw(BAR1 BAR2) ] } ], + [ '?:foo=&:@bar=' => { foo => 'FOO', bar => [ qw(BAR1 BAR2) ] } ], + [ '?:baz=&:evil=' => { baz => 'one two', evil => '/' } ], + [ '?*' => \%all_single ], + [ '?@*' => \%all_multi ], + [ '?foo=&@*' => 'FOO', do { my %h = %all_multi; delete $h{foo}; \%h } ], + [ '?:foo=&@*' => { %all_multi, foo => 'FOO' } ], + [ '?:@bar=&*' => { %all_single, bar => [ qw(BAR1 BAR2) ] } ], +) { + my ($spec, @res) = @$win; + my $match = $dp->parse_dispatch_specification($spec); +#use Data::Dump::Streamer; warn Dump($match); + is_deeply( + [ $match->({ QUERY_STRING => $q }) ], + [ {}, @res ], + "${spec} matches correctly" + ); +}