#4921 - updated screenshots and fix some tests

epam · Jun 28, 2024 · 3b35e92 · 3b35e92
1 parent 450c3dc
commit 3b35e92
Show file tree

Hide file tree

Showing 29 changed files with 39 additions and 42 deletions.
diff --git a/...er-autotests/tests/Macromolecule-editor/Macro-Micro-Switcher/macro-micro-switcher.spec.ts b/...er-autotests/tests/Macromolecule-editor/Macro-Micro-Switcher/macro-micro-switcher.spec.ts
@@ -565,7 +565,6 @@ test.describe('Macro-Micro-Switcher', () => {
       Test fail because we have a bug https://github.com/epam/ketcher/issues/4881
       After fix we need update snapshots.
       */
-      test.fail();
       await openFileAndAddToCanvasMacro(testInfo.fileName, page);
       await turnOnMicromoleculesEditor(page);
       await takeEditorScreenshot(page);
@@ -1483,47 +1482,45 @@ test.describe('Macro-Micro-Switcher', () => {
   ];
 
   for (const data of testData4) {
-    test.fail(
-      `Delete macro structure ${data.description} in micro mode and Undo deletion`,
-      async ({ page }) => {
-        /*
+    test(`Delete macro structure ${data.description} in micro mode and Undo deletion`, async ({
+      page,
+    }) => {
+      /*
       Test case: Macro-Micro-Switcher/#4530
-      Description: The test does not work properly because we have a bug https://github.com/epam/ketcher/issues/4734
-      After fix we need add snapshots and remove test.fail()
+      Description: Macro structure deleted.
       */
-        const x = 750;
-        const y = 370;
-        const firstMonomer = await page.getByText('F1').locator('..');
-        const secondMonomer = await page
-          .getByText(data.monomer)
-          .locator('..')
-          .first();
-        await openFileAndAddToCanvas(
-          'KET/one-attachment-point-added-in-micro-mode.ket',
-          page,
-        );
-        await turnOnMacromoleculesEditor(page);
-        await page.getByTestId('RNA-TAB').click();
-        await page.getByTestId(data.summaryTestId).click();
-        await page.getByTestId(data.monomerTestId).click();
-        await page.mouse.click(x, y);
-        await bondTwoMonomersPointToPoint(
-          page,
-          firstMonomer,
-          secondMonomer,
-          data.bondEndpoints.first,
-          data.bondEndpoints.second,
-        );
-        await turnOnMicromoleculesEditor(page);
-        await selectEraseTool(page);
-        await page.getByText(data.monomer).locator('..').click();
-        await takeEditorScreenshot(page);
-        await waitForRender(page, async () => {
-          await selectTopPanelButton(TopPanelButton.Undo, page);
-        });
-        await takeEditorScreenshot(page);
-      },
-    );
+      const x = 750;
+      const y = 370;
+      const firstMonomer = await page.getByText('F1').locator('..');
+      const secondMonomer = await page
+        .getByText(data.monomer)
+        .locator('..')
+        .first();
+      await openFileAndAddToCanvas(
+        'KET/one-attachment-point-added-in-micro-mode.ket',
+        page,
+      );
+      await turnOnMacromoleculesEditor(page);
+      await page.getByTestId('RNA-TAB').click();
+      await page.getByTestId(data.summaryTestId).click();
+      await page.getByTestId(data.monomerTestId).click();
+      await page.mouse.click(x, y);
+      await bondTwoMonomersPointToPoint(
+        page,
+        firstMonomer,
+        secondMonomer,
+        data.bondEndpoints.first,
+        data.bondEndpoints.second,
+      );
+      await turnOnMicromoleculesEditor(page);
+      await selectEraseTool(page);
+      await page.getByText(data.monomer).click();
+      await takeEditorScreenshot(page);
+      await waitForRender(page, async () => {
+        await selectTopPanelButton(TopPanelButton.Undo, page);
+      });
+      await takeEditorScreenshot(page);
+    });
   }
 
   test('Delete bond between micro structure with attachment point and CHEM in micro mode and Undo deletion', async ({
@@ -1558,7 +1555,7 @@ test.describe('Macro-Micro-Switcher', () => {
     );
     await turnOnMicromoleculesEditor(page);
     await selectEraseTool(page);
-    await page.mouse.click(690, 350);
+    await page.mouse.click(675, 330);
     await takeEditorScreenshot(page);
     await selectTopPanelButton(TopPanelButton.Undo, page);
     await takeEditorScreenshot(page);
@@ -1596,7 +1593,7 @@ test.describe('Macro-Micro-Switcher', () => {
     );
     await turnOnMicromoleculesEditor(page);
     await selectDropdownTool(page, 'bonds', 'bond-double');
-    await page.mouse.click(690, 350);
+    await page.mouse.click(675, 330);
     await takeEditorScreenshot(page);
   });
 

diff --git a/...ing-Layout-button-not-erase-all-macromolecules-from-canvas-1-chromium-linux.png b/...ing-Layout-button-not-erase-all-macromolecules-from-canvas-1-chromium-linux.png
diff --git a/...nt-poi-e88ec-as-attachment-point-when-switch-to-macro-mode-2-chromium-linux.png b/...nt-poi-e88ec-as-attachment-point-when-switch-to-macro-mode-2-chromium-linux.png
diff --git a/...t-poss-5053b-tween-monomer-and-micromolecule-in-micro-mode-1-chromium-linux.png b/...t-poss-5053b-tween-monomer-and-micromolecule-in-micro-mode-1-chromium-linux.png
diff --git a/...-attac-2ff17-as-attachment-point-when-switch-to-macro-mode-2-chromium-linux.png b/...-attac-2ff17-as-attachment-point-when-switch-to-macro-mode-2-chromium-linux.png
diff --git a/...g-betw-13aa9-A-with-modified-monomer-with-modyfied-monomer-2-chromium-linux.png b/...g-betw-13aa9-A-with-modified-monomer-with-modyfied-monomer-2-chromium-linux.png
diff --git a/...g-betw-3653d-A-with-modified-monomer-with-modyfied-monomer-2-chromium-linux.png b/...g-betw-3653d-A-with-modified-monomer-with-modyfied-monomer-2-chromium-linux.png
diff --git a/...icro-structure-with-attachment-point-to-Base-in-macro-mode-1-chromium-linux.png b/...icro-structure-with-attachment-point-to-Base-in-macro-mode-1-chromium-linux.png
diff --git a/...icro-structure-with-attachment-point-to-CHEM-in-macro-mode-1-chromium-linux.png b/...icro-structure-with-attachment-point-to-CHEM-in-macro-mode-1-chromium-linux.png
diff --git a/...structure-with-attachment-point-to-Phosphate-in-macro-mode-1-chromium-linux.png b/...structure-with-attachment-point-to-Phosphate-in-macro-mode-1-chromium-linux.png
diff --git a/...cro-structure-with-attachment-point-to-Sugar-in-macro-mode-1-chromium-linux.png b/...cro-structure-with-attachment-point-to-Sugar-in-macro-mode-1-chromium-linux.png
diff --git a/...cule-t-e6098-el-and-it-remain-back-after-delete-connection-3-chromium-linux.png b/...cule-t-e6098-el-and-it-remain-back-after-delete-connection-3-chromium-linux.png
diff --git a/...-micro-4aff7-and-Phosphate-in-macro-mode-and-Undo-deletion-2-chromium-linux.png b/...-micro-4aff7-and-Phosphate-in-macro-mode-and-Undo-deletion-2-chromium-linux.png
diff --git a/...-micro-5521a-oint-and-Base-in-macro-mode-and-Undo-deletion-1-chromium-linux.png b/...-micro-5521a-oint-and-Base-in-macro-mode-and-Undo-deletion-1-chromium-linux.png
diff --git a/...-micro-6c7c2-oint-and-CHEM-in-macro-mode-and-Undo-deletion-2-chromium-linux.png b/...-micro-6c7c2-oint-and-CHEM-in-macro-mode-and-Undo-deletion-2-chromium-linux.png
diff --git a/...-micro-8e03f-oint-and-CHEM-in-macro-mode-and-Undo-deletion-1-chromium-linux.png b/...-micro-8e03f-oint-and-CHEM-in-macro-mode-and-Undo-deletion-1-chromium-linux.png
diff --git a/...-micro-97674-int-and-Sugar-in-macro-mode-and-Undo-deletion-1-chromium-linux.png b/...-micro-97674-int-and-Sugar-in-macro-mode-and-Undo-deletion-1-chromium-linux.png
diff --git a/...-micro-9963d-int-and-Sugar-in-macro-mode-and-Undo-deletion-2-chromium-linux.png b/...-micro-9963d-int-and-Sugar-in-macro-mode-and-Undo-deletion-2-chromium-linux.png
diff --git a/...-micro-d15d5-and-Phosphate-in-macro-mode-and-Undo-deletion-1-chromium-linux.png b/...-micro-d15d5-and-Phosphate-in-macro-mode-and-Undo-deletion-1-chromium-linux.png
diff --git a/...-micro-d4be6-oint-and-CHEM-in-micro-mode-and-Undo-deletion-2-chromium-linux.png b/...-micro-d4be6-oint-and-CHEM-in-micro-mode-and-Undo-deletion-2-chromium-linux.png
diff --git a/...-micro-ea7ee-oint-and-Base-in-macro-mode-and-Undo-deletion-2-chromium-linux.png b/...-micro-ea7ee-oint-and-Base-in-macro-mode-and-Undo-deletion-2-chromium-linux.png
diff --git a/...-micro-ece7d-oint-and-CHEM-in-micro-mode-and-Undo-deletion-1-chromium-linux.png b/...-micro-ece7d-oint-and-CHEM-in-micro-mode-and-Undo-deletion-1-chromium-linux.png
diff --git a/...elete-macro-structure-Base-in-micro-mode-and-Undo-deletion-1-chromium-linux.png b/...elete-macro-structure-Base-in-micro-mode-and-Undo-deletion-1-chromium-linux.png
diff --git a/...elete-macro-structure-Base-in-micro-mode-and-Undo-deletion-2-chromium-linux.png b/...elete-macro-structure-Base-in-micro-mode-and-Undo-deletion-2-chromium-linux.png
diff --git a/...-macro-structure-Phosphate-in-micro-mode-and-Undo-deletion-1-chromium-linux.png b/...-macro-structure-Phosphate-in-micro-mode-and-Undo-deletion-1-chromium-linux.png
diff --git a/...-macro-structure-Phosphate-in-micro-mode-and-Undo-deletion-2-chromium-linux.png b/...-macro-structure-Phosphate-in-micro-mode-and-Undo-deletion-2-chromium-linux.png
diff --git a/...lete-macro-structure-Sugar-in-micro-mode-and-Undo-deletion-1-chromium-linux.png b/...lete-macro-structure-Sugar-in-micro-mode-and-Undo-deletion-1-chromium-linux.png
diff --git a/...lete-macro-structure-Sugar-in-micro-mode-and-Undo-deletion-2-chromium-linux.png b/...lete-macro-structure-Sugar-in-micro-mode-and-Undo-deletion-2-chromium-linux.png
diff --git a/...fy-attachment-points-can-be-added-removed-via-context-menu-3-chromium-linux.png b/...fy-attachment-points-can-be-added-removed-via-context-menu-3-chromium-linux.png