@squiz/dxp-cli-next 5.15.0-develop.1 → 5.15.0-develop.2

package/lib/porter/project/get/get.spec.js

@@ -31,22 +31,15 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
-const nock_1 = __importDefault(require("nock"));
-const get_1 = __importDefault(require("./get"));
 const axios_1 = require("axios");
 const ApiService = __importStar(require("../../../ApiService"));
-const BuildPorterUrl = __importStar(require("../../utils/BuildPorterUrl/BuildPorterUrl"));
-const ErrorUtils = __importStar(require("../../utils/ErrorUtils/ErrorUtils"));
-const GetProjectConfig = __importStar(require("../../utils/GetProjectConfig/GetProjectConfig"));
+const PorterUtils = __importStar(require("../../utils"));
 const logSpy = jest.spyOn(global.console, 'log');
+jest.mock('../../utils');
 describe('getProject', () => {
     let baseUrl;
     let projectName;
-    let projectConfig;
     let mockHandleCommandError;
     let mockHandleError;
     let mockBuildPorterUrl;
@@ -55,14 +48,8 @@ describe('getProject', () => {
     beforeEach(() => {
         jest.resetAllMocks();
         jest.clearAllMocks();
-        nock_1.default.restore();
         projectName = 'porter-project';
         baseUrl = 'mock-url';
-        projectConfig = {
-            service: 'dxp-porter',
-            projectName,
-            version: '0.1.0',
-        };
         mockHandleCommandError = jest.fn(() => Promise.resolve());
         mockHandleError = jest.fn();
         mockBuildPorterUrl = jest.fn(() => Promise.resolve(baseUrl));
@@ -73,14 +60,18 @@ describe('getProject', () => {
         mockClient = {
             get: jest.fn().mockImplementationOnce(() => mockResponse),
         };
+        jest.spyOn(PorterUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
+        jest
+            .spyOn(PorterUtils, 'getProjectName')
+            .mockImplementationOnce(() => Promise.resolve(projectName));
         jest
-            .spyOn(BuildPorterUrl, 'buildPorterUrl')
+            .spyOn(PorterUtils, 'buildPorterUrl')
             .mockImplementationOnce(mockBuildPorterUrl);
         jest
-            .spyOn(ErrorUtils, 'handleCommandError')
+            .spyOn(PorterUtils, 'handleCommandError')
             .mockImplementationOnce(mockHandleCommandError);
         jest
-            .spyOn(ErrorUtils, 'handleError')
+            .spyOn(PorterUtils, 'handleError')
             .mockImplementationOnce(mockHandleError);
         jest.spyOn(process, 'exit').mockImplementation();
         jest.spyOn(ApiService, 'ApiService').mockImplementationOnce(() => {
@@ -92,47 +83,34 @@ describe('getProject', () => {
     it('should throw an error if not logged in', () => __awaiter(void 0, void 0, void 0, function* () {
         const mockError = new Error('Error not logged in');
         jest
-            .spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn')
-            .mockImplementationOnce(() => {
+            .spyOn(PorterUtils, 'throwErrorIfNotLoggedIn')
+            .mockImplementation(() => {
             throw mockError;
         });
-        (0, nock_1.default)('http://localhost:9999')
-            .get('/__dxp/au/porter/SquizRoot-0000/health')
-            .reply(401);
-        const program = (0, get_1.default)();
-        yield expect(() => program.parseAsync(['dxp-next', 'porter', 'project', 'getProject'])).rejects.toThrow(mockError);
+        const { default: createGetCommand } = require('./get');
+        const program = createGetCommand();
+        yield expect(() => program.parseAsync(['project', 'getProject'])).rejects.toThrow(mockError);
         const opts = program.opts();
         expect(opts).toStrictEqual({});
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).not.toHaveBeenCalled();
     }));
-    it('should throw an error if not in a project directory', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
+    it('should throw an error if project name could not be found', () => __awaiter(void 0, void 0, void 0, function* () {
         jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => {
-            throw new Error('You must be within a porter project directory to use this command');
+            .spyOn(PorterUtils, 'getProjectName')
+            .mockReset()
+            .mockImplementation(() => {
+            throw new Error();
         });
         const { default: createGetCommand } = require('./get');
         const program = createGetCommand();
         yield program.parseAsync(['dxp-next', 'porter', 'project', 'getProject']);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).toHaveBeenCalled();
-    }));
-    it('should throw an error if project name is missing in the config.json file', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(Object.assign(Object.assign({}, projectConfig), { projectName: '' })));
-        const { default: createGetCommand } = require('./get');
-        const program = createGetCommand();
-        yield program.parseAsync(['dxp-next', 'porter', 'project', 'getProject']);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).toHaveBeenCalled();
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
+        expect(mockClient.get).not.toHaveBeenCalled();
+        expect(PorterUtils.handleCommandError).toHaveBeenCalled();
     }));
     it('should throw an error if the project does not exist in the cloud', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(projectConfig));
         mockClient.get = jest.fn(() => {
             throw new axios_1.AxiosError('Project does not exist', '404', undefined, undefined, {
                 data: { status: 404 },
@@ -140,16 +118,13 @@ describe('getProject', () => {
         });
         const { default: createGetCommand } = require('./get');
         const program = createGetCommand();
-        yield program.parseAsync(['dxp-next', 'porter', 'project', 'getProject']);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
+        yield program.parseAsync(['project', 'getProject']);
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
         expect(mockClient.get).toHaveBeenCalled();
-        expect(ErrorUtils.handleError).toHaveBeenCalled();
+        expect(PorterUtils.handleError).toHaveBeenCalled();
     }));
     it('should handle unexpected errors', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(projectConfig));
         mockClient.get = jest.fn(() => {
             throw new axios_1.AxiosError('Internal server error', '500', undefined, undefined, {
                 data: { status: 500 },
@@ -157,23 +132,17 @@ describe('getProject', () => {
         });
         const { default: createGetCommand } = require('./get');
         const program = createGetCommand();
-        yield program.parseAsync(['dxp-next', 'porter', 'project', 'getProject']);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
+        yield program.parseAsync(['project', 'getProject']);
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
         expect(mockClient.get).toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).toHaveBeenCalled();
+        expect(PorterUtils.handleCommandError).toHaveBeenCalled();
     }));
     it('should successfully retrieve the project details (without generated links) and log this to the console', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(projectConfig));
         const mockGetResponse = {
-            portid: '1234-5678-abcd-efgh',
-            stages: {},
-            status: {},
-            'artifacts-link': null,
-            'report-link': null,
-            'stage-settings': {},
+            projectid: projectName,
+            'last-artifacts-link': null,
+            'last-report-link': null,
         };
         mockResponse = {
             data: mockGetResponse,
@@ -181,28 +150,22 @@ describe('getProject', () => {
         };
         const { default: createGetCommand } = require('./get');
         const program = createGetCommand();
-        yield program.parseAsync(['dxp-next', 'porter', 'project', 'getProject']);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
+        yield program.parseAsync(['project', 'getProject']);
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
         expect(mockClient.get).toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).not.toHaveBeenCalled();
+        expect(PorterUtils.handleCommandError).not.toHaveBeenCalled();
         expect(logSpy).toHaveBeenNthCalledWith(2, `Project name: ${projectName}`);
         expect(logSpy).toHaveBeenNthCalledWith(3, 'Download last port report: Not generated');
         expect(logSpy).toHaveBeenNthCalledWith(4, 'Download last port artifacts: Not generated');
     }));
     it('should successfully retrieve the project details and log this to the console', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(projectConfig));
         const mockReportLink = 'mock-report-link';
         const mockArtifactsLink = 'mock-artifacts-link';
         const mockGetResponse = {
-            portid: '1234-5678-abcd-efgh',
-            stages: {},
-            status: {},
-            'artifacts-link': mockArtifactsLink,
-            'report-link': mockReportLink,
-            'stage-settings': {},
+            projectid: '1234-5678-abcd-efgh',
+            'last-artifacts-link': mockArtifactsLink,
+            'last-report-link': mockReportLink,
         };
         mockResponse = {
             data: mockGetResponse,
@@ -210,10 +173,11 @@ describe('getProject', () => {
         };
         const { default: createGetCommand } = require('./get');
         const program = createGetCommand();
-        yield program.parseAsync(['dxp-next', 'porter', 'project', 'getProject']);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
+        yield program.parseAsync(['project', 'getProject']);
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
         expect(mockClient.get).toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).not.toHaveBeenCalled();
+        expect(PorterUtils.handleCommandError).not.toHaveBeenCalled();
         expect(logSpy).toHaveBeenNthCalledWith(2, `Project name: ${projectName}`);
         expect(logSpy).toHaveBeenNthCalledWith(3, `Download last port report: ${mockReportLink}`);
         expect(logSpy).toHaveBeenNthCalledWith(4, `Download last port artifacts: ${mockArtifactsLink}`);

package/lib/porter/project/remove/remove.js

@@ -18,7 +18,6 @@ const inquirer_1 = require("inquirer");
 const ora_1 = __importDefault(require("ora"));
 const ApiService_1 = require("../../../ApiService");
 const utils_1 = require("../../utils");
-const constants_1 = require("../../constants");
 const createRemoveProjectCommand = () => {
     const removeCommand = new commander_1.Command('removeProject')
         .description('Deletes the project')
@@ -34,11 +33,7 @@ const createRemoveProjectCommand = () => {
         let projectName = '';
         const spinner = (0, ora_1.default)();
         try {
-            const projectConfig = yield (0, utils_1.getProjectConfig)();
-            projectName = projectConfig.projectName;
-            if (!projectName) {
-                throw new Error(constants_1.PORTER_ERRORS.projectNameMissing);
-            }
+            projectName = yield (0, utils_1.getProjectName)();
             const confirmation = yield (0, inquirer_1.prompt)({
                 message: `Are you sure you wish to remove all data about the project "${projectName}" from the cloud permanently? [y/N]`,
                 name: 'answer',

package/lib/porter/project/remove/remove.spec.js

@@ -31,27 +31,20 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
-const nock_1 = __importDefault(require("nock"));
-const remove_1 = __importDefault(require("./remove"));
 const axios_1 = require("axios");
-const ApiService = __importStar(require("../../../ApiService"));
 const Inquirer = __importStar(require("inquirer"));
-const BuildPorterUrl = __importStar(require("../../utils/BuildPorterUrl/BuildPorterUrl"));
-const ErrorUtils = __importStar(require("../../utils/ErrorUtils/ErrorUtils"));
-const GetProjectConfig = __importStar(require("../../utils/GetProjectConfig/GetProjectConfig"));
+const ApiService = __importStar(require("../../../ApiService"));
+const PorterUtils = __importStar(require("../../utils"));
 jest.mock('inquirer', () => {
     return {
         prompt: jest.fn(),
     };
 });
+jest.mock('../../utils');
 describe('removeProject', () => {
     let baseUrl;
     let projectName;
-    let projectConfig;
     let mockHandleCommandError;
     let mockHandleError;
     let mockBuildPorterUrl;
@@ -60,14 +53,8 @@ describe('removeProject', () => {
     beforeEach(() => {
         jest.resetAllMocks();
         jest.clearAllMocks();
-        nock_1.default.restore();
         projectName = 'porter-project';
         baseUrl = 'mock-url';
-        projectConfig = {
-            service: 'dxp-porter',
-            projectName,
-            version: '0.1.0',
-        };
         mockHandleCommandError = jest.fn(() => Promise.resolve());
         mockHandleError = jest.fn();
         mockBuildPorterUrl = jest.fn(() => Promise.resolve(baseUrl));
@@ -78,14 +65,18 @@ describe('removeProject', () => {
         mockClient = {
             delete: jest.fn().mockImplementationOnce(() => mockResponse),
         };
+        jest.spyOn(PorterUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
         jest
-            .spyOn(BuildPorterUrl, 'buildPorterUrl')
+            .spyOn(PorterUtils, 'getProjectName')
+            .mockImplementationOnce(() => Promise.resolve(projectName));
+        jest
+            .spyOn(PorterUtils, 'buildPorterUrl')
             .mockImplementationOnce(mockBuildPorterUrl);
         jest
-            .spyOn(ErrorUtils, 'handleCommandError')
+            .spyOn(PorterUtils, 'handleCommandError')
             .mockImplementationOnce(mockHandleCommandError);
         jest
-            .spyOn(ErrorUtils, 'handleError')
+            .spyOn(PorterUtils, 'handleError')
             .mockImplementationOnce(mockHandleError);
         jest.spyOn(process, 'exit').mockImplementation();
         jest.spyOn(ApiService, 'ApiService').mockImplementationOnce(() => {
@@ -97,57 +88,32 @@ describe('removeProject', () => {
     it('should throw an error if not logged in', () => __awaiter(void 0, void 0, void 0, function* () {
         const mockError = new Error('Error not logged in');
         jest
-            .spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn')
+            .spyOn(PorterUtils, 'throwErrorIfNotLoggedIn')
             .mockImplementationOnce(() => {
             throw mockError;
         });
-        (0, nock_1.default)('http://localhost:9999')
-            .get('/__dxp/au/porter/SquizRoot-0000/health')
-            .reply(401);
-        const program = (0, remove_1.default)();
+        const { default: createRemoveCommand } = require('./remove');
+        const program = createRemoveCommand();
         yield expect(() => program.parseAsync(['dxp-next', 'porter', 'project', 'removeProject'])).rejects.toThrow(mockError);
         const opts = program.opts();
         expect(opts).toStrictEqual({});
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).not.toHaveBeenCalled();
     }));
-    it('should throw an error if not in a project directory', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => {
-            throw new Error('You must be within a porter project directory to use this command');
+    it('should throw an error if project name could not be found', () => __awaiter(void 0, void 0, void 0, function* () {
+        jest.spyOn(PorterUtils, 'getProjectName').mockImplementationOnce(() => {
+            throw new Error();
         });
         const { default: createRemoveCommand } = require('./remove');
         const program = createRemoveCommand();
-        yield program.parseAsync([
-            'dxp-next',
-            'porter',
-            'project',
-            'removeProject',
-        ]);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).toHaveBeenCalled();
-    }));
-    it('should throw an error if project name is missing in the config.json file', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(Object.assign(Object.assign({}, projectConfig), { projectName: '' })));
-        const { default: createRemoveCommand } = require('./remove');
-        const program = createRemoveCommand();
-        yield program.parseAsync([
-            'dxp-next',
-            'porter',
-            'project',
-            'removeProject',
-        ]);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).toHaveBeenCalled();
+        yield program.parseAsync(['dxp-next', 'porter', 'project', 'getProject']);
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.getProjectName).toHaveBeenCalled();
+        expect(mockClient.delete).not.toHaveBeenCalled();
+        expect(PorterUtils.handleCommandError).toHaveBeenCalled();
     }));
     it('should end the program if the prompt was input with anything other than "y"', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(projectConfig));
         jest.spyOn(Inquirer, 'prompt').mockImplementationOnce(() => Promise.resolve({
             answer: 'not-y',
         }));
@@ -159,15 +125,13 @@ describe('removeProject', () => {
             'project',
             'removeProject',
         ]);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.getProjectName).toHaveBeenCalled();
         expect(mockClient.delete).not.toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).not.toHaveBeenCalled();
+        expect(PorterUtils.handleCommandError).not.toHaveBeenCalled();
     }));
     it('should throw an error if the project does not exist in the cloud', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(projectConfig));
         jest.spyOn(Inquirer, 'prompt').mockImplementationOnce(() => Promise.resolve({
             answer: 'y',
         }));
@@ -184,15 +148,16 @@ describe('removeProject', () => {
             'project',
             'removeProject',
         ]);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.getProjectName).toHaveBeenCalled();
         expect(mockClient.delete).toHaveBeenCalled();
-        expect(ErrorUtils.handleError).toHaveBeenCalled();
+        expect(PorterUtils.handleError).toHaveBeenCalled();
     }));
     it('should successfully delete the project from the cloud', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
         jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(projectConfig));
+            .spyOn(PorterUtils, 'getProjectName')
+            .mockImplementationOnce(() => Promise.resolve(projectName));
         jest.spyOn(Inquirer, 'prompt').mockImplementationOnce(() => Promise.resolve({
             answer: 'y',
         }));
@@ -204,15 +169,13 @@ describe('removeProject', () => {
             'project',
             'removeProject',
         ]);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.getProjectName).toHaveBeenCalled();
         expect(mockClient.delete).toHaveBeenCalled();
-        expect(ErrorUtils.handleError).not.toHaveBeenCalled();
+        expect(PorterUtils.handleError).not.toHaveBeenCalled();
     }));
     it('should handle unexpected errors', () => __awaiter(void 0, void 0, void 0, function* () {
-        jest.spyOn(ErrorUtils, 'throwErrorIfNotLoggedIn').mockImplementation();
-        jest
-            .spyOn(GetProjectConfig, 'getProjectConfig')
-            .mockImplementationOnce(() => Promise.resolve(projectConfig));
         jest.spyOn(Inquirer, 'prompt').mockImplementationOnce(() => Promise.resolve({
             answer: 'y',
         }));
@@ -229,8 +192,10 @@ describe('removeProject', () => {
             'project',
             'removeProject',
         ]);
-        expect(BuildPorterUrl.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.throwErrorIfNotLoggedIn).toHaveBeenCalled();
+        expect(PorterUtils.buildPorterUrl).toHaveBeenCalled();
+        expect(PorterUtils.getProjectName).toHaveBeenCalled();
         expect(mockClient.delete).toHaveBeenCalled();
-        expect(ErrorUtils.handleCommandError).toHaveBeenCalled();
+        expect(PorterUtils.handleCommandError).toHaveBeenCalled();
     }));
 });

package/lib/porter/utils/ErrorUtils/ErrorUtils.js

@@ -41,7 +41,10 @@ function handleCommandError(command, error) {
     if (error.response.data) {
         let errorMessage;
         if (error.response.data['invalid-params']) {
-            errorMessage = JSON.stringify(error.response.data['invalid-params']);
+            errorMessage = error.response.data['invalid-params']
+                .map((item) => `${item.name} - ${item.reason}`)
+                .join('\n');
+            return command.error(chalk_1.default.red(errorMessage));
         }
         else {
             errorMessage = error.response.data.title;

package/lib/porter/utils/ErrorUtils/ErrorUtils.spec.js

@@ -97,7 +97,7 @@ describe('handleCommandError', () => {
         // read the mock call strings directly otherwise colours cause test failures with .toEqual()
         expect(stderrSpy.mock.calls[0][0]).toContain('You must be logged in as a DXP super admin or Squiz support staff to use this service');
     }));
-    it('correctly displays the console logs for axios errors from the porter service', () => __awaiter(void 0, void 0, void 0, function* () {
+    it('correctly displays the console logs for invalid-params errors from the porter service', () => __awaiter(void 0, void 0, void 0, function* () {
         const command = new commander_1.Command();
         const errorStatus = 400;
         const errorTitle = 'Bad request';
@@ -115,7 +115,22 @@ describe('handleCommandError', () => {
         (0, ErrorUtils_1.handleCommandError)(command, new axios_1.AxiosError(errorTitle, `${errorStatus}`, undefined, undefined, errorResponse));
         expect(stderrSpy).toHaveBeenCalledTimes(1);
         // read the mock call strings directly otherwise colours cause test failures with .toEqual()
-        expect(stderrSpy.mock.calls[0][0]).toContain(JSON.stringify(errorResponse.data['invalid-params']));
+        expect(stderrSpy.mock.calls[0][0]).toContain(`${errorResponse.data['invalid-params'][0].name} - ${errorResponse.data['invalid-params'][0].reason}`);
+    }));
+    it('correctly displays the console logs for errors (using the title) from the porter service', () => __awaiter(void 0, void 0, void 0, function* () {
+        const errorStatus = 400;
+        const errorTitle = 'Bad request';
+        const command = new commander_1.Command();
+        const errorResponse = {
+            data: {
+                title: errorTitle,
+                status: errorStatus,
+            },
+        };
+        (0, ErrorUtils_1.handleCommandError)(command, new axios_1.AxiosError(errorTitle, `${errorStatus}`, undefined, undefined, errorResponse));
+        expect(stderrSpy).toHaveBeenCalledTimes(1);
+        // read the mock call strings directly otherwise colours cause test failures with .toEqual()
+        expect(stderrSpy.mock.calls[0][0]).toContain(errorTitle);
     }));
 });
 describe('handleError', () => {

package/lib/porter/utils/GetProjectConfig/GetProjectConfig.js

@@ -16,6 +16,7 @@ exports.getProjectConfig = void 0;
 const path_1 = __importDefault(require("path"));
 const promises_1 = __importDefault(require("fs/promises"));
 const CoreUtils_1 = require("../CoreUtils/CoreUtils");
+const constants_1 = require("../../constants");
 function getProjectConfig() {
     return __awaiter(this, void 0, void 0, function* () {
         try {
@@ -25,7 +26,7 @@ function getProjectConfig() {
         }
         catch (err) {
             (0, CoreUtils_1.logDebug)(err);
-            throw new Error('You must be within a porter project directory to use this command');
+            throw new Error(constants_1.PORTER_ERRORS.projectDirectoryInvalid);
         }
     });
 }

package/lib/porter/utils/GetProjectConfig/GetProjectConfig.spec.js

@@ -14,24 +14,25 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 const GetProjectConfig_1 = require("./GetProjectConfig");
 const promises_1 = __importDefault(require("fs/promises"));
+const constants_1 = require("../../constants");
 jest.mock('fs/promises', () => {
     return {
         readFile: jest.fn(),
     };
 });
 describe('getProjectConfig', () => {
-    it('should throw an error if cannot config file does not exist', () => __awaiter(void 0, void 0, void 0, function* () {
+    it('should throw an error if config file does not exist', () => __awaiter(void 0, void 0, void 0, function* () {
         jest.spyOn(promises_1.default, 'readFile').mockImplementationOnce(() => {
             throw new Error('ENOENT');
         });
-        yield expect((0, GetProjectConfig_1.getProjectConfig)()).rejects.toThrowError('You must be within a porter project directory to use this command');
+        yield expect((0, GetProjectConfig_1.getProjectConfig)()).rejects.toThrowError(constants_1.PORTER_ERRORS.projectDirectoryInvalid);
         expect(promises_1.default.readFile).toHaveBeenCalled();
     }));
     it('should throw an error if cannot parse config file', () => __awaiter(void 0, void 0, void 0, function* () {
         jest
             .spyOn(promises_1.default, 'readFile')
             .mockImplementationOnce(() => Promise.resolve('invalid-json'));
-        yield expect((0, GetProjectConfig_1.getProjectConfig)()).rejects.toThrowError('You must be within a porter project directory to use this command');
+        yield expect((0, GetProjectConfig_1.getProjectConfig)()).rejects.toThrowError(constants_1.PORTER_ERRORS.projectDirectoryInvalid);
         expect(promises_1.default.readFile).toHaveBeenCalled();
     }));
     it('should return the project config json', () => __awaiter(void 0, void 0, void 0, function* () {

package/lib/porter/utils/GetProjectName/GetProjectName.d.ts

@@ -0,0 +1,4 @@
+/**
+ * The project name set in the project config file.
+ */
+export declare function getProjectName(): Promise<string>;

package/lib/porter/utils/GetProjectName/GetProjectName.js

@@ -0,0 +1,30 @@
+"use strict";
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.getProjectName = void 0;
+const GetProjectConfig_1 = require("../GetProjectConfig/GetProjectConfig");
+const constants_1 = require("../../constants");
+const CoreUtils_1 = require("../CoreUtils/CoreUtils");
+/**
+ * The project name set in the project config file.
+ */
+function getProjectName() {
+    return __awaiter(this, void 0, void 0, function* () {
+        const config = yield (0, GetProjectConfig_1.getProjectConfig)();
+        const { projectName } = config;
+        if (!projectName) {
+            (0, CoreUtils_1.logDebug)(`Missing project name. Config: ${JSON.stringify(config)}`);
+            throw new Error(constants_1.PORTER_ERRORS.projectNameMissing);
+        }
+        return projectName;
+    });
+}
+exports.getProjectName = getProjectName;

package/lib/porter/utils/GetProjectName/GetProjectName.spec.d.ts

@@ -0,0 +1 @@
+export {};