From 5e3b211d932fbaca32a89a94c103bff202452251 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sun, 15 Feb 2026 23:41:41 +0000
Subject: [PATCH] perf(test): fold gmail watcher assertions into hooks install
 suite

---
 src/hooks/gmail-watcher.test.ts | 12 ------------
 src/hooks/install.test.ts       | 11 +++++++++++
 2 files changed, 11 insertions(+), 12 deletions(-)
 delete mode 100644 src/hooks/gmail-watcher.test.ts

diff --git a/src/hooks/gmail-watcher.test.ts b/src/hooks/gmail-watcher.test.ts
deleted file mode 100644
index 8fb42247804..00000000000
--- a/src/hooks/gmail-watcher.test.ts
+++ /dev/null
@@ -1,12 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { isAddressInUseError } from "./gmail-watcher.js";
-
-describe("gmail watcher", () => {
-  it("detects address already in use errors", () => {
-    expect(isAddressInUseError("listen tcp 127.0.0.1:8788: bind: address already in use")).toBe(
-      true,
-    );
-    expect(isAddressInUseError("EADDRINUSE: address already in use")).toBe(true);
-    expect(isAddressInUseError("some other error")).toBe(false);
-  });
-});
diff --git a/src/hooks/install.test.ts b/src/hooks/install.test.ts
index 579bff41726..4b862a35769 100644
--- a/src/hooks/install.test.ts
+++ b/src/hooks/install.test.ts
@@ -3,6 +3,7 @@ import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
 import { afterAll, beforeEach, describe, expect, it, vi } from "vitest";
+import { isAddressInUseError } from "./gmail-watcher.js";
 
 const fixtureRoot = path.join(os.tmpdir(), `openclaw-hook-install-${randomUUID()}`);
 let tempDirIndex = 0;
@@ -296,3 +297,13 @@ describe("installHooksFromNpmSpec", () => {
     expect(result.error).toContain("unsupported npm spec");
   });
 });
+
+describe("gmail watcher", () => {
+  it("detects address already in use errors", () => {
+    expect(isAddressInUseError("listen tcp 127.0.0.1:8788: bind: address already in use")).toBe(
+      true,
+    );
+    expect(isAddressInUseError("EADDRINUSE: address already in use")).toBe(true);
+    expect(isAddressInUseError("some other error")).toBe(false);
+  });
+});