Skip to content

Commit

Permalink
Fixing unit test cases for env vars
Browse files Browse the repository at this point in the history
  • Loading branch information
@karanshah-browserstack
karanshah-browserstack committed Aug 10, 2021
1 parent 08f5190 commit 9df01ac
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 9 deletions.
19 changes: 17 additions & 2 deletions test/unit/bin/commands/runs.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ describe("runs", () => {
setCypressConfigFilenameStub = sandbox.stub();
setUserSpecsStub = sandbox.stub();
setTestEnvsStub = sandbox.stub();
setSystemEnvsStub = sandbox.stub();
getConfigPathStub = sandbox.stub();
setupLocalTestingStub = sandbox.stub();
setUsageReportingFlagStub = sandbox.stub().returns(undefined);
Expand Down Expand Up @@ -141,7 +142,8 @@ describe("runs", () => {
setupLocalTesting: setupLocalTestingStub,
isJSONInvalid: isJSONInvalidStub,
setLocalMode: setLocalModeStub,
setLocalConfigFile: setLocalConfigFileStub
setLocalConfigFile: setLocalConfigFileStub,
setSystemEnvs: setSystemEnvsStub
},
'../helpers/capabilityHelper': {
validate: capabilityValidatorStub
Expand All @@ -168,6 +170,7 @@ describe("runs", () => {
sinon.assert.calledOnce(setCypressConfigFilenameStub);
sinon.assert.calledOnce(setUserSpecsStub);
sinon.assert.calledOnce(setTestEnvsStub);
sinon.assert.calledOnce(setSystemEnvsStub);
sinon.assert.calledOnce(setLocalStub);
sinon.assert.calledOnce(setLocalModeStub);
sinon.assert.calledOnce(setLocalConfigFileStub);
Expand Down Expand Up @@ -203,6 +206,7 @@ describe("runs", () => {
setCypressConfigFilenameStub = sandbox.stub();
setUserSpecsStub = sandbox.stub();
setTestEnvsStub = sandbox.stub();
setSystemEnvsStub = sandbox.stub();
validateBstackJsonStub = sandbox.stub();
setUsageReportingFlagStub = sandbox.stub().returns(undefined);
sendUsageReportStub = sandbox.stub().callsFake(function () {
Expand Down Expand Up @@ -256,7 +260,8 @@ describe("runs", () => {
deleteResults: deleteResultsStub,
setDefaults: setDefaultsStub,
getNumberOfSpecFiles: getNumberOfSpecFilesStub,
setLocalConfigFile: setLocalConfigFileStub
setLocalConfigFile: setLocalConfigFileStub,
setSystemEnvs: setSystemEnvsStub
},
'../helpers/capabilityHelper': {
validate: capabilityValidatorStub,
Expand Down Expand Up @@ -301,6 +306,7 @@ describe("runs", () => {
sinon.assert.calledOnce(deleteZipStub);
sinon.assert.calledOnce(deleteResultsStub);
sinon.assert.calledOnce(setDefaultsStub);
sinon.assert.calledOnce(setSystemEnvsStub);
sinon.assert.calledOnceWithExactly(
sendUsageReportStub,
bsConfig,
Expand All @@ -327,6 +333,7 @@ describe("runs", () => {
setCypressConfigFilenameStub = sandbox.stub();
setUserSpecsStub = sandbox.stub();
setTestEnvsStub = sandbox.stub();
setSystemEnvsStub = sandbox.stub();
getConfigPathStub = sandbox.stub();
setUsageReportingFlagStub = sandbox.stub().returns(undefined);
sendUsageReportStub = sandbox.stub().callsFake(function () {
Expand Down Expand Up @@ -370,6 +377,7 @@ describe("runs", () => {
setCypressConfigFilename: setCypressConfigFilenameStub,
setUserSpecs: setUserSpecsStub,
setTestEnvs: setTestEnvsStub,
setSystemEnvs: setSystemEnvsStub,
setUsageReportingFlag: setUsageReportingFlagStub,
getConfigPath: getConfigPathStub,
setLocal: setLocalStub,
Expand Down Expand Up @@ -426,6 +434,7 @@ describe("runs", () => {
sinon.assert.calledOnce(zipUploadStub);
sinon.assert.calledOnce(deleteResultsStub);
sinon.assert.calledOnce(setDefaultsStub);
sinon.assert.calledOnce(setSystemEnvsStub);
sinon.assert.calledOnceWithExactly(
sendUsageReportStub,
bsConfig,
Expand Down Expand Up @@ -455,6 +464,7 @@ describe("runs", () => {
setCypressConfigFilenameStub = sandbox.stub();
setUserSpecsStub = sandbox.stub();
setTestEnvsStub = sandbox.stub();
setSystemEnvsStub = sandbox.stub();
getConfigPathStub = sandbox.stub();
setUsageReportingFlagStub = sandbox.stub().returns(undefined);
sendUsageReportStub = sandbox.stub().callsFake(function () {
Expand Down Expand Up @@ -500,6 +510,7 @@ describe("runs", () => {
setCypressConfigFilename: setCypressConfigFilenameStub,
setUserSpecs: setUserSpecsStub,
setTestEnvs: setTestEnvsStub,
setSystemEnvs: setSystemEnvsStub,
setUsageReportingFlag: setUsageReportingFlagStub,
getConfigPath: getConfigPathStub,
setLocal: setLocalStub,
Expand Down Expand Up @@ -568,6 +579,7 @@ describe("runs", () => {
sinon.assert.calledOnce(sendUsageReportStub);
sinon.assert.calledOnce(deleteResultsStub);
sinon.assert.calledOnce(setDefaultsStub);
sinon.assert.calledOnce(setSystemEnvsStub);

sinon.assert.calledOnceWithExactly(
sendUsageReportStub,
Expand Down Expand Up @@ -596,6 +608,7 @@ describe("runs", () => {
setCypressConfigFilenameStub = sandbox.stub();
setUserSpecsStub = sandbox.stub();
setTestEnvsStub = sandbox.stub();
setSystemEnvsStub = sandbox.stub();
getConfigPathStub = sandbox.stub();
setUsageReportingFlagStub = sandbox.stub().returns(undefined);
sendUsageReportStub = sandbox.stub().callsFake(function () {
Expand Down Expand Up @@ -647,6 +660,7 @@ describe("runs", () => {
setCypressConfigFilename: setCypressConfigFilenameStub,
setUserSpecs: setUserSpecsStub,
setTestEnvs: setTestEnvsStub,
setSystemEnvs: setSystemEnvsStub,
setUsageReportingFlag: setUsageReportingFlagStub,
setParallels: setParallelsStub,
warnSpecLimit: warnSpecLimitStub,
Expand Down Expand Up @@ -725,6 +739,7 @@ describe("runs", () => {
sinon.assert.calledOnce(exportResultsStub);
sinon.assert.calledOnce(deleteResultsStub);
sinon.assert.calledOnce(setDefaultsStub);
sinon.assert.calledOnce(setSystemEnvsStub);
sinon.assert.match(
sendUsageReportStub.getCall(0).args,
[
Expand Down
14 changes: 7 additions & 7 deletions test/unit/bin/helpers/capabilityHelper.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ describe("capabilityHelper.js", () => {
return capabilityHelper
.caps(bsConfig, { zip_url: zip_url })
.then(function (data) {
chai.assert.equal(JSON.parse(data).cypress_version, cypress_version);
chai.assert.equal(JSON.parse(JSON.parse(data).run_settings).cypress_version, cypress_version);
})
.catch((error) => {
chai.assert.fail("Promise error");
Expand Down Expand Up @@ -309,7 +309,7 @@ describe("capabilityHelper.js", () => {
return capabilityHelper
.caps(bsConfig, { zip_url: zip_url })
.then(function (data) {
let parsed_data = JSON.parse(data);
let parsed_data = JSON.parse(JSON.parse(data).run_settings);
chai.assert.equal(parsed_data.specs, specsList);
chai.assert.equal(parsed_data.env, undefined);
})
Expand Down Expand Up @@ -341,7 +341,7 @@ describe("capabilityHelper.js", () => {
return capabilityHelper
.caps(bsConfig, { zip_url: zip_url })
.then(function (data) {
let parsed_data = JSON.parse(data);
let parsed_data = JSON.parse(JSON.parse(data).run_settings);
chai.assert.equal(parsed_data.env, envList);
chai.assert.equal(parsed_data.specs, undefined);
})
Expand Down Expand Up @@ -375,7 +375,7 @@ describe("capabilityHelper.js", () => {
return capabilityHelper
.caps(bsConfig, { zip_url: zip_url })
.then(function (data) {
let parsed_data = JSON.parse(data);
let parsed_data = JSON.parse(JSON.parse(data).run_settings);
chai.assert.equal(parsed_data.specs, specsList);
chai.assert.equal(parsed_data.env, envList);
})
Expand Down Expand Up @@ -405,7 +405,7 @@ describe("capabilityHelper.js", () => {
return capabilityHelper
.caps(bsConfig, { zip_url: zip_url })
.then(function (data) {
let parsed_data = JSON.parse(data);
let parsed_data = JSON.parse(JSON.parse(data).run_settings);
chai.assert.equal(parsed_data.specs, undefined);
chai.assert.equal(parsed_data.env, undefined);
})
Expand Down Expand Up @@ -439,7 +439,7 @@ describe("capabilityHelper.js", () => {
return capabilityHelper
.caps(bsConfig, { zip_url: zip_url })
.then(function (data) {
let parsed_data = JSON.parse(data);
let parsed_data = JSON.parse(JSON.parse(data).run_settings);
chai.assert.equal(parsed_data.headless, headless);
chai.assert.equal(parsed_data.env, undefined);
})
Expand Down Expand Up @@ -471,7 +471,7 @@ describe("capabilityHelper.js", () => {
return capabilityHelper
.caps(bsConfig, { zip_url: zip_url })
.then(function (data) {
let parsed_data = JSON.parse(data);
let parsed_data = JSON.parse(JSON.parse(data).run_settings);
chai.assert.equal(parsed_data.headless, headless);
chai.assert.equal(parsed_data.env, undefined);
})
Expand Down

0 comments on commit 9df01ac

Please sign in to comment.