Highlighting - Add more tests.

computablefacts · Feb 5, 2024 · 90fb537 · 90fb537
1 parent b106ba3
commit 90fb537
Show file tree

Hide file tree

Showing 2 changed files with 52 additions and 2 deletions.
diff --git a/src/strings.js b/src/strings.js
@@ -294,7 +294,7 @@ strings.highlight = function (text, patterns) {
     let highlightedSnippet = `${prefix.substring(
         begin)}<mark style="border-radius:3px;background:${position.color}">${infix}</mark>${suffix.substring(0, end)}`;
     const pages = prefix.split('\f' /* page separator */).map((page, index) => index);
-    const beginMark = highlightedSnippet.lastIndexOf('<mark');
+    const beginMark = highlightedSnippet.lastIndexOf('<m');
     const endMark = highlightedSnippet.lastIndexOf('</mark>');
 
     if (beginMark && (!endMark || beginMark > endMark)) {

diff --git a/src/strings.test.js b/src/strings.test.js
@@ -133,7 +133,7 @@ test('highlight', () => {
   });
 });
 
-test('highlight_overlaps', () => {
+test('highlight_overlaps_style', () => {
 
   const text = "Le stationnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx stationner ici.";
   const textHighlighted = 'Le <mark style="border-radius:3px;background:#fffec8">statio</mark>nnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx <mark style="border-radius:3px;background:#fffec8">statio</mark>nner ici.';
@@ -153,6 +153,56 @@ test('highlight_overlaps', () => {
     highlightedSnippet: 'Le <mark style="border-radius:3px;background:#fffec8">statio</mark>nnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx ',
   };
 
+  expect(strings.highlight(text, [pattern])).toEqual({
+    snippets: [snippet1, snippet2], text: textHighlighted
+  });
+});
+
+test('highlight_overlaps_begin_tag', () => {
+
+  const text = "Le stationnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx stationner ici.";
+  const textHighlighted = 'Le <mark style="border-radius:3px;background:#fffec8">statio</mark>nnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx <mark style="border-radius:3px;background:#fffec8">statio</mark>nner ici.';
+  const pattern = {
+    regexp: /statio/gims, color: "#fffec8"
+  };
+  const snippet1 = {
+    matchedText: "statio",
+    matchedPage: 1,
+    rawSnippet: 'x xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx stationner ici.',
+    highlightedSnippet: 'x xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx <mark style=\"border-radius:3px;background:#fffec8\">statio</mark>nner ici.',
+  };
+  const snippet2 = {
+    matchedText: "statio",
+    matchedPage: 1,
+    rawSnippet: 'Le stationnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx <mar',
+    highlightedSnippet: 'Le <mark style=\"border-radius:3px;background:#fffec8\">statio</mark>nnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx ',
+  };
+
+  expect(strings.highlight(text, [pattern])).toEqual({
+    snippets: [snippet1, snippet2], text: textHighlighted
+  });
+});
+
+test('highlight_overlaps_end_tag', () => {
+
+  const text = "Le stationnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx stationner ici.";
+  const textHighlighted = 'Le <mark style="border-radius:3px;background:#fffec8">statio</mark>nnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx <mark style="border-radius:3px;background:#fffec8">statio</mark>nner ici.';
+  const pattern = {
+    regexp: /statio/gims, color: "#fffec8"
+  };
+  const snippet1 = {
+    matchedText: "statio",
+    matchedPage: 1,
+    rawSnippet: 'x xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx stationner ici.',
+    highlightedSnippet: 'x xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx <mark style=\"border-radius:3px;background:#fffec8\">statio</mark>nner ici.',
+  };
+  const snippet2 = {
+    matchedText: "statio",
+    matchedPage: 1,
+    rawSnippet: 'Le stationnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx <mark style=\"border-radius:3px;background:#fffec8\">statio</m',
+    highlightedSnippet: 'Le <mark style=\"border-radius:3px;background:#fffec8\">statio</mark>nnement est interdit. Merci de ne pas xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx xxx ',
+  };
+
   expect(strings.highlight(text, [pattern])).toEqual({
     snippets: [snippet1, snippet2], text: textHighlighted
   });