hermes-admin/hermes-agent
1
0
Fork 0
Code Issues Pull Requests Actions 28 Packages Projects Releases Wiki Activity

test(desktop): load @hermes/shared once in the connection-registry contract test

Browse Source 
A cold import inside the test body can outlast vitest's 5s per-test timeout
under CI load (seen on main 9df628615c and #121523).
This commit is contained in:
Hermes Agent
2026-09-25 14:02:49 -05:00
committed by brooklyn!
parent a6578fcaa5
commit 4c286ae7a0
1 changed files with 11 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
apps/desktop/electron/connection-registry.test.ts
View File

@@ -45,6 +45,16 @@ import {
upsertConnection
} from './connection-registry'
// Non-literal specifier on purpose: tsconfig.electron.json's project boundary
// excludes apps/shared sources, but vitest resolves the workspace package fine
// at runtime. Loaded once here, not in the test body, where a cold import under
// CI load can outlast the 5s test timeout.
const shared = (await import(String('@hermes/shared'))) as {
backendScopeKey: typeof backendScopeKey
backendScopePrefix: typeof backendScopePrefix
LOCAL_CONNECTION_ID: string
}
function emptyRegistry(): ConnectionRegistry {
return normalizeRegistry(null)
}
@@ -655,16 +665,7 @@ test('uniqueLabel counts up (never "X 2 2") and clamps long candidates', () => {
// the renderer keys its socket registry with the shared copy while the main
// process keys the backend pool with this one. This contract test is the
// enforcement (see the NOTE on backendScopeKey).
test('backendScopeKey: electron and shared implementations agree everywhere', async () => {
// Non-literal specifier on purpose: tsconfig.electron.json's project
// boundary excludes apps/shared sources, but vitest resolves the workspace
// package fine at runtime — which is exactly what this test needs.
const shared = (await import(String('@hermes/shared'))) as {
backendScopeKey: typeof backendScopeKey
backendScopePrefix: typeof backendScopePrefix
LOCAL_CONNECTION_ID: string
}
test('backendScopeKey: electron and shared implementations agree everywhere', () => {
const cases: [null | string | undefined, null | string | undefined][] = [
[null, null],
[undefined, undefined],