Merge pull request #236 from LambdaTest/dev

3.0.12

Author: asad9711

commands/init.js

@@ -5,11 +5,11 @@ const { config } = require('yargs');
 const constants = require('./utils/constants.js')
 
 function create_file(file_path, content) {
-    fs.writeFile(file_path, content, function (err) {
+    fs.writeFileSync(file_path, content, function (err) {
         if (err) throw err;
         console.log('Saved at ',file_path);
     });
-}
+};
 
 function create_ltconfig_file(args) {
     let cv=9.6
@@ -47,14 +47,6 @@ function create_ltconfig_file(args) {
     }
 };
 
-function create_base_reporter_config_file(args) {
-    let config = require('./utils/default_reporter_config.js')
-    let content = JSON.stringify(config, null, 3);
-    if (args._.length == 1) {
-        create_file(constants.LT_BASE_REPORTER_CONFIG_FILE_NAME, content)
-    }
-};
-
 function create_custom_support_file(args){
     const pathToFile = path.join(__dirname, "default_custom_support_file.js");
     const pathToNewDestination = constants.LT_BASE_CUSTOM_SUPPORT_FILE_NAME;
@@ -68,7 +60,16 @@ function create_custom_support_file(args){
       });
 }
 
-module.exports = function (args) {
+function create_base_reporter_config_file(args) {
+    let config = require('./utils/default_reporter_config.js')
+    let content = JSON.stringify(config, null, 3);
+
+    if (args._.length == 1) {
+        create_file(constants.LT_BASE_REPORTER_CONFIG_FILE_NAME, content)
+    }
+};
+
+function init_implementation(args){
     create_ltconfig_file(args);
     create_base_reporter_config_file(args);
     if ("cv" in args){
@@ -77,3 +78,7 @@ module.exports = function (args) {
         }
     }
 };
+module.exports = {
+    create_base_reporter_config_file:create_base_reporter_config_file,
+    init_implementation:init_implementation,
+};

commands/utils/batch/batcher.js

@@ -52,11 +52,11 @@ function get_spec_files(files, exclude_files) {
         remaining_files.push(matched_files[i]);
       }
       if (!fs.existsSync(matched_files[i])) {
-        reject("Spec files are not present");
+        reject("Error! Spec files are not present");
       }
     }
     if (remaining_files.length == 0) {
-      reject("Spec files are not present");
+      reject("Error! Spec files are not present");
     }
     resolve(remaining_files);
   });

commands/utils/constants.js

@@ -37,6 +37,7 @@ module.exports = {
     REPORT_URL:
       "https://api-envi-dev.lambdatestinternal.com/automation/api/v1/cypress/artefacts/test/",
   },
+  
   stage: {
     INTEGRATION_BASE_URL: "https://stage-api.lambdatestinternal.com/liis",
     BUILD_BASE_URL:

commands/utils/default_reporter_config.js

@@ -5,5 +5,5 @@ module.exports = {
         "overwrite": true,
         "html": true,
         "json": true
-        }
-};
+    }
+}

commands/utils/set_args.js

@@ -1,4 +1,6 @@
 const constants = require("./constants.js");
+var init_commands = require("./../init.js");
+
 const fs = require("fs");
 const path = require("path");
 const process = require("process");
@@ -237,6 +239,13 @@ function sync_args_from_cmd(args) {
       console.log(
         "Warning !! Value of reporter_config_file parameter missing. Proceeding with default reporter config"
       );
+      // make sure that the default file exists on user system. If not, place it. There is a possibility that 
+      // user may have deleted it from her system/or not ran init command.
+      if (!fs.existsSync(lt_config["run_settings"]["reporter_config_file"])) {
+        console.log("!! Warning, Creating the default reporter config file");
+        init_commands.create_base_reporter_config_file(args);
+
+      }
       lt_config["run_settings"]["reporter_config_file"] =
         constants.LT_BASE_REPORTER_CONFIG_FILE_NAME;
     }

commands/utils/validate.js

@@ -4,7 +4,7 @@ const semverCompare = require("semver/functions/compare");
 
 const constants = require("./constants.js");
 module.exports = validate_config = function (lt_config, validation_configs) {
-  console.log("validating config");
+  console.log("Validating lambdatest config");
   return new Promise(function (resolve, reject) {
     //validate auth keys are present
     if (
@@ -251,9 +251,9 @@ module.exports = validate_config = function (lt_config, validation_configs) {
               );
             }
           }
-        } catch {
+        } catch(e) {
           console.log(
-            "reporter_config_file could not be parsed, please provide a valid json in Reporter Config"
+            "reporter_config_file could not be parsed, please provide a valid json in Reporter Config - ", e
           );
           reject("Error!! Reporter JSON File does not have correct json");
         }

index.js

@@ -1,12 +1,10 @@
 #!/usr/bin/env node
 
-const { init } = require("./commands/init");
-
 const argv = require("yargs")
   .usage("Usage: $0 <command> [options]")
   .command(
     "init",
-    "create an intial config file",
+    "create an initial config file",
     function (yargs) {
       return yargs
         .option("cv", {
@@ -21,7 +19,7 @@ const argv = require("yargs")
         });
     },
     function (argv) {
-      require("./commands/init")(argv);
+      require("./commands/init").init_implementation(argv);
     }
   )
   .command(

package.json

@@ -1,6 +1,6 @@
 {
   "name": "lambdatest-cypress-cli",
-  "version": "3.0.11",
+  "version": "3.0.12",
   "description": "The lambdatest-cypress-cli is LambdaTest's command-line interface (CLI) aimed to help you run your Cypress tests on LambdaTest platform.",
   "homepage": "https://github.com/LambdaTest/lambdatest-cypress-cli",
   "author": "LambdaTest <keys@lambdatest.com>",