|
| 1 | +import Rename from './rename.js' |
| 2 | +import {themeRenameJsonOutputSchema} from '../../services/rename/types.js' |
| 3 | +import {findOrSelectTheme} from '../../utilities/theme-selector.js' |
| 4 | +import {Config} from '@oclif/core' |
| 5 | +import {ensureAuthenticatedThemes} from '@shopify/cli-kit/node/session' |
| 6 | +import {themeUpdate} from '@shopify/cli-kit/node/themes/api' |
| 7 | +import {withCapturedStandardStreams} from '@shopify/cli-kit/node/testing/output' |
| 8 | +import {runWithCommandEventsForCommand} from '@shopify/cli-kit/node/command-events' |
| 9 | +import {inTemporaryDirectory, writeFile} from '@shopify/cli-kit/node/fs' |
| 10 | +import {joinPath} from '@shopify/cli-kit/node/path' |
| 11 | +import {outputWarn} from '@shopify/cli-kit/node/output' |
| 12 | +import {describe, expect, test, vi} from 'vitest' |
| 13 | + |
| 14 | +vi.mock('@shopify/cli-kit/node/session') |
| 15 | +vi.mock('@shopify/cli-kit/node/themes/api') |
| 16 | +vi.mock('@shopify/cli-kit/node/metadata') |
| 17 | +vi.mock('../../utilities/theme-selector.js') |
| 18 | +vi.mock('../../utilities/theme-store.js', () => ({ensureThemeStore: ({store}: {store: string}) => store})) |
| 19 | + |
| 20 | +const originalTheme = {id: 1, name: 'Original', role: 'unpublished', processing: false, createdAtRuntime: false} |
| 21 | +const renamedTheme = {...originalTheme, name: 'Renamed Theme'} |
| 22 | +const store = 'test.myshopify.com' |
| 23 | + |
| 24 | +async function run(argv: string[]) { |
| 25 | + const config = new Config({root: __dirname}) |
| 26 | + await config.load() |
| 27 | + vi.mocked(ensureAuthenticatedThemes).mockImplementation(async (storeFqdn) => ({token: 'token', storeFqdn})) |
| 28 | + return runWithCommandEventsForCommand(argv, () => new Rename(argv, config).run()) |
| 29 | +} |
| 30 | + |
| 31 | +async function inEnvironments(run: () => Promise<void>) { |
| 32 | + await inTemporaryDirectory(async (directory) => { |
| 33 | + await writeFile( |
| 34 | + joinPath(directory, 'shopify.theme.toml'), |
| 35 | + ` |
| 36 | +[environments.first] |
| 37 | +store = "first.myshopify.com" |
| 38 | +password = "token" |
| 39 | +[environments.second] |
| 40 | +store = "second.myshopify.com" |
| 41 | +password = "token" |
| 42 | +`, |
| 43 | + ) |
| 44 | + const cwd = vi.spyOn(process, 'cwd').mockReturnValue(directory) |
| 45 | + try { |
| 46 | + await run() |
| 47 | + } finally { |
| 48 | + cwd.mockRestore() |
| 49 | + } |
| 50 | + }) |
| 51 | +} |
| 52 | + |
| 53 | +describe('theme rename JSON output', () => { |
| 54 | + test('exposes its schema and JSON flag in help', () => { |
| 55 | + expect(Rename.jsonOutputSchema).toBe(themeRenameJsonOutputSchema) |
| 56 | + expect(Rename.flags.json).toBeDefined() |
| 57 | + expect(Rename.description).toContain('ThemeRenameResult') |
| 58 | + }) |
| 59 | + |
| 60 | + test.each([undefined, '', 'https://example.com/theme.zip'])( |
| 61 | + 'returns the updated theme and omits missing src (%s)', |
| 62 | + async (src) => { |
| 63 | + vi.mocked(findOrSelectTheme).mockResolvedValue(originalTheme) |
| 64 | + vi.mocked(themeUpdate).mockResolvedValue({...renamedTheme, src}) |
| 65 | + await withCapturedStandardStreams(async ({stdout, stderr}) => { |
| 66 | + await run(['--store', store, '--theme', '1', '--name', 'Renamed Theme', '--json']) |
| 67 | + expect(JSON.parse(stdout())).toEqual({ |
| 68 | + theme: {...renamedTheme, ...(src === undefined ? {} : {src}), shop: store}, |
| 69 | + }) |
| 70 | + expect(stderr()).toBe('') |
| 71 | + }) |
| 72 | + }, |
| 73 | + ) |
| 74 | + |
| 75 | + test('routes diagnostics to stderr', async () => { |
| 76 | + vi.mocked(findOrSelectTheme).mockResolvedValue(originalTheme) |
| 77 | + vi.mocked(themeUpdate).mockImplementation(async () => { |
| 78 | + outputWarn('Retrying request') |
| 79 | + return renamedTheme |
| 80 | + }) |
| 81 | + await withCapturedStandardStreams(async ({stdout, stderr}) => { |
| 82 | + await run(['--store', store, '--theme', '1', '--name', 'Renamed Theme', '--json']) |
| 83 | + expect(JSON.parse(stdout()).theme.name).toBe('Renamed Theme') |
| 84 | + expect(JSON.parse(stderr())).toMatchObject({type: 'diagnostic', level: 'warning', message: 'Retrying request'}) |
| 85 | + }) |
| 86 | + }) |
| 87 | + |
| 88 | + test('leaves execution failures to the shared error handler without writing a result', async () => { |
| 89 | + vi.mocked(findOrSelectTheme).mockResolvedValue(originalTheme) |
| 90 | + vi.mocked(themeUpdate).mockRejectedValue(new Error('Renameing failed')) |
| 91 | + await withCapturedStandardStreams(async ({stdout}) => { |
| 92 | + await expect(run(['--store', store, '--theme', '1', '--name', 'Renamed Theme', '--json'])).rejects.toThrow( |
| 93 | + 'Renameing failed', |
| 94 | + ) |
| 95 | + expect(stdout()).toBe('') |
| 96 | + }) |
| 97 | + }) |
| 98 | + |
| 99 | + test.each(['none', 'partial', 'all'])('collects one document in configured order (%s failures)', async (failures) => { |
| 100 | + vi.mocked(findOrSelectTheme).mockResolvedValue(originalTheme) |
| 101 | + vi.mocked(themeUpdate).mockImplementation(async (_id, _params, session) => { |
| 102 | + if (failures === 'all' || (failures === 'partial' && session.storeFqdn.startsWith('first.'))) { |
| 103 | + throw new Error('Renameing failed') |
| 104 | + } |
| 105 | + return renamedTheme |
| 106 | + }) |
| 107 | + const exitCode = process.exitCode |
| 108 | + await inEnvironments(async () => { |
| 109 | + await withCapturedStandardStreams(async ({stdout, stderr}) => { |
| 110 | + await run([ |
| 111 | + '--environment', |
| 112 | + 'first', |
| 113 | + '--environment', |
| 114 | + 'second', |
| 115 | + '--theme', |
| 116 | + '1', |
| 117 | + '--name', |
| 118 | + 'Renamed Theme', |
| 119 | + '--json', |
| 120 | + ]) |
| 121 | + const environments: string[] = [] |
| 122 | + if (failures === 'none') environments.push('first') |
| 123 | + if (failures !== 'all') environments.push('second') |
| 124 | + expect(JSON.parse(stdout())).toEqual({ |
| 125 | + environments: environments.map((environment) => ({ |
| 126 | + environment, |
| 127 | + result: {theme: {...renamedTheme, shop: `${environment}.myshopify.com`}}, |
| 128 | + })), |
| 129 | + }) |
| 130 | + if (failures === 'none') { |
| 131 | + expect(stderr()).toBe('') |
| 132 | + } else { |
| 133 | + const errors = stderr() |
| 134 | + .trim() |
| 135 | + .split('\n') |
| 136 | + .map((line) => JSON.parse(line)) |
| 137 | + expect(errors).toHaveLength(failures === 'all' ? 2 : 1) |
| 138 | + expect(errors[0]).toMatchObject({type: 'diagnostic', level: 'error', code: 'theme-environment-failed'}) |
| 139 | + } |
| 140 | + expect(process.exitCode).toBe(exitCode) |
| 141 | + }) |
| 142 | + }) |
| 143 | + }) |
| 144 | + |
| 145 | + test.each([ |
| 146 | + {theme: {...renamedTheme, id: '1', shop: store}}, |
| 147 | + {theme: {...renamedTheme, shop: null}}, |
| 148 | + {theme: {...renamedTheme, src: false, shop: store}}, |
| 149 | + {environments: [{environment: 'first', result: {theme: {...renamedTheme, role: null, shop: store}}}]}, |
| 150 | + ])('rejects malformed results %#', (result) => { |
| 151 | + expect(() => themeRenameJsonOutputSchema.validate(result)).toThrow() |
| 152 | + }) |
| 153 | +}) |
0 commit comments