restructured fahrplanparser.getRowContents and added some tests

source/app.d

@@ -16,10 +16,12 @@ void main(string[] args)
     string fileName;
     string busStop = "Universität Regensburg";
     string substitutionFileName = "replacement.txt";
+
     auto helpInformation = getopt(args,
             "file|f", "The file that the data is written to.", &fileName,
             "stop|s", "The bus stop for which to fetch data.", &busStop,
             "replacement-file|r", "The file that contais the direction name replacement info.", &substitutionFileName);
+
     if (helpInformation.helpWanted)
     {
         defaultGetoptPrinter("Some information about the program.", helpInformation.options);
@@ -43,7 +45,8 @@ void main(string[] args)
     if (fileName !is null)
     {
         auto outfile = File(fileName, "w");
-        scope(exit) outfile.close;
+        scope (exit)
+            outfile.close;
         outfile.writeln(output);
     }
     else

source/fahrplanparser.d

@@ -6,16 +6,28 @@ import std.conv : to;
 import std.datetime : dur, TimeOfDay;
 import std.regex : ctRegex, matchAll;
 import std.string : strip;
-import std.typecons : tuple;
+import std.typecons : tuple, Tuple;
 
 import kxml.xml : readDocument, XmlNode;
 
 import substitution;
 
+private:
+
+enum ScheduleHeadings
+{
+    date,
+    departure,
+    line,
+    direction,
+    platform
+}
+
 public:
 
 auto parsedFahrplan(in string data)
 {
+    // dfmt off
     return data.readDocument
         .parseXPath(`//table[@id="departureMonitor"]/tbody/tr`)[1 .. $]
         .getRowContents
@@ -24,6 +36,7 @@ auto parsedFahrplan(in string data)
                     "delay" : a[0].parseTime[1].total!"minutes".to!string,
                     "line" : a[1],
                     "direction" : a[2].substitute]);
+    // dfmt on
 }
 
 private:
@@ -65,6 +78,7 @@ auto parseTime(in string input) @safe
 @safe unittest
 {
     import std.exception : assertThrown;
+
     assertThrown(parseTime(""));
     assertThrown(parseTime("lkeqf"));
     assertThrown(parseTime(":"));
@@ -88,9 +102,59 @@ auto parseTime(in string input) @safe
 
 auto getRowContents(XmlNode[] rows)
 {
-    return rows.map!(row => row.parseXPath("//td")[1 .. $ - 1].map!((column) {
-            auto link = column.parseXPath("//a");
-            if (!link.empty)
-                return link.front.getCData.replace("...", "");
-            return column.getCData;}));
+    return rows.map!(x => getRowContent(x));
+}
+
+auto getRowContent(XmlNode row)
+{
+    return row.parseXPath("//td")[ScheduleHeadings.departure .. ScheduleHeadings.direction + 1].map!(
+            cell => stripLinks(cell));
+}
+
+auto stripLinks(XmlNode cell)
+{
+    auto links = cell.parseXPath("//a");
+    if (links.empty)
+    {
+        return cell.getCData;
+    }
+    else
+    {
+        return links.front.getCData.replace("...", "");
+    }
+}
+
+@system unittest
+{
+    auto foo = new XmlNode("foo");
+    assert(foo.stripLinks == "");
+
+    auto link = new XmlNode("a");
+    link.setCData("test");
+    foo.addChild(link);
+    assert(foo.stripLinks == "test");
+
+    link.setCData("test2...");
+    assert(foo.stripLinks == "test2");
+
+    auto bar = new XmlNode("bar");
+    bar.setCData("test3");
+    assert(bar.stripLinks == "test3");
+
+    bar.addChild(link);
+    assert(bar.stripLinks == "test2");
+
+    auto baz = new XmlNode("baz");
+    auto subNode = new XmlNode("subNode");
+    baz.addChild(subNode);
+    assert(baz.stripLinks == "");
+
+    baz.addChild(link);
+    assert(baz.stripLinks == "test2");
+
+    baz.addCData("test4");
+    assert(baz.stripLinks == "test2");
+
+    baz.removeChild(link);
+    assert(baz.stripLinks == "test4");
 }