testcase for completion

Author: Elimpizza

src/annotator/anchoring/test/html-boundary-test.js

@@ -40,4 +40,32 @@ describe('annotator/anchoring/html boundary handling', () => {
 
     assert.equal(anchoredRange.toString(), 'bar');
   });
+
+  it('describes selection spanning <br> with a space in exact', () => {
+    // This is the bug the PR fixes: a selection that crosses a `<br>` used
+    // to produce `exact: "foobar"` because `textContent` doesn't include a
+    // character for the `<br>`. With the substitution, the stored quote
+    // reflects what the user actually sees in the rendered page.
+    const range = document.createRange();
+    range.selectNodeContents(container.querySelector('p'));
+
+    const selectors = html.describe(container, range);
+    const quoteSel = selectors.find(s => s.type === 'TextQuoteSelector');
+
+    assert.equal(quoteSel.exact, 'foo bar');
+  });
+
+  it('anchors a selector with the substituted space back to the original range', async () => {
+    const range = document.createRange();
+    range.selectNodeContents(container.querySelector('p'));
+
+    const selectors = html.describe(container, range);
+    const anchoredRange = await html.anchor(container, selectors);
+
+    // The anchored range covers the same DOM content as the original.
+    // `toString()` on either Range returns "foobar" because `<br>` has no
+    // character contribution to `textContent` — the substituted space lives
+    // only in the stored `exact`.
+    assert.equal(anchoredRange.toString(), 'foobar');
+  });
 });