[REF] component: remove onDestroyed, implement onWillDestroy

tests/components/__snapshots__/lifecycle.test.ts.snap

@@ -107,36 +107,6 @@ exports[`lifecycle hooks component semantics 6`] = `
 }"
 `;
 
-exports[`lifecycle hooks components are unmounted and destroyed if no longer in DOM 1`] = `
-"function anonymous(bdom, helpers
-) {
-  let { text, createBlock, list, multi, html, toggler, component } = bdom;
-  let { withDefault, getTemplate, prepareList, withKey, zero, call, callSlot, capture, isBoundary, shallowEqual, setContextValue, toNumber, safeOutput } = helpers;
-  
-  let block1 = createBlock(\`<div/>\`);
-  
-  return function template(ctx, node, key = \\"\\") {
-    return block1();
-  }
-}"
-`;
-
-exports[`lifecycle hooks components are unmounted and destroyed if no longer in DOM 2`] = `
-"function anonymous(bdom, helpers
-) {
-  let { text, createBlock, list, multi, html, toggler, component } = bdom;
-  let { withDefault, getTemplate, prepareList, withKey, zero, call, callSlot, capture, isBoundary, shallowEqual, setContextValue, toNumber, safeOutput } = helpers;
-  
-  return function template(ctx, node, key = \\"\\") {
-    let b2;
-    if (ctx['state'].ok) {
-      b2 = component(\`Child\`, {}, key + \`__1\`, node, ctx);
-    }
-    return multi([b2]);
-  }
-}"
-`;
-
 exports[`lifecycle hooks components are unmounted and destroyed if no longer in DOM, even after updateprops 1`] = `
 "function anonymous(bdom, helpers
 ) {
@@ -171,6 +141,36 @@ exports[`lifecycle hooks components are unmounted and destroyed if no longer in
 }"
 `;
 
+exports[`lifecycle hooks components are unmounted destroyed if no longer in DOM 1`] = `
+"function anonymous(bdom, helpers
+) {
+  let { text, createBlock, list, multi, html, toggler, component } = bdom;
+  let { withDefault, getTemplate, prepareList, withKey, zero, call, callSlot, capture, isBoundary, shallowEqual, setContextValue, toNumber, safeOutput } = helpers;
+  
+  let block1 = createBlock(\`<div/>\`);
+  
+  return function template(ctx, node, key = \\"\\") {
+    return block1();
+  }
+}"
+`;
+
+exports[`lifecycle hooks components are unmounted destroyed if no longer in DOM 2`] = `
+"function anonymous(bdom, helpers
+) {
+  let { text, createBlock, list, multi, html, toggler, component } = bdom;
+  let { withDefault, getTemplate, prepareList, withKey, zero, call, callSlot, capture, isBoundary, shallowEqual, setContextValue, toNumber, safeOutput } = helpers;
+  
+  return function template(ctx, node, key = \\"\\") {
+    let b2;
+    if (ctx['state'].ok) {
+      b2 = component(\`Child\`, {}, key + \`__1\`, node, ctx);
+    }
+    return multi([b2]);
+  }
+}"
+`;
+
 exports[`lifecycle hooks hooks are called in proper order in widget creation/destruction 1`] = `
 "function anonymous(bdom, helpers
 ) {

tests/components/concurrency.test.ts

@@ -64,7 +64,7 @@ describe("async rendering", () => {
     def.resolve();
     await nextTick();
     expect(status(w)).toBe("destroyed");
-    expect(["W:setup", "W:willStart", "W:destroyed"]).toBeLogged();
+    expect(["W:setup", "W:willStart", "W:willDestroy"]).toBeLogged();
   });
 });
 
@@ -112,7 +112,7 @@ test("destroying/recreating a subwidget with different props (if start is not ov
 
   expect([
     "W:willRender",
-    "Child:destroyed",
+    "Child:willDestroy",
     "Child:setup",
     "Child:willStart",
     "W:rendered",
@@ -190,7 +190,7 @@ test("creating two async components, scenario 1", async () => {
   expect(fixture.innerHTML).toBe("");
   expect([
     "Parent:willRender",
-    "ChildA:destroyed",
+    "ChildA:willDestroy",
     "ChildA:setup",
     "ChildA:willStart",
     "ChildB:setup",
@@ -583,8 +583,8 @@ test("properly behave when destroyed/unmounted while rendering ", async () => {
     "Parent:willPatch",
     "Child:willUnmount",
     "SubChild:willUnmount",
-    "SubChild:destroyed",
-    "Child:destroyed",
+    "SubChild:willDestroy",
+    "Child:willDestroy",
     "Parent:patched",
   ]).toBeLogged();
 
@@ -642,7 +642,7 @@ test("rendering component again in next microtick", async () => {
     "Child:willStart",
     "Parent:rendered",
     "Parent:willRender",
-    "Child:destroyed",
+    "Child:willDestroy",
     "Child:setup",
     "Child:willStart",
     "Parent:rendered",
@@ -1694,7 +1694,7 @@ test("concurrent renderings scenario 10", async () => {
   expect(rendered).toBe(1);
   expect([
     "ComponentB:willRender",
-    "ComponentC:destroyed",
+    "ComponentC:willDestroy",
     "ComponentC:setup",
     "ComponentC:willStart",
     "ComponentB:rendered",
@@ -2249,7 +2249,7 @@ test.skip("concurrent renderings scenario 16", async () => {
     "C:willUpdateProps",
     "C:willRender",
     "C:rendered",
-    "D:destroyed",
+    "D:willDestroy",
     "D:setup",
     "D:willStart",
     "D:willRender",
@@ -2324,7 +2324,7 @@ test("calling render in destroy", async () => {
     "B:willRender",
     "B:rendered",
     "B:willUnmount",
-    "B:destroyed",
+    "B:willDestroy",
     "B:mounted",
     "B:willRender",
     "B:rendered",
@@ -2522,7 +2522,7 @@ test("two renderings initiated between willPatch and patched", async () => {
     "Panel:rendered",
     "Parent:willPatch",
     "Panel:willUnmount",
-    "Panel:destroyed",
+    "Panel:willDestroy",
     "Panel:mounted",
     "Parent:patched",
   ]).toBeLogged();
@@ -2535,7 +2535,7 @@ test("two renderings initiated between willPatch and patched", async () => {
     "Parent:rendered",
     "Parent:willPatch",
     "Panel:willUnmount",
-    "Panel:destroyed",
+    "Panel:willDestroy",
     "Parent:patched",
   ]).toBeLogged();
 });

tests/components/lifecycle.test.ts

@@ -313,7 +313,7 @@ describe("lifecycle hooks", () => {
     Object.freeze(steps);
   });
 
-  test("components are unmounted and destroyed if no longer in DOM", async () => {
+  test("components are unmounted destroyed if no longer in DOM", async () => {
     let steps: string[] = [];
 
     class Child extends Component {
@@ -410,7 +410,7 @@ describe("lifecycle hooks", () => {
       "Parent:rendered",
       "Parent:willPatch",
       "Child:willUnmount",
-      "Child:destroyed",
+      "Child:willDestroy",
       "Parent:patched",
     ]).toBeLogged();
   });
@@ -450,8 +450,8 @@ describe("lifecycle hooks", () => {
     expect([
       "Parent:willUnmount",
       "Child:willUnmount",
-      "Child:destroyed",
-      "Parent:destroyed",
+      "Child:willDestroy",
+      "Parent:willDestroy",
     ]).toBeLogged();
   });
 
@@ -568,8 +568,8 @@ describe("lifecycle hooks", () => {
     expect([
       "Parent:willUnmount",
       "Child:willUnmount",
-      "Child:destroyed",
-      "Parent:destroyed",
+      "Child:willDestroy",
+      "Parent:willDestroy",
     ]).toBeLogged();
   });
 
@@ -631,9 +631,9 @@ describe("lifecycle hooks", () => {
       "Parent:willUnmount",
       "Child:willUnmount",
       "GrandChild:willUnmount",
-      "GrandChild:destroyed",
-      "Child:destroyed",
-      "Parent:destroyed",
+      "GrandChild:willDestroy",
+      "Child:willDestroy",
+      "Parent:willDestroy",
     ]).toBeLogged();
   });
 
@@ -675,7 +675,7 @@ describe("lifecycle hooks", () => {
     // immediately destroy everything
     app.destroy();
     await nextTick();
-    expect(["Parent:willUnmount", "Parent:destroyed"]).toBeLogged();
+    expect(["Parent:willUnmount", "Parent:willDestroy"]).toBeLogged();
   });
 
   test("lifecycle semantics, part 4", async () => {
@@ -732,9 +732,9 @@ describe("lifecycle hooks", () => {
     app.destroy();
     expect([
       "Parent:willUnmount",
-      "GrandChild:destroyed",
-      "Child:destroyed",
-      "Parent:destroyed",
+      "GrandChild:willDestroy",
+      "Child:willDestroy",
+      "Parent:willDestroy",
     ]).toBeLogged();
   });
 
@@ -776,7 +776,7 @@ describe("lifecycle hooks", () => {
       "Parent:rendered",
       "Parent:willPatch",
       "Child:willUnmount",
-      "Child:destroyed",
+      "Child:willDestroy",
       "Parent:patched",
     ]).toBeLogged();
   });
@@ -1016,7 +1016,7 @@ describe("lifecycle hooks", () => {
       "C:willPatch",
       "D:willPatch",
       "E:willUnmount",
-      "E:destroyed",
+      "E:willDestroy",
       "F:mounted",
       "D:patched",
       "C:patched",
@@ -1061,7 +1061,7 @@ describe("lifecycle hooks", () => {
       "Parent:rendered",
       "Parent:willPatch",
       "Child:willUnmount",
-      "Child:destroyed",
+      "Child:willDestroy",
       "Parent:patched",
     ]).toBeLogged();
 

tests/components/t_component.test.ts

@@ -95,7 +95,7 @@ describe("t-component", () => {
       "ChildB:rendered",
       "Parent:willPatch",
       "ChildA:willUnmount",
-      "ChildA:destroyed",
+      "ChildA:willDestroy",
       "ChildB:mounted",
       "Parent:patched",
     ]).toBeLogged();

tests/helpers.ts

@@ -1,7 +1,7 @@
 import {
   App,
   Component,
-  onDestroyed,
+  onWillDestroy,
   onMounted,
   onPatched,
   onWillRender,
@@ -210,9 +210,9 @@ export function useLogLifecycle() {
     logStep(`${name}:willUnmount`);
   });
 
-  onDestroyed(() => {
-    expect(name + ": " + status(component)).toBe(name + ": " + "destroyed");
-    logStep(`${name}:destroyed`);
+  onWillDestroy(() => {
+    expect(status(component)).not.toBe("destroyed");
+    logStep(`${name}:willDestroy`);
   });
 }
 

tests/reactivity.test.ts

@@ -1492,7 +1492,7 @@ describe("Reactivity: useState", () => {
       "Parent:rendered",
       "Parent:willPatch",
       "Child:willUnmount",
-      "Child:destroyed",
+      "Child:willDestroy",
       "Parent:patched",
     ]).toBeLogged();