Added Node.js unit tests for core JavaScript utilities (#636)

* Added Node.js unit tests for core JavaScript utilities

Added tests for utils.js (unaccented, UpdateQueryString, locationOf,
sortedInsert, Ranges) and buggytrack.js (cyrb53) using Node's built-in
test runner. Zero new dependencies.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Address review feedback: relocate tests, improve assertions, add coverage

- Moved test files from test/ to assets/js/tests/ (closer to source)
- Added assets/js/tests/ to _config.yml exclude list
- Updated categoryName tests to validate HTML structure instead of specific names
- Added tests for addMatchHighlights and getBlurbForResult
- Added inline comments explaining why vm.createContext needs explicit globals

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: load search_functions.js directly, add handleSearchMessage tests

Address review feedback:
- Remove extractFunction parser since search_functions.js now exists
  as a standalone file (after #637 merged)
- Load search_functions.js directly via vm sandbox
- Add handleSearchMessage tests covering: stop word handling,
  +prefix logic, fallback search, filterquery filtering,
  and result structure validation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* test: add +-, --, -+ prefix combination tests for handleSearchMessage

Address review feedback from khemarato: add test cases covering
all prefix combinations to verify existing +/- prefixes are preserved.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* test: address review feedback — add html-validate and missing test cases

* fix(test): add HTML validation in categoryName loop + flip assertion order

Address review nits:
- Add assertValidHtml() for every category in the knownCategories loop
- Flip assertion order in "finds the section with the most matches" to
  catch incorrect greedy implementations by checking the negative first

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix(search): use numeric for-loop instead of for-in on positions array

for (var i in positions) yields string keys ("0", "1", ...),
causing j = i + 1 to produce "01" (string concatenation) instead
of 1 (numeric addition). This made the inner while-loop skip
positions[1] for i=0 and exit immediately for all other starting
indices, so the algorithm always selected the first match's window
regardless of where the densest cluster was.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* test(search): flip test data to catch greedy blurb selection

Place the lone IGNORE match before the cluster (MATCH, ME, INSTEAD)
so that a greedy algorithm picking the first match would fail.
Also add assertValidHtml() in the categoryName loop per review.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: xr843

_config.yml

@@ -88,6 +88,7 @@ exclude:
   - scripts/
   - .github/
   - .obsidian/
+  - assets/js/tests/
 
 permalink: pretty
 

assets/js/search_functions.js

@@ -67,7 +67,7 @@ function getBlurbForResult(result, item, positions) {
     // Calculate the best section of the content to blurb
     var best_i = -1;
     var best_n = 0;
-    for (var i in positions) {
+    for (var i = 0; i < positions.length; i++) {
         var n = 1;
         var j = i+1;
         while (j < positions.length) {

assets/js/tests/buggytrack.test.js

@@ -0,0 +1,77 @@
+const { describe, it } = require('node:test');
+const assert = require('node:assert/strict');
+const vm = require('node:vm');
+const fs = require('node:fs');
+const path = require('node:path');
+
+// Load only the cyrb53 function from buggytrack.js
+// The file also creates a BuggyTracker that needs `document`, so we
+// provide minimal DOM stubs to let the file evaluate without errors.
+const src = fs.readFileSync(
+  path.join(__dirname, '..', 'buggytrack.js'),
+  'utf-8'
+);
+
+// vm.createContext() creates a bare sandbox with no built-in globals.
+// Unlike the main Node.js runtime, Math, Array, etc. must be provided explicitly.
+const sandbox = {
+  Math,
+  console,
+  localStorage: {
+    getItem() { return null; },
+    setItem() {},
+  },
+  window: { WEBSITE_SECTION: 'test' },
+  gtag() {},
+  document: {
+    createElement() {
+      return { href: '', host: '', hostname: '', pathname: '' };
+    },
+    location: { host: 'example.com', hostname: 'example.com', pathname: '/', search: '', href: 'https://example.com/' },
+    addEventListener() {},
+    referrer: '',
+    cookie: '',
+  },
+};
+// BuggyTracker references `document` as a bare global
+sandbox.self = sandbox;
+vm.createContext(sandbox);
+vm.runInContext(src + '\nthis.cyrb53 = cyrb53;\n', sandbox);
+
+const { cyrb53 } = sandbox;
+
+describe('cyrb53', () => {
+  it('returns a number', () => {
+    assert.equal(typeof cyrb53('hello'), 'number');
+  });
+
+  it('is deterministic (same input gives same output)', () => {
+    const a = cyrb53('test string');
+    const b = cyrb53('test string');
+    assert.equal(a, b);
+  });
+
+  it('produces different hashes for different inputs', () => {
+    const a = cyrb53('hello');
+    const b = cyrb53('world');
+    assert.notEqual(a, b);
+  });
+
+  it('handles empty string', () => {
+    const h = cyrb53('');
+    assert.equal(typeof h, 'number');
+    assert.ok(Number.isFinite(h));
+  });
+
+  it('handles unicode strings', () => {
+    const h = cyrb53('āgama nibbāna');
+    assert.equal(typeof h, 'number');
+    assert.ok(Number.isFinite(h));
+  });
+
+  it('returns non-negative values', () => {
+    for (const s of ['a', 'bb', 'ccc', '', 'test123']) {
+      assert.ok(cyrb53(s) >= 0, `Expected non-negative hash for "${s}"`);
+    }
+  });
+});