@sap-ux/deploy-tooling 0.9.18 → 0.9.20

package/dist/base/deploy.js

@@ -207,8 +207,9 @@ function tryDeploy(provider, config, logger, archive) {
                 // Reset as we dont want other flows kicking it off again!
                 config.createTransport = false;
             }
+            let validateOutput;
             if (config.test === true) {
-                const validateOutput = yield (0, validate_1.validateBeforeDeploy)({
+                validateOutput = yield (0, validate_1.validateBeforeDeploy)({
                     appName: config.app.name,
                     description: (_a = config.app.description) !== null && _a !== void 0 ? _a : '',
                     package: (_b = config.app.package) !== null && _b !== void 0 ? _b : '',
@@ -216,7 +217,9 @@ function tryDeploy(provider, config, logger, archive) {
                     client: (_d = config.target.client) !== null && _d !== void 0 ? _d : '',
                     url: (_e = config.target.url) !== null && _e !== void 0 ? _e : ''
                 }, provider, logger);
-                logger.info((0, validate_1.formatSummary)(validateOutput.summary));
+                if (!validateOutput.result) {
+                    logger.info(`Results of validating the deployment configuration settings:${(0, validate_1.formatSummary)(validateOutput.summary)}`);
+                }
             }
             const service = getUi5AbapRepositoryService(provider, config, logger);
             yield service.deploy({ archive, bsp: config.app, testMode: config.test, safeMode: config.safe });

package/dist/base/validate.js

@@ -76,7 +76,7 @@ function formatSummary(summary) {
                 statusSymbol = (0, chalk_1.yellow)('?');
                 break;
         }
-        return `${statusSymbol} ${next.message}`;
+        return `${'    '}${statusSymbol} ${next.message}`;
     })
         .reduce((aggregated, current) => {
         return `${aggregated}${os_1.EOL}${current}`;
@@ -189,6 +189,11 @@ function validatePackageWithAdt(input, output, provider, logger) {
         if (output.result === false) {
             return;
         }
+        // ADT expects input package
+        let inputPackage = input.package;
+        if (inputPackage.toUpperCase() === '$TMP') {
+            inputPackage = '$TMP';
+        }
         try {
             const adtService = yield provider.getAdtService(axios_extension_1.ListPackageService);
             if (!adtService) {
@@ -199,8 +204,8 @@ function validatePackageWithAdt(input, output, provider, logger) {
                 output.result = false;
                 return;
             }
-            const packages = yield adtService.listPackages({ phrase: input.package });
-            const isValidPackage = packages.findIndex((packageName) => packageName === input.package) >= 0;
+            const packages = yield adtService.listPackages({ phrase: inputPackage });
+            const isValidPackage = packages.findIndex((packageName) => packageName === inputPackage) >= 0;
             if (isValidPackage) {
                 output.summary.push({
                     message: exports.summaryMessage.packageCheckPass,

package/package.json

@@ -9,7 +9,7 @@
   "bugs": {
     "url": "https://github.com/SAP/open-ux-tools/issues?q=is%3Aopen+is%3Aissue+label%3Abug+label%3Adeploy-tooling"
   },
-  "version": "0.9.18",
+  "version": "0.9.20",
   "license": "Apache-2.0",
   "author": "@SAP/ux-tools-team",
   "main": "dist/index.js",
@@ -35,14 +35,14 @@
     "@sap-ux/axios-extension": "1.5.1",
     "@sap-ux/btp-utils": "0.11.9",
     "@sap-ux/logger": "0.3.9",
-    "@sap-ux/system-access": "0.2.10",
+    "@sap-ux/system-access": "0.2.11",
     "@sap-ux/ui5-config": "0.19.5",
-    "@sap-ux/project-input-validator": "0.1.0"
+    "@sap-ux/project-input-validator": "0.1.2"
   },
   "devDependencies": {
     "@types/prompts": "2.4.4",
     "@types/yazl": "2.4.2",
-    "@sap-ux/store": "0.3.15"
+    "@sap-ux/store": "0.3.16"
   },
   "ui5": {
     "dependencies": []