X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/cc0d875507671498e7cf9cd782357d32dbd88f66..b5b8c5cec:/t/podcast.t?ds=inline

diff --git a/t/podcast.t b/t/podcast.t
index a50b7ff2a..f036c3113 100755
--- a/t/podcast.t
+++ b/t/podcast.t
@@ -3,49 +3,56 @@ use warnings;
 use strict;
 
 BEGIN {
-	eval q{use XML::Feed; use HTML::Parser};
+	eval q{use XML::Feed; use HTML::Parser; use HTML::LinkExtor; use File::MimeInfo};
 	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 {
-		eval q{use Test::More tests => 77};
+		eval q{use Test::More tests => 136};
 	}
 }
 
+use Cwd;
+use File::Basename;
+
 my $tmp = 't/tmp';
 my $statedir = 't/tinypodcast/.ikiwiki';
 
-sub simple_podcast {
+sub podcast {
+	my $podcast_style = shift;
+
 	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";
 
-	ok(! system("mkdir $tmp"));
-	ok(! system(@command));
+	ok(! system("mkdir $tmp"),
+		q{setup});
+	ok(! system(@command),
+		q{build});
 
 	my %media_types = (
-		'post'		=> undef,
+		'simplepost'	=> undef,
 		'piano.mp3'	=> 'audio/mpeg',
 		'scroll.3gp'	=> 'video/3gpp',
 		'walter.ogg'	=> 'video/x-theora+ogg',
 	);
 
 	for my $format (qw(atom rss)) {
-		my $feed = XML::Feed->parse("$tmp/out/index.$format");
+		my $feed = XML::Feed->parse("$tmp/out/$podcast_style/index.$format");
 
-		is($feed->title, 'wiki',
+		is($feed->title, $podcast_style,
 			qq{$format feed title});
-		is($feed->link, "$baseurl/",
+		is($feed->link, "$baseurl/$podcast_style/",
 			qq{$format feed link});
-		is($feed->description, $feed->title,
+		is($feed->description, 'wiki',
 			qq{$format feed description});
 		if ('atom' eq $format) {
-			is($feed->author, $feed->title,
+			is($feed->author, $feed->description,
 				qq{$format feed author});
-			is($feed->id, "$baseurl/",
+			is($feed->id, $feed->link,
 				qq{$format feed id});
 			is($feed->generator, "ikiwiki",
 				qq{$format feed generator});
@@ -76,17 +83,33 @@ sub simple_podcast {
 					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});
-				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"));
+	ok(! system("rm -rf $tmp $statedir"), q{teardown});
 }
 
 sub single_page_html {
@@ -95,20 +118,73 @@ sub single_page_html {
 	push @command, qw(-set underlaydirbase=underlays -templatedir=templates);
 	push @command, qw(t/tinypodcast), "$tmp/out";
 
-	ok(! system("mkdir $tmp"));
-	ok(! system(@command));
-	my $html = "$tmp/out/podcast/index.html";
+	ok(! system("mkdir $tmp"),
+		q{setup});
+	ok(! system(@command),
+		q{build});
+
+	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/Download/m,
+		q{html enclosure});
+	my ($href) = _extract_html_links($html, 'piano');
+	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});
+	like(_extract_html_content($html, 'enclosure'), qr/Download/m,
+		q{html enclosure});
+	($href) = _extract_html_links($html, 'walter');
+	is($href, '/walter.ogg',
+		q{html enclosure sans -url is site-absolute});
 
-	my $body = _extract_html_content($html, 'content');
-	like($body, qr/article has content and/m, q{html body text});
+	my $baseurl = 'http://example.com';
+	ok(! system(@command, "-url=$baseurl", q{--rebuild}));
 
-	my $enclosure = _extract_html_content($html, 'enclosure');
-	like($enclosure, qr/Download this episode/m, q{html enclosure});
+	$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});
 
-	# XXX die if specified enclosure doesn't exist
-	# XXX die if more than one enclosure is specified
+	$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 = (qw(./ikiwiki.out -plugin inline));
+	push @command, qw(-underlaydir=underlays/basewiki);
+	push @command, qw(-set underlaydirbase=underlays -templatedir=templates);
+	push @command, qw(t/tinypodcast), "$tmp/out";
 
-	ok(! system("rm -rf $tmp $statedir"));
+	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});
 }
 
 sub _extract_html_content {
@@ -127,11 +203,6 @@ sub _extract_html_content {
 			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 $!;
@@ -139,5 +210,24 @@ sub _extract_html_content {
 	return $content;
 }
 
-simple_podcast();
+sub _extract_html_links {
+	my ($file, $desired_value) = @_;
+
+	my @hrefs = ();
+
+	my $p = HTML::LinkExtor->new(sub {
+		my ($tag, %attr) = @_;
+		return if $tag ne 'a';
+		return unless $attr{href} =~ qr/$desired_value/;
+		push(@hrefs, values %attr);
+	}, getcwd() . '/' . $file);
+
+	$p->parse_file($file);
+
+	return @hrefs;
+}
+
+podcast('simple');
 single_page_html();
+inlined_pages_html();
+podcast('fancy');