test(fixtures): rename single-letter fixture placeholders to descriptive fakes

Author: jdalton

test/define-guarded.test.mts

@@ -15,7 +15,7 @@ import { defineGuardedPlugin } from '../.config/repo/rolldown/define-guarded.mts
 // Exported `function` declaration: satisfies both fleet rules —
 // socket/prefer-function-declaration (no module-scope arrow) and
 // socket/export-top-level-functions (top-level functions must be exported).
-export function run(code: string, id = 'src/x.ts'): string | undefined {
+export function run(code: string, id = 'src/example.ts'): string | undefined {
   const plugin = defineGuardedPlugin({
     'process.env.INLINED_EXTENSION_VERSION': JSON.stringify('2.0.3'),
   })
@@ -60,7 +60,7 @@ describe('defineGuardedPlugin', () => {
   test('parses .tsx source', () => {
     const out = run(
       'const e = <div>{process.env["INLINED_EXTENSION_VERSION"]}</div>',
-      'src/x.tsx',
+      'src/example.tsx',
     )
     expect(out).toContain('"2.0.3"')
   })

test/scripts/install-claude-plugins.test.mts

@@ -27,24 +27,26 @@ const OUR = 'socket-wheelhouse'
 
 test('extractInstalledSha returns 12-char prefix for SHA-pinned cache path', () => {
   const got = extractInstalledSha(
-    '/Users/x/.claude/plugins/cache/socket-wheelhouse/codex/9cb4fe409919-deadbeef',
+    '/Users/<user>/.claude/plugins/cache/socket-wheelhouse/codex/9cb4fe409919-deadbeef',
   )
   expect(got).toBe('9cb4fe409919')
 })
 
 test('extractInstalledSha handles content-hash of various lengths', () => {
-  const got = extractInstalledSha('/x/cache/m/p/abcdef012345-fedcba98')
+  const got = extractInstalledSha(
+    '/cache/marketplace/plugins/abcdef012345-fedcba98',
+  )
   expect(got).toBe('abcdef012345')
 })
 
 test('extractInstalledSha returns undefined for directory-source install (version-tagged)', () => {
-  const got = extractInstalledSha('/Users/x/projects/codex-plugin-cc')
+  const got = extractInstalledSha('/Users/<user>/projects/codex-plugin-cc')
   expect(got).toBe(undefined)
 })
 
 test('extractInstalledSha returns undefined for version-tagged install', () => {
   const got = extractInstalledSha(
-    '/Users/x/.claude/plugins/cache/openai-codex/codex/1.0.1',
+    '/Users/<user>/.claude/plugins/cache/openai-codex/codex/1.0.1',
   )
   expect(got).toBe(undefined)
 })
@@ -59,15 +61,17 @@ test('extractInstalledSha returns undefined for empty string', () => {
 
 test('extractInstalledSha rejects shapes that almost-match but are not 12 + 8+', () => {
   // 11 chars instead of 12.
-  expect(extractInstalledSha('/x/cache/m/p/9cb4fe40991-deadbeef')).toBe(
-    undefined,
-  )
+  expect(
+    extractInstalledSha('/cache/marketplace/plugins/9cb4fe40991-deadbeef'),
+  ).toBe(undefined)
   // No content-hash suffix.
-  expect(extractInstalledSha('/x/cache/m/p/9cb4fe409919')).toBe(undefined)
-  // Non-hex chars.
-  expect(extractInstalledSha('/x/cache/m/p/zzzzzzzzzzzz-deadbeef')).toBe(
+  expect(extractInstalledSha('/cache/marketplace/plugins/9cb4fe409919')).toBe(
     undefined,
   )
+  // Non-hex chars.
+  expect(
+    extractInstalledSha('/cache/marketplace/plugins/zzzzzzzzzzzz-deadbeef'),
+  ).toBe(undefined)
 })
 
 const fakePlugin = (
@@ -82,7 +86,7 @@ const fakePlugin = (
 
 test('findForeignInstall finds plugin under non-canonical marketplace', () => {
   const plugins = [
-    fakePlugin('codex@openai-codex', '/Users/x/projects/codex-plugin-cc'),
+    fakePlugin('codex@openai-codex', '/Users/<user>/projects/codex-plugin-cc'),
     fakePlugin('clangd-lsp@claude-plugins-official'),
   ]
   const got = findForeignInstall('codex', plugins, OUR)
@@ -93,7 +97,7 @@ test('findForeignInstall returns undefined when plugin is under our marketplace'
   const plugins = [
     fakePlugin(
       'codex@socket-wheelhouse',
-      '/x/cache/socket-wheelhouse/codex/9cb4fe409919-aa',
+      '/cache/example/socket-wheelhouse/codex/9cb4fe409919-aa',
     ),
   ]
   const got = findForeignInstall('codex', plugins, OUR)
@@ -189,7 +193,7 @@ test('lookupInstalledSha extracts gitCommitSha from installed_plugins.json shape
       'codex@socket-wheelhouse': [
         {
           scope: 'user',
-          installPath: '/x/y/z',
+          installPath: '/home/<user>/install',
           version: '1.0.1',
           gitCommitSha: FULL_SHA,
         },
@@ -209,7 +213,11 @@ test('lookupInstalledSha returns undefined when entry has no gitCommitSha', () =
     version: 2,
     plugins: {
       'codex@socket-wheelhouse': [
-        { scope: 'user', installPath: '/x/y/z', version: '1.0.1' },
+        {
+          scope: 'user',
+          installPath: '/home/<user>/install',
+          version: '1.0.1',
+        },
       ],
     },
   }
@@ -322,14 +330,14 @@ test('stripPatchHeader only matches --- at line start (not mid-line)', () => {
 })
 
 test('patchSidecarDir maps <x>.patch → <x>.files', () => {
-  expect(patchSidecarDir('/a/b/codex-1.0.1-stdin-eagain.patch')).toBe(
-    '/a/b/codex-1.0.1-stdin-eagain.files',
-  )
+  expect(
+    patchSidecarDir('/home/<user>/build/codex-1.0.1-stdin-eagain.patch'),
+  ).toBe('/home/<user>/build/codex-1.0.1-stdin-eagain.files')
 })
 
 test('patchSidecarDir only rewrites a trailing .patch extension', () => {
   // A `.patch` mid-path must not be rewritten — only the final extension.
-  expect(patchSidecarDir('/a/.patch-stuff/codex-1.0.1-x.patch')).toBe(
-    '/a/.patch-stuff/codex-1.0.1-x.files',
+  expect(patchSidecarDir('/home/<user>/.patch-stuff/codex-1.0.1-x.patch')).toBe(
+    '/home/<user>/.patch-stuff/codex-1.0.1-x.files',
   )
 })

test/select-artifacts.test.mts

@@ -82,15 +82,15 @@ describe('worstArtifactsByPurl', () => {
   })
 
   test('keeps each distinct PURL independently', () => {
-    const purlA = 'pkg:pypi/a' as SimPURL
-    const purlB = 'pkg:pypi/b' as SimPURL
-    const aLow = makeArtifact(purlA, 0.1)
-    const aHigh = makeArtifact(purlA, 0.9)
-    const bOnly = makeArtifact(purlB, 0.7)
-    const result = worstArtifactsByPurl([aHigh, bOnly, aLow])
+    const purlExample = 'pkg:pypi/example' as SimPURL
+    const purlOther = 'pkg:pypi/other' as SimPURL
+    const exampleLow = makeArtifact(purlExample, 0.1)
+    const exampleHigh = makeArtifact(purlExample, 0.9)
+    const otherOnly = makeArtifact(purlOther, 0.7)
+    const result = worstArtifactsByPurl([exampleHigh, otherOnly, exampleLow])
     expect(result.size).toBe(2)
-    expect(result.get(purlA)).toBe(aLow)
-    expect(result.get(purlB)).toBe(bOnly)
+    expect(result.get(purlExample)).toBe(exampleLow)
+    expect(result.get(purlOther)).toBe(otherOnly)
   })
 
   test('returns an empty map for no entries', () => {