Skip to content

Commit eaef5fc

Browse files
authored
update utils test (Azure#16)
* update utils test * update utils test * update serviceprincipallogin test
1 parent cf9ec15 commit eaef5fc

2 files changed

Lines changed: 5 additions & 9 deletions

File tree

__tests__/PowerShell/ServicePrinicipalLogin.test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,7 @@ describe('Testing login', () => {
3131
loginSpy = jest.spyOn(spnlogin, 'login');
3232
});
3333
test('ServicePrincipal login should pass', async () => {
34-
loginSpy.mockImplementationOnce(() => Promise.resolve(
35-
console.log('Azure PowerShell session successfully initialized')));
34+
loginSpy.mockImplementationOnce(() => Promise.resolve());
3635
await spnlogin.login();
3736
expect(loginSpy).toHaveBeenCalled();
3837
});

__tests__/PowerShell/Utilities/Utils.test.ts

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,17 +20,14 @@ describe('Testing isValidVersion', () => {
2020
});
2121

2222
describe('Testing setPSModulePath', () => {
23-
test('PSModulepath with azPSVersion non-empty', () => {
24-
if(!process.env.PSModulePath) {
25-
process.env.PSModulePath = process.env.PSModulePath + "modulePath";
26-
}
23+
test('PSModulePath with azPSVersion non-empty', () => {
2724
Utils.setPSModulePath(version);
28-
expect(process.env.PSModulepath).toContain(version);
25+
expect(process.env.PSModulePath).toContain(version);
2926
});
3027
test('PSModulePath with azPSVersion empty', () => {
31-
const currPSModulePath = process.env.PSModulepath;
28+
const prevPSModulePath = process.env.PSModulePath;
3229
Utils.setPSModulePath();
33-
expect(process.env.PSModulePath).not.toEqual(currPSModulePath);
30+
expect(process.env.PSModulePath).not.toEqual(prevPSModulePath);
3431
});
3532
});
3633

0 commit comments

Comments
 (0)