✨ Add additional errors for other cases.

lustre-labs · Aug 19, 2023 · 8ddc115 · 8ddc115
1 parent 37564e4
commit 8ddc115
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/lib/src/lustre.ffi.mjs b/lib/src/lustre.ffi.mjs
@@ -1,6 +1,8 @@
 import {
-  ElementNotFound,
+  AppAlreadyStarted,
+  AppNotYetStarted,
   ComponentAlreadyRegistered,
+  ElementNotFound,
   NotABrowser,
 } from "./lustre.mjs";
 import { from } from "./lustre/effect.mjs";
@@ -31,7 +33,7 @@ export class App {
 
   start(selector, flags) {
     if (!is_browser()) return new Error(new NotABrowser());
-    if (this.#root) return this;
+    if (this.#root) return new Error(new AppAlreadyStarted());
 
     const el = document.querySelector(selector);
 
@@ -65,6 +67,8 @@ export class App {
   }
 
   destroy() {
+    if (!this.#root) return new Error(new AppNotYetStarted());
+
     this.#root.remove();
     this.#state = null;
     this.#queue = [];

diff --git a/lib/src/lustre.gleam b/lib/src/lustre.gleam
@@ -50,6 +50,8 @@ import lustre/element.{Element}
 pub type App(flags, model, msg)
 
 pub type Error {
+  AppAlreadyStarted
+  AppNotYetStarted
   ComponentAlreadyRegistered
   ElementNotFound
   NotABrowser