Fixed more tests

refs 8630be5a87

- referenced commit skipped current user and proxy service population when `skipAuthSuccessHandler` was set but the population is needed during setup
- `skipAuthSuccessHandler` is used for skipping the post-authentication transition rather than skipping all post-auth behaviour so the the post-auth behaviour was reintroduced but put in a guard so we don't try to do it again when it's already been run

ghost/admin/app/services/session.js

@@ -56,11 +56,7 @@ export default class SessionService extends ESASessionService {
     }
 
     async handleAuthentication() {
-        if (this.skipAuthSuccessHandler) {
+        if (!this.user) {
-            this.skipAuthSuccessHandler = false;
-            return;
-        }
-
             try {
                 await this.populateUser();
             } catch (err) {
@@ -68,6 +64,12 @@ export default class SessionService extends ESASessionService {
             }
 
             await this.postAuthPreparation();
+        }
+
+        if (this.skipAuthSuccessHandler) {
+            this.skipAuthSuccessHandler = false;
+            return;
+        }
 
         super.handleAuthentication('home');
     }