add regex testing for non-alpha characters

translate/src/modules/placeable/components/Highlight.test.jsx

@@ -60,6 +60,33 @@ describe('mark terms', () => {
     expect(container.querySelector('mark')).toBeNull();
   });
 
+  it('does not mark a term followed by _', () => {
+    const string = 'foo_bar text';
+    const terms = {
+      terms: [{ text: 'foo' }],
+    };
+
+    const { container } = mountMarker(string, terms);
+
+    expect(container.querySelector('mark')).toBeNull();
+  });
+
+  it.each(['[', '\\', ']', '^', '`'])(
+    'does not extend a term match past %s',
+    (char) => {
+      const string = `foo${char}bar text`;
+      const terms = {
+        terms: [{ text: 'foo' }],
+      };
+
+      const { container } = mountMarker(string, terms);
+      const marks = container.querySelectorAll('mark.term');
+
+      expect(marks).toHaveLength(1);
+      expect(marks[0].textContent).toBe('foo');
+    },
+  );
+
   it('marks longer terms first', () => {
     const string = 'This is a translation tool.';
     const terms = {

translate/src/modules/placeable/components/Highlight.tsx

@@ -112,7 +112,7 @@ export function Highlight({
       .map((t) => t.text)
       .sort((a, b) => (a.length < b.length ? 1 : -1));
     for (const term of sourceTerms) {
-      const re = new RegExp(`\\b${escapeRegExp(term)}[a-zA-z]*\\b`, 'gi');
+      const re = new RegExp(`\\b${escapeRegExp(term)}[a-zA-Z]*\\b`, 'gi');
       for (const match of source.matchAll(re)) {
         marks.push({
           index: match.index ?? -1,