finished testing

Author: levizwannah

package.json

@@ -72,6 +72,7 @@
     "jsdom": "^27.2.0",
     "postcss": "^8.4.32",
     "tailwindcss": "^3.4.0",
+    "terser": "^5.44.1",
     "vite": "^5.0.7",
     "vitest": "^4.0.13"
   },

src/broker/BrokerRegistrar.js

@@ -1,79 +1,70 @@
-import { broker } from "../index.js";
+import { container } from "../core/Container.js";
+import { isClass } from "../utils/helpers.js";
 
 /**
  * Registers events on the broker
  */
 export default class BrokerRegistrar {
-    async registerNamespace(namespace, events, obj) {
-        if (typeof events !== "object") {
-            console.error(
-                `Namespace has incorrect declaration syntax: '${namespace}' with value: `,
-                events,
-                `in ${obj.constructor.name}`
-            );
+  async registerNamespace(namespace, events, obj) {
+    if (typeof events !== "object") {
+      console.error(
+        `Namespace has incorrect declaration syntax: '${namespace}' with value: `,
+        events,
+        `in ${obj.constructor.name}`
+      );
 
-            return;
-        }
+      return;
+    }
 
-        for (let event in events) {
-            if (
-                event.startsWith("$$") ||
-                (typeof events[event] === "object" &&
-                    !(typeof events[event] === "function"))
-            ) {
-                this.registerNamespace(
-                    `${namespace}:${
-                        event.startsWith("$$") ? event.substring(2) : event
-                    }`,
-                    events[event],
-                    obj
-                );
-            } else {
-                let ev = event.split(/_/g).filter((a) => a.length > 0);
+    for (let event in events) {
+      if (
+        event.startsWith("$$") ||
+        (typeof events[event] === "object" &&
+          !(typeof events[event] === "function"))
+      ) {
+        this.registerNamespace(
+          `${namespace}:${event.startsWith("$$") ? event.substring(2) : event}`,
+          events[event],
+          obj
+        );
+      } else {
+        let ev = event.split(/_/g).filter((a) => a.length > 0);
 
-                for (let e of ev) {
-                    this.registerMethod(
-                        `${namespace}:${e}`,
-                        events[event],
-                        obj
-                    );
-                }
-            }
+        for (let e of ev) {
+          this.registerMethod(`${namespace}:${e}`, events[event], obj);
         }
+      }
     }
+  }
 
-    async register(o) {
-        let obj = o;
-        let seen = new Set();
+  async register(o) {
+    let obj = o;
+    let seen = new Set();
 
-        do {
-            for (let method of Object.getOwnPropertyNames(obj)) {
-                if (seen.has(method)) continue;
-                if (method.length < 3) continue;
-                if (!method.startsWith("$$")) continue;
+    do {
+      for (let method of Object.getOwnPropertyNames(obj)) {
+        if (seen.has(method)) continue;
+        if (method.length < 3) continue;
+        if (!method.startsWith("$$")) continue;
 
-                if (typeof obj[method] !== "function") {
-                    await this.registerNamespace(
-                        method.substring(2),
-                        obj[method],
-                        obj
-                    );
-                    continue;
-                }
+        if (typeof obj[method] !== "function") {
+          await this.registerNamespace(method.substring(2), obj[method], obj);
+          continue;
+        }
 
-                this.registerMethod(method.substring(2), obj[method], obj);
+        this.registerMethod(method.substring(2), obj[method], obj);
 
-                seen.add(method);
-            }
-        } while ((obj = Object.getPrototypeOf(obj)));
-    }
+        seen.add(method);
+      }
+    } while ((obj = Object.getPrototypeOf(obj)));
+  }
 
-    async registerMethod(method, listener, object) {
-        let events = method.split(/_/g).filter((a) => a.length > 0);
+  async registerMethod(method, listener, object) {
+    let events = method.split(/_/g).filter((a) => a.length > 0);
 
-        for (let ev of events) {
-            if (ev.length === 0) continue;
-            broker.on(ev, listener.bind(object));
-        }
+    for (let ev of events) {
+      if (ev.length === 0) continue;
+      container.resolve("broker").on(ev, listener.bind(object));
     }
+  }
 }

src/broker/Listener.js

@@ -4,11 +4,22 @@ import BrokerRegistrar from "./BrokerRegistrar.js";
  * A Broker Listener
  */
 export default class Listener {
-    /**
-     * Registers with the broker
-     */
-    async register() {
-        let br = new BrokerRegistrar();
-        br.register(this);
+  /**
+   * Registers with the broker
+   */
+  async register() {
+    // Prevent duplicate registration
+    if (this.__ojsRegistered) {
+      console.warn(
+        `Listener "${this.constructor.name}" is already registered. Skipping duplicate registration.`
+      );
+      return;
     }
+
+    let br = new BrokerRegistrar();
+    br.register(this);
+
+    // Mark as registered
+    this.__ojsRegistered = true;
+  }
 }