mrc-2797 Remove calibration plot when user changes calibration options

NEWS.md

@@ -1,3 +1,7 @@
+# hint 1.73.2
+
+* Remove calibration plot when user changes calibration options
+
 # hint 1.73.1
 
 * fix issues with displaying Input Time Series for DRC data

src/app/static/src/app/components/modelCalibrate/ModelCalibrate.vue

@@ -44,7 +44,7 @@
             <span v-translate="'genCalibResults'"></span>
         </div>
         <calibration-results
-            v-if="calibrationPlotGenerated"
+            v-if="calibrationPlotGenerated && complete"
         ></calibration-results>
     </div>
 </template>
@@ -62,13 +62,11 @@
 
     import {
         mapActionByName,
-        mapGetterByName,
         mapMutationByName,
         mapStateProp,
         mapStateProps,
     } from "../../utils";
     import { ModelCalibrateMutation } from "../../store/modelCalibrate/mutations";
-    import { StepDescription } from "../../store/stepper/stepper";
     import { RootState } from "../../root";
     import { Language } from "../../store/translations/locales";
     import { ModelCalibrateState } from "../../store/modelCalibrate/modelCalibrate";
@@ -113,18 +111,18 @@
         },
         computed: {
             ...mapStateProps<ModelCalibrateState, keyof Computed>(namespace, {
-                loading: (s) => s.fetching,
-                calibrating: (s) => s.calibrating,
-                generatingCalibrationPlot: (s) => s.generatingCalibrationPlot,
-                calibrationPlotGenerated: (s) => !!s.calibratePlotResult,
-                error: (s) => s.error,
-                progressMessage: (s) => {
+                loading: (state) => state.fetching,
+                calibrating: (state) => state.calibrating,
+                generatingCalibrationPlot: (state) => state.generatingCalibrationPlot,
+                calibrationPlotGenerated: (state) => !!state.calibratePlotResult,
+                error: (state) => state.error,
+                progressMessage: (state) => {
                     if (
-                        s.status &&
-                        s.status.progress &&
-                        s.status.progress.length > 0
+                        state.status &&
+                        state.status.progress &&
+                        state.status.progress.length > 0
                     ) {
-                        const p = s.status.progress[0]; //This may be either a string or ProgressPhase
+                        const p = state.status.progress[0]; //This may be either a string or ProgressPhase
                         return typeof p == "string"
                             ? p
                             : `${p.name}: ${p.helpText}`;
@@ -179,6 +177,6 @@
         },
         mounted() {
             this.fetchOptions();
-        },
+        }
     });
 </script>

src/app/static/src/app/hintVersion.ts

@@ -1 +1 @@
-export const currentHintVersion = "1.73.1";
+export const currentHintVersion = "1.73.2";

src/app/static/src/tests/components/modelCalibrate/modelCalibrate.test.ts

@@ -1,8 +1,8 @@
-import Vuex, {ActionTree, MutationTree, Store} from "vuex";
+import Vuex, {Store} from "vuex";
 import {mockError, mockModelCalibrateState, mockRootState} from "../../mocks";
 import registerTranslations from "../../../app/store/translations/registerTranslations";
 import {RootState} from "../../../app/root";
-import {ModelCalibrateState} from "../../../app/store/modelCalibrate/modelCalibrate";;
+import {ModelCalibrateState} from "../../../app/store/modelCalibrate/modelCalibrate";
 import {mount, shallowMount} from "@vue/test-utils";
 import ModelCalibrate from "../../../app/components/modelCalibrate/ModelCalibrate.vue";
 import CalibrationResults from "../../../app/components/modelCalibrate/CalibrationResults.vue";
@@ -74,7 +74,7 @@ describe("Model calibrate component", () => {
     it("invokes fetch options action on mount", () => {
         const mockFetch = jest.fn();
         const store = getStore({}, mockFetch);
-        const wrapper = getWrapper(store);
+        getWrapper(store);
         expect(mockFetch.mock.calls.length).toBe(1);
     });
 
@@ -176,11 +176,17 @@ describe("Model calibrate component", () => {
         "Générer des résultats d'étalonnage", "Gerando resultados de calibração", store);
     });
 
-    it("renders calibration plot", () => {
+    it("renders calibration plot if calibration is complete", () => {
         const store = getStore({complete: true, calibratePlotResult: {}});
         const wrapper = getWrapper(store);
         expect(wrapper.find(CalibrationResults).exists()).toBe(true);
         expectTranslated(wrapper.find("#reviewResults"), "(Review results below)",
         "(Consultez les résultats ci-dessous)", "(Analise os resultados abaixo)", store);
     });
+
+    it("it does not render calibration plot if calibration is incomplete", () => {
+        const store = getStore({complete: false, calibratePlotResult: {}});
+        const wrapper = getWrapper(store);
+        expect(wrapper.find(CalibrationResults).exists()).toBe(false);
+    });
 });