Changes in args to reporter and fixing specs

Author: suryart

bin/commands/generateReport.js

@@ -27,8 +27,9 @@ module.exports = function generateReport(args) {
 
     let messageType = Constants.messageTypes.INFO;
     let errorCode = null;
+    let buildId = args._[1];
 
-    reportGenerator(bsConfig, args);
+    reportGenerator(bsConfig, buildId, args);
     utils.sendUsageReport(bsConfig, args, 'generate-report called', messageType, errorCode);
   }).catch(function (err) {
     logger.error(err);

bin/helpers/reporterHTML.js

@@ -1,6 +1,9 @@
 const fs = require('fs'),
       path = require('path'),
-      logger = require('./logger').winstonLogger;
+      logger = require('./logger').winstonLogger,
+      utils = require("./utils"),
+      Constants = require('./constants'),
+      config = require("./config");
 
 let templatesDir = path.join(__dirname, '../', 'templates');
 
@@ -84,7 +87,7 @@ function buildSpecStats(specMeta) {
   return specStats;
 }
 
-reportGenerator = (bsConfig, buildId) => {
+reportGenerator = (bsConfig, buildId, args) => {
   let options = {
     url: `${config.buildUrl}${buildId}/custom_report`,
     auth: {

test/unit/bin/commands/generateReport.js

@@ -66,7 +66,7 @@ describe("generateReport", () => {
 
       generateReport(args)
       .then(function (_bsConfig) {
-        sinon.assert.calledWith(reportGeneratorSpy, bsConfig, args);
+        sinon.assert.calledWith(reportGeneratorSpy, bsConfig, args._[1], args);
         sinon.assert.calledOnce(getConfigPathStub);
         sinon.assert.calledOnceWithExactly(sendUsageReportStub, bsConfig, args, 'generate-report called', Constants.messageTypes.INFO, null);
       })