@thegetty/quire-cli 1.0.0-rc.44 → 1.0.0-rc.46

package/src/commands/pdf.test.js

@@ -1,464 +0,0 @@
-import test from 'ava'
-import sinon from 'sinon'
-import esmock from 'esmock'
-
-test.beforeEach((t) => {
-  t.context.sandbox = sinon.createSandbox()
-
-  // Create mock reporter
-  t.context.mockReporter = {
-    configure: t.context.sandbox.stub().returnsThis(),
-    start: t.context.sandbox.stub().returnsThis(),
-    update: t.context.sandbox.stub().returnsThis(),
-    succeed: t.context.sandbox.stub().returnsThis(),
-    fail: t.context.sandbox.stub().returnsThis(),
-    stop: t.context.sandbox.stub().returnsThis(),
-  }
-})
-
-test.afterEach.always((t) => {
-  t.context.sandbox.restore()
-})
-
-test('pdf command should call generatePdf with pagedjs engine', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await command.action({ engine: 'pagedjs' }, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.is(mockGeneratePdf.firstCall.args[0].lib, 'pagedjs', 'should pass lib to generatePdf')
-  // Nota bene: reporter lifecycle (start/succeed/fail) is handled by the façade, not the command
-})
-
-test('pdf command should call generatePdf with prince engine', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await command.action({ engine: 'prince' }, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.is(mockGeneratePdf.firstCall.args[0].lib, 'prince', 'should pass lib to generatePdf')
-})
-
-test('pdf command should open PDF when --open flag is provided', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const outputPath = '/project/_site/_pdf/test-book.pdf'
-  const mockGeneratePdf = sandbox.stub().resolves(outputPath)
-  const mockOpen = sandbox.stub()
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: mockOpen
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await command.action({ engine: 'pagedjs', open: true }, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.true(mockOpen.called, 'open should be called when --open flag is provided')
-  t.is(mockOpen.firstCall.args[0], outputPath, 'should open the generated PDF path')
-})
-
-test('pdf command should not open PDF when --open flag is not provided', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-  const mockOpen = sandbox.stub()
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: mockOpen
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await command.action({ engine: 'pagedjs' }, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.false(mockOpen.called, 'open should not be called without --open flag')
-})
-
-test('pdf command should pass debug option to generatePdf', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await command.action({ engine: 'pagedjs', debug: true }, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.is(mockGeneratePdf.firstCall.args[0].lib, 'pagedjs', 'should pass lib to generatePdf')
-  t.true(mockGeneratePdf.firstCall.args[0].debug, 'should pass debug option')
-})
-
-test('pdf command should propagate MissingBuildOutputError from façade', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  // Simulate the façade throwing MissingBuildOutputError when pdf.html is missing
-  const missingBuildError = new Error('Cannot generate pdf.html: build output not found')
-  missingBuildError.code = 'BUILD_OUTPUT_MISSING'
-  const mockGeneratePdf = sandbox.stub().rejects(missingBuildError)
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true // Command doesn't check this for validation anymore
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  const error = await t.throwsAsync(() => command.action({ engine: 'pagedjs' }, command))
-
-  t.is(error.code, 'BUILD_OUTPUT_MISSING', 'should propagate error code from façade')
-  t.true(mockGeneratePdf.called, 'generatePdf should be called (validation is in façade)')
-})
-
-test('pdf command should run build first when --build flag is set and output missing', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-  const mockBuild = sandbox.stub().resolves()
-  let buildCalled = false
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => {
-        // Return false first (before build), true after build
-        if (!buildCalled) return false
-        return true
-      }
-    },
-    '#lib/11ty/index.js': {
-      default: {
-        build: async (opts) => {
-          buildCalled = true
-          return mockBuild(opts)
-        }
-      }
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await command.action({ engine: 'pagedjs', build: true }, command)
-
-  t.true(mockBuild.called, 'build should be called when --build flag is set')
-  t.true(mockGeneratePdf.called, 'generatePdf should be called after build')
-})
-
-test('pdf command should support deprecated --lib option', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  // Use deprecated --lib option (should still work)
-  await command.action({ lib: 'prince' }, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called with deprecated --lib')
-  t.is(mockGeneratePdf.firstCall.args[0].lib, 'prince', 'should pass lib to generatePdf from deprecated option')
-})
-
-test('pdf command should propagate errors from generatePdf', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const pdfError = new Error('PDF generation failed')
-  const mockGeneratePdf = sandbox.stub().rejects(pdfError)
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await t.throwsAsync(() => command.action({ engine: 'pagedjs' }, command), { message: 'PDF generation failed' })
-
-  // Note: reporter lifecycle (start/succeed/fail) is handled by the façade, not the command
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-})
-
-test('pdf command should configure reporter with quiet option', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await command.action({ engine: 'pagedjs', quiet: true }, command)
-
-  t.true(
-    mockReporter.configure.calledWith(sinon.match({ quiet: true })),
-    'reporter.configure should be called with quiet option'
-  )
-})
-
-test('pdf command should pass output option to generatePdf', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const customOutput = '/custom/path/my-book.pdf'
-  const mockGeneratePdf = sandbox.stub().resolves(customOutput)
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-
-  await command.action({ engine: 'pagedjs', output: customOutput }, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.is(mockGeneratePdf.firstCall.args[0].output, customOutput, 'should pass output option to generatePdf')
-})
-
-test('pdf command should use pdfEngine from config when --engine not specified', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-  // Mock config to return 'prince' for pdfEngine
-  command.config = { get: (key) => key === 'pdfEngine' ? 'prince' : undefined }
-
-  // No --engine specified, should use config value
-  await command.action({}, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.is(mockGeneratePdf.firstCall.args[0].lib, 'prince', 'should use pdfEngine from config')
-})
-
-test('pdf command should use default engine when --engine not specified and config not set', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-  // Mock config to return undefined (no pdfEngine set)
-  command.config = { get: () => undefined }
-
-  // No --engine specified, no config, should use default 'pagedjs'
-  await command.action({}, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.is(mockGeneratePdf.firstCall.args[0].lib, 'pagedjs', 'should use default pagedjs when no config')
-})
-
-test('pdf command --engine flag should override config pdfEngine', async (t) => {
-  const { sandbox, mockReporter } = t.context
-
-  const mockGeneratePdf = sandbox.stub().resolves('/project/_site/_pdf/test-book.pdf')
-
-  const PDFCommand = await esmock('./pdf.js', {
-    '#lib/pdf/index.js': {
-      default: mockGeneratePdf
-    },
-    '#lib/project/index.js': {
-      hasSiteOutput: () => true
-    },
-    '#lib/reporter/index.js': {
-      default: mockReporter
-    },
-    open: {
-      default: sandbox.stub()
-    }
-  })
-
-  const command = new PDFCommand()
-  command.name = sandbox.stub().returns('pdf')
-  // Mock config to return 'prince' for pdfEngine
-  command.config = { get: (key) => key === 'pdfEngine' ? 'prince' : undefined }
-
-  // --engine pagedjs should override config's 'prince'
-  await command.action({ engine: 'pagedjs' }, command)
-
-  t.true(mockGeneratePdf.called, 'generatePdf should be called')
-  t.is(mockGeneratePdf.firstCall.args[0].lib, 'pagedjs', 'CLI --engine should override config')
-})

package/src/commands/preview.spec.js

@@ -1,97 +0,0 @@
-import { Command, Option } from 'commander'
-import program from '#src/main.js'
-import test from 'ava'
-
-/**
- * Command Contract/Interface Tests
- *
- * Verifies the command's public API and Commander.js integration.
- * @see docs/testing-commands.md
- */
-
-test.before((t) => {
-  // Get the registered command (from program.commands) once and share across all tests
-  t.context.command = program.commands.find((cmd) => cmd.name() === 'preview')
-})
-
-test('command is registered in CLI program', (t) => {
-  const { command } = t.context
-
-  t.truthy(command, 'command "preview" should be registered in program')
-  t.true(command instanceof Command, 'registered command should be Commander.js Command instance')
-})
-
-test('registered command has correct metadata', (t) => {
-  const { command } = t.context
-
-  t.is(command.name(), 'preview')
-  t.truthy(command.description())
-  t.is(typeof command._actionHandler, 'function', 'command should have action handler')
-})
-
-test('registered command has no arguments', (t) => {
-  const { command } = t.context
-  const registeredArguments = command.registeredArguments
-
-  t.is(registeredArguments.length, 0, 'preview command should have no arguments')
-})
-
-test('registered command has correct options', (t) => {
-  const { command } = t.context
-
-  // Get all options
-  const portOption = command.options.find((opt) => opt.long === '--port')
-  const quietOption = command.options.find((opt) => opt.long === '--quiet')
-  const verboseOption = command.options.find((opt) => opt.long === '--verbose')
-  const eleventyOption = command.options.find((opt) => opt.long === '--11ty')
-  const debugOption = command.options.find((opt) => opt.long === '--debug')
-
-  // Verify all options exist
-  t.truthy(portOption, '--port option should exist')
-  t.truthy(quietOption, '--quiet option should exist')
-  t.truthy(verboseOption, '--verbose option should exist')
-  t.truthy(eleventyOption, '--11ty option should exist')
-  t.truthy(debugOption, '--debug option should exist')
-
-  // Verify they are Option instances
-  t.true(portOption instanceof Option, '--port should be Option instance')
-  t.true(quietOption instanceof Option, '--quiet should be Option instance')
-  t.true(verboseOption instanceof Option, '--verbose should be Option instance')
-  t.true(eleventyOption instanceof Option, '--11ty should be Option instance')
-  t.true(debugOption instanceof Option, '--debug should be Option instance')
-
-  // Verify option properties
-  t.is(portOption.long, '--port')
-  t.is(portOption.short, '-p')
-  t.truthy(portOption.description)
-  t.true(portOption.required, '--port should require a value')
-
-  t.is(quietOption.long, '--quiet')
-  t.is(quietOption.short, '-q')
-  t.truthy(quietOption.description)
-
-  t.is(verboseOption.long, '--verbose')
-  t.is(verboseOption.short, '-v')
-  t.truthy(verboseOption.description)
-
-  t.is(eleventyOption.long, '--11ty')
-  t.truthy(eleventyOption.description)
-  t.true(eleventyOption.required, '--11ty should require a value')
-
-  t.is(debugOption.long, '--debug')
-  t.truthy(debugOption.description)
-  t.false(debugOption.required, '--debug should not require a value')
-})
-
-test('command options are accessible via public API', (t) => {
-  const { command } = t.context
-
-  // Test that options can be accessed the way Commander.js does
-  const optionNames = command.options.map((opt) => opt.long)
-
-  t.true(optionNames.includes('--port'))
-  t.true(optionNames.includes('--quiet'))
-  t.true(optionNames.includes('--verbose'))
-  t.true(optionNames.includes('--11ty'))
-  t.true(optionNames.includes('--debug'))
-})