From bdeb07a6affe5df23b592a434211c9d4b54a00a1 Mon Sep 17 00:00:00 2001
From: Anders Kaseorg <anders@zulip.com>
Date: Mon, 20 Jul 2020 23:14:03 -0700
Subject: [PATCH] zjquery: Refactor generic_event to wrap trigger.

Signed-off-by: Anders Kaseorg <anders@zulip.com>
---
 frontend_tests/zjsunit/zjquery.js | 23 ++++++++---------------
 1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/frontend_tests/zjsunit/zjquery.js b/frontend_tests/zjsunit/zjquery.js
index b814e428c0..9d525b9362 100644
--- a/frontend_tests/zjsunit/zjquery.js
+++ b/frontend_tests/zjsunit/zjquery.js
@@ -24,22 +24,7 @@ exports.make_event_store = (selector) => {
     const on_functions = new Map();
     const child_on_functions = new Map();
 
-    function generic_event($element, event_name, arg) {
-        if (typeof arg === "function") {
-            on_functions.set(event_name, arg);
-        } else {
-            const handler = on_functions.get(event_name);
-            if (!handler) {
-                const error = "Cannot find " + event_name + " handler for " + selector;
-                throw Error(error);
-            }
-            handler.call($element, arg);
-        }
-    }
-
     const self = {
-        generic_event: generic_event,
-
         get_on_handler: function (name, child_selector) {
             let handler;
 
@@ -130,6 +115,14 @@ exports.make_event_store = (selector) => {
 
             func.call($element, ev, data);
         },
+
+        generic_event($element, event_name, arg) {
+            if (typeof arg === "function") {
+                on_functions.set(event_name, arg);
+            } else {
+                self.trigger($element, event_name, arg);
+            }
+        },
     };
 
     return self;