From 13d59eb04dd941d0e2423499059bf2a47e3651f9 Mon Sep 17 00:00:00 2001
From: David Kalmakoff <david@extendedstep.com>
Date: Tue, 5 Nov 2024 16:01:26 -0600
Subject: [PATCH] bug: fixed router navigating to homepage on page loads

---
 ui/src/CurrentUser.ts     | 21 +++++++++------------
 ui/src/tests/user.test.ts |  4 ++--
 2 files changed, 11 insertions(+), 14 deletions(-)

diff --git a/ui/src/CurrentUser.ts b/ui/src/CurrentUser.ts
index 6de4edb..0d657f6 100644
--- a/ui/src/CurrentUser.ts
+++ b/ui/src/CurrentUser.ts
@@ -15,7 +15,7 @@ export class CurrentUser {
     @observable
     public loggedIn = false;
     @observable
-    public authenticating = false;
+    public authenticating = true;
     @observable
     public user: IUser = {name: 'unknown', admin: false, id: -1};
     @observable
@@ -80,17 +80,11 @@ export class CurrentUser {
             .then((resp: AxiosResponse<IClient>) => {
                 this.snack(`A client named '${name}' was created for your session.`);
                 this.setToken(resp.data.token);
-                this.tryAuthenticate()
-                    .then(() => {
-                        this.authenticating = false;
-                        this.loggedIn = true;
-                    })
-                    .catch(() => {
-                        this.authenticating = false;
-                        console.log(
-                            'create client succeeded, but authenticated with given token failed'
-                        );
-                    });
+                this.tryAuthenticate().catch(() => {
+                    console.log(
+                        'create client succeeded, but authenticated with given token failed'
+                    );
+                });
             })
             .catch(() => {
                 this.authenticating = false;
@@ -100,6 +94,7 @@ export class CurrentUser {
 
     public tryAuthenticate = async (): Promise<AxiosResponse<IUser>> => {
         if (this.token() === '') {
+            this.authenticating = false;
             return Promise.reject();
         }
 
@@ -111,11 +106,13 @@ export class CurrentUser {
                 .then((passThrough) => {
                     this.user = passThrough.data;
                     this.loggedIn = true;
+                    this.authenticating = false;
                     this.connectionErrorMessage = null;
                     this.reconnectTime = 7500;
                     return passThrough;
                 })
                 .catch((error: AxiosError) => {
+                    this.authenticating = false;
                     if (!error || !error.response) {
                         this.connectionError('No network connection or server unavailable.');
                         return Promise.reject(error);
diff --git a/ui/src/tests/user.test.ts b/ui/src/tests/user.test.ts
index c412e22..8531e58 100644
--- a/ui/src/tests/user.test.ts
+++ b/ui/src/tests/user.test.ts
@@ -24,8 +24,8 @@ const $dialog = selector.form('#add-edit-user-dialog');
 
 describe('User', () => {
     it('does login', async () => await auth.login(page));
-    it('navigates to users', async () => {
-        await page.click('#navigate-users');
+    it('navigates to users through window location', async () => {
+        await page.goto(gotify.url + '/#/users');
         await waitForExists(page, selector.heading(), 'Users');
     });
     it('has changed url', async () => {