fix(FN-2402): preserve onboarding stepper done state on completion
- Treat the completion view as an effective final step so all prior indicators/connectors remain marked done - Mark the first-task step as completed before tracking onboarding completion and clear it from skipped state - Replace hardcoded success colors in setup wizard stepper done styles with theme-aware done status tokens - Add regression assertions verifying all step indicators and connectors stay in done state on the completion screen
This commit is contained in:
@@ -2094,6 +2094,15 @@ describe("ModelOnboardingModal", () => {
|
||||
expect(screen.getByText("All Set!")).toBeTruthy();
|
||||
});
|
||||
|
||||
// Completed indicators should remain in done state on the completion screen.
|
||||
const aiSetupIndicator = screen.getByRole("button", { name: "Go back to AI Setup" });
|
||||
const githubIndicator = screen.getByRole("button", { name: "Go back to GitHub" });
|
||||
const firstTaskIndicator = screen.getByRole("button", { name: "Go back to First Task" });
|
||||
expect(aiSetupIndicator).toHaveClass("done");
|
||||
expect(githubIndicator).toHaveClass("done");
|
||||
expect(firstTaskIndicator).toHaveClass("done");
|
||||
expect(document.querySelectorAll(".model-onboarding-step-connector.done")).toHaveLength(2);
|
||||
|
||||
// Click Get Started to close
|
||||
fireEvent.click(screen.getByText("Get Started"));
|
||||
expect(onComplete).toHaveBeenCalled();
|
||||
|
||||
Reference in New Issue
Block a user