test: add missing skips for closing the browser (#804)

* test: update tests to fix CI

* test: add more missing var declarations

* test: fix missing skip condition

* refactor: rework how we get session payload for newSessionCreated

* test: add missing skips for closing the browser

Author: porcellus

test/end-to-end/mfa.chooserscreen.test.js

@@ -87,7 +87,9 @@ describe("SuperTokens SignIn w/ MFA", function () {
     });
 
     after(async function () {
-        await browser.close();
+        if (!skipped) {
+            await browser.close();
+        }
 
         await fetch(`${TEST_SERVER_BASE_URL}/after`, {
             method: "POST",

test/end-to-end/mfa.factorscreen.totp.test.js

@@ -88,7 +88,9 @@ describe("SuperTokens SignIn w/ MFA", function () {
     });
 
     after(async function () {
-        await browser.close();
+        if (skipped) {
+            await browser.close();
+        }
 
         await fetch(`${TEST_SERVER_BASE_URL}/after`, {
             method: "POST",