@thegetty/quire-cli 1.0.0-rc.45 → 1.0.0-rc.47

package/src/lib/doctor/index.test.js

@@ -1,797 +0,0 @@
-import test from 'ava'
-import sinon from 'sinon'
-import esmock from 'esmock'
-
-/**
- * Index module tests
- *
- * Tests the barrel export, checkSections structure, and runner functions.
- * Individual check function tests are in the domain test files:
- * - checks/environment/*.test.js
- * - checks/tools/*.test.js
- * - checks/project/*.test.js
- * - checks/outputs/*.test.js
- */
-
-test.beforeEach((t) => {
-  t.context.sandbox = sinon.createSandbox()
-})
-
-test.afterEach.always((t) => {
-  t.context.sandbox.restore()
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Barrel export tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('checks array exports all check definitions', async (t) => {
-  const { checks } = await import('./index.js')
-
-  t.true(Array.isArray(checks))
-  t.is(checks.length, 15)
-
-  const checkNames = checks.map((c) => c.name)
-  t.true(checkNames.includes('Operating system'))
-  t.true(checkNames.includes('Quire CLI version'))
-  t.true(checkNames.includes('Node.js version'))
-  t.true(checkNames.includes('Node.js runtime'))
-  t.true(checkNames.includes('npm'))
-  t.true(checkNames.includes('Git'))
-  t.true(checkNames.includes('PrinceXML'))
-  t.true(checkNames.includes('Pandoc'))
-  t.true(checkNames.includes('Quire project'))
-  t.true(checkNames.includes('Dependencies'))
-  t.true(checkNames.includes('quire-11ty version'))
-  t.true(checkNames.includes('Data files'))
-  t.true(checkNames.includes('Build status'))
-  t.true(checkNames.includes('PDF output'))
-  t.true(checkNames.includes('EPUB output'))
-
-  // Verify each check has a function
-  for (const { check } of checks) {
-    t.is(typeof check, 'function')
-  }
-})
-
-test('checkSections exports checks organized by 4 sections', async (t) => {
-  const { checkSections } = await import('./index.js')
-
-  t.true(Array.isArray(checkSections))
-  t.is(checkSections.length, 4)
-
-  const sectionNames = checkSections.map((s) => s.name)
-  t.true(sectionNames.includes('Environment'))
-  t.true(sectionNames.includes('Tools'))
-  t.true(sectionNames.includes('Project'))
-  t.true(sectionNames.includes('Outputs'))
-
-  // Environment section should have 6 checks
-  const envSection = checkSections.find((s) => s.name === 'Environment')
-  t.is(envSection.checks.length, 6)
-  const envCheckNames = envSection.checks.map((c) => c.name)
-  t.true(envCheckNames.includes('Operating system'))
-  t.true(envCheckNames.includes('Quire CLI version'))
-  t.true(envCheckNames.includes('Node.js version'))
-  t.true(envCheckNames.includes('Node.js runtime'))
-  t.true(envCheckNames.includes('npm'))
-  t.true(envCheckNames.includes('Git'))
-
-  // Tools section should have 2 checks
-  const toolsSection = checkSections.find((s) => s.name === 'Tools')
-  t.is(toolsSection.checks.length, 2)
-  const toolCheckNames = toolsSection.checks.map((c) => c.name)
-  t.true(toolCheckNames.includes('PrinceXML'))
-  t.true(toolCheckNames.includes('Pandoc'))
-
-  // Project section should have 4 checks
-  const projectSection = checkSections.find((s) => s.name === 'Project')
-  t.is(projectSection.checks.length, 4)
-  const projectCheckNames = projectSection.checks.map((c) => c.name)
-  t.true(projectCheckNames.includes('Quire project'))
-  t.true(projectCheckNames.includes('Dependencies'))
-  t.true(projectCheckNames.includes('quire-11ty version'))
-  t.true(projectCheckNames.includes('Data files'))
-
-  // Outputs section should have 3 checks
-  const outputsSection = checkSections.find((s) => s.name === 'Outputs')
-  t.is(outputsSection.checks.length, 3)
-  const outputCheckNames = outputsSection.checks.map((c) => c.name)
-  t.true(outputCheckNames.includes('Build status'))
-  t.true(outputCheckNames.includes('PDF output'))
-  t.true(outputCheckNames.includes('EPUB output'))
-})
-
-test('default export includes all expected functions', async (t) => {
-  const doctor = await import('./index.js')
-
-  // Runner functions
-  t.is(typeof doctor.default.runAllChecks, 'function')
-  t.is(typeof doctor.default.runAllChecksWithSections, 'function')
-
-  // Check functions
-  t.is(typeof doctor.default.checkOsInfo, 'function')
-  t.is(typeof doctor.default.checkCliVersion, 'function')
-  t.is(typeof doctor.default.checkNodeVersion, 'function')
-  t.is(typeof doctor.default.checkRuntimeInfo, 'function')
-  t.is(typeof doctor.default.checkNpmAvailable, 'function')
-  t.is(typeof doctor.default.checkGitAvailable, 'function')
-  t.is(typeof doctor.default.checkPrinceAvailable, 'function')
-  t.is(typeof doctor.default.checkPandocAvailable, 'function')
-  t.is(typeof doctor.default.checkQuireProject, 'function')
-  t.is(typeof doctor.default.checkDependencies, 'function')
-  t.is(typeof doctor.default.checkOutdatedQuire11ty, 'function')
-  t.is(typeof doctor.default.checkDataFiles, 'function')
-  t.is(typeof doctor.default.checkStaleBuild, 'function')
-  t.is(typeof doctor.default.checkPdfOutput, 'function')
-  t.is(typeof doctor.default.checkEpubOutput, 'function')
-
-  // Collections
-  t.truthy(doctor.default.checks)
-  t.truthy(doctor.default.checkSections)
-})
-
-test('named exports match default export', async (t) => {
-  const doctor = await import('./index.js')
-
-  t.is(doctor.checkOsInfo, doctor.default.checkOsInfo)
-  t.is(doctor.checkCliVersion, doctor.default.checkCliVersion)
-  t.is(doctor.checkNodeVersion, doctor.default.checkNodeVersion)
-  t.is(doctor.checkRuntimeInfo, doctor.default.checkRuntimeInfo)
-  t.is(doctor.checkNpmAvailable, doctor.default.checkNpmAvailable)
-  t.is(doctor.checkGitAvailable, doctor.default.checkGitAvailable)
-  t.is(doctor.checkPrinceAvailable, doctor.default.checkPrinceAvailable)
-  t.is(doctor.checkPandocAvailable, doctor.default.checkPandocAvailable)
-  t.is(doctor.checkQuireProject, doctor.default.checkQuireProject)
-  t.is(doctor.checkDependencies, doctor.default.checkDependencies)
-  t.is(doctor.checkOutdatedQuire11ty, doctor.default.checkOutdatedQuire11ty)
-  t.is(doctor.checkDataFiles, doctor.default.checkDataFiles)
-  t.is(doctor.checkStaleBuild, doctor.default.checkStaleBuild)
-  t.is(doctor.checkPdfOutput, doctor.default.checkPdfOutput)
-  t.is(doctor.checkEpubOutput, doctor.default.checkEpubOutput)
-  t.is(doctor.runAllChecks, doctor.default.runAllChecks)
-  t.is(doctor.runAllChecksWithSections, doctor.default.runAllChecksWithSections)
-})
-
-test('constants are exported', async (t) => {
-  const doctor = await import('./index.js')
-
-  t.is(doctor.DOCS_BASE_URL, 'https://quire.getty.edu/docs-v1')
-  t.is(doctor.REQUIRED_NODE_VERSION, 22)
-  t.is(doctor.QUIRE_11TY_PACKAGE, '@thegetty/quire-11ty')
-  t.deepEqual(doctor.SECTION_NAMES, ['environment', 'tools', 'project', 'outputs'])
-  t.deepEqual(doctor.CHECK_IDS, [
-    'os', 'cli', 'node', 'runtime', 'npm', 'git',
-    'prince', 'pandoc',
-    'project', 'deps', '11ty', 'data',
-    'build', 'pdf', 'epub',
-  ])
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Runner function tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('runAllChecks runs all checks and returns results array', async (t) => {
-  const { sandbox } = t.context
-
-  // Create mock check functions
-  const mockCheckOs = sandbox.stub().returns({ ok: true, message: 'macOS 14 (arm64)' })
-  const mockCheckCli = sandbox.stub().returns({ ok: true, message: 'v1.0.0-rc.33' })
-  const mockCheckNode = sandbox.stub().returns({ ok: true, message: 'v22' })
-  const mockCheckRuntime = sandbox.stub().returns({ ok: true, message: 'Heap limit: 4096 MB' })
-  const mockCheckNpm = sandbox.stub().resolves({ ok: true, message: '10.2.4' })
-  const mockCheckGit = sandbox.stub().resolves({ ok: true, message: '2.43.0' })
-  const mockCheckPrince = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckPandoc = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckProject = sandbox.stub().returns({ ok: true, message: '.quire' })
-  const mockCheckDeps = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckQuire11ty = sandbox.stub().resolves({ ok: true, message: 'v1.0.0' })
-  const mockCheckData = sandbox.stub().returns({ ok: true, message: '3 files' })
-  const mockCheckStale = sandbox.stub().returns({ ok: true, message: 'up to date' })
-  const mockCheckPdf = sandbox.stub().returns({ ok: true, message: 'no PDF' })
-  const mockCheckEpub = sandbox.stub().returns({ ok: true, message: 'no EPUB' })
-
-  const { runAllChecks } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: mockCheckOs,
-      checkCliVersion: mockCheckCli,
-      checkNodeVersion: mockCheckNode,
-      checkRuntimeInfo: mockCheckRuntime,
-      checkNpmAvailable: mockCheckNpm,
-      checkGitAvailable: mockCheckGit,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: mockCheckPrince,
-      checkPandocAvailable: mockCheckPandoc,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: mockCheckProject,
-      checkDependencies: mockCheckDeps,
-      checkOutdatedQuire11ty: mockCheckQuire11ty,
-      checkDataFiles: mockCheckData,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: mockCheckStale,
-      checkPdfOutput: mockCheckPdf,
-      checkEpubOutput: mockCheckEpub,
-    },
-  })
-
-  const results = await runAllChecks()
-
-  t.true(Array.isArray(results))
-  t.is(results.length, 15)
-
-  // Verify each result has expected shape
-  for (const result of results) {
-    t.is(typeof result.name, 'string')
-    t.is(typeof result.ok, 'boolean')
-  }
-
-  // Verify all checks were called
-  t.true(mockCheckOs.calledOnce)
-  t.true(mockCheckCli.calledOnce)
-  t.true(mockCheckNode.calledOnce)
-  t.true(mockCheckRuntime.calledOnce)
-  t.true(mockCheckNpm.calledOnce)
-  t.true(mockCheckGit.calledOnce)
-  t.true(mockCheckPrince.calledOnce)
-  t.true(mockCheckPandoc.calledOnce)
-  t.true(mockCheckProject.calledOnce)
-  t.true(mockCheckDeps.calledOnce)
-  t.true(mockCheckQuire11ty.calledOnce)
-  t.true(mockCheckData.calledOnce)
-  t.true(mockCheckStale.calledOnce)
-  t.true(mockCheckPdf.calledOnce)
-  t.true(mockCheckEpub.calledOnce)
-})
-
-test('runAllChecksWithSections returns results organized by 4 sections', async (t) => {
-  const { sandbox } = t.context
-
-  // Create mock check functions
-  const mockCheckOs = sandbox.stub().returns({ ok: true, message: 'macOS 14 (arm64)' })
-  const mockCheckCli = sandbox.stub().returns({ ok: true, message: 'v1.0.0-rc.33' })
-  const mockCheckNode = sandbox.stub().returns({ ok: true, message: 'v22' })
-  const mockCheckRuntime = sandbox.stub().returns({ ok: true, message: 'Heap limit: 4096 MB' })
-  const mockCheckNpm = sandbox.stub().resolves({ ok: true, message: '10.2.4' })
-  const mockCheckGit = sandbox.stub().resolves({ ok: true, message: '2.43.0' })
-  const mockCheckPrince = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckPandoc = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckProject = sandbox.stub().returns({ ok: true, message: '.quire' })
-  const mockCheckDeps = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckQuire11ty = sandbox.stub().resolves({ ok: true, message: 'v1.0.0' })
-  const mockCheckData = sandbox.stub().returns({ ok: true, message: '3 files' })
-  const mockCheckStale = sandbox.stub().returns({ ok: true, message: 'up to date' })
-  const mockCheckPdf = sandbox.stub().returns({ ok: true, message: 'no PDF' })
-  const mockCheckEpub = sandbox.stub().returns({ ok: true, message: 'no EPUB' })
-
-  const { runAllChecksWithSections } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: mockCheckOs,
-      checkCliVersion: mockCheckCli,
-      checkNodeVersion: mockCheckNode,
-      checkRuntimeInfo: mockCheckRuntime,
-      checkNpmAvailable: mockCheckNpm,
-      checkGitAvailable: mockCheckGit,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: mockCheckPrince,
-      checkPandocAvailable: mockCheckPandoc,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: mockCheckProject,
-      checkDependencies: mockCheckDeps,
-      checkOutdatedQuire11ty: mockCheckQuire11ty,
-      checkDataFiles: mockCheckData,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: mockCheckStale,
-      checkPdfOutput: mockCheckPdf,
-      checkEpubOutput: mockCheckEpub,
-    },
-  })
-
-  const sections = await runAllChecksWithSections()
-
-  t.true(Array.isArray(sections))
-  t.is(sections.length, 4)
-
-  // Verify section structure
-  const sectionNames = sections.map((s) => s.section)
-  t.deepEqual(sectionNames, ['Environment', 'Tools', 'Project', 'Outputs'])
-
-  // Verify each section has results
-  for (const { section, results } of sections) {
-    t.is(typeof section, 'string')
-    t.true(Array.isArray(results))
-    for (const result of results) {
-      t.is(typeof result.name, 'string')
-      t.is(typeof result.ok, 'boolean')
-    }
-  }
-
-  // Verify section sizes
-  const envSection = sections.find((s) => s.section === 'Environment')
-  t.is(envSection.results.length, 6)
-
-  const toolsSection = sections.find((s) => s.section === 'Tools')
-  t.is(toolsSection.results.length, 2)
-
-  const projectSection = sections.find((s) => s.section === 'Project')
-  t.is(projectSection.results.length, 4)
-
-  const outputsSection = sections.find((s) => s.section === 'Outputs')
-  t.is(outputsSection.results.length, 3)
-})
-
-test('runAllChecksWithSections filters by section name', async (t) => {
-  const { sandbox } = t.context
-
-  // Create mock check functions
-  const mockCheckOs = sandbox.stub().returns({ ok: true, message: 'macOS 14 (arm64)' })
-  const mockCheckCli = sandbox.stub().returns({ ok: true, message: 'v1.0.0-rc.33' })
-  const mockCheckNode = sandbox.stub().returns({ ok: true, message: 'v22' })
-  const mockCheckRuntime = sandbox.stub().returns({ ok: true, message: 'Heap limit: 4096 MB' })
-  const mockCheckNpm = sandbox.stub().resolves({ ok: true, message: '10.2.4' })
-  const mockCheckGit = sandbox.stub().resolves({ ok: true, message: '2.43.0' })
-  const mockCheckPrince = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckPandoc = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckProject = sandbox.stub().returns({ ok: true, message: '.quire' })
-  const mockCheckDeps = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckQuire11ty = sandbox.stub().resolves({ ok: true, message: 'v1.0.0' })
-  const mockCheckData = sandbox.stub().returns({ ok: true, message: '3 files' })
-  const mockCheckStale = sandbox.stub().returns({ ok: true, message: 'up to date' })
-  const mockCheckPdf = sandbox.stub().returns({ ok: true, message: 'no PDF' })
-  const mockCheckEpub = sandbox.stub().returns({ ok: true, message: 'no EPUB' })
-
-  const { runAllChecksWithSections } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: mockCheckOs,
-      checkCliVersion: mockCheckCli,
-      checkNodeVersion: mockCheckNode,
-      checkRuntimeInfo: mockCheckRuntime,
-      checkNpmAvailable: mockCheckNpm,
-      checkGitAvailable: mockCheckGit,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: mockCheckPrince,
-      checkPandocAvailable: mockCheckPandoc,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: mockCheckProject,
-      checkDependencies: mockCheckDeps,
-      checkOutdatedQuire11ty: mockCheckQuire11ty,
-      checkDataFiles: mockCheckData,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: mockCheckStale,
-      checkPdfOutput: mockCheckPdf,
-      checkEpubOutput: mockCheckEpub,
-    },
-  })
-
-  // Filter to environment section only (legacy array syntax)
-  const sections = await runAllChecksWithSections(['environment'])
-
-  t.is(sections.length, 1)
-  t.is(sections[0].section, 'Environment')
-  t.is(sections[0].results.length, 6)
-
-  // Verify only environment checks were called
-  t.true(mockCheckOs.calledOnce)
-  t.true(mockCheckCli.calledOnce)
-  t.true(mockCheckNode.calledOnce)
-  t.true(mockCheckRuntime.calledOnce)
-  t.true(mockCheckNpm.calledOnce)
-  t.true(mockCheckGit.calledOnce)
-
-  // Verify tools, project, and output checks were NOT called
-  t.false(mockCheckPrince.called)
-  t.false(mockCheckPandoc.called)
-  t.false(mockCheckProject.called)
-  t.false(mockCheckDeps.called)
-  t.false(mockCheckQuire11ty.called)
-  t.false(mockCheckData.called)
-  t.false(mockCheckStale.called)
-  t.false(mockCheckPdf.called)
-  t.false(mockCheckEpub.called)
-})
-
-test('runAllChecksWithSections filters by check ID', async (t) => {
-  const { sandbox } = t.context
-
-  // Create mock check functions
-  const mockCheckOs = sandbox.stub().returns({ ok: true, message: 'macOS 14 (arm64)' })
-  const mockCheckCli = sandbox.stub().returns({ ok: true, message: 'v1.0.0-rc.33' })
-  const mockCheckNode = sandbox.stub().returns({ ok: true, message: 'v22' })
-  const mockCheckRuntime = sandbox.stub().returns({ ok: true, message: 'Heap limit: 4096 MB' })
-  const mockCheckNpm = sandbox.stub().resolves({ ok: true, message: '10.2.4' })
-  const mockCheckGit = sandbox.stub().resolves({ ok: true, message: '2.43.0' })
-  const mockCheckPrince = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckPandoc = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckProject = sandbox.stub().returns({ ok: true, message: '.quire' })
-  const mockCheckDeps = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckQuire11ty = sandbox.stub().resolves({ ok: true, message: 'v1.0.0' })
-  const mockCheckData = sandbox.stub().returns({ ok: true, message: '3 files' })
-  const mockCheckStale = sandbox.stub().returns({ ok: true, message: 'up to date' })
-  const mockCheckPdf = sandbox.stub().returns({ ok: true, message: 'no PDF' })
-  const mockCheckEpub = sandbox.stub().returns({ ok: true, message: 'no EPUB' })
-
-  const { runAllChecksWithSections } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: mockCheckOs,
-      checkCliVersion: mockCheckCli,
-      checkNodeVersion: mockCheckNode,
-      checkRuntimeInfo: mockCheckRuntime,
-      checkNpmAvailable: mockCheckNpm,
-      checkGitAvailable: mockCheckGit,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: mockCheckPrince,
-      checkPandocAvailable: mockCheckPandoc,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: mockCheckProject,
-      checkDependencies: mockCheckDeps,
-      checkOutdatedQuire11ty: mockCheckQuire11ty,
-      checkDataFiles: mockCheckData,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: mockCheckStale,
-      checkPdfOutput: mockCheckPdf,
-      checkEpubOutput: mockCheckEpub,
-    },
-  })
-
-  // Filter to specific checks by ID (node and git from Environment)
-  const sections = await runAllChecksWithSections({ checks: ['node', 'git'] })
-
-  t.is(sections.length, 1)
-  t.is(sections[0].section, 'Environment')
-  t.is(sections[0].results.length, 2)
-
-  // Verify result names match the requested checks
-  const resultNames = sections[0].results.map((r) => r.name)
-  t.true(resultNames.includes('Node.js version'))
-  t.true(resultNames.includes('Git'))
-
-  // Verify only requested checks were called
-  t.true(mockCheckNode.calledOnce)
-  t.true(mockCheckGit.calledOnce)
-
-  // Verify other checks were NOT called
-  t.false(mockCheckOs.called)
-  t.false(mockCheckCli.called)
-  t.false(mockCheckRuntime.called)
-  t.false(mockCheckNpm.called)
-  t.false(mockCheckPrince.called)
-  t.false(mockCheckPandoc.called)
-  t.false(mockCheckProject.called)
-  t.false(mockCheckDeps.called)
-  t.false(mockCheckQuire11ty.called)
-  t.false(mockCheckData.called)
-  t.false(mockCheckStale.called)
-  t.false(mockCheckPdf.called)
-  t.false(mockCheckEpub.called)
-})
-
-test('runAllChecksWithSections filters checks across multiple sections', async (t) => {
-  const { sandbox } = t.context
-
-  // Create mock check functions
-  const mockCheckOs = sandbox.stub().returns({ ok: true, message: 'macOS 14 (arm64)' })
-  const mockCheckCli = sandbox.stub().returns({ ok: true, message: 'v1.0.0-rc.33' })
-  const mockCheckNode = sandbox.stub().returns({ ok: true, message: 'v22' })
-  const mockCheckRuntime = sandbox.stub().returns({ ok: true, message: 'Heap limit: 4096 MB' })
-  const mockCheckNpm = sandbox.stub().resolves({ ok: true, message: '10.2.4' })
-  const mockCheckGit = sandbox.stub().resolves({ ok: true, message: '2.43.0' })
-  const mockCheckPrince = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckPandoc = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckProject = sandbox.stub().returns({ ok: true, message: '.quire' })
-  const mockCheckDeps = sandbox.stub().returns({ ok: true, message: 'installed' })
-  const mockCheckQuire11ty = sandbox.stub().resolves({ ok: true, message: 'v1.0.0' })
-  const mockCheckData = sandbox.stub().returns({ ok: true, message: '3 files' })
-  const mockCheckStale = sandbox.stub().returns({ ok: true, message: 'up to date' })
-  const mockCheckPdf = sandbox.stub().returns({ ok: true, message: 'no PDF' })
-  const mockCheckEpub = sandbox.stub().returns({ ok: true, message: 'no EPUB' })
-
-  const { runAllChecksWithSections } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: mockCheckOs,
-      checkCliVersion: mockCheckCli,
-      checkNodeVersion: mockCheckNode,
-      checkRuntimeInfo: mockCheckRuntime,
-      checkNpmAvailable: mockCheckNpm,
-      checkGitAvailable: mockCheckGit,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: mockCheckPrince,
-      checkPandocAvailable: mockCheckPandoc,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: mockCheckProject,
-      checkDependencies: mockCheckDeps,
-      checkOutdatedQuire11ty: mockCheckQuire11ty,
-      checkDataFiles: mockCheckData,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: mockCheckStale,
-      checkPdfOutput: mockCheckPdf,
-      checkEpubOutput: mockCheckEpub,
-    },
-  })
-
-  // Filter to checks from different sections (node from Environment, pdf from Outputs)
-  const sections = await runAllChecksWithSections({ checks: ['node', 'pdf'] })
-
-  t.is(sections.length, 2)
-
-  // Verify Environment section has only node check
-  const envSection = sections.find((s) => s.section === 'Environment')
-  t.truthy(envSection)
-  t.is(envSection.results.length, 1)
-  t.is(envSection.results[0].name, 'Node.js version')
-
-  // Verify Outputs section has only pdf check
-  const outputsSection = sections.find((s) => s.section === 'Outputs')
-  t.truthy(outputsSection)
-  t.is(outputsSection.results.length, 1)
-  t.is(outputsSection.results[0].name, 'PDF output')
-
-  // Verify only requested checks were called
-  t.true(mockCheckNode.calledOnce)
-  t.true(mockCheckPdf.calledOnce)
-
-  // Project and Tools sections should not appear (no checks from them)
-  const projectSection = sections.find((s) => s.section === 'Project')
-  t.falsy(projectSection)
-  const toolsSection = sections.find((s) => s.section === 'Tools')
-  t.falsy(toolsSection)
-})
-
-test('runAllChecks handles async checks correctly', async (t) => {
-  const { sandbox } = t.context
-
-  // Mix of sync and async checks
-  const mockSyncCheck = sandbox.stub().returns({ ok: true, message: 'sync' })
-  const mockAsyncCheck = sandbox.stub().resolves({ ok: false, level: 'warn', message: 'async' })
-
-  const { runAllChecks } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: mockSyncCheck,
-      checkCliVersion: mockSyncCheck,
-      checkNodeVersion: mockSyncCheck,
-      checkRuntimeInfo: mockSyncCheck,
-      checkNpmAvailable: mockAsyncCheck,
-      checkGitAvailable: mockAsyncCheck,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: mockSyncCheck,
-      checkPandocAvailable: mockSyncCheck,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: mockSyncCheck,
-      checkDependencies: mockSyncCheck,
-      checkOutdatedQuire11ty: mockAsyncCheck,
-      checkDataFiles: mockSyncCheck,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: mockSyncCheck,
-      checkPdfOutput: mockSyncCheck,
-      checkEpubOutput: mockSyncCheck,
-    },
-  })
-
-  const results = await runAllChecks()
-
-  t.is(results.length, 15)
-
-  // Verify async results are properly awaited
-  const asyncResults = results.filter((r) => r.message === 'async')
-  t.is(asyncResults.length, 3)
-  for (const result of asyncResults) {
-    t.false(result.ok)
-    t.is(result.level, 'warn')
-  }
-})
-
-test('runAllChecks preserves all check result properties', async (t) => {
-  const { sandbox } = t.context
-
-  const fullResult = {
-    ok: false,
-    level: 'warn',
-    message: 'Test message',
-    remediation: 'Fix steps',
-    docsUrl: 'https://example.com',
-  }
-
-  const { runAllChecks } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: sandbox.stub().returns({ ok: true, message: 'macOS 14' }),
-      checkCliVersion: sandbox.stub().returns(fullResult),
-      checkNodeVersion: sandbox.stub().returns({ ok: true, message: null }),
-      checkRuntimeInfo: sandbox.stub().returns({ ok: true, message: 'Heap limit: 4096 MB' }),
-      checkNpmAvailable: sandbox.stub().resolves({ ok: true, message: '10.2.4' }),
-      checkGitAvailable: sandbox.stub().resolves({ ok: true, message: '2.43.0' }),
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: sandbox.stub().returns({ ok: true, message: 'installed' }),
-      checkPandocAvailable: sandbox.stub().returns({ ok: true, message: 'installed' }),
-    },
-    './checks/project/index.js': {
-      checkQuireProject: sandbox.stub().returns({ ok: true, message: null }),
-      checkDependencies: sandbox.stub().returns({ ok: true, message: 'installed' }),
-      checkOutdatedQuire11ty: sandbox.stub().resolves({ ok: true, message: null }),
-      checkDataFiles: sandbox.stub().returns({ ok: true, message: null }),
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: sandbox.stub().returns({ ok: true, message: null }),
-      checkPdfOutput: sandbox.stub().returns({ ok: true, message: null }),
-      checkEpubOutput: sandbox.stub().returns({ ok: true, message: null }),
-    },
-  })
-
-  const results = await runAllChecks()
-  const cliResult = results.find((r) => r.name === 'Quire CLI version')
-
-  t.false(cliResult.ok)
-  t.is(cliResult.level, 'warn')
-  t.is(cliResult.message, 'Test message')
-  t.is(cliResult.remediation, 'Fix steps')
-  t.is(cliResult.docsUrl, 'https://example.com')
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Timeout tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('DEFAULT_CHECK_TIMEOUT is exported and has reasonable value', async (t) => {
-  const { DEFAULT_CHECK_TIMEOUT } = await import('./index.js')
-
-  t.is(typeof DEFAULT_CHECK_TIMEOUT, 'number')
-  t.is(DEFAULT_CHECK_TIMEOUT, 10000, 'default timeout should be 10 seconds')
-})
-
-test('runAllChecksWithSections handles slow checks with timeout', async (t) => {
-  const { sandbox } = t.context
-
-  // Create a check that takes longer than the timeout
-  const slowCheck = sandbox.stub().callsFake(() =>
-    new Promise((resolve) => setTimeout(() => resolve({ ok: true, message: 'slow' }), 200))
-  )
-  const fastCheck = sandbox.stub().returns({ ok: true, message: 'fast' })
-
-  const { runAllChecksWithSections } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: fastCheck,
-      checkCliVersion: fastCheck,
-      checkNodeVersion: slowCheck,
-      checkRuntimeInfo: fastCheck,
-      checkNpmAvailable: fastCheck,
-      checkGitAvailable: fastCheck,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: fastCheck,
-      checkPandocAvailable: fastCheck,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: fastCheck,
-      checkDependencies: fastCheck,
-      checkOutdatedQuire11ty: fastCheck,
-      checkDataFiles: fastCheck,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: fastCheck,
-      checkPdfOutput: fastCheck,
-      checkEpubOutput: fastCheck,
-    },
-  })
-
-  // Use a very short timeout (50ms) to trigger the timeout
-  const sections = await runAllChecksWithSections({ checks: ['node'], timeout: 50 })
-
-  t.is(sections.length, 1)
-  const nodeResult = sections[0].results[0]
-
-  // Should have timed out
-  t.false(nodeResult.ok)
-  t.is(nodeResult.level, 'timeout')
-  t.regex(nodeResult.message, /skipped.*timed out/)
-  t.truthy(nodeResult.remediation)
-})
-
-test('runAllChecksWithSections completes checks within timeout', async (t) => {
-  const { sandbox } = t.context
-
-  // Create a check that completes quickly
-  const fastCheck = sandbox.stub().returns({ ok: true, message: 'fast' })
-
-  const { runAllChecksWithSections } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: fastCheck,
-      checkCliVersion: fastCheck,
-      checkNodeVersion: fastCheck,
-      checkRuntimeInfo: fastCheck,
-      checkNpmAvailable: fastCheck,
-      checkGitAvailable: fastCheck,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: fastCheck,
-      checkPandocAvailable: fastCheck,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: fastCheck,
-      checkDependencies: fastCheck,
-      checkOutdatedQuire11ty: fastCheck,
-      checkDataFiles: fastCheck,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: fastCheck,
-      checkPdfOutput: fastCheck,
-      checkEpubOutput: fastCheck,
-    },
-  })
-
-  // Use a long timeout - check should complete well before
-  const sections = await runAllChecksWithSections({ checks: ['node'], timeout: 5000 })
-
-  t.is(sections.length, 1)
-  const nodeResult = sections[0].results[0]
-
-  // Should have completed successfully
-  t.true(nodeResult.ok)
-  t.is(nodeResult.message, 'fast')
-})
-
-test('runAllChecksWithSections includes check id in results', async (t) => {
-  const { sandbox } = t.context
-
-  const fastCheck = sandbox.stub().returns({ ok: true, message: 'fast' })
-
-  const { runAllChecksWithSections } = await esmock('./index.js', {
-    './checks/environment/index.js': {
-      checkOsInfo: fastCheck,
-      checkCliVersion: fastCheck,
-      checkNodeVersion: fastCheck,
-      checkRuntimeInfo: fastCheck,
-      checkNpmAvailable: fastCheck,
-      checkGitAvailable: fastCheck,
-    },
-    './checks/tools/index.js': {
-      checkPrinceAvailable: fastCheck,
-      checkPandocAvailable: fastCheck,
-    },
-    './checks/project/index.js': {
-      checkQuireProject: fastCheck,
-      checkDependencies: fastCheck,
-      checkOutdatedQuire11ty: fastCheck,
-      checkDataFiles: fastCheck,
-    },
-    './checks/outputs/index.js': {
-      checkStaleBuild: fastCheck,
-      checkPdfOutput: fastCheck,
-      checkEpubOutput: fastCheck,
-    },
-  })
-
-  const sections = await runAllChecksWithSections({ checks: ['node', 'npm'] })
-
-  // Each result should have an id property
-  for (const { results } of sections) {
-    for (const result of results) {
-      t.truthy(result.id, 'result should have id property')
-      t.is(typeof result.id, 'string')
-    }
-  }
-
-  // Verify specific IDs
-  const envSection = sections.find((s) => s.section === 'Environment')
-  const nodeResult = envSection.results.find((r) => r.name === 'Node.js version')
-  const npmResult = envSection.results.find((r) => r.name === 'npm')
-
-  t.is(nodeResult.id, 'node')
-  t.is(npmResult.id, 'npm')
-})