Correct Snippet interpretation (but leave old marker-split algorithm as fallback and for testing)
Change-Id: I9c93b3b881a9e548b064145787d83de98a2a5576
diff --git a/plugin/src/test/java/de/ids_mannheim/korap/plkexport/RtfExportTest.java b/plugin/src/test/java/de/ids_mannheim/korap/plkexport/RtfExportTest.java
index c191f18..e979cc1 100644
--- a/plugin/src/test/java/de/ids_mannheim/korap/plkexport/RtfExportTest.java
+++ b/plugin/src/test/java/de/ids_mannheim/korap/plkexport/RtfExportTest.java
@@ -23,7 +23,7 @@
Response resp = rtf.serve().build();
String x = (String) resp.getEntity();
resp.close();
- assertTrue(x.contains("\\footer\\pard\\ql\\fs18\\f0 @ Institut"));
+ assertTrue(x.contains("\\footer\\pard\\qr\\fs18\\f0 @ Institut"));
assertTrue(x.contains("Institut f\\u252\\'fcr Deutsche"));
};
@@ -31,21 +31,22 @@
public void testInitFull () throws IOException {
RtfExporter rtf = new RtfExporter();
rtf.init("{\"meta\":\"ja\",\"collection\":\"hm\",\"query\":\"cool\"," +
- "\"matches\":["+
- "{\"author\":\"Goethe\","+
- "\"title\":\"Title1\","+
- "\"pubDate\":\"20051103\","+
- "\"textSigle\":\"RTF/G59/34284\","+
- "\"snippet\":\"Simple <mark>match1</mark> Snippet\"}"+
- ","+
- "{\"author\":\"Schiller\","+
- "\"title\":\"Title2\","+
- "\"pubDate\":\"20051104\","+
- "\"textSigle\":\"RTF/G59/34285\","+
- "\"snippet\":\"<span class=\\\"context-left\\\"><span class=\\\"more\\\"></span>"+
- "Simpler <mark>"match2"</mark> Snippet"+
- "<span class=\\\"more\\\"></span></span>\"}"+
- "]}");
+ "\"matches\":["+
+ "{\"author\":\"Goethe\","+
+ "\"title\":\"Title1\","+
+ "\"pubDate\":\"20051103\","+
+ "\"textSigle\":\"RTF/G59/34284\","+
+ "\"snippet\":\"Simple <mark>match1</mark> Snippet\"}"+
+ ","+
+ "{\"author\":\"Schiller\","+
+ "\"title\":\"Title2\","+
+ "\"pubDate\":\"20051104\","+
+ "\"textSigle\":\"RTF/G59/34285\","+
+ "\"snippet\":\"<span class=\\\"context-left\\\"><span class=\\\"more\\\"></span>"+
+ "Simpler </span><span class=\\\"match\\\"><mark>"match2"</mark></span>"+
+ "<span class=\\\"context-right\\\"> Snippet"+
+ "<span class=\\\"more\\\"></span></span>\"}"+
+ "]}");
Response resp = rtf.serve().build();
String x = (String) resp.getEntity();
diff --git a/plugin/src/test/java/de/ids_mannheim/korap/plkexport/SnippetTest.java b/plugin/src/test/java/de/ids_mannheim/korap/plkexport/SnippetTest.java
index 18ded61..746d3dc 100644
--- a/plugin/src/test/java/de/ids_mannheim/korap/plkexport/SnippetTest.java
+++ b/plugin/src/test/java/de/ids_mannheim/korap/plkexport/SnippetTest.java
@@ -16,8 +16,8 @@
@Test
public void testSimple () {
Snippet s = new Snippet("<span class=\"context-left\">Der </span><span class=\"match\"><mark>Plagegeist</mark></span><span class=\"context-right\"> ging um</span>");
- assertEquals(s.getLeft(),"Der");
- assertEquals(s.getRight(),"ging um");
+ assertEquals(s.getLeft(),"Der ");
+ assertEquals(s.getRight()," ging um");
assertEquals(s.getMark(),"Plagegeist");
assertFalse(s.hasMoreLeft());
assertFalse(s.hasMoreRight());
@@ -39,4 +39,14 @@
assertTrue(s.hasMoreLeft());
assertTrue(s.hasMoreRight());
};
+
+ @Test
+ public void testMultipleMarks () {
+ Snippet s = new Snippet("<span class=\"context-left\"><span class=\"more\"></span>Figueras (gegen 1030, Kopialbuch der Abtei von Saint-Pé-de-Bigorre), Figeres (1154, laut Pierre de Marcas Buch Histoire de Béarn), Figueres (1421, Urkunden der Vicomté von Béarn), Higueres (1750, Karte von Cassini), Higueres (1793, Notice Communale) und Higueres und Higuères (1801, Bulletin </span><span class=\"match\"><mark><mark class=\"class-2 level-0\">des <mark class=\"class-1 level-1\">lois</mark></mark><mark class=\"class-1 level-1\">). Toponyme</mark></mark></span><span class=\"context-right\"> und Erwähnungen von Souye waren: Soyge und Soya (1538 bzw. 1547, Manuskriptsammlung des 16. bis 18. Jahrhunderts), Souia (1645, Volkszählung von Morlaàs), Souge und Souie (1675 bzw. 1682, Manuskriptsammlung des 16. bis 18. Jahrhunderts), Souge (1750, Karte von Cassini), Souye<span class=\"more\"></span></span>");
+ assertEquals(s.getLeft(), "Figueras (gegen 1030, Kopialbuch der Abtei von Saint-Pé-de-Bigorre), Figeres (1154, laut Pierre de Marcas Buch Histoire de Béarn), Figueres (1421, Urkunden der Vicomté von Béarn), Higueres (1750, Karte von Cassini), Higueres (1793, Notice Communale) und Higueres und Higuères (1801, Bulletin ");
+ assertEquals(s.getRight()," und Erwähnungen von Souye waren: Soyge und Soya (1538 bzw. 1547, Manuskriptsammlung des 16. bis 18. Jahrhunderts), Souia (1645, Volkszählung von Morlaàs), Souge und Souie (1675 bzw. 1682, Manuskriptsammlung des 16. bis 18. Jahrhunderts), Souge (1750, Karte von Cassini), Souye");
+ assertEquals(s.getMark(),"des lois). Toponyme");
+ assertTrue(s.hasMoreLeft());
+ assertTrue(s.hasMoreRight());
+ };
};