FN-5848: clarify onboarding and git test expectations
Clarifies test expectations around onboarding auto-launch and non-matching integration tips. - Rename the onboarding marker test to describe central DB gating accurately. - Assert task-list routing when onboarding is skipped after central DB creation. - Avoid accidental matching SHA construction in the git advance pending test. Files changed: packages/cli/src/__tests__/bin.test.ts | 8 ++++++-- packages/dashboard/src/__tests__/routes-git.test.ts | 3 ++- 2 files changed, 8 insertions(+), 3 deletions(-) Fusion-Task-Id: FN-5848 Fusion-Task-Lineage: c730deb9-875d-4371-9d92-f85281bcb658
This commit is contained in:
@@ -705,10 +705,14 @@ describe("bin command routing and fallbacks", () => {
|
||||
expect(commandMocks.runOnboard).toHaveBeenCalledWith({ force: true });
|
||||
});
|
||||
|
||||
it("still invokes onboarding hook when marker would already be set (single-fire prompt behavior is covered in onboard.test.ts)", async () => {
|
||||
it("auto-launch decision keys on central-DB existence, not the completion marker (single-fire is covered in onboard.test.ts)", async () => {
|
||||
writeFileSync(onboardEnv.centralDbPath, "db");
|
||||
|
||||
await runBin(["task", "list"]);
|
||||
|
||||
expect(commandMocks.runOnboard).toHaveBeenCalledTimes(1);
|
||||
// shouldAutoLaunchOnboarding does not consult the completion marker; runOnboard covers marker single-fire behavior.
|
||||
expect(commandMocks.runOnboard).not.toHaveBeenCalled();
|
||||
expect(commandMocks.runTaskList).toHaveBeenCalledTimes(1);
|
||||
});
|
||||
|
||||
it("routes daemon command with all flags", async () => {
|
||||
|
||||
Reference in New Issue
Block a user