add TODO notes

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

diff --git a/t/autoindex.t b/t/autoindex.t
index 04d668fbc8fc159b54194852a9db5119b4b52ae6..d16e44ca8ebca7a14f5d3bcc601b6add1aeae47d 100755 (executable)
--- a/t/autoindex.t
+++ b/t/autoindex.t
@@ -3,7 +3,7 @@ package IkiWiki;
 
 use warnings;
 use strict;
 
 use warnings;
 use strict;

-use Test::More tests => 28;
+use Test::More tests => 38;

 
 BEGIN { use_ok("IkiWiki"); }
 BEGIN { use_ok("IkiWiki::Render"); }
 
 BEGIN { use_ok("IkiWiki"); }
 BEGIN { use_ok("IkiWiki::Render"); }


@@ -26,6 +26,7 @@ $config{userdir} = "users";
 $config{tagbase} = "tags";
 $config{default_pageext} = "mdwn";
 $config{wiki_file_prune_regexps} = [qr/^\./];
 $config{tagbase} = "tags";
 $config{default_pageext} = "mdwn";
 $config{wiki_file_prune_regexps} = [qr/^\./];

+$config{autoindex_commit} = 0;

 
 is(checkconfig(), 1);
 
 
 is(checkconfig(), 1);
 


@@ -33,7 +34,8 @@ is(checkconfig(), 1);
 %pagesources=%pagemtime=%oldlinks=%links=%depends=%typedlinks=%oldtypedlinks=
 %destsources=%renderedfiles=%pagecase=%pagestate=();
 
 %pagesources=%pagemtime=%oldlinks=%links=%depends=%typedlinks=%oldtypedlinks=
 %destsources=%renderedfiles=%pagecase=%pagestate=();
 

-# pages that (we claim) were deleted in an earlier pass
+# Pages that (we claim) were deleted in an earlier pass. We're using deleted,
+# not autofile, to test backwards compat.

 $wikistate{autoindex}{deleted}{deleted} = 1;
 $wikistate{autoindex}{deleted}{expunged} = 1;
 $wikistate{autoindex}{deleted}{reinstated} = 1;
 $wikistate{autoindex}{deleted}{deleted} = 1;
 $wikistate{autoindex}{deleted}{expunged} = 1;
 $wikistate{autoindex}{deleted}{reinstated} = 1;


@@ -66,44 +68,67 @@ my @del;
 
 IkiWiki::Plugin::autoindex::refresh();
 
 
 IkiWiki::Plugin::autoindex::refresh();
 

-# these pages are still on record as having been deleted, because they have
+# this page is still on record as having been deleted, because it has

 # a reason to be re-created
 # a reason to be re-created

-is($wikistate{autoindex}{deleted}{deleted}, 1);
-is($wikistate{autoindex}{deleted}{gone}, 1);
+is($wikistate{autoindex}{autofile}{"deleted.mdwn"}, 1);
+is($autofiles{"deleted.mdwn"}{plugin}, "autoindex");
+%pages = ();
+@del = ();
+IkiWiki::gen_autofile("deleted.mdwn", \%pages, \@del);
+is_deeply(\%pages, {});
+is_deeply(\@del, []);

 ok(! -f "t/tmp/deleted.mdwn");
 ok(! -f "t/tmp/deleted.mdwn");

+
+# this page is tried as an autofile, but because it'll be in @del, it's not
+# actually created
+ok(! exists $wikistate{autoindex}{autofile}{"gone.mdwn"});
+%pages = ();
+@del = ("gone.mdwn");
+is($autofiles{"gone.mdwn"}{plugin}, "autoindex");
+IkiWiki::gen_autofile("gone.mdwn", \%pages, \@del);
+is_deeply(\%pages, {});
+is_deeply(\@del, ["gone.mdwn"]);

 ok(! -f "t/tmp/gone.mdwn");
 
 ok(! -f "t/tmp/gone.mdwn");
 

-# this page does not exist and has no reason to be re-created, so we forget
-# about it - it will be re-created if it gains sub-pages
-ok(! exists $wikistate{autoindex}{deleted}{expunged});
+# this page does not exist and has no reason to be re-created, but we no longer
+# have a special case for that - see
+# [[todo/autoindex_should_use_add__95__autofile]] - so it won't be created
+# even if it gains subpages later
+is($wikistate{autoindex}{autofile}{"expunged.mdwn"}, 1);
+ok(! exists $autofiles{"expunged.mdwn"});

 ok(! -f "t/tmp/expunged.mdwn");
 
 # a directory containing only an internal page shouldn't be indexed
 ok(! -f "t/tmp/expunged.mdwn");
 
 # a directory containing only an internal page shouldn't be indexed

-ok(! exists $wikistate{autoindex}{deleted}{has_internal});
+ok(! exists $wikistate{autoindex}{autofile}{"has_internal.mdwn"});
+ok(! exists $autofiles{"has_internal.mdwn"});

 ok(! -f "t/tmp/has_internal.mdwn");
 
 ok(! -f "t/tmp/has_internal.mdwn");
 

-# this page was re-created, so it drops off the radar
-ok(! exists $wikistate{autoindex}{deleted}{reinstated});
+# this page was re-created, but that no longer gets a special case
+# (see [[todo/autoindex_should_use_add__95__autofile]]) so it's the same as
+# deleted
+is($wikistate{autoindex}{autofile}{"reinstated.mdwn"}, 1);
+ok(! exists $autofiles{"reinstated.mdwn"});

 ok(! -f "t/tmp/reinstated.mdwn");
 
 # needs creating (deferred; part of the autofile mechanism now)
 ok(! -f "t/tmp/reinstated.mdwn");
 
 # needs creating (deferred; part of the autofile mechanism now)

-ok(! exists $wikistate{autoindex}{deleted}{tags});
+ok(! exists $wikistate{autoindex}{autofile}{"tags.mdwn"});

 %pages = ();
 @del = ();
 is($autofiles{"tags.mdwn"}{plugin}, "autoindex");
 IkiWiki::gen_autofile("tags.mdwn", \%pages, \@del);
 %pages = ();
 @del = ();
 is($autofiles{"tags.mdwn"}{plugin}, "autoindex");
 IkiWiki::gen_autofile("tags.mdwn", \%pages, \@del);

-is_deeply(\%pages, {"t/tmp/tags" => 1}) || diag explain \%pages;
-is_deeply(\@del, []) || diag explain \@del;
-ok(-s "t/tmp/tags.mdwn");
+is_deeply(\%pages, {"t/tmp/tags" => 1});
+is_deeply(\@del, []);
+ok(! -s "t/tmp/tags.mdwn");
+ok(-s "t/tmp/.ikiwiki/transient/tags.mdwn");

 
 # needs creating because of an attachment
 
 # needs creating because of an attachment

-ok(! exists $wikistate{autoindex}{deleted}{attached});
+ok(! exists $wikistate{autoindex}{autofile}{"attached.mdwn"});

 %pages = ();
 @del = ();
 is($autofiles{"attached.mdwn"}{plugin}, "autoindex");
 IkiWiki::gen_autofile("attached.mdwn", \%pages, \@del);
 %pages = ();
 @del = ();
 is($autofiles{"attached.mdwn"}{plugin}, "autoindex");
 IkiWiki::gen_autofile("attached.mdwn", \%pages, \@del);

-is_deeply(\%pages, {"t/tmp/attached" => 1}) || diag explain \%pages;
-is_deeply(\@del, []) || diag explain \@del;
-ok(-s "t/tmp/attached.mdwn");
+is_deeply(\%pages, {"t/tmp/attached" => 1});
+is_deeply(\@del, []);
+ok(-s "t/tmp/.ikiwiki/transient/attached.mdwn");

 
 1;
 
 1;