]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/plack.t
www: redirect /$MESSAGE_ID/f/ endpoints
[public-inbox.git] / t / plack.t
index 3bc4433f9ae3ce73cedf1d124790a8e238c8ed27..1ae58731ec370bb2a8fe01547023db9861026748 100644 (file)
--- a/t/plack.t
+++ b/t/plack.t
@@ -1,4 +1,4 @@
-# Copyright (C) 2014, Eric Wong <normalperson@yhbt.net> and all contributors
+# Copyright (C) 2014-2015 all contributors <meta@public-inbox.org>
 # License: AGPLv3 or later (https://www.gnu.org/licenses/agpl-3.0.txt)
 use strict;
 use warnings;
@@ -9,7 +9,7 @@ use Cwd;
 use IPC::Run qw/run/;
 my $psgi = "examples/public-inbox.psgi";
 my $mda = "blib/script/public-inbox-mda";
-my $tmpdir = tempdir(CLEANUP => 1);
+my $tmpdir = tempdir('pi-plack-XXXXXX', TMPDIR => 1, CLEANUP => 1);
 my $home = "$tmpdir/pi-home";
 my $pi_home = "$home/.public-inbox";
 my $pi_config = "$pi_home/config";
@@ -20,15 +20,15 @@ my $addr = 'test-public@example.com';
 my $cfgpfx = "publicinbox.test";
 my $failbox = "$home/fail.mbox";
 local $ENV{PI_EMERGENCY} = $failbox;
+my @mods = qw(HTTP::Request::Common Plack::Request Plack::Test
+       Mail::Thread URI::Escape);
+foreach my $mod (@mods) {
+       eval "require $mod";
+       plan skip_all => "$mod missing for plack.t" if $@;
+}
 
-our $have_plack;
-eval {
-       require Plack::Request;
-       eval 'use Plack::Test; use HTTP::Request::Common';
-       $have_plack = 1;
-};
-SKIP: {
-       skip 'Plack not installed', 1 unless $have_plack;
+foreach my $mod (@mods) { use_ok $mod; }
+{
        ok(-f $psgi, "psgi example file found");
        ok(-x "$main_bin/spamc",
                "spamc ham mock found (run in top of source tree");
@@ -81,26 +81,112 @@ EOF
                is($to, $res->header('Location'), 'redirect location matches');
        });
 
+       my $pfx = 'http://example.com/test';
+       foreach my $t (qw(t T)) {
+               test_psgi($app, sub {
+                       my ($cb) = @_;
+                       my $u = $pfx . "/blah%40example.com/$t";
+                       my $res = $cb->(GET($u));
+                       is(301, $res->code, "redirect for missing /");
+                       my $location = $res->header('Location');
+                       like($location, qr!/\Q$t\E/#u\z!,
+                               'redirected with missing /');
+               });
+       }
+       foreach my $t (qw(f)) {
+               test_psgi($app, sub {
+                       my ($cb) = @_;
+                       my $u = $pfx . "/blah%40example.com/$t";
+                       my $res = $cb->(GET($u));
+                       is(301, $res->code, "redirect for legacy /f");
+                       my $location = $res->header('Location');
+                       like($location, qr!/blah%40example\.com/\z!,
+                               'redirected with missing /');
+               });
+       }
+
        test_psgi($app, sub {
                my ($cb) = @_;
-               my $atomurl = 'http://example.com/test/atom.xml';
+               my $atomurl = 'http://example.com/test/new.atom';
                my $res = $cb->(GET('http://example.com/test/'));
                is(200, $res->code, 'success response received');
                like($res->content, qr!href="\Q$atomurl\E"!,
                        'atom URL generated');
-               like($res->content, qr!href="m/blah%40example\.com\.html"!,
+               like($res->content, qr!href="blah%40example\.com/"!,
                        'index generated');
        });
 
        test_psgi($app, sub {
                my ($cb) = @_;
-               my $pfx = 'http://example.com/test';
                my $res = $cb->(GET($pfx . '/atom.xml'));
                is(200, $res->code, 'success response received for atom');
                like($res->content,
-                       qr!link href="\Q$pfx\E/m/blah%40example\.com\.html"!,
+                       qr!link\s+href="\Q$pfx\E/blah%40example\.com/"!s,
                        'atom feed generated correct URL');
        });
+
+       test_psgi($app, sub {
+               my ($cb) = @_;
+               my $path = '/blah%40example.com/';
+               my $res = $cb->(GET($pfx . $path));
+               is(200, $res->code, "success for $path");
+               like($res->content, qr!<title>hihi - Me</title>!,
+                       "HTML returned");
+
+               $path .= 'f/';
+               $res = $cb->(GET($pfx . $path));
+               is(301, $res->code, "redirect for $path");
+               my $location = $res->header('Location');
+               like($location, qr!/blah%40example\.com/\z!,
+                       '/$MESSAGE_ID/f/ redirected to /$MESSAGE_ID/');
+       });
+
+       test_psgi($app, sub {
+               my ($cb) = @_;
+               my $res = $cb->(GET($pfx . '/blah%40example.com/raw'));
+               is(200, $res->code, 'success response received for /*/raw');
+               like($res->content, qr!^From !sm, "mbox returned");
+       });
+
+       # legacy redirects
+       foreach my $t (qw(m f)) {
+               test_psgi($app, sub {
+                       my ($cb) = @_;
+                       my $res = $cb->(GET($pfx . "/$t/blah%40example.com.txt"));
+                       is(301, $res->code, "redirect for old $t .txt link");
+                       my $location = $res->header('Location');
+                       like($location, qr!/blah%40example\.com/raw\z!,
+                               ".txt redirected to /raw");
+               });
+       }
+
+       my %umap = (
+               'm' => '',
+               'f' => '',
+               't' => 't/',
+       );
+       while (my ($t, $e) = each %umap) {
+               test_psgi($app, sub {
+                       my ($cb) = @_;
+                       my $res = $cb->(GET($pfx . "/$t/blah%40example.com.html"));
+                       is(301, $res->code, "redirect for old $t .html link");
+                       my $location = $res->header('Location');
+                       like($location,
+                               qr!/blah%40example\.com/$e(?:#u)?\z!,
+                               ".html redirected to new location");
+               });
+       }
+       foreach my $sfx (qw(mbox mbox.gz)) {
+               test_psgi($app, sub {
+                       my ($cb) = @_;
+                       my $res = $cb->(GET($pfx . "/t/blah%40example.com.$sfx"));
+                       is(301, $res->code, 'redirect for old thread link');
+                       my $location = $res->header('Location');
+                       like($location,
+                            qr!/blah%40example\.com/t\.mbox(?:\.gz)?\z!,
+                            "$sfx redirected to /mbox.gz");
+               });
+       }
 }
 
 done_testing();