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="" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No
This commit is contained in:
parent
ddefab7306
commit
593577da43
|
|
@ -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.",
|
||||
);
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user