diff --git a/src/commands/env/env-set.ts b/src/commands/env/env-set.ts index 9c4b670c674..d1cf426cdea 100644 --- a/src/commands/env/env-set.ts +++ b/src/commands/env/env-set.ts @@ -1,5 +1,3 @@ -import process from 'process' - import { OptionValues } from 'commander' import { chalk, error, log, logJson } from '../../utils/command-helpers.js' diff --git a/src/utils/prompts/blob-delete-prompts.ts b/src/utils/prompts/blob-delete-prompts.ts index 42e37b8428a..cf319586208 100644 --- a/src/utils/prompts/blob-delete-prompts.ts +++ b/src/utils/prompts/blob-delete-prompts.ts @@ -4,14 +4,11 @@ import { confirmPrompt } from './confirm-prompt.js' import { destructiveCommandMessages } from './prompt-messages.js' export const promptBlobDelete = async (key: string, storeName: string): Promise => { - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, overwriteConfirmationMessage } = destructiveCommandMessages.blobDelete - - const warningMessage = generateWarningMessage(key, storeName) + const warningMessage = destructiveCommandMessages.blobDelete.generateWarning(key, storeName) log() log(warningMessage) log() - log(overwriteNoticeMessage) - await confirmPrompt(overwriteConfirmationMessage) + log(destructiveCommandMessages.overwriteNotice) + await confirmPrompt(destructiveCommandMessages.blobDelete.overwriteConfirmation) } diff --git a/src/utils/prompts/blob-set-prompt.ts b/src/utils/prompts/blob-set-prompt.ts index 7d7ee58b12e..df84022db60 100644 --- a/src/utils/prompts/blob-set-prompt.ts +++ b/src/utils/prompts/blob-set-prompt.ts @@ -4,14 +4,11 @@ import { confirmPrompt } from './confirm-prompt.js' import { destructiveCommandMessages } from './prompt-messages.js' export const promptBlobSetOverwrite = async (key: string, storeName: string): Promise => { - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, overwriteConfirmationMessage } = destructiveCommandMessages.blobSet - - const warningMessage = generateWarningMessage(key, storeName) + const warningMessage = destructiveCommandMessages.blobDelete.generateWarning(key, storeName) log() log(warningMessage) log() - log(overwriteNoticeMessage) - await confirmPrompt(overwriteConfirmationMessage) + log(destructiveCommandMessages.overwriteNotice) + await confirmPrompt(destructiveCommandMessages.blobDelete.overwriteConfirmation) } diff --git a/src/utils/prompts/env-clone-prompt.ts b/src/utils/prompts/env-clone-prompt.ts index b72d2c73d26..69a5719afc6 100644 --- a/src/utils/prompts/env-clone-prompt.ts +++ b/src/utils/prompts/env-clone-prompt.ts @@ -14,23 +14,21 @@ export const generateEnvVarsList = (envVarsToDelete: EnvVar[]) => envVarsToDelet * @returns {Promise} A promise that resolves when the user has confirmed the overwriting of the variables. */ export async function promptEnvCloneOverwrite(siteId: string, existingEnvVars: EnvVar[]): Promise { - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, noticeEnvVarsMessage, overwriteConfirmationMessage } = - destructiveCommandMessages.envClone + const { generateWarning } = destructiveCommandMessages.envClone const existingEnvVarKeys = generateEnvVarsList(existingEnvVars) - const warningMessage = generateWarningMessage(siteId) + const warningMessage = generateWarning(siteId) log() log(warningMessage) log() - log(noticeEnvVarsMessage) + log(destructiveCommandMessages.envClone.noticeEnvVars) log() existingEnvVarKeys.forEach((envVar) => { log(envVar) }) log() - log(overwriteNoticeMessage) + log(destructiveCommandMessages.overwriteNotice) - await confirmPrompt(overwriteConfirmationMessage) + await confirmPrompt(destructiveCommandMessages.envClone.overwriteConfirmation) } diff --git a/src/utils/prompts/env-set-prompts.ts b/src/utils/prompts/env-set-prompts.ts index e256b333b4d..981867bced7 100644 --- a/src/utils/prompts/env-set-prompts.ts +++ b/src/utils/prompts/env-set-prompts.ts @@ -3,21 +3,12 @@ import { log } from '../command-helpers.js' import { confirmPrompt } from './confirm-prompt.js' import { destructiveCommandMessages } from './prompt-messages.js' -/** - * Prompts the user to confirm overwriting an existing environment variable. - * - * @param {string} existingKey - The key of the existing environment variable. - * @returns {Promise} A promise that resolves when the user confirms overwriting the variable. - */ -export const promptOverwriteEnvVariable = async (existingKey: string): Promise => { - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, overwriteConfirmationMessage } = destructiveCommandMessages.envSet - - const warningMessage = generateWarningMessage(existingKey) +export const promptOverwriteEnvVariable = async (key: string): Promise => { + const warningMessage = destructiveCommandMessages.envSet.generateWarning(key) log() log(warningMessage) log() - log(overwriteNoticeMessage) - await confirmPrompt(overwriteConfirmationMessage) + log(destructiveCommandMessages.overwriteNotice) + await confirmPrompt(destructiveCommandMessages.envSet.overwriteConfirmation) } diff --git a/src/utils/prompts/env-unset-prompts.ts b/src/utils/prompts/env-unset-prompts.ts index 8128922636e..73af7886889 100644 --- a/src/utils/prompts/env-unset-prompts.ts +++ b/src/utils/prompts/env-unset-prompts.ts @@ -10,13 +10,12 @@ import { destructiveCommandMessages } from './prompt-messages.js' * @returns {Promise} A promise that resolves when the user has confirmed overwriting the variable */ export const promptOverwriteEnvVariable = async (existingKey: string): Promise => { - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, overwriteConfirmationMessage } = destructiveCommandMessages.envUnset + const { generateWarning } = destructiveCommandMessages.envUnset - const warningMessage = generateWarningMessage(existingKey) + const warningMessage = generateWarning(existingKey) log(warningMessage) log() - log(overwriteNoticeMessage) - await confirmPrompt(overwriteConfirmationMessage) + log(destructiveCommandMessages.overwriteNotice) + await confirmPrompt(destructiveCommandMessages.envUnset.overwriteConfirmation) } diff --git a/src/utils/prompts/prompt-messages.ts b/src/utils/prompts/prompt-messages.ts index f8b8fcf02d0..29014136499 100644 --- a/src/utils/prompts/prompt-messages.ts +++ b/src/utils/prompts/prompt-messages.ts @@ -1,48 +1,46 @@ import { chalk } from '../command-helpers.js' export const destructiveCommandMessages = { - overwriteNoticeMessage: `${chalk.yellowBright( - 'Notice', - )}: To overwrite without this warning, you can use the --force flag.`, + overwriteNotice: `${chalk.yellowBright('Notice')}: To overwrite without this warning, you can use the --force flag.`, blobSet: { - generateWarningMessage: (key: string, storeName: string) => + generateWarning: (key: string, storeName: string) => `${chalk.redBright('Warning')}: The blob key ${chalk.cyan(key)} already exists in store ${chalk.cyan( storeName, )}!`, - overwriteConfirmationMessage: 'Do you want to proceed with overwriting this blob key existing value?', + overwriteConfirmation: 'Do you want to proceed with overwriting this blob key existing value?', }, blobDelete: { - generateWarningMessage: (key: string, storeName: string) => + generateWarning: (key: string, storeName: string) => `${chalk.redBright('Warning')}: The following blob key ${chalk.cyan(key)} will be deleted from store ${chalk.cyan( storeName, )}!`, - overwriteConfirmationMessage: 'Do you want to proceed with deleting the value at this key?', + overwriteConfirmation: 'Do you want to proceed with deleting the value at this key?', }, envSet: { - generateWarningMessage: (variableName: string) => + generateWarning: (variableName: string) => `${chalk.redBright('Warning')}: The environment variable ${chalk.bgBlueBright(variableName)} already exists!`, - overwriteConfirmationMessage: 'Do you want to overwrite it?', + overwriteConfirmation: 'Do you want to overwrite it?', }, envUnset: { - generateWarningMessage: (variableName: string) => + generateWarning: (variableName: string) => `${chalk.redBright('Warning')}: The environment variable ${chalk.bgBlueBright( variableName, )} will be removed from all contexts!`, - overwriteConfirmationMessage: 'Do you want to remove it?', + overwriteConfirmation: 'Do you want to remove it?', }, envClone: { - generateWarningMessage: (siteId: string) => + generateWarning: (siteId: string) => `${chalk.redBright( 'Warning', )}: The following environment variables are already set on the site with ID ${chalk.bgBlueBright( siteId, )}. They will be overwritten!`, - noticeEnvVarsMessage: `${chalk.yellowBright('Notice')}: The following variables will be overwritten:`, - overwriteConfirmationMessage: 'The environment variables already exist. Do you want to overwrite them?', + noticeEnvVars: `${chalk.yellowBright('Notice')}: The following variables will be overwritten:`, + overwriteConfirmation: 'The environment variables already exist. Do you want to overwrite them?', }, } diff --git a/tests/integration/commands/blobs/blobs-delete.test.ts b/tests/integration/commands/blobs/blobs-delete.test.ts index 739670c8834..d4652830161 100644 --- a/tests/integration/commands/blobs/blobs-delete.test.ts +++ b/tests/integration/commands/blobs/blobs-delete.test.ts @@ -51,10 +51,10 @@ describe('blob:delete command', () => { const storeName = 'my-store' const key = 'my-key' - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, overwriteConfirmationMessage } = destructiveCommandMessages.blobDelete + const { overwriteNotice } = destructiveCommandMessages + const { generateWarning, overwriteConfirmation } = destructiveCommandMessages.blobDelete - const warningMessage = generateWarningMessage(key, storeName) + const warningMessage = generateWarning(key, storeName) const successMessage = `${chalk.greenBright('Success')}: Blob ${chalk.yellow(key)} deleted from store ${chalk.yellow( storeName, @@ -89,12 +89,12 @@ describe('blob:delete command', () => { expect(promptSpy).toHaveBeenCalledWith({ type: 'confirm', name: 'confirm', - message: expect.stringContaining(overwriteConfirmationMessage), + message: expect.stringContaining(overwriteConfirmation), default: false, }) expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -124,12 +124,12 @@ describe('blob:delete command', () => { expect(promptSpy).toHaveBeenCalledWith({ type: 'confirm', name: 'confirm', - message: expect.stringContaining(overwriteConfirmationMessage), + message: expect.stringContaining(overwriteConfirmation), default: false, }) expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).not.toHaveBeenCalledWith(successMessage) }) }) @@ -154,7 +154,7 @@ describe('blob:delete command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -188,7 +188,7 @@ describe('blob:delete command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).not.toHaveBeenCalledWith(successMessage) }) } catch (error) { @@ -229,7 +229,7 @@ describe('blob:delete command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -255,7 +255,7 @@ describe('blob:delete command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) diff --git a/tests/integration/commands/blobs/blobs-set.test.ts b/tests/integration/commands/blobs/blobs-set.test.ts index 206fbdd0f1c..4e043a843fc 100644 --- a/tests/integration/commands/blobs/blobs-set.test.ts +++ b/tests/integration/commands/blobs/blobs-set.test.ts @@ -52,10 +52,10 @@ describe('blob:set command', () => { const value = 'my-value' const newValue = 'my-new-value' - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, overwriteConfirmationMessage } = destructiveCommandMessages.blobSet + const { overwriteNotice } = destructiveCommandMessages + const { generateWarning, overwriteConfirmation } = destructiveCommandMessages.blobSet - const warningMessage = generateWarningMessage(key, storeName) + const warningMessage = generateWarning(key, storeName) const successMessage = `${chalk.greenBright('Success')}: Blob ${chalk.yellow(key)} set in store ${chalk.yellow( storeName, @@ -89,7 +89,7 @@ describe('blob:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).toHaveBeenCalledWith(successMessage) expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) }) }) @@ -116,13 +116,13 @@ describe('blob:set command', () => { expect(promptSpy).toHaveBeenCalledWith({ type: 'confirm', name: 'confirm', - message: expect.stringContaining(overwriteConfirmationMessage), + message: expect.stringContaining(overwriteConfirmation), default: false, }) expect(log).toHaveBeenCalledWith(successMessage) expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) }) }) @@ -154,12 +154,12 @@ describe('blob:set command', () => { expect(promptSpy).toHaveBeenCalledWith({ type: 'confirm', name: 'confirm', - message: expect.stringContaining(overwriteConfirmationMessage), + message: expect.stringContaining(overwriteConfirmation), default: false, }) expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).not.toHaveBeenCalledWith(successMessage) }) }) @@ -187,7 +187,7 @@ describe('blob:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -216,7 +216,7 @@ describe('blob:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).not.toHaveBeenCalledWith(successMessage) }) }) @@ -256,7 +256,7 @@ describe('blob:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -285,7 +285,7 @@ describe('blob:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) diff --git a/tests/integration/commands/env/env-clone.test.ts b/tests/integration/commands/env/env-clone.test.ts index 3ae6d0a5433..17fe47f6632 100644 --- a/tests/integration/commands/env/env-clone.test.ts +++ b/tests/integration/commands/env/env-clone.test.ts @@ -22,12 +22,11 @@ describe('env:clone command', () => { const sharedEnvVars = [existingVar, existingVar] const siteIdTwo = secondSiteInfo.id - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, noticeEnvVarsMessage, overwriteConfirmationMessage } = - destructiveCommandMessages.envClone + const { overwriteNotice } = destructiveCommandMessages + const { generateWarning, noticeEnvVars, overwriteConfirmation } = destructiveCommandMessages.envClone const envVarsList = generateEnvVarsList(sharedEnvVars) - const warningMessage = generateWarningMessage(siteIdTwo) + const warningMessage = generateWarning(siteIdTwo) const successMessage = `Successfully cloned environment variables from ${chalk.green('site-name')} to ${chalk.green( 'site-name-2', @@ -53,16 +52,16 @@ describe('env:clone command', () => { expect(promptSpy).toHaveBeenCalledWith({ type: 'confirm', name: 'confirm', - message: expect.stringContaining(overwriteConfirmationMessage), + message: expect.stringContaining(overwriteConfirmation), default: false, }) expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(noticeEnvVarsMessage) + expect(log).toHaveBeenCalledWith(noticeEnvVars) envVarsList.forEach((envVar) => { expect(log).toHaveBeenCalledWith(envVar) }) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -84,8 +83,8 @@ describe('env:clone command', () => { envVarsList.forEach((envVar) => { expect(log).not.toHaveBeenCalledWith(envVar) }) - expect(log).not.toHaveBeenCalledWith(noticeEnvVarsMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(noticeEnvVars) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -109,11 +108,11 @@ describe('env:clone command', () => { expect(promptSpy).toHaveBeenCalled() expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(noticeEnvVarsMessage) + expect(log).toHaveBeenCalledWith(noticeEnvVars) envVarsList.forEach((envVar) => { expect(log).toHaveBeenCalledWith(envVar) }) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).not.toHaveBeenCalledWith(successMessage) }) }) @@ -135,11 +134,11 @@ describe('env:clone command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(noticeEnvVarsMessage) + expect(log).not.toHaveBeenCalledWith(noticeEnvVars) envVarsList.forEach((envVar) => { expect(log).not.toHaveBeenCalledWith(envVar) }) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessageSite3) }) }) @@ -171,7 +170,7 @@ describe('env:clone command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -192,7 +191,7 @@ describe('env:clone command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) diff --git a/tests/integration/commands/env/env-set.test.ts b/tests/integration/commands/env/env-set.test.ts index 26b11f9bafb..e01f399ed0e 100644 --- a/tests/integration/commands/env/env-set.test.ts +++ b/tests/integration/commands/env/env-set.test.ts @@ -22,10 +22,10 @@ describe('env:set command', () => { // already exists as value in withMockApi const existingVar = 'EXISTING_VAR' const newEnvValue = 'value' - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, overwriteConfirmationMessage } = destructiveCommandMessages.envSet + const { overwriteNotice } = destructiveCommandMessages + const { generateWarning, overwriteConfirmation } = destructiveCommandMessages.envSet - const warningMessage = generateWarningMessage(existingVar) + const warningMessage = generateWarning(existingVar) const successMessage = `Set environment variable ${chalk.yellow( `${existingVar}=${newEnvValue}`, @@ -302,12 +302,12 @@ describe('env:set command', () => { expect(promptSpy).toHaveBeenCalledWith({ type: 'confirm', name: 'confirm', - message: expect.stringContaining(overwriteConfirmationMessage), + message: expect.stringContaining(overwriteConfirmation), default: false, }) expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -326,7 +326,7 @@ describe('env:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith( `Set environment variable ${chalk.yellow(`${'NEW_ENV_VAR'}=${'NEW_VALUE'}`)} in the ${chalk.magenta( 'all', @@ -349,7 +349,7 @@ describe('env:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -373,7 +373,7 @@ describe('env:set command', () => { expect(promptSpy).toHaveBeenCalled() expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).not.toHaveBeenCalledWith(successMessage) }) }) @@ -404,7 +404,7 @@ describe('env:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) @@ -425,7 +425,7 @@ describe('env:set command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(successMessage) }) }) diff --git a/tests/integration/commands/env/env-unset.test.ts b/tests/integration/commands/env/env-unset.test.ts index d91c9c94a2d..00b8c11a329 100644 --- a/tests/integration/commands/env/env-unset.test.ts +++ b/tests/integration/commands/env/env-unset.test.ts @@ -19,12 +19,12 @@ vi.mock('../../../../src/utils/command-helpers.js', async () => ({ })) describe('env:unset command', () => { - const { overwriteNoticeMessage } = destructiveCommandMessages - const { generateWarningMessage, overwriteConfirmationMessage } = destructiveCommandMessages.envUnset + const { overwriteNotice } = destructiveCommandMessages + const { generateWarning, overwriteConfirmation } = destructiveCommandMessages.envUnset // already exists as value in withMockApi const existingVar = 'EXISTING_VAR' - const warningMessage = generateWarningMessage(existingVar) + const warningMessage = generateWarning(existingVar) const expectedSuccessMessage = `Unset environment variable ${chalk.yellow(`${existingVar}`)} in the ${chalk.magenta( 'all', )} context` @@ -106,12 +106,12 @@ describe('env:unset command', () => { expect(promptSpy).toHaveBeenCalledWith({ type: 'confirm', name: 'confirm', - message: expect.stringContaining(overwriteConfirmationMessage), + message: expect.stringContaining(overwriteConfirmation), default: false, }) expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(expectedSuccessMessage) }) }) @@ -130,7 +130,7 @@ describe('env:unset command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(expectedSuccessMessage) }) }) @@ -154,7 +154,7 @@ describe('env:unset command', () => { expect(promptSpy).toHaveBeenCalled() expect(log).toHaveBeenCalledWith(warningMessage) - expect(log).toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).toHaveBeenCalledWith(overwriteNotice) expect(log).not.toHaveBeenCalledWith(expectedSuccessMessage) }) }) @@ -173,7 +173,7 @@ describe('env:unset command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).not.toHaveBeenCalledWith(expectedSuccessMessage) }) }) @@ -205,7 +205,7 @@ describe('env:unset command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(expectedSuccessMessage) }) }) @@ -226,7 +226,7 @@ describe('env:unset command', () => { expect(promptSpy).not.toHaveBeenCalled() expect(log).not.toHaveBeenCalledWith(warningMessage) - expect(log).not.toHaveBeenCalledWith(overwriteNoticeMessage) + expect(log).not.toHaveBeenCalledWith(overwriteNotice) expect(log).toHaveBeenCalledWith(expectedSuccessMessage) }) })