fix(backend-tests): un-skip api/ and admin/ subdirectory specs (#7789)

* fix(backend-tests): un-skip tests/backend/specs/{api,admin}/*

The pnpm test script's glob `tests/backend/specs/**.ts` only matched
files at the top level of tests/backend/specs/. Every spec under
tests/backend/specs/api/ (14 files) and tests/backend/specs/admin/
(2 files) has been silently skipped by CI — including the failing
tests reported in #7785, #7786, #7787, #7788.

Switch to passing the directories with `--extension ts --recursive`,
which makes mocha walk the tree the way --recursive is documented to.

Local run after this change picks up 16 additional spec files and
surfaces 6 newly-visible failures: 4 already filed (#7785–#7788) plus
one that wasn't yet filed (appendTextAuthor.ts:
"appendText without authorId does not attribute to any author").

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

* test(backend): regression check for tests/backend/specs/{api,admin} discovery

Read the pnpm test script from src/package.json, hand mocha the
same arguments under --dry-run --list-files, and assert that a
representative spec from tests/backend/specs/api/ and
tests/backend/specs/admin/ appears in the discovered list.

Locks in the glob fix from this PR: if anyone re-narrows the
script back to the previous tests/backend/specs/**.ts pattern
(which only matches depth 1), this vitest fails with a clear
"glob missed ..." message instead of letting the affected specs
silently drop out of CI.

Verified the test FAILS when the script is reverted to the
broken glob.

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

---------

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

Author: JohnMcLear

src/package.json

@@ -147,7 +147,7 @@
   },
   "scripts": {
     "lint": "eslint .",
-    "test": "cross-env NODE_ENV=production mocha --import=tsx --require ./tests/backend/diagnostics.ts --timeout 120000 --recursive tests/backend/specs/**.ts ../node_modules/ep_*/static/tests/backend/specs/**",
+    "test": "cross-env NODE_ENV=production mocha --import=tsx --require ./tests/backend/diagnostics.ts --timeout 120000 --extension ts --recursive tests/backend/specs ../node_modules/ep_*/static/tests/backend/specs",
     "test-utils": "cross-env NODE_ENV=production mocha --import=tsx --timeout 5000 --recursive tests/backend/specs/*utils.ts",
     "test-container": "mocha --import=tsx --timeout 5000 tests/container/specs/api",
     "dev": "cross-env NODE_ENV=development  node --require tsx/cjs node/server.ts",

src/tests/backend-new/specs/backend-tests-glob.test.ts

@@ -0,0 +1,54 @@
+'use strict';
+
+// Regression check for the `pnpm test` glob. The previous spec script
+// `tests/backend/specs/**.ts` only matched files at depth 1 under
+// tests/backend/specs/, silently skipping every spec under api/ and
+// admin/ — including the failures filed in #7785–#7788, #7790. This
+// test asserts that mocha (running the exact arguments from
+// src/package.json's "test" script) still discovers a representative
+// file in each of those subdirectories.
+//
+// If the glob is ever narrowed again, this test fails loudly instead
+// of letting the affected specs slip out of CI.
+
+import {execFileSync} from 'child_process';
+import {readFileSync} from 'fs';
+import {join} from 'path';
+import {describe, it, expect} from 'vitest';
+
+const srcRoot = join(__dirname, '..', '..', '..');
+const pkg = JSON.parse(readFileSync(join(srcRoot, 'package.json'), 'utf8'));
+
+// Strip `cross-env NAME=value` prefixes and the leading binary name so we
+// can pass the remaining tokens straight to npx mocha --dry-run.
+const tokens = String(pkg.scripts.test).split(/\s+/);
+while (tokens[0] && /^[A-Z_][A-Z0-9_]*=/.test(tokens[0])) tokens.shift();
+if (tokens[0] === 'cross-env') {
+  tokens.shift();
+  while (tokens[0] && /^[A-Z_][A-Z0-9_]*=/.test(tokens[0])) tokens.shift();
+}
+// Drop the binary itself (mocha) — npx will re-add it.
+if (tokens[0] === 'mocha') tokens.shift();
+
+const REQUIRED = [
+  'tests/backend/specs/api/pad.ts',
+  'tests/backend/specs/api/importexportGetPost.ts',
+  'tests/backend/specs/admin/authorSearch.ts',
+];
+
+describe('backend test glob', () => {
+  it('discovers nested specs under tests/backend/specs/{api,admin}/', () => {
+    const out = execFileSync(
+        'npx', ['mocha', '--dry-run', '--list-files', ...tokens],
+        {cwd: srcRoot, encoding: 'utf8', env: {...process.env, NODE_ENV: 'production'}},
+    );
+    // mocha --list-files prints absolute paths. Normalise to repo-relative.
+    const seen = out.split('\n')
+        .map((l) => l.trim())
+        .filter(Boolean)
+        .map((l) => l.replace(`${srcRoot}/`, ''));
+    for (const required of REQUIRED) {
+      expect(seen, `mocha test glob missed ${required}`).toContain(required);
+    }
+  }, 60000);
+});