From 07f08f6ccbdb0e1c0c98551bcb53a2e10cdd932d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Grand?= <francois.grand@inrae.fr>
Date: Thu, 2 Jun 2022 14:47:07 +0200
Subject: [PATCH 1/2] update jalhyd_branch

refs #547
---
 jalhyd_branch | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/jalhyd_branch b/jalhyd_branch
index d64531f13..f4a6ca09b 100644
--- a/jalhyd_branch
+++ b/jalhyd_branch
@@ -1 +1 @@
-devel
+289-liens-inaccessibles-pour-certains-modules
-- 
GitLab


From a10638d61356232a2d56b71f6a75dc77dfe7095b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Grand?= <francois.grand@inrae.fr>
Date: Wed, 29 Jun 2022 14:27:43 +0200
Subject: [PATCH 2/2] fix: make e2e tests pass after modification setting
 parameters from linked to fixed mode in case of session loading error

see jalhyd#289/adf6009, nghyd#551

refs #547
---
 e2e/compute-reset-chained-links.e2e-spec.ts |  3 --
 e2e/load-linked-params.e2e-spec.ts          | 34 ++++++++++++++-------
 e2e/session/session-cascade-results.json    |  2 +-
 3 files changed, 24 insertions(+), 15 deletions(-)

diff --git a/e2e/compute-reset-chained-links.e2e-spec.ts b/e2e/compute-reset-chained-links.e2e-spec.ts
index 65e2e4e95..cd97f196a 100644
--- a/e2e/compute-reset-chained-links.e2e-spec.ts
+++ b/e2e/compute-reset-chained-links.e2e-spec.ts
@@ -98,9 +98,6 @@ describe("ngHyd − compute then reset chained results − ", () => {
         await browser.sleep(500);
         expect(await navbar.getAllCalculatorTabs().count()).toBe(3);
 
-        // dirty hack : get unwanted module first to prevent Calc button
-        // of PAB Dimensions to be greyed-out @WTF
-        await navbar.clickCalculatorTabForUid("Y2s1cG");
         // 1. get down-most module (PAB Dimensions)
         await navbar.clickCalculatorTabForUid("eHh5YX");
 
diff --git a/e2e/load-linked-params.e2e-spec.ts b/e2e/load-linked-params.e2e-spec.ts
index 00b262f24..dd8253971 100644
--- a/e2e/load-linked-params.e2e-spec.ts
+++ b/e2e/load-linked-params.e2e-spec.ts
@@ -80,13 +80,24 @@ describe("ngHyd − load session with multiple linked parameters − ", () => {
         await browser.sleep(500);
 
         // check target params values
-        const lo_z2 = calcPage.getSelectById("linked_Z2");
-        const lo_z2v = await calcPage.getSelectValueText(lo_z2);
-        expect(lo_z2v).toContain("Cote de fond aval (Macro-rugo.)");
-
-        const lo_l = calcPage.getSelectById("1_linked_L");
-        const lo_lv = await calcPage.getSelectValueText(lo_l);
-        expect(lo_lv).toContain("Largeur au miroir (Sec. param.)");
+        // const lo_z2 = calcPage.getSelectById("linked_Z2");
+        // const lo_z2v = await calcPage.getSelectValueText(lo_z2);
+        // expect(lo_z2v).toContain("Cote de fond aval (Macro-rugo.)");
+
+        /**
+         * depuis qu'on interdit les liens vers des paramètres cachés/liés eux mêmes,
+         * l'appli les remplace par un paramètre en mode fixé (valeur undefined) lors du chargement d'une session
+         * jalhyd#289/adf6009
+         * nghyd#551
+         */
+        const inpZ2 = calcPage.getInputById("Z2");
+        expect(await inpZ2.getAttribute("value")).toEqual("");
+
+        // const lo_l = calcPage.getSelectById("1_linked_L");
+        // const lo_lv = await calcPage.getSelectValueText(lo_l);
+        // expect(lo_lv).toContain("Largeur au miroir (Sec. param.)");
+        const inpL = calcPage.getInputById("1_L");
+        expect(await inpL.getAttribute("value")).toEqual("");
 
         const lo_w = calcPage.getSelectById("2_linked_W");
         const lo_wv = await calcPage.getSelectValueText(lo_w);
@@ -97,9 +108,10 @@ describe("ngHyd − load session with multiple linked parameters − ", () => {
         await browser.sleep(500);
 
         // check target params values
-        const lo_br = calcPage.getSelectById("linked_BR");
-        const lo_brv = await calcPage.getSelectValueText(lo_br);
-        expect(lo_brv).toContain("Largeur au miroir (Sec. param.)");
+        // const lo_br = calcPage.getSelectById("linked_BR");
+        // const lo_brv = await calcPage.getSelectValueText(lo_br);
+        // expect(lo_brv).toContain("Largeur au miroir (Sec. param.)");
+        const inpBR = calcPage.getInputById("BR");
+        expect(await inpBR.getAttribute("value")).toEqual("");
     });
-
 });
diff --git a/e2e/session/session-cascade-results.json b/e2e/session/session-cascade-results.json
index eebf487fc..8602fd4f3 100644
--- a/e2e/session/session-cascade-results.json
+++ b/e2e/session/session-cascade-results.json
@@ -120,7 +120,7 @@
                     "symbol": "Q",
                     "mode": "LINK",
                     "targetNub": "MGg5OH",
-                    "targetParam": "Q_GuideTech"
+                    "targetParam": "Q"
                 },
                 {
                     "symbol": "D",
-- 
GitLab