#5670 – Update UI tests

epam · Dec 27, 2024 · 6d60724 · 6d60724
1 parent 80717f0
commit 6d60724
Show file tree

Hide file tree

Showing 20 changed files with 9 additions and 8 deletions.
diff --git a/...o-dele-01d9b-e-removing-the-expanded-monomer-and-its-bonds-6-chromium-linux.png b/...o-dele-01d9b-e-removing-the-expanded-monomer-and-its-bonds-6-chromium-linux.png
diff --git a/...o-dele-13832--removing-the-expanded-monomer-and-its-bonds-14-chromium-linux.png b/...o-dele-13832--removing-the-expanded-monomer-and-its-bonds-14-chromium-linux.png
diff --git a/...o-dele-23647-e-removing-the-expanded-monomer-and-its-bonds-8-chromium-linux.png b/...o-dele-23647-e-removing-the-expanded-monomer-and-its-bonds-8-chromium-linux.png
diff --git a/...o-dele-49c75--removing-the-expanded-monomer-and-its-bonds-17-chromium-linux.png b/...o-dele-49c75--removing-the-expanded-monomer-and-its-bonds-17-chromium-linux.png
diff --git a/...o-dele-5d341-e-removing-the-expanded-monomer-and-its-bonds-2-chromium-linux.png b/...o-dele-5d341-e-removing-the-expanded-monomer-and-its-bonds-2-chromium-linux.png
diff --git a/...o-dele-926b8--removing-the-expanded-monomer-and-its-bonds-11-chromium-linux.png b/...o-dele-926b8--removing-the-expanded-monomer-and-its-bonds-11-chromium-linux.png
diff --git a/...o-dele-98ab7--removing-the-expanded-monomer-and-its-bonds-12-chromium-linux.png b/...o-dele-98ab7--removing-the-expanded-monomer-and-its-bonds-12-chromium-linux.png
diff --git a/...o-dele-c21dc-e-removing-the-expanded-monomer-and-its-bonds-9-chromium-linux.png b/...o-dele-c21dc-e-removing-the-expanded-monomer-and-its-bonds-9-chromium-linux.png
diff --git a/...o-dele-ea013-e-removing-the-expanded-monomer-and-its-bonds-5-chromium-linux.png b/...o-dele-ea013-e-removing-the-expanded-monomer-and-its-bonds-5-chromium-linux.png
diff --git a/...o-dele-ec1f0--removing-the-expanded-monomer-and-its-bonds-15-chromium-linux.png b/...o-dele-ec1f0--removing-the-expanded-monomer-and-its-bonds-15-chromium-linux.png
diff --git a/...o-dele-f92a6-e-removing-the-expanded-monomer-and-its-bonds-3-chromium-linux.png b/...o-dele-f92a6-e-removing-the-expanded-monomer-and-its-bonds-3-chromium-linux.png
diff --git a/...o-dele-fec19--removing-the-expanded-monomer-and-its-bonds-18-chromium-linux.png b/...o-dele-fec19--removing-the-expanded-monomer-and-its-bonds-18-chromium-linux.png
diff --git a/...-in-a--851ba-ool-cause-Edit-Abbreviations-dialog-to-appear-4-chromium-linux.png b/...-in-a--851ba-ool-cause-Edit-Abbreviations-dialog-to-appear-4-chromium-linux.png
diff --git a/...-in-a--a98d0-ool-cause-Edit-Abbreviations-dialog-to-appear-6-chromium-linux.png b/...-in-a--a98d0-ool-cause-Edit-Abbreviations-dialog-to-appear-6-chromium-linux.png
diff --git a/...-in-a--b3191-ool-cause-Edit-Abbreviations-dialog-to-appear-2-chromium-linux.png b/...-in-a--b3191-ool-cause-Edit-Abbreviations-dialog-to-appear-2-chromium-linux.png
diff --git a/...-in-a--edd10-ool-cause-Edit-Abbreviations-dialog-to-appear-5-chromium-linux.png b/...-in-a--edd10-ool-cause-Edit-Abbreviations-dialog-to-appear-5-chromium-linux.png
diff --git a/...-in-a--f16d1-ool-cause-Edit-Abbreviations-dialog-to-appear-7-chromium-linux.png b/...-in-a--f16d1-ool-cause-Edit-Abbreviations-dialog-to-appear-7-chromium-linux.png
diff --git a/...-in-a--f7638-ool-cause-Edit-Abbreviations-dialog-to-appear-3-chromium-linux.png b/...-in-a--f7638-ool-cause-Edit-Abbreviations-dialog-to-appear-3-chromium-linux.png
diff --git a/...mircom-2ef95--behaves-correctly-without-breaking-the-chain-2-chromium-linux.png b/...mircom-2ef95--behaves-correctly-without-breaking-the-chain-2-chromium-linux.png
diff --git a/packages/ketcher-core/src/application/editor/actions/sgroup.ts b/packages/ketcher-core/src/application/editor/actions/sgroup.ts
@@ -355,13 +355,17 @@ export function setExpandMonomerSGroup(
     });
   });
 
-  atomsToMove.forEach((atomIds, key) => {
-    if (handledAtoms.has(key)) {
+  atomsToMove.forEach((atomIds) => {
+    const intactAtoms = atomIds.filter((aid) => !handledAtoms.has(aid));
+    if (intactAtoms.length === 0) {
       return;
     }
 
-    const sGroups = sGroupsToMove.get(key) ?? [];
-    const subStructBBox = SGroup.getObjBBox(atomIds, restruct.molecule, true);
+    const subStructBBox = SGroup.getObjBBox(
+      intactAtoms,
+      restruct.molecule,
+      true,
+    );
     const subStructCenter = new Vec2(
       subStructBBox.p0.x + (subStructBBox.p1.x - subStructBBox.p0.x) / 2,
       subStructBBox.p0.y + (subStructBBox.p1.y - subStructBBox.p0.y) / 2,
@@ -378,12 +382,9 @@ export function setExpandMonomerSGroup(
 
     const finalMoveVector = attrs.expanded ? moveVector : moveVector.negated();
 
-    atomIds.forEach((atomId) => {
+    intactAtoms.forEach((atomId) => {
       action.addOp(new AtomMove(atomId, finalMoveVector));
     });
-    sGroups.forEach((sGroupId) => {
-      action.addOp(new SGroupDataMove(sGroupId, finalMoveVector));
-    });
   });
 
   sGroupAtoms.forEach((aid) => {