test: fix tests

Author: nattb8

src/Packages/Passport/Runtime/Scripts/Private/Core/BrowserCommunicationsManager.cs

@@ -163,9 +163,7 @@ private void HandleResponse(string message)
             // Validate the deserialised response object
             if (response == null || string.IsNullOrEmpty(response.responseFor) || string.IsNullOrEmpty(response.requestId))
             {
-                string responseError = "Response from browser is incorrect. Check game bridge file.";
-                PassportLogger.Error(responseError);
-                throw new PassportException(responseError);
+                throw new PassportException("Response from browser is incorrect. Check game bridge file.");
             }
 
             string logMessage = $"{TAG} Response for: {response.responseFor} (request ID: {response.requestId}) : {message}";

src/Packages/Passport/Runtime/Scripts/Private/PassportImpl.cs

@@ -533,7 +533,7 @@ public async UniTask<string> GetLogoutUrl()
         {
             string response = await communicationsManager.Call(PassportFunction.LOGOUT);
             string logoutUrl = response.GetStringResult();
-            if (String.IsNullOrEmpty(logoutUrl))
+            if (string.IsNullOrEmpty(logoutUrl))
             {
                 throw new PassportException("Failed to get logout URL", PassportErrorType.AUTHENTICATION_ERROR);
             }

src/Packages/Passport/Tests/Runtime/Scripts/PassportTests.cs

@@ -9,6 +9,7 @@
 using Immutable.Browser.Core;
 using UnityEngine;
 using Immutable.Passport.Helpers;
+using UnityEngine.TestTools;
 
 namespace Immutable.Passport
 {
@@ -1072,6 +1073,7 @@ public async Task Logout_FailedGetLogoutUrl()
                 e = ex;
             }
 
+            LogAssert.Expect(LogType.Error, "[Immutable] [Passport Implementation] Failed to log out: Response is invalid!");
             Assert.NotNull(e);
 
             Assert.AreEqual(0, urlsOpened.Count);
@@ -1103,6 +1105,7 @@ public async Task Logout_EmptyGetLogoutUrl()
                 e = ex;
             }
 
+            LogAssert.Expect(LogType.Error, "[Immutable] [Passport Implementation] Failed to log out: Failed to get logout URL");
             Assert.NotNull(e);
 
             Assert.AreEqual(0, urlsOpened.Count);