fix(core): ensure all components are properly closed

Author: darrachequesne

packages/core/src/App.ts

@@ -146,7 +146,7 @@ export class App {
     app.close = () => {
       debug("closing all components");
       return Promise.all(
-        components.map((component) => component.close()),
+        allComponents.map((component) => component.close()),
       ).then();
     };
 

packages/core/test/base/App.test.ts

@@ -0,0 +1,41 @@
+import { describe, it } from "node:test";
+import * as assert from "node:assert";
+import { App, type AppInit, Component } from "../../src";
+import { createTestDB } from "../setup.js";
+
+describe("App", () => {
+  it("should initialize and close components", async () => {
+    let initCalled = false;
+    let closeCalled = false;
+
+    class TestComponent extends Component {
+      override async init() {
+        initCalled = true;
+      }
+
+      override async close() {
+        closeCalled = true;
+      }
+    }
+
+    const app = await App.create({
+      components: [createTestDB()],
+      modules: [
+        {
+          name: "test",
+          version: "0.0.1",
+          init(app: AppInit) {
+            app.component(TestComponent);
+          },
+        },
+      ],
+    });
+
+    assert.equal(initCalled, true);
+    assert.equal(closeCalled, false);
+
+    await app.close();
+
+    assert.equal(closeCalled, true);
+  });
+});