inadvertent improvement

[git.ikiwiki.info.git] / t / podcast.t

diff --git a/t/podcast.t b/t/podcast.t
index 3125a7e55d79d4a1ae253effbfe5a634ec2c7d60..7b0987e21321f2e7738b71903a8381da3361a0b1 100755 (executable)
--- a/t/podcast.t
+++ b/t/podcast.t
@@ -3,27 +3,45 @@ use warnings;
 use strict;
 
 BEGIN {
 use strict;
 
 BEGIN {

-       eval q{use XML::Feed; use HTML::Parser; use HTML::LinkExtor};
+       eval q{use XML::Feed; use HTML::Parser; use HTML::LinkExtor; use File::MimeInfo};

        if ($@) {
                eval q{use Test::More skip_all =>
        if ($@) {
                eval q{use Test::More skip_all =>

-                       "XML::Feed and/or HTML::Parser not available"};
+                       "XML::Feed and/or HTML::Parser or File::MimeInfo not available"};

        }
        else {
        }
        else {

-               eval q{use Test::More tests => 81};
+               eval q{use Test::More};

        }
 }
 
 use Cwd;
        }
 }
 
 use Cwd;

+use File::Basename;
+
+my $installed = $ENV{INSTALLED_TESTS};
+
+my @base_command;
+if ($installed) {
+       ok(1, "running installed");
+       @base_command = qw(ikiwiki);
+}
+else {
+       ok(! system("make -s ikiwiki.out"));
+       @base_command = ("perl", "-I".getcwd, qw(./ikiwiki.out
+               --underlaydir=underlays/basewiki
+               --set underlaydirbase=underlays
+               --templatedir=templates));
+}

 
 my $tmp = 't/tmp';
 my $statedir = 't/tinypodcast/.ikiwiki';
 
 
 my $tmp = 't/tmp';
 my $statedir = 't/tinypodcast/.ikiwiki';
 

-sub simple_podcast {
+sub podcast {
+       my $podcast_style = shift;
+

        my $baseurl = 'http://example.com';
        my $baseurl = 'http://example.com';

-       my @command = (qw(./ikiwiki.out -plugin inline -rss -atom));
-       push @command, qw(-underlaydir=underlays/basewiki);
-       push @command, qw(-set underlaydirbase=underlays -templatedir=templates);
-       push @command, "-url=$baseurl", qw(t/tinypodcast), "$tmp/out";
+       my @command = (@base_command, qw(--plugin inline --rss --atom));
+       push @command, "--url=$baseurl", qw(t/tinypodcast), "$tmp/out";
+
+       ok(! system("rm -rf $tmp $statedir"), q{setup});

 
        ok(! system("mkdir $tmp"),
                q{setup});
 
        ok(! system("mkdir $tmp"),
                q{setup});


@@ -34,15 +52,15 @@ sub simple_podcast {
                'simplepost'    => undef,
                'piano.mp3'     => 'audio/mpeg',
                'scroll.3gp'    => 'video/3gpp',
                'simplepost'    => undef,
                'piano.mp3'     => 'audio/mpeg',
                'scroll.3gp'    => 'video/3gpp',

-               'walter.ogg'    => 'video/x-theora+ogg',
+               'walter.ogg'    => 'audio/ogg',

        );
 
        for my $format (qw(atom rss)) {
        );
 
        for my $format (qw(atom rss)) {

-               my $feed = XML::Feed->parse("$tmp/out/simple/index.$format");
+               my $feed = XML::Feed->parse("$tmp/out/$podcast_style/index.$format");

 
 

-               is($feed->title, 'simple',
+               is($feed->title, $podcast_style,

                        qq{$format feed title});
                        qq{$format feed title});

-               is($feed->link, "$baseurl/simple/",
+               is($feed->link, "$baseurl/$podcast_style/",

                        qq{$format feed link});
                is($feed->description, 'wiki',
                        qq{$format feed description});
                        qq{$format feed link});
                is($feed->description, 'wiki',
                        qq{$format feed description});


@@ -80,25 +98,39 @@ sub simple_podcast {
                                        qq{$format $title enclosure length});
                        }
                        else {
                                        qq{$format $title enclosure length});
                        }
                        else {

-                               is($url, "$baseurl/$title/",
+                               # XXX hack hack hack
+                               my $expected_id = "$baseurl/$title/";
+                               $expected_id =~ s/\ /_/g;
+
+                               is($url, $expected_id,

                                        qq{$format $title id});
                                isnt($body, undef,
                                        qq{$format $title body text});
                                        qq{$format $title id});
                                isnt($body, undef,
                                        qq{$format $title body text});

-                               is($enclosure, undef,
-                                       qq{$format $title no enclosure});
+
+                               if ('fancy' eq $podcast_style) {
+                                       isnt($enclosure, undef,
+                                               qq{$format $title enclosure});
+                                       my $filename = basename($enclosure->url);
+                                       is($enclosure->type, $media_types{$filename},
+                                               qq{$format $title enclosure type});
+                                       cmp_ok($enclosure->length, '>', 0,
+                                               qq{$format $title enclosure length});
+                               }
+                               else {
+                                       is($enclosure, undef,
+                                               qq{$format $title no enclosure});
+                               }

                        }
                }
        }
                        }
                }
        }

-
-       ok(! system("rm -rf $tmp $statedir"), q{teardown});

 }
 
 sub single_page_html {
 }
 
 sub single_page_html {

-       my @command = (qw(./ikiwiki.out));
-       push @command, qw(-underlaydir=underlays/basewiki);
-       push @command, qw(-set underlaydirbase=underlays -templatedir=templates);
+       my @command = @base_command;

        push @command, qw(t/tinypodcast), "$tmp/out";
 
        push @command, qw(t/tinypodcast), "$tmp/out";
 

+       ok(! system("rm -rf $tmp $statedir"), q{setup});
+

        ok(! system("mkdir $tmp"),
                q{setup});
        ok(! system(@command),
        ok(! system("mkdir $tmp"),
                q{setup});
        ok(! system(@command),


@@ -107,20 +139,63 @@ sub single_page_html {
        my $html = "$tmp/out/pianopost/index.html";
        like(_extract_html_content($html, 'content'), qr/has content and/m,
                q{html body text});
        my $html = "$tmp/out/pianopost/index.html";
        like(_extract_html_content($html, 'content'), qr/has content and/m,
                q{html body text});

-       like(_extract_html_content($html, 'enclosure'), qr/this episode/m,
+       like(_extract_html_content($html, 'enclosure'), qr/Download/m,

                q{html enclosure});
        my ($href) = _extract_html_links($html, 'piano');
                q{html enclosure});
        my ($href) = _extract_html_links($html, 'piano');

-       ok(-f $href,
-               q{html enclosure exists});
+       is($href, '/piano.mp3',
+               q{html enclosure sans --url is site-absolute});

 
        $html = "$tmp/out/attempted_multiple_enclosures/index.html";
        like(_extract_html_content($html, 'content'), qr/has content and/m,
                q{html body text});
 
        $html = "$tmp/out/attempted_multiple_enclosures/index.html";
        like(_extract_html_content($html, 'content'), qr/has content and/m,
                q{html body text});

-       like(_extract_html_content($html, 'enclosure'), qr/this episode/m,
+       like(_extract_html_content($html, 'enclosure'), qr/Download/m,

                q{html enclosure});
        ($href) = _extract_html_links($html, 'walter');
                q{html enclosure});
        ($href) = _extract_html_links($html, 'walter');

-       ok(-f $href,
-               q{html enclosure exists});
+       is($href, '/walter.ogg',
+               q{html enclosure sans --url is site-absolute});
+
+       my $baseurl = 'http://example.com';
+       ok(! system(@command, "--url=$baseurl", q{--rebuild}));
+
+       $html = "$tmp/out/pianopost/index.html";
+       ($href) = _extract_html_links($html, 'piano');
+       is($href, "$baseurl/piano.mp3",
+               q{html enclosure with --url is fully absolute});
+
+       $html = "$tmp/out/attempted_multiple_enclosures/index.html";
+       ($href) = _extract_html_links($html, 'walter');
+       is($href, "$baseurl/walter.ogg",
+               q{html enclosure with --url is fully absolute});
+
+       ok(! system("rm -rf $tmp $statedir"), q{teardown});
+}
+
+sub inlined_pages_html {
+       my @command = (@base_command, qw(--plugin inline));
+       push @command, qw(t/tinypodcast), "$tmp/out";
+
+       ok(! system("rm -rf $tmp $statedir"), q{setup});
+
+       ok(! system("mkdir $tmp"),
+               q{setup});
+       ok(! system(@command),
+               q{build});
+
+       my $html = "$tmp/out/fancy/index.html";
+       my $contents = _extract_html_content($html, 'content');
+       like($contents, qr/has content and an/m,
+               q{html body text from pianopost});
+       like($contents, qr/has content and only one/m,
+               q{html body text from attempted_multiple_enclosures});
+       my $enclosures = _extract_html_content($html, 'inlineenclosure');
+       like($enclosures, qr/Download/m,
+               q{html enclosure});
+       my ($href) = _extract_html_links($html, 'piano.mp3');
+       is($href, '/piano.mp3',
+               q{html enclosure from pianopost sans --url});
+       ($href) = _extract_html_links($html, 'walter.ogg');
+       is($href, '/walter.ogg',
+               q{html enclosure from attempted_multiple_enclosures sans --url});

 
        ok(! system("rm -rf $tmp $statedir"), q{teardown});
 }
 
        ok(! system("rm -rf $tmp $statedir"), q{teardown});
 }


@@ -141,11 +216,6 @@ sub _extract_html_content {
                        my ($dtext) = @_;
                        $content .= $dtext;
                }, "dtext");
                        my ($dtext) = @_;
                        $content .= $dtext;
                }, "dtext");

-
-               $self->handler(end  => sub {
-                       my ($tag, $self) = @_;
-                       $self->eof if $tag eq $desired_tag;
-               }, "tagname,self");

        }, "tagname,self,attr");
 
        $p->parse_file($file) || die $!;
        }, "tagname,self,attr");
 
        $p->parse_file($file) || die $!;


@@ -170,5 +240,9 @@ sub _extract_html_links {
        return @hrefs;
 }
 
        return @hrefs;
 }
 

-simple_podcast();
+podcast('simple');

 single_page_html();
 single_page_html();

+inlined_pages_html();
+podcast('fancy');
+
+done_testing;