TA-2807: Batch export/import fixes and release

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@celonis/content-cli",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "CLI Tool to help manage content in Celonis EMS",
   "main": "content-cli.js",
   "bin": {

src/content-cli-config.ts

@@ -25,7 +25,9 @@ export class Config {
 
     public static listVariables(program: CommanderStatic): CommanderStatic {
         program
-            .command("listVariables")
+            .command("variables")
+            .description("Commands related to variable configs")
+            .command("list")
             .description("Command to list versioned variables of packages")
             .option("-p, --profile <profile>", "Profile which you want to use to list packages")
             .option("--json", "Return response as json type", "")
@@ -43,7 +45,7 @@ export class Config {
         program
             .command("export")
             .description("Command to export package configs")
-            .option("-p, --profile <profile>", "Profile which you want to use to list packages")
+            .option("-p, --profile <profile>", "Profile which you want to use to export packages")
             .requiredOption("--packageKeys <packageKeys...>", "Keys of packages to export")
             .option("--withDependencies", "Include variables and dependencies", "")
             .action(async cmd => {
@@ -70,10 +72,6 @@ export class Config {
     }
 }
 
-process.on("unhandledRejection", (e, promise) => {
-    logger.error(e.toString());
-});
-
 const loadAllCommands = () => {
     Config.list(commander);
     Config.listVariables(commander);
@@ -83,7 +81,7 @@ const loadAllCommands = () => {
 };
 
 ContextInitializer.initContext()
-    .then(loadAllCommands)
+    .then(loadAllCommands, loadAllCommands)
     .catch(e => {
         logger.error(e);
     });

src/content-cli-list.ts

@@ -100,10 +100,6 @@ export class List {
     }
 }
 
-process.on("unhandledRejection", (e, promise) => {
-    logger.error(e.toString());
-});
-
 const loadAllCommands = () => {
     List.packages(commander);
     List.spaces(commander);
@@ -115,7 +111,7 @@ const loadAllCommands = () => {
 };
 
 ContextInitializer.initContext()
-    .then(loadAllCommands)
+    .then(loadAllCommands, loadAllCommands)
     .catch(e => {
         logger.error(e);
     });

src/content-cli-set.ts

@@ -24,11 +24,13 @@ class Set {
     }
 }
 
+const loadAllCommands = () => {
+    Set.connection(commander);
+    commander.parse(process.argv);
+};
+
 ContextInitializer.initContext()
-    .then(() => {
-        Set.connection(commander);
-        commander.parse(process.argv);
-    })
+    .then(loadAllCommands, loadAllCommands)
     .catch(e => {
         logger.error(e);
     });

src/content-cli.ts

@@ -30,7 +30,7 @@ program.command("get", "Commands to get configuration properties.");
 
 program.command("set", "Commands to set configuration properties.");
 
-program.command("config", "Commands related to config management")
+program.command("config", "Commands related to config management.")
 
 program.version(VersionUtils.getCurrentCliVersion());
 program.parse(process.argv);

src/services/package-manager/variable-service.ts

@@ -80,9 +80,9 @@ class VariableService {
     private async buildKeyVersionPairs(keysByVersion: string[], keysByVersionFile: string): Promise<PackageKeyAndVersionPair[]> {
         let variablesExportRequest: PackageKeyAndVersionPair[] = [];
 
-        if (keysByVersion.length) {
+        if (keysByVersion.length !== 0) {
             variablesExportRequest = this.buildKeyAndVersionPairsFromArrayInput(keysByVersion);
-        } else if (!keysByVersion.length && keysByVersionFile.length) {
+        } else if (keysByVersion.length === 0 && keysByVersionFile !== "") {
             variablesExportRequest = await fileService.readFileToJson(keysByVersionFile);
         } else {
             throw new FatalError("Please provide keysByVersion mappings or file path!");

tests/config/config-list-variables.spec.ts

@@ -163,7 +163,7 @@ describe("Config listVariables", () => {
         expect(variableExportRequest).toEqual(packageKeyAndVersionPairs);
     })
 
-    it("Should throw error if no mapping or file path is provided", async () => {
+    it("Should throw error if no mapping and no file path is provided", async () => {
         try {
             await new ConfigCommand().listVariables(true, [], "");
         } catch (e) {

tests/jest.setup.ts

@@ -14,8 +14,7 @@ const mockWriteSync = jest.fn();
 (fs.writeSync as jest.Mock).mockImplementation(mockWriteSync);
 
 afterEach(() => {
-    mockWriteFileSync.mockClear();
-    mockWriteSync.mockClear();
+    jest.clearAllMocks();
 })
 
 beforeAll(() => {
@@ -25,6 +24,7 @@ beforeAll(() => {
 let testTransport;
 
 beforeEach(() => {
+    jest.clearAllMocks();
     testTransport = new TestTransport({})
     logger.add(testTransport);
 })