@sasjs/cli 6.5.0 → 6.7.0

package/build/commands/compile/internal/spec/compileTestFile.spec.js

@@ -375,7 +375,7 @@ describe('compileTestFile', function () {
             });
         }); });
         it('should not log 0/0 coverage', function () { return __awaiter(void 0, void 0, void 0, function () {
-            var totalCalls;
+            var loggedLines;
             return __generator(this, function (_a) {
                 switch (_a.label) {
                     case 0:
@@ -383,11 +383,14 @@ describe('compileTestFile', function () {
                         return [4 /*yield*/, (0, compile_1.compile)(target)];
                     case 1:
                         _a.sent();
-                        totalCalls = 16;
-                        expect(process.logger.info).toHaveBeenCalledTimes(totalCalls);
-                        expect(process.logger.info).toHaveBeenNthCalledWith(13, "Test coverage:");
-                        expect(process.logger.info).toHaveBeenNthCalledWith(14, "Services coverage: 0/4 (".concat(chalk_1.default.greenBright('0%'), ")"));
-                        expect(process.logger.info).toHaveBeenNthCalledWith(totalCalls - 1, "Overall coverage: 0/4 (".concat(chalk_1.default.greenBright('0%'), ")"));
+                        loggedLines = process.logger.info.mock.calls.map(function (call) { return String(call[0]); });
+                        // The intent is the absence of a 0/0 line, not a particular number of
+                        // log calls - asserting a total turns every future log line into a
+                        // failure of this test.
+                        expect(loggedLines.some(function (line) { return line.includes('0/0'); })).toBeFalse();
+                        expect(loggedLines).toContain('Test coverage:');
+                        expect(loggedLines).toContain("Services coverage: 0/4 (".concat(chalk_1.default.greenBright('0%'), ")"));
+                        expect(loggedLines).toContain("Overall coverage: 0/4 (".concat(chalk_1.default.greenBright('0%'), ")"));
                         return [2 /*return*/];
                 }
             });

package/build/commands/compile/spec/checkCompileStatus.spec.d.ts

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

package/build/commands/compile/spec/checkCompileStatus.spec.js

@@ -0,0 +1,221 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+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());
+    });
+};
+var __generator = (this && this.__generator) || function (thisArg, body) {
+    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g = Object.create((typeof Iterator === "function" ? Iterator : Object).prototype);
+    return g.next = verb(0), g["throw"] = verb(1), g["return"] = verb(2), typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
+    function verb(n) { return function (v) { return step([n, v]); }; }
+    function step(op) {
+        if (f) throw new TypeError("Generator is already executing.");
+        while (g && (g = 0, op[0] && (_ = 0)), _) try {
+            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
+            if (y = 0, t) op = [op[0] & 2, t.value];
+            switch (op[0]) {
+                case 0: case 1: t = op; break;
+                case 4: _.label++; return { value: op[1], done: false };
+                case 5: _.label++; y = op[1]; op = [0]; continue;
+                case 7: op = _.ops.pop(); _.trys.pop(); continue;
+                default:
+                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
+                    if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
+                    if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
+                    if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
+                    if (t[2]) _.ops.pop();
+                    _.trys.pop(); continue;
+            }
+            op = body.call(thisArg, _);
+        } catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
+        if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
+    }
+};
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+var path_1 = __importDefault(require("path"));
+var getAllFoldersModule = __importStar(require("../internal/getAllFolders"));
+var compareFoldersModule = __importStar(require("../internal/compareFolders"));
+var compileManifestModule = __importStar(require("../internal/compileManifest"));
+var utils_1 = require("../../../utils");
+var checkCompileStatus_1 = require("../internal/checkCompileStatus");
+jest.mock('../internal/getAllFolders');
+jest.mock('../internal/compareFolders');
+jest.mock('../internal/compileManifest');
+describe('checkCompileStatus', function () {
+    var target = { name: 'test' };
+    var folderMatch = { equal: true, reason: '' };
+    beforeAll(function () { return __awaiter(void 0, void 0, void 0, function () {
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0: return [4 /*yield*/, (0, utils_1.setConstants)(false)];
+                case 1:
+                    _a.sent();
+                    return [2 /*return*/];
+            }
+        });
+    }); });
+    beforeEach(function () {
+        jest.resetAllMocks();
+        jest
+            .spyOn(getAllFoldersModule, 'getAllFolders')
+            .mockImplementation(function () { return Promise.resolve(['/app/sasjs/services']); });
+        jest
+            .spyOn(compareFoldersModule, 'compareFolders')
+            .mockImplementation(function () { return Promise.resolve(folderMatch); });
+        jest
+            .spyOn(compileManifestModule, 'getManifestFolders')
+            .mockImplementation(function () { return Promise.resolve([]); });
+        jest
+            .spyOn(compileManifestModule, 'getManifestStamps')
+            .mockImplementation(function () { return Promise.resolve([]); });
+        jest
+            .spyOn(compileManifestModule, 'getChangedFiles')
+            .mockImplementation(function () { return Promise.resolve([]); });
+    });
+    it('reports a build folder that does not exist', function () { return __awaiter(void 0, void 0, void 0, function () {
+        var previous, result;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    previous = process.sasjsConstants.buildDestinationFolder;
+                    process.sasjsConstants.buildDestinationFolder = path_1.default.join(__dirname, 'cli-tests-no-such-build-folder');
+                    return [4 /*yield*/, (0, checkCompileStatus_1.checkCompileStatus)(target)];
+                case 1:
+                    result = _a.sent();
+                    expect(result.compiled).toBe(false);
+                    expect(result.message).toContain("doesn't exist");
+                    process.sasjsConstants.buildDestinationFolder = previous;
+                    return [2 /*return*/];
+            }
+        });
+    }); });
+    it('says the project is compiled when the folders match and nothing changed', function () { return __awaiter(void 0, void 0, void 0, function () {
+        var result;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0: return [4 /*yield*/, (0, checkCompileStatus_1.checkCompileStatus)(target)];
+                case 1:
+                    result = _a.sent();
+                    expect(result.compiled).toBe(true);
+                    return [2 /*return*/];
+            }
+        });
+    }); });
+    it('names only the services side when the services are missing', function () { return __awaiter(void 0, void 0, void 0, function () {
+        var result;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    jest
+                        .spyOn(getAllFoldersModule, 'getAllFolders')
+                        .mockImplementation(function (_target, type) {
+                        return Promise.resolve([
+                            type === 'job' ? '/app/sasjs/jobs' : '/app/sasjs/services'
+                        ]);
+                    });
+                    jest
+                        .spyOn(compareFoldersModule, 'compareFolders')
+                        .mockImplementation(function (source) {
+                        return Promise.resolve(source.includes('services')
+                            ? { equal: false, reason: 'Files missing from build: getdata.sas' }
+                            : folderMatch);
+                    });
+                    return [4 /*yield*/, (0, checkCompileStatus_1.checkCompileStatus)(target)];
+                case 1:
+                    result = _a.sent();
+                    expect(result.compiled).toBe(false);
+                    expect(result.message).toContain('Missing Services:');
+                    expect(result.message).not.toContain('Missing jobs:');
+                    return [2 /*return*/];
+            }
+        });
+    }); });
+    it('names only the jobs side when the jobs are missing', function () { return __awaiter(void 0, void 0, void 0, function () {
+        var result;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    jest
+                        .spyOn(getAllFoldersModule, 'getAllFolders')
+                        .mockImplementation(function (_target, type) {
+                        return Promise.resolve([
+                            type === 'job' ? '/app/sasjs/jobs' : '/app/sasjs/services'
+                        ]);
+                    });
+                    jest
+                        .spyOn(compareFoldersModule, 'compareFolders')
+                        .mockImplementation(function (source) {
+                        return Promise.resolve(source.includes('jobs')
+                            ? { equal: false, reason: 'Subfolders missing from build: nightly' }
+                            : folderMatch);
+                    });
+                    return [4 /*yield*/, (0, checkCompileStatus_1.checkCompileStatus)(target)];
+                case 1:
+                    result = _a.sent();
+                    expect(result.compiled).toBe(false);
+                    expect(result.message).toContain('Missing jobs:');
+                    expect(result.message).not.toContain('Missing Services:');
+                    return [2 /*return*/];
+            }
+        });
+    }); });
+    it('reports the files that changed since the last compile', function () { return __awaiter(void 0, void 0, void 0, function () {
+        var result;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    jest
+                        .spyOn(compileManifestModule, 'getChangedFiles')
+                        .mockImplementation(function () {
+                        return Promise.resolve(['/app/sasjs/services/getdata.sas']);
+                    });
+                    return [4 /*yield*/, (0, checkCompileStatus_1.checkCompileStatus)(target)];
+                case 1:
+                    result = _a.sent();
+                    expect(result.compiled).toBe(false);
+                    expect(result.message).toContain('Changed since the last compile: /app/sasjs/services/getdata.sas');
+                    return [2 /*return*/];
+            }
+        });
+    }); });
+});

package/build/commands/compile/spec/compileCommand.spec.js

@@ -185,10 +185,17 @@ describe('CompileCommand', function () {
                 case 0:
                     args = __spreadArray(__spreadArray([], __read(defaultArgs), false), ['compile', '-t', 'test'], false);
                     command = new compileCommand_1.CompileCommand(args);
-                    return [4 /*yield*/, command.execute()];
+                    return [4 /*yield*/, command.execute()
+                        // The command no longer forces a recompile: checkCompileStatus compares a
+                        // content manifest, so an unchanged project is skipped and a changed one is
+                        // recompiled without the caller passing anything.
+                    ];
                 case 1:
                     _a.sent();
-                    expect(compileModule.compile).toHaveBeenCalledWith(target, true);
+                    // The command no longer forces a recompile: checkCompileStatus compares a
+                    // content manifest, so an unchanged project is skipped and a changed one is
+                    // recompiled without the caller passing anything.
+                    expect(compileModule.compile).toHaveBeenCalledWith(target);
                     return [2 /*return*/];
             }
         });

package/build/commands/compile/spec/compileManifest.spec.d.ts

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