From eaef5fc7a43f02aa2c06973836cd774d0d192bf3 Mon Sep 17 00:00:00 2001
From: aksm-ms <58936966+aksm-ms@users.noreply.github.com>
Date: Wed, 15 Apr 2020 12:04:04 +0530
Subject: [PATCH] update utils test (#16)

* update utils test

* update utils test

* update serviceprincipallogin test
---
 __tests__/PowerShell/ServicePrinicipalLogin.test.ts |  3 +--
 __tests__/PowerShell/Utilities/Utils.test.ts        | 11 ++++-------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/__tests__/PowerShell/ServicePrinicipalLogin.test.ts b/__tests__/PowerShell/ServicePrinicipalLogin.test.ts
index f8ba1f10..f8fdbb24 100644
--- a/__tests__/PowerShell/ServicePrinicipalLogin.test.ts
+++ b/__tests__/PowerShell/ServicePrinicipalLogin.test.ts
@@ -31,8 +31,7 @@ describe('Testing login', () => {
         loginSpy = jest.spyOn(spnlogin, 'login');
     });
     test('ServicePrincipal login should pass', async () => {
-        loginSpy.mockImplementationOnce(() => Promise.resolve(
-            console.log('Azure PowerShell session successfully initialized')));
+        loginSpy.mockImplementationOnce(() => Promise.resolve());
         await spnlogin.login();
         expect(loginSpy).toHaveBeenCalled();
     });
diff --git a/__tests__/PowerShell/Utilities/Utils.test.ts b/__tests__/PowerShell/Utilities/Utils.test.ts
index 10baa772..f6e2825b 100644
--- a/__tests__/PowerShell/Utilities/Utils.test.ts
+++ b/__tests__/PowerShell/Utilities/Utils.test.ts
@@ -20,17 +20,14 @@ describe('Testing isValidVersion', () => {
 });
 
 describe('Testing setPSModulePath', () => {
-    test('PSModulepath with azPSVersion non-empty', () => {
-        if(!process.env.PSModulePath) {
-            process.env.PSModulePath = process.env.PSModulePath + "modulePath";
-        }
+    test('PSModulePath with azPSVersion non-empty', () => {
         Utils.setPSModulePath(version);
-        expect(process.env.PSModulepath).toContain(version);
+        expect(process.env.PSModulePath).toContain(version);
     });
     test('PSModulePath with azPSVersion empty', () => {
-        const currPSModulePath = process.env.PSModulepath;
+        const prevPSModulePath = process.env.PSModulePath;
         Utils.setPSModulePath();
-        expect(process.env.PSModulePath).not.toEqual(currPSModulePath);
+        expect(process.env.PSModulePath).not.toEqual(prevPSModulePath);
     });
 });