fix(clerk-js): Fixed bugs in setActive caused by new conditions on session

The `setActive` contains a few new conditions on the
`session` param, given that in this case `session` can
also be undefined (which is something we didn't have
in the previous `setSession`).

However, given that we use the current session if we
don't pass a `session`, a lot of these checks can be
removed.

They were wrong in the first place, because they were
also preventing the execution for `session === null`
so the behaviour was different.

https://www.notion.so/clerkdev/Add-ClerkJS-method-to-set-the-active-organization-2f551fafdbd14834a7b5519e87c6f254

Author: alex-ntousias

packages/clerk-js/src/core/clerk.ts

@@ -295,9 +295,7 @@ export default class Clerk implements ClerkInterface {
       }
     }
 
-    if (session) {
-      this.#authService?.setAuthCookiesFromSession(session);
-    }
+    this.#authService?.setAuthCookiesFromSession(session);
 
     // If this.session exists, then signout was triggered by the current tab
     // and should emit. Other tabs should not emit the same event again
@@ -308,7 +306,7 @@ export default class Clerk implements ClerkInterface {
     //1. setLastActiveSession to passed usersession (add a param).
     //   Note that this will also update the session's active organization
     //   id.
-    if (session && typeof document != 'undefined' && document.hasFocus()) {
+    if (typeof document != 'undefined' && document.hasFocus()) {
       await this.#touchLastActiveSession(session);
     }
 
@@ -317,7 +315,7 @@ export default class Clerk implements ClerkInterface {
     //   When undefined, neither SignedIn nor SignedOut renders, which avoids flickers or
     //   automatic reloading when reloading shouldn't be happening.
     const beforeUnloadTracker = createBeforeUnloadTracker();
-    if (session && beforeEmit) {
+    if (beforeEmit) {
       beforeUnloadTracker.startTracking();
       this.session = undefined;
       this.organization = undefined;