Merge pull request #120 from MongooseMoo/fix/master-ci-microtask-tests

Fix lifecycle tests after batched store merge

Author: ctoth

src/client.test.ts

@@ -403,12 +403,14 @@ describe('MudClient lifecycle cleanup', () => {
     expect(socket.send).toHaveBeenCalledWith('look\r\n');
   });
 
-  it('reports a disconnected command without adding local echo', () => {
+  it('reports a disconnected command without adding local echo', async () => {
     mockPreferencesState.general.localEcho = true;
     const client = new MudClient('example.test', 443);
 
     expect(() => client.sendCommand('look')).not.toThrow();
 
+    // outputStore batches appends into a microtask flush; let it run.
+    await Promise.resolve();
     expect(useOutputStore.getState().entries).toEqual([
       {
         id: 1,

src/createConfiguredClient.test.ts

@@ -224,7 +224,7 @@ describe("createConfiguredClient", () => {
     expect(useUserlistStore.getState().hasReceivedList).toBe(true);
   });
 
-  it("resets session-owned feature state while preserving user history and input", () => {
+  it("resets session-owned feature state while preserving user history and input", async () => {
     client = createConfiguredClient();
     const closeListeners: Array<() => void> = [];
     const stream = {
@@ -297,6 +297,8 @@ describe("createConfiguredClient", () => {
     useOutputStore.getState().addMessage("connection history");
 
     client.close();
+    // History stores batch appends into a microtask flush; let them run.
+    await Promise.resolve();
 
     expect(useCharacterStatusStore.getState().vitals).toBeNull();
     expect(useItemsStore.getState().itemsByLocation).toEqual({});