fix: undefined arg when request fails

browserstack · Dec 7, 2023 · 0ca8b1a · 0ca8b1a
1 parent 50a8bf0
commit 0ca8b1a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/bin/commands/runs.js b/bin/commands/runs.js
@@ -154,7 +154,7 @@ module.exports = function run(args, rawArgs) {
         // Hence whenever running on ATS, need to make local as false
         bsConfig.connection_settings.local = false;
 
-        const gridDetails = await getTurboScaleGridDetails(bsConfig);
+        const gridDetails = await getTurboScaleGridDetails(bsConfig, args, rawArgs);
 
         if (gridDetails && Object.keys(gridDetails).length > 0) {
           Constants.turboScaleObj.gridDetails = gridDetails;

diff --git a/bin/helpers/atsHelper.js b/bin/helpers/atsHelper.js
@@ -38,7 +38,7 @@ exports.getTurboScaleGridName = (bsConfig) => {
   return 'NO_GRID_NAME_PASSED';
 };
 
-exports.getTurboScaleGridDetails = async (bsConfig) => {
+exports.getTurboScaleGridDetails = async (bsConfig, args, rawArgs) => {
   try {
     const gridName = this.getTurboScaleGridName(bsConfig);
 

diff --git a/test/unit/bin/helpers/atsHelper.js b/test/unit/bin/helpers/atsHelper.js
@@ -87,7 +87,7 @@ describe('#atsHelper', () => {
 
       requestStub.yields(null, { statusCode: 200 }, body);
       sendUsageReportStub.notCalled;
-      atsHelper.getTurboScaleGridDetails(testObjects.sampleATSBsConfigWithOptions).then((result) => {
+      atsHelper.getTurboScaleGridDetails(testObjects.sampleATSBsConfigWithOptions, {}, {}).then((result) => {
         expect(result).to.eq(body);
       })
     });
@@ -104,7 +104,7 @@ describe('#atsHelper', () => {
       };
 
       requestStub.yields(null, { statusCode: 422 }, body);
-      atsHelper.getTurboScaleGridDetails(testObjects.sampleATSBsConfigWithOptions).then((result) => {
+      atsHelper.getTurboScaleGridDetails(testObjects.sampleATSBsConfigWithOptions, {}, {}).then((result) => {
         expect(result).to.eq({});
         expect(sendUsageReportStub.calledOnce).to.be.true;
       })