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

package/lib/page/layouts/deploy/deploy.js

@@ -35,7 +35,7 @@ const createDeployCommand = () => {
         .addOption(new commander_1.Option('-t, --tenant <string>', 'Tenant ID to deploy to. If not provided will use configured tenant from login').env('SQUIZ_DXP_TENANT_ID'))
         .addOption(new commander_1.Option('--dry-run', 'Run all pre-deployment processes without deploying').default(false))
         .action((options) => __awaiter(void 0, void 0, void 0, function* () {
-        var _a, _b, _c, _d;
+        var _a, _b, _c, _d, _e;
         if (options.contentServiceUrl) {
             console.log(`NOTICE: CONTENT_SERVICE_URL is set and will deploy to ${options.contentServiceUrl}`);
         }
@@ -59,6 +59,9 @@ const createDeployCommand = () => {
                 const response = yield uploadLayout(apiService.client, layout, contentServiceUrl, options.dryRun);
                 if (!options.dryRun) {
                     exports.logger.info(`Layout "${layout.name}" version ${response.data.version} deployed successfully.`);
+                    // Log the deployed layout URL (clickable)
+                    const layoutUrl = `${baseUrl}/organization/${(_e = options.tenant) !== null && _e !== void 0 ? _e : maybeConfig === null || maybeConfig === void 0 ? void 0 : maybeConfig.tenant}/component-service/all-layouts/${encodeURIComponent(layout.name)}`;
+                    exports.logger.info(`Deployed layout URL: \u001b]8;;${layoutUrl}\u001b\\${layoutUrl}\u001b]8;;\u001b\\`);
                 }
                 else {
                     exports.logger.info(`Layout "${layout.name}" dry run successful.`);
@@ -87,6 +90,7 @@ const createDeployCommand = () => {
 };
 exports.default = createDeployCommand;
 function uploadLayout(client, layout, contentServiceUrl, dryRun) {
+    var _a, _b;
     return __awaiter(this, void 0, void 0, function* () {
         try {
             const queryParam = dryRun ? '?_dryRun=true' : '';
@@ -96,7 +100,11 @@ function uploadLayout(client, layout, contentServiceUrl, dryRun) {
             if (response.status === 200) {
                 return response;
             }
-            throw new Error(response.data.message);
+            const error = new Error(response.data.message);
+            if ((_b = (_a = response.data.details) === null || _a === void 0 ? void 0 : _a.input) === null || _b === void 0 ? void 0 : _b.message) {
+                error.message += `: ${response.data.details.input.message}`;
+            }
+            throw error;
         }
         catch (error) {
             throw error;

package/lib/page/layouts/deploy/deploy.spec.js

@@ -93,13 +93,13 @@ describe('deployCommand', () => {
         const file = './src/__tests__/layouts/page-layout.yaml';
         const dxpBaseUrl = 'http://dxp-base-url.com';
         const mockLayout = {
-            name: 'Test Layout',
+            name: 'test-layout',
             zones: {
                 content: { displayName: 'Content', description: 'Main content' },
             },
             template: '<div>{{zones.content}}</div>',
         };
-        const mockResponse = { name: 'Test Layout', version: '12345' };
+        const mockResponse = { name: 'test-layout', version: '12345' };
         definitions_1.loadLayoutDefinition.mockResolvedValue(mockLayout);
         (0, nock_1.default)(dxpBaseUrl + '/__dxp/service/components-content')
             .post('/page-layout', mockLayout)
@@ -107,20 +107,21 @@ describe('deployCommand', () => {
         const program = (0, deploy_1.default)();
         yield program.parseAsync(createMockArgs({ config: file, dxpBaseUrl }));
         expect(mockLoggerInfoFn).toHaveBeenNthCalledWith(1, `Loading layout data from the file ${file}`);
-        expect(mockLoggerInfoFn).toHaveBeenNthCalledWith(2, 'Layout "Test Layout" version 12345 deployed successfully.');
+        expect(mockLoggerInfoFn).toHaveBeenNthCalledWith(2, 'Layout "test-layout" version 12345 deployed successfully.');
+        expect(mockLoggerInfoFn).toHaveBeenNthCalledWith(3, 'Deployed layout URL: \u001b]8;;http://dxp-base-url.com/organization/myTenant/component-service/all-layouts/test-layout\u001b\\http://dxp-base-url.com/organization/myTenant/component-service/all-layouts/test-layout\u001b]8;;\u001b\\');
     }));
     it('deploys a layout with dry-run option', () => __awaiter(void 0, void 0, void 0, function* () {
         const file = './src/__tests__/layout.yaml';
         const dxpBaseUrl = 'http://dxp-base-url.com';
         const dryRun = true;
         const mockLayout = {
-            name: 'Test Layout',
+            name: 'test-layout',
             zones: {
                 content: { displayName: 'Content', description: 'Main content' },
             },
             template: '<div>{{zones.content}}</div>',
         };
-        const mockResponse = { name: 'Test Layout', version: '12345' };
+        const mockResponse = { name: 'test-layout', version: '12345' };
         definitions_1.loadLayoutDefinition.mockResolvedValue(mockLayout);
         (0, nock_1.default)(dxpBaseUrl + '/__dxp/service/components-content')
             .post('/page-layout', mockLayout)
@@ -129,7 +130,7 @@ describe('deployCommand', () => {
         const program = (0, deploy_1.default)();
         yield program.parseAsync(createMockArgs({ config: file, dxpBaseUrl, dryRun }));
         expect(mockLoggerInfoFn).toHaveBeenNthCalledWith(1, `Loading layout data from the file ${file}`);
-        expect(mockLoggerInfoFn).toHaveBeenNthCalledWith(2, 'Layout "Test Layout" dry run successful.');
+        expect(mockLoggerInfoFn).toHaveBeenNthCalledWith(2, 'Layout "test-layout" dry run successful.');
     }));
     it('handles InvalidLoginSessionError', () => __awaiter(void 0, void 0, void 0, function* () {
         const dxpBaseUrl = 'http://dxp-base-url.com';
@@ -171,13 +172,30 @@ describe('deployCommand', () => {
         yield program.parseAsync(createMockArgs({ contentServiceUrl }));
         expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('An unknown error occurred'));
     }));
+    it('should log additional details when layout validation fails', () => __awaiter(void 0, void 0, void 0, function* () {
+        const dxpBaseUrl = 'http://dxp-base-url.com';
+        (0, nock_1.default)(dxpBaseUrl + '/__dxp/service/components-content')
+            .post('/page-layout')
+            .reply(400, {
+            message: 'Layout validation failed',
+            details: {
+                input: {
+                    message: 'ERROR: Validation failed: "version" is an excess property and therefore is not allowed',
+                },
+            },
+        });
+        const program = (0, deploy_1.default)();
+        errorSpy = jest.spyOn(program, 'error').mockImplementation();
+        yield program.parseAsync(createMockArgs({ dxpBaseUrl }));
+        expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('Layout validation failed: ERROR: Validation failed: "version" is an excess property and therefore is not allowed'));
+    }));
     describe('zone consistency validation', () => {
         it('should handle zone consistency validation errors where zones are used but not defined in the layout', () => __awaiter(void 0, void 0, void 0, function* () {
             const file = './src/__tests__/layout.yaml';
             const dxpBaseUrl = 'http://dxp-base-url.com';
             // Mock layout with zones that don't match the template
             const mockLayout = {
-                name: 'Test Layout',
+                name: 'test-layout',
                 zones: {
                     col1: { displayName: 'Column 1', description: 'The first column' },
                 },
@@ -193,7 +211,7 @@ describe('deployCommand', () => {
             const file = './src/__tests__/layout.yaml';
             const dxpBaseUrl = 'http://dxp-base-url.com';
             const mockLayout = {
-                name: 'Test Layout',
+                name: 'test-layout',
                 zones: {
                     col1: { displayName: 'Column 1', description: 'The first column' },
                     col2: { displayName: 'Column 2', description: 'The second column' }, // col2 is defined but not used in the template

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@squiz/dxp-cli-next",
-  "version": "5.30.0-develop.1",
+  "version": "5.30.0-develop.2",
   "repository": {
     "url": "https://gitlab.squiz.net/dxp/dxp-cli-next"
   },