adding tests back in and tidying up

Author: daz-yozu

src/core/controller.ts

@@ -101,7 +101,7 @@ export class Controller<ElementType extends Element = Element> {
       prefix = this.identifier,
       bubbles = true,
       cancelable = true,
-    }: DispatchOptions = {},
+    }: DispatchOptions = {}
   ) {
     const type = prefix ? `${prefix}:${eventName}` : eventName
     const event = new CustomEvent(type, { detail, bubbles, cancelable })

src/tests/modules/core/find_element_test.ts

@@ -11,5 +11,28 @@ export default class FindElementTests extends ControllerTestCase(ClassController
       <div id="inside-id" class="busy"></div>
     </div>
     <div id="outside-id" class="busy"></div>
+
+    "test findElement finds element by id inside scope"() {
+      const result = this.controller.findElement("inside-id")
+      this.assert.equal(result?.id, "inside-id")
+    }
+
+    "test findElement does not find element by id outside scope"() {
+      const result = this.controller.findElement("outside-id")
+      this.assert.equal(result, undefined)
+    }
+
+    "test findElement finds element by defined class"() {
+      const result = this.controller.findElement(this.controller.loadingClass)
+      this.assert.ok(result instanceof Element)
+      this.assert.ok(result?.classList.contains("busy"))
+    }
+
+    "test findAllElements returns multiple matches for defined class"() {
+      const results = this.controller.findAllElements(this.controller.loadingClass)
+      this.assert.ok(Array.isArray(results))
+      this.assert.equal(results.length, 1)
+      this.assert.ok(results[0].classList.contains("busy"))
+    }
   `
 }