fix(FN-1690): update test expectations for experimentalFeatures merge behavior
- Fix store.test.ts to expect experimentalFeatures merging behavior (not replacement) - Update SettingsModal.test.tsx nav item count from 14 to 15 for Experimental Features section - Note: RoadmapStore implementation was already merged to main via previous squash
This commit is contained in:
@@ -1199,7 +1199,7 @@ describe("TaskStore", () => {
|
||||
expect(settings.experimentalFeatures).toEqual({ "my-feature": false });
|
||||
});
|
||||
|
||||
it("can add a new experimental feature (replaces entire object)", async () => {
|
||||
it("can add a new experimental feature (merges with existing)", async () => {
|
||||
await store.updateSettings({
|
||||
experimentalFeatures: { "feature-a": true },
|
||||
});
|
||||
@@ -1209,9 +1209,8 @@ describe("TaskStore", () => {
|
||||
});
|
||||
|
||||
const settings = await store.getSettings();
|
||||
// Note: updateSettings replaces experimentalFeatures entirely, not merged
|
||||
// To preserve existing features, pass all features in a single update
|
||||
expect(settings.experimentalFeatures).toEqual({ "feature-b": true });
|
||||
// Note: updateSettings merges experimentalFeatures, preserving existing features
|
||||
expect(settings.experimentalFeatures).toEqual({ "feature-a": true, "feature-b": true });
|
||||
});
|
||||
|
||||
it("can remove an experimental feature by setting it to null (selective removal)", async () => {
|
||||
|
||||
@@ -1694,7 +1694,7 @@ describe("SettingsModal", () => {
|
||||
expect(layout!.querySelector(".settings-content")).toBeTruthy();
|
||||
});
|
||||
|
||||
it("has .settings-sidebar with 14 .settings-nav-item buttons for all sections", async () => {
|
||||
it("has .settings-sidebar with 15 .settings-nav-item buttons for all sections", async () => {
|
||||
const { container } = render(<SettingsModal onClose={onClose} addToast={addToast} />);
|
||||
await waitFor(() => expect(fetchSettings).toHaveBeenCalled());
|
||||
|
||||
|
||||
Reference in New Issue
Block a user