unit tests

mrc-ide · Aug 31, 2023 · 8add76e · 8add76e
1 parent 543a1ff
commit 8add76e
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 2 deletions.
diff --git a/app/static/src/app/components/options/SensitivityOptions.vue b/app/static/src/app/components/options/SensitivityOptions.vue
@@ -176,6 +176,7 @@ export default defineComponent({
             canDeleteSettings,
             showOptions,
             compileModelMessage,
+            addingParamSettings,
             closeEdit,
             openEdit,
             editSettings,

diff --git a/app/static/tests/unit/components/options/sensitivityOptions.test.ts b/app/static/tests/unit/components/options/sensitivityOptions.test.ts
@@ -322,18 +322,46 @@ describe("SensitivityOptions", () => {
         expect(mockSensitivitySetParamSettings).not.toHaveBeenCalled();
     });
 
-    it("adds param settings when click Add button", async () => {
+    it("opens editor with default new settings when click Add button", async () => {
         const settings = [customSettings, percentSettings];
         const wrapper = getWrapperForMultiSensitivity(settings);
         await wrapper.find("#add-param-settings").trigger("click");
+
+        const editParamSettings = wrapper.findComponent(EditParamSettings);
+        expect(editParamSettings.props("open")).toBe(true);
+        expect(editParamSettings.props("paramSettings")).toStrictEqual({
+            ...defaultSensitivityParamSettings(),
+            parameterToVary: "B"
+        });
+        expect(mockMultiSensitivitySetParamSettings).not.toHaveBeenCalled();
+    });
+
+    it("saves added multi-sensitivity param settings", async () => {
+        const settings = [customSettings, percentSettings];
+        const wrapper = getWrapperForMultiSensitivity(settings);
+        await wrapper.find("#add-param-settings").trigger("click");
+        const editParamSettings = wrapper.findComponent(EditParamSettings);
+        await editParamSettings.vm.$emit("update", { ...rangeSettings });
         expect(mockMultiSensitivitySetParamSettings).toHaveBeenCalledTimes(1);
         // The first parameter not already in the settings should be B
         expect(mockMultiSensitivitySetParamSettings.mock.calls[0][1]).toStrictEqual([
             ...settings,
-            { ...defaultSensitivityParamSettings(), parameterToVary: "B" }
+            rangeSettings
         ]);
     });
 
+    it("can edit setting after closing editor on add", async () => {
+        const settings = [customSettings, percentSettings];
+        const wrapper = getWrapperForMultiSensitivity(settings);
+        await wrapper.find("#add-param-settings").trigger("click");
+        expect((wrapper.vm as any).addingParamSettings).toBe(true);
+        const editParamSettings = wrapper.findComponent(EditParamSettings);
+        editParamSettings.vm.$emit("close");
+        expect((wrapper.vm as any).addingParamSettings).toBe(false);
+        await wrapper.find(".edit-param-settings").trigger("click");
+        expect(editParamSettings.props("paramSettings")).toStrictEqual(customSettings);
+    });
+
     it("deletes param settings when click Delete button", async () => {
         const settings = [customSettings, percentSettings, rangeSettings];
         const wrapper = getWrapperForMultiSensitivity(settings);