From 593577da4387a71f0f899f1b0cf1a30fb36158de Mon Sep 17 00:00:00 2001 From: Apeksha Bhosale <7846888+ApekshaBhosale@users.noreply.github.com> Date: Mon, 20 May 2024 18:13:12 +0530 Subject: [PATCH] test: WidgetPropertySetters1_spec - lint warning displacement fix (#33582) ## Description In this spec, on hover of lint error line of JS object, error was not shown on the right line but instead was on the left corner at the top. The hunch was the product flakiness but it's a spec problem. Hover on an element used with realTouch is causing the problem. Tooltip gets calculated from top most to the linted line but on CI it gets stuck at top most area when done really fast if realTouch is used. RealTouch is taking incorrect position as it is not finished calculating the position of the tooltip to be displayed is causing the problem i think. Link fore reference https://github.com/dmtrKovalenko/cypress-real-events/pull/15/files EE PR Runs - https://github.com/appsmithorg/appsmith-ee/pull/4204 Fixes #33107 _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="" ### :mag: Cypress test results > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No --- .../ClientSide/SetProperty/WidgetPropertySetters1_spec.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/client/cypress/e2e/Regression/ClientSide/SetProperty/WidgetPropertySetters1_spec.ts b/app/client/cypress/e2e/Regression/ClientSide/SetProperty/WidgetPropertySetters1_spec.ts index 7991b42e99..8ad9dc2ed7 100644 --- a/app/client/cypress/e2e/Regression/ClientSide/SetProperty/WidgetPropertySetters1_spec.ts +++ b/app/client/cypress/e2e/Regression/ClientSide/SetProperty/WidgetPropertySetters1_spec.ts @@ -189,7 +189,7 @@ describe( ); agHelper.AssertElementVisibility(locators._lintErrorElement); - agHelper.HoverElement(locators._lintErrorElement); + agHelper.GetElement(locators._lintErrorElement).realHover(); agHelper.AssertContains( "Direct mutation of widget properties is not supported. Use Button1.setVisibility(value) instead.", );