Skip to content

Commit ade11bb

Browse files
thymikeethymikee
committed
refactor(ios): keep the prep-ledger register call off the barrel
registerRunnerPrepProcess has no production consumer through runner-xctestrun.ts - the build spawns through it in-process and the stoppers read only the two children selectors. Tests import it from the owning runner-artifact.ts, so the barrel exports only what its real consumers read.
1 parent b6cdd3c commit ade11bb

3 files changed

Lines changed: 4 additions & 3 deletions

File tree

‎packages/platform-apple/src/runner/__tests__/runner-session-close.test.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,8 @@ import {
107107
readRunnerSessionLiveness,
108108
releaseIosRunnerOnClose,
109109
} from '../runner-session.ts';
110-
import { registerRunnerPrepProcess, runnerPrepProcessChildren } from '../runner-xctestrun.ts';
110+
import { registerRunnerPrepProcess } from '../runner-artifact.ts';
111+
import { runnerPrepProcessChildren } from '../runner-xctestrun.ts';
111112

112113
// Test-only stand-in for the daemon's runtime lease-owner-state-dir setter (root-only; the package
113114
// cannot import it). Backs the host.leaseOwnerStateDir() getter the package reads instead.

‎packages/platform-apple/src/runner/__tests__/runner-start-budget.test.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,8 @@ import {
1010
import { IOS_SIMULATOR } from './device-fixtures.ts';
1111
import { appleRunnerTestHost } from '../test-host.ts';
1212
import { raceRunnerStartAgainstCaller } from '../runner-start-budget.ts';
13-
import { registerRunnerPrepProcess, runnerPrepProcessChildren } from '../runner-xctestrun.ts';
13+
import { registerRunnerPrepProcess } from '../runner-artifact.ts';
14+
import { runnerPrepProcessChildren } from '../runner-xctestrun.ts';
1415

1516
const mockSignalPidsBestEffort = vi.fn();
1617
const mockSignalProcessGroupBestEffort = vi.fn();

‎packages/platform-apple/src/runner/runner-xctestrun.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ export {
33
forgetRunnerPrepProcess,
44
hasCachedAppleRunnerArtifact,
55
prepareXctestrunWithEnv,
6-
registerRunnerPrepProcess,
76
runnerPrepProcessChildren,
87
runnerPrepProcessChildrenWithoutActiveOwner,
98
type ExternalXctestRunnerOptions,

0 commit comments

Comments
 (0)