Skip to content

fix: Inline editing flaky test fix #36366

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
yatinappsmith merged 13 commits into from
Sep 20, 2024
Merged

fix: Inline editing flaky test fix #36366

Changes from 3 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
7ab6847
Inline editing fix
sagar-qa007 Sep 17, 2024
71849e9
Merge remote-tracking branch 'origin/release' into fix/inlinefailedcase
sagar-qa007 Sep 17, 2024
fa07816
reverted test case
sagar-qa007 Sep 17, 2024
87630ec
testing another fix
sagar-qa007 Sep 17, 2024
922f300
unused import remove
sagar-qa007 Sep 17, 2024
9c8ae31
Fix the case
sagar-qa007 Sep 18, 2024
b87e5a2
Reverted some changes
sagar-qa007 Sep 18, 2024
19d60c1
Reverted click outside
sagar-qa007 Sep 18, 2024
0b84fd1
revert limited changes
sagar-qa007 Sep 19, 2024
cc41e00
Locator update
sagar-qa007 Sep 19, 2024
eb88d7c
Prettier
sagar-qa007 Sep 19, 2024
c659f76
Ran limited test
sagar-qa007 Sep 19, 2024
32d1be4
reverted changes
sagar-qa007 Sep 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 1 addition & 4 deletions app/client/cypress/e2e/Regression/ClientSide/Widgets/TableV2/Inline_editing_2_spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,6 @@ describe(
"Table widget inline editing functionality",
{ tags: ["@tag.Widget", "@tag.Table"] },
() => {
afterEach(() => {
agHelper.SaveLocalStorageCache();
});

beforeEach(() => {
agHelper.RestoreLocalStorageCache();
agHelper.AddDsl("Table/InlineEditingDSL");
Expand Down Expand Up @@ -161,6 +157,7 @@ describe(
});

it("7. should check if updatedRowIndex is getting updated for multi row update mode", () => {
agHelper.AddDsl("Table/InlineEditingDSL");
Comment thread
sagar-qa007 marked this conversation as resolved.
Outdated
cy.dragAndDropToCanvas("textwidget", { x: 400, y: 400 });
cy.get(".t--widget-textwidget").should("exist");
cy.updateCodeInput(
Expand Down