Skip to content

Commit b9c2c30

Browse files
committed
test(cli): assert on runner-agnostic suffix of migrateNotImplemented
`migrateNotImplemented` was converted from a string to a runner-aware factory in 6cc47d2; the existing E2E assertion still passed it as a string to `toContain`, which then matched against the printed function name instead of the warning text. Local unit tests use a separate vitest config and didn't catch it; the CLI E2E job did. Switch the assertion to the stable runner-agnostic suffix `'stash db migrate" is not yet implemented.'` — works under any runner the test environment detects (pnpm dlx in CI, npm/npx locally).
1 parent 6cc47d2 commit b9c2c30

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

packages/cli/tests/e2e/smoke.e2e.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,8 @@ describe('stash CLI — non-interactive smoke', () => {
6969
const r = render(['db', 'migrate'])
7070
const { exitCode } = await r.exit
7171
expect(exitCode).toBe(0)
72-
expect(r.output).toContain(messages.db.migrateNotImplemented)
72+
// `migrateNotImplemented` is a runner-aware factory; the runner-agnostic
73+
// suffix is the stable assertion target.
74+
expect(r.output).toContain('stash db migrate" is not yet implemented.')
7375
})
7476
})

0 commit comments

Comments
 (0)