test(gateway): avoid brittle shutdown timer assertion · openclaw/openclaw@b7232db
hansolo949
·
2026-05-31
·
via Recent Commits to openclaw:main
| Original file line number | Diff line number | Diff line change |
|---|
@@ -937,11 +937,12 @@ describe("createGatewayCloseHandler", () => {
|
937 | 937 | it("fails shutdown when http server close still hangs after force close", async () => { |
938 | 938 | vi.useFakeTimers(); |
939 | 939 | |
| 940 | +const closeAllConnections = vi.fn(); |
940 | 941 | const close = createGatewayCloseHandler( |
941 | 942 | createGatewayCloseTestDeps({ |
942 | 943 | httpServer: { |
943 | 944 | close: () => undefined, |
944 | | -closeAllConnections: vi.fn(), |
| 945 | + closeAllConnections, |
945 | 946 | closeIdleConnections: vi.fn(), |
946 | 947 | } as never, |
947 | 948 | }), |
@@ -953,7 +954,13 @@ describe("createGatewayCloseHandler", () => {
|
953 | 954 | ); |
954 | 955 | await vi.advanceTimersByTimeAsync(HTTP_CLOSE_GRACE_MS + HTTP_CLOSE_FORCE_WAIT_MS); |
955 | 956 | await closeExpectation; |
956 | | -expect(vi.getTimerCount()).toBe(0); |
| 957 | + |
| 958 | +expect(closeAllConnections).toHaveBeenCalledTimes(1); |
| 959 | +expect( |
| 960 | +mocks.logWarn.mock.calls.some(([message]) => |
| 961 | +String(message).includes("http-server close exceeded 1000ms"), |
| 962 | +), |
| 963 | +).toBe(true); |
957 | 964 | }); |
958 | 965 | |
959 | 966 | it("labels warnings for multiple HTTP servers with their index", async () => { |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。
