fix: fix click event target and add message to login fail event (#19)

Author: n1tsua

src/initialize.ts

@@ -63,10 +63,20 @@ function rewriteLinks() {
     "click",
     (event) => {
       rewriteTargets();
+      let target = "";
+      if (event.target instanceof Element) {
+        if (event.target.id) {
+          target = `#${event.target.id}`;
+        } else if (event.target.classList.length) {
+          target = `.${event.target.classList.value.replaceAll(" ", ".")}`;
+        } else {
+          target = event.target.tagName;
+        }
+      }
       sendMessageToTestBox(CLICK, {
         x: event.x,
         y: event.y,
-        target: event.target,
+        target,
       });
     },
     false

src/messaging/outgoing.ts

@@ -22,7 +22,11 @@ type HealthCheckEvent = {
 type ClickEvent = {
   x: number;
   y: number;
-  target: any;
+  target?: string;
+};
+
+type LoginFailEvent = {
+  message?: string;
 };
 
 export interface TestBoxOutgoingEvents {
@@ -35,7 +39,7 @@ export interface TestBoxOutgoingEvents {
   [INITIALIZE_FAIL]: undefined;
   [LOGIN_ACK]: undefined;
   [LOGIN_SUCCESS]: undefined;
-  [LOGIN_FAIL]: undefined;
+  [LOGIN_FAIL]: LoginFailEvent;
   [LOGIN_NO_CREDS]: undefined;
   [NAVIGATE_ACK]: undefined;
 }

src/utils/login.ts

@@ -1,6 +1,7 @@
 import { sendMessageToTestBox } from "../messaging";
 import { LOGIN_FAIL, LOGIN_SUCCESS } from "../messaging/outgoing";
 import { IncomingEventHandlers, LoginEvent } from "../messaging/incoming";
+import { error, info } from "./logging";
 
 export let loggingIn = false;
 
@@ -9,22 +10,24 @@ export async function autoLogin(
   router: Partial<IncomingEventHandlers>
 ) {
   let nextUrl: string | boolean;
+  let errorMessage: string;
   loggingIn = true;
   try {
     const func = router["login"];
     if (!func) {
-      console.log("No login callback exists!");
+      info("No login callback exists!");
       sendMessageToTestBox(LOGIN_FAIL);
       return;
     }
     nextUrl = await func(data);
   } catch (e) {
-    console.log(e);
+    error(e);
+    errorMessage = e.message;
     nextUrl = false;
   }
 
   if (nextUrl === false) {
-    sendMessageToTestBox(LOGIN_FAIL);
+    sendMessageToTestBox(LOGIN_FAIL, { message: errorMessage });
   } else {
     sendMessageToTestBox(LOGIN_SUCCESS);
   }