From: Amitai Schlair <schmonz-web-ikiwiki@schmonz.com>
Date: Sat, 21 Jan 2012 19:37:24 +0000 (-0500)
Subject: Extract subs: startup/shutdown, and tests for web and manual commits.
X-Git-Tag: 3.20120202~26^2~9^2~7
X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/commitdiff_plain/94c1f3bd337f53119d8e28fead322e85f000253e?hp=700d39d765e59f45d15d5b6dd81c187947bd308d

Extract subs: startup/shutdown, and tests for web and manual commits.

No functional change intended.
---

diff --git a/t/cvs.t b/t/cvs.t
index 5ed377ed5..2937ac2ed 100755
--- a/t/cvs.t
+++ b/t/cvs.t
@@ -29,60 +29,77 @@ use Test::More tests => 12;
 
 BEGIN { use_ok("IkiWiki"); }
 
-%config=IkiWiki::defaultconfig();
-$config{rcs} = "cvs";
-$config{srcdir} = "$dir/src";
-$config{cvsrepo} = "$dir/repo";
-$config{cvspath} = "ikiwiki";
-IkiWiki::loadplugins();
-IkiWiki::checkconfig();
+sub _startup {
+	_generate_minimal_config();
+	_create_test_repo();
+}
+
+sub _shutdown {
+	system "rm -rf $dir";
+}
+
+sub _generate_minimal_config {
+	%config=IkiWiki::defaultconfig();
+	$config{rcs} = "cvs";
+	$config{srcdir} = "$dir/src";
+	$config{cvsrepo} = "$dir/repo";
+	$config{cvspath} = "ikiwiki";
+	IkiWiki::loadplugins();
+	IkiWiki::checkconfig();
+}
 
-my $cvsrepo = "$dir/repo";
+sub _create_test_repo {
+	my $cvsrepo = "$dir/repo";
 
-system "cvs -d $cvsrepo init >/dev/null";
-system "mkdir $dir/ikiwiki >/dev/null";
-system "cd $dir/ikiwiki && cvs -d $cvsrepo import -m import ikiwiki VENDOR RELEASE >/dev/null";
-system "rm -rf $dir/ikiwiki >/dev/null";
-system "cvs -d $cvsrepo co -d $config{srcdir} ikiwiki >/dev/null";
+	system "cvs -d $cvsrepo init >/dev/null";
+	system "mkdir $dir/ikiwiki >/dev/null";
+	system "cd $dir/ikiwiki && cvs -d $cvsrepo import -m import ikiwiki VENDOR RELEASE >/dev/null";
+	system "rm -rf $dir/ikiwiki >/dev/null";
+	system "cvs -d $cvsrepo co -d $config{srcdir} ikiwiki >/dev/null";
+}
 
-# Web commit
-my $test1 = readfile("t/test1.mdwn");
-writefile('test1.mdwn', $config{srcdir}, $test1);
-IkiWiki::rcs_add("test1.mdwn");
-IkiWiki::rcs_commit(
-	files => "test1.mdwn",
-	message => "Added the first page",
-	token => "moo"
-);
+sub test_web_commit {
+	my $test1 = readfile("t/test1.mdwn");
+	writefile('test1.mdwn', $config{srcdir}, $test1);
+	IkiWiki::rcs_add("test1.mdwn");
+	IkiWiki::rcs_commit(
+		files => "test1.mdwn",
+		message => "Added the first page",
+		token => "moo"
+	);
 
-my @changes;
-@changes = IkiWiki::rcs_recentchanges(3);
+	my @changes = IkiWiki::rcs_recentchanges(3);
 
-is($#changes, 0);
-is($changes[0]{message}[0]{"line"}, "Added the first page");
-is($changes[0]{pages}[0]{"page"}, "test1");
+	is($#changes, 0);
+	is($changes[0]{message}[0]{"line"}, "Added the first page");
+	is($changes[0]{pages}[0]{"page"}, "test1");
+}
 
-# Manual commit
-my $message = "Added the second page";
+sub test_manual_commit {
+	my $message = "Added the second page";
 
-my $test2 = readfile("t/test2.mdwn");
-writefile('test2.mdwn', $config{srcdir}, $test2);
-system "cd $config{srcdir} && cvs add test2.mdwn >/dev/null 2>&1";
-system "cd $config{srcdir} && cvs commit -m \"$message\" test2.mdwn >/dev/null";
+	my $test2 = readfile("t/test2.mdwn");
+	writefile('test2.mdwn', $config{srcdir}, $test2);
+	system "cd $config{srcdir} && cvs add test2.mdwn >/dev/null 2>&1";
+	system "cd $config{srcdir} && cvs commit -m \"$message\" test2.mdwn >/dev/null";
 
-@changes = IkiWiki::rcs_recentchanges(3);
-is($#changes, 1);
-is($changes[0]{message}[0]{"line"}, $message);
-is($changes[0]{pages}[0]{"page"}, "test2");
-is($changes[1]{pages}[0]{"page"}, "test1");
+	my @changes = IkiWiki::rcs_recentchanges(3);
+	is($#changes, 1);
+	is($changes[0]{message}[0]{"line"}, $message);
+	is($changes[0]{pages}[0]{"page"}, "test2");
+	is($changes[1]{pages}[0]{"page"}, "test1");
 
-# extra slashes in the path shouldn't break things
-$config{cvspath} = "/ikiwiki//";
-IkiWiki::checkconfig();
-@changes = IkiWiki::rcs_recentchanges(3);
-is($#changes, 1);
-is($changes[0]{message}[0]{"line"}, $message);
-is($changes[0]{pages}[0]{"page"}, "test2");
-is($changes[1]{pages}[0]{"page"}, "test1");
+	# extra slashes in the path shouldn't break things
+	$config{cvspath} = "/ikiwiki//";
+	IkiWiki::checkconfig();
+	@changes = IkiWiki::rcs_recentchanges(3);
+	is($#changes, 1);
+	is($changes[0]{message}[0]{"line"}, $message);
+	is($changes[0]{pages}[0]{"page"}, "test2");
+	is($changes[1]{pages}[0]{"page"}, "test1");
+}
 
-system "rm -rf $dir";
+_startup();
+test_web_commit();
+test_manual_commit();
+_shutdown();