提交: c1d9d4ea0b520b74c4bfdd259acde91f3fe17350 - medialist-cli (git)

CLI interface to medialist (fossil mirror)

Commit MetaInfo

• 修訂: c1d9d4ea0b520b74c4bfdd259acde91f3fe17350 (tree)
• 時間: 2021-11-13 17:35:56
• 作者: mio <stigma@disr...>
• Commiter: mio

Log Message

Add a unittest for ticket 5b28c4e8174fb4dd

FossilOrigin-Name: e86c92b52f314e15f7d33676123c6c11e6f381d8500eabe87d9d979898b51df6

Change Summary

• modified: update.d

差異

--- a/update.d
+++ b/update.d
@@ -144,7 +144,7 @@ handle_update(string program_name, string[] args, string data_dir)
   string oldStatus = "";
 
   import std.array : replicate;
-  
+
   string currentDateString = currentDate.toISOExtString();
 
   foreach (sectionIndent, section; sections)
@@ -215,7 +215,7 @@ handle_update(string program_name, string[] args, string data_dir)
   File f = File(filename, "w");
   foreach(l; contents)
     if (0 != l.length) f.writeln(l);
-  
+
   stdout.writefln("Successfully updated %s (%s, %s)", title, status, progress);
 
   return true;
@@ -248,3 +248,80 @@ example:
     10: Title -> COMPLETE -> 12/12", program_name, program_name,
                                      program_name);
 }
+
+/*
+ * Ticket 5b28c4e8174fb4dd
+ * After creating a new list and adding a new item to it, calling subsequent modifications
+ * (e.g. update and delete) would cause the "new" headers to be removed.
+ *
+ * This issue occurred when the "start_date", "end_date", and "last_updated" headers were
+ * first added, so the previous code assumes the header line was TITLE\tSTATUS\tPROGRESS,
+ * while the new code shouldn't assume the position of anything.
+ */
+unittest
+{
+	/*
+	 * I would like to make a "medialist library" so that you would
+	 * call something like ml_list_create (MediaList*, string) to
+	 * create a new list, but for now, we just "re-run" the program.
+	 *
+	 * Perhaps a command-based interface?
+	 */
+
+	import std.algorithm.searching : countUntil;
+	import std.file : getcwd, remove;
+	import std.stdio : File, writeln;
+	import std.string : split, strip;
+
+	import add : handle_add;
+	import create : handle_create;
+	import del : handle_delete;
+	import update : handle_update;
+	import util : NUMBER_OF_ML_HEADERS;
+
+	string[] addArgs = ["ml_testing", "item01"];
+	string[] updateArgs = ["ml_testing", "1", "-s", "COMPLETE"];
+	string[] deleteArgs = ["ml_testing", "1"];
+
+	handle_create("medialist-cli-unittest", "ml_testing", getcwd());
+	handle_add ("medialist-cli-unittest", addArgs, getcwd());
+
+	File ml_testing = File("ml_testing.tsv");
+	scope(exit) remove("ml_testing.tsv");
+
+	string firstLine = ml_testing.readln.strip;
+	assert (NUMBER_OF_ML_HEADERS == firstLine.split("\t").length, "Different number of headers");
+
+	ptrdiff_t lastUpdatedIndex = firstLine.split("\t").countUntil("last_updated");
+	assert (-1 != lastUpdatedIndex, "Couldn't find 'last_updated' header");
+
+	string secondLine = ml_testing.readln.strip;
+	assert ("" != secondLine.split("\t")[lastUpdatedIndex], "'last_updated' is initially empty");
+
+	ml_testing.close();
+
+	handle_update ("medialist-cli-unittest", updateArgs, getcwd());
+
+	ml_testing.open("ml_testing.tsv");
+
+	firstLine = ml_testing.readln.strip;
+	assert (NUMBER_OF_ML_HEADERS == firstLine.split("\t").length,
+		"Some headers were removed when updating list");
+	lastUpdatedIndex = firstLine.split("\t").countUntil("last_updated");
+	assert (-1 != lastUpdatedIndex,
+		"'last_updated' header was removed when updating the list");
+
+	ml_testing.close();
+
+	handle_delete ("medialist-cli-unittest", deleteArgs, getcwd());
+
+	ml_testing.open("ml_testing.tsv");
+
+	firstLine = ml_testing.readln.strip;
+	assert (NUMBER_OF_ML_HEADERS == firstLine.split("\t").length,
+		"Some headers were removed when deleting from list");
+	lastUpdatedIndex = firstLine.split("\t").countUntil("last_updated");
+	assert (-1 != lastUpdatedIndex,
+		"'last_updated' header was removed when deleting from the list");
+}
+