Skip to content
Merged
Show file tree
Hide file tree
Changes from 73 commits
Commits
Show all changes
76 commits
Select commit Hold shift + click to select a range
1682ff2
removing feature flag for using new Entity Item component from ADS te…
ankitakinger Feb 6, 2025
75b36b7
Merge branch 'release' of https://github.com/appsmithorg/appsmith int…
ankitakinger Feb 6, 2025
304b6d7
fixing failing unit tests
ankitakinger Feb 6, 2025
e068b56
Merge branch 'release' of https://github.com/appsmithorg/appsmith int…
ankitakinger Feb 6, 2025
edbba40
fixing another unit test
ankitakinger Feb 6, 2025
5c13d0e
Merge branch 'release' of https://github.com/appsmithorg/appsmith int…
ankitakinger Feb 7, 2025
345fd93
updating locators
ankitakinger Feb 7, 2025
a5b55a1
fixing unit tests
ankitakinger Feb 7, 2025
a4551fa
updating selected entity item selector
ankitakinger Feb 7, 2025
e320299
fixing some more tests
ankitakinger Feb 7, 2025
672e056
fixing some more tests
ankitakinger Feb 7, 2025
bdc0de9
Merge branch 'release' of https://github.com/appsmithorg/appsmith int…
ankitakinger Feb 11, 2025
0182006
fixing a couple of integration tests
ankitakinger Feb 11, 2025
e864601
minor change
ankitakinger Feb 11, 2025
500002f
Merge branch 'release' of https://github.com/appsmithorg/appsmith int…
ankitakinger Feb 11, 2025
675a44e
fixing more tests
ankitakinger Feb 11, 2025
dc62804
fix for failing tests
ankitakinger Feb 12, 2025
29ad43c
fix for failing tests
ankitakinger Feb 12, 2025
f1452fb
reverting double click changes on entity item
ankitakinger Feb 12, 2025
7e84052
Merge branch 'release' of https://github.com/appsmithorg/appsmith int…
ankitakinger Feb 12, 2025
521dc3e
updating a test
ankitakinger Feb 12, 2025
73bfc57
fixing remaining tests
ankitakinger Feb 12, 2025
76f393a
fixing another set of failing tests
ankitakinger Feb 12, 2025
8acb96f
fixing client build
ankitakinger Feb 12, 2025
78fbcc4
Merge branch 'release' of https://github.com/appsmithorg/appsmith int…
ankitakinger Feb 19, 2025
9bacf69
feat: Add useActiveDoubleClick hook for improved double-click handling
hetunandu Feb 27, 2025
33461e2
Merge branch 'release' of https://github.com/appsmithorg/appsmith int…
ankitakinger Feb 27, 2025
69c15bc
chore: Improvements to useActiveDoubleClick
hetunandu Feb 27, 2025
7aa10e2
Merge branch 'chore/entity-item-double-click' of https://github.com/a…
ankitakinger Feb 27, 2025
2b591c2
Merge branch 'release' into chore/entity-item-double-click
hetunandu Feb 27, 2025
239a945
Merge branch 'chore/entity-item-double-click' of https://github.com/a…
ankitakinger Feb 27, 2025
4e1852e
fixing client build
ankitakinger Feb 27, 2025
6d6a15a
fixing unit test
ankitakinger Feb 27, 2025
a4eefd0
fixing a few tests
ankitakinger Feb 28, 2025
6c232ca
fixing a few tests
ankitakinger Feb 28, 2025
0c51244
fixing some tests
ankitakinger Feb 28, 2025
531a76d
fixing yet another test
ankitakinger Feb 28, 2025
9473f99
and another
ankitakinger Feb 28, 2025
bfa6253
Merge branch 'release' into chore/remove-ff-entity-item-usage
hetunandu Mar 3, 2025
3a8c91b
fix: build issue
hetunandu Mar 3, 2025
e299af4
Merge remote-tracking branch 'origin/release' into chore/remove-ff-en…
hetunandu Mar 3, 2025
d88dfd7
Merge branch 'chore/remove-ff-entity-item-usage' of github.com:/appsm…
hetunandu Mar 3, 2025
13814dc
fix: Null check
hetunandu Mar 3, 2025
ea1bd3e
chore: Fix widget name fisasco adjustment
hetunandu Mar 3, 2025
b02398b
fix: Build issue with type change
hetunandu Mar 3, 2025
5a90ebc
fix: DSL parent ids
hetunandu Mar 3, 2025
dc24d29
fix: Page Element hook usage
hetunandu Mar 4, 2025
2d726be
Merge branch 'release' into chore/remove-ff-entity-item-usage
hetunandu Mar 4, 2025
29711d5
Sync changes from EE excluding enterprise directory
hetunandu Mar 4, 2025
69a3ddc
Merge branch 'release' into chore/remove-ff-entity-item-usage
hetunandu Mar 7, 2025
dabfa7d
Discard changes to app/client/src/ce/pages/AppIDE/components/JSListIt…
hetunandu Mar 7, 2025
e35c689
Discard changes to app/client/src/ce/pages/AppIDE/components/QueryEnt…
hetunandu Mar 7, 2025
b74ccf9
Discard changes to app/client/src/ee/pages/AppIDE/components/JSListIt…
hetunandu Mar 7, 2025
0e996ab
Discard changes to app/client/src/ee/pages/AppIDE/components/QueryEnt…
hetunandu Mar 7, 2025
d9e3bad
Discard changes to app/client/src/pages/AppIDE/components/JSExplorer/…
hetunandu Mar 7, 2025
6344db4
Discard changes to app/client/src/pages/AppIDE/components/PageList/Ol…
hetunandu Mar 7, 2025
042daf7
Discard changes to app/client/src/pages/AppIDE/components/QueryEntity…
hetunandu Mar 7, 2025
77f4b13
Discard changes to app/client/src/pages/AppIDE/components/QueryExplor…
hetunandu Mar 7, 2025
627aafd
Discard changes to app/client/src/pages/AppIDE/components/UIEntityLis…
hetunandu Mar 7, 2025
6eefeb4
Discard changes to app/client/src/pages/AppIDE/components/UIList/UILi…
hetunandu Mar 7, 2025
b39b620
Discard changes to app/client/src/pages/AppIDE/components/UIList/UILi…
hetunandu Mar 7, 2025
f086dc2
Discard changes to app/client/src/pages/Editor/Explorer/Widgets/OldWi…
hetunandu Mar 7, 2025
88cb2f7
Discard changes to app/client/src/pages/Editor/Explorer/Widgets/Widge…
hetunandu Mar 7, 2025
3ea586c
Discard changes to app/client/src/pages/Editor/Explorer/Widgets/Widge…
hetunandu Mar 7, 2025
eb9a9d2
chore: Revert few changes
hetunandu Mar 7, 2025
8908314
Merge branch 'chore/remove-ff-entity-item-usage' of github.com:/appsm…
hetunandu Mar 7, 2025
2435fd1
Discard changes to app/client/src/ce/entities/FeatureFlag.ts
hetunandu Mar 7, 2025
c19af30
enable feature flag in tests
hetunandu Mar 7, 2025
c1a36af
Revert "Discard changes to app/client/src/pages/AppIDE/components/UIL…
hetunandu Mar 7, 2025
22d8d6b
Merge remote-tracking branch 'origin/release' into chore/remove-ff-en…
hetunandu Mar 10, 2025
43e0b73
fix: some locators post new merge
hetunandu Mar 10, 2025
94f0fc5
Merge remote-tracking branch 'origin/release' into chore/remove-ff-en…
hetunandu Mar 10, 2025
0183f03
fix: Delete all queries in DB
hetunandu Mar 10, 2025
68579f6
fix: PageActions_spec
hetunandu Mar 11, 2025
08bb0b9
fix: Unit tests
hetunandu Mar 12, 2025
8a4a200
Merge branch 'release' into chore/remove-ff-entity-item-usage
hetunandu Mar 12, 2025
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
2 changes: 1 addition & 1 deletion app/client/cypress/e2e/Regression/Apps/EchoApiCMS_spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ describe(
cy.get(appPage.closeButton).closest("div").click({ force: true });
PageLeftPane.switchSegment(PagePaneSegment.UI);
PageLeftPane.switchSegment(PagePaneSegment.Queries);
cy.xpath(appPage.postApi).click({ force: true });
cy.get(appPage.postApi).click({ force: true });
cy.ResponseCheck("Test");
// cy.ResponseCheck("Task completed");
cy.ResponseCheck("Curt50@gmail.com");
Comment on lines 96 to 98

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Use structured assertions instead of plain strings.

Direct string assertions should be replaced with more structured expectations.

-cy.ResponseCheck("Test");
-cy.ResponseCheck("Curt50@gmail.com");
+cy.ResponseCheck({
+  subject: "Test",
+  recipient: "Curt50@gmail.com"
+});
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
cy.ResponseCheck("Test");
// cy.ResponseCheck("Task completed");
cy.ResponseCheck("Curt50@gmail.com");
cy.ResponseCheck({
subject: "Test",
recipient: "Curt50@gmail.com"
});
// cy.ResponseCheck("Task completed");

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,8 @@ describe(
"8. No autocomplete for Removed libraries",
{ tags: ["@tag.excludeForAirgap"] },
function () {
AppSidebar.navigate(AppSidebarButton.Editor);
EditorNavigation.SelectEntityByName("Text1Copy", EntityType.Widget);
entityExplorer.RenameEntityFromExplorer("Text1Copy", "UUIDTEXT");
AppSidebar.navigate(AppSidebarButton.Libraries);
installer.uninstallLibrary("uuidjs");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ describe(
prettify: false,
});

jsEditor.EnableDisableAsyncFuncSettings("myFun1", true, false);
jsEditor.EnableDisableAsyncFuncSettings("myFun1", true);

ee.DragDropWidgetNVerify(WIDGET.TEXT, 200, 600);
EditorNavigation.SelectEntityByName("Text1", EntityType.Widget);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -108,11 +108,6 @@ describe(
action: "Rename",
});
cy.EditApiNameFromExplorer("SecondAPI");
cy.xpath(apiwidget.popover)
.last()
.should("be.hidden")
.invoke("show")
.click({ force: true });
ee.ActionContextMenuByEntityName({
entityNameinLeftSidebar: "SecondAPI",
action: "Move to page",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ describe(
toastToValidate: "moved to page",
entityType: EntityItems.Page,
});
EditorNavigation.SelectEntityByName(newPageId, EntityType.Page);
PageList.VerifyIsCurrentPage(newPageId);
PageLeftPane.switchSegment(PagePaneSegment.JS);
PageLeftPane.assertPresence("RenamedJSObjectCopy");
jsEditor.ValidateDefaultJSObjProperties("RenamedJSObjectCopy");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,10 @@ describe(
cy.get(".t--context-menu").click({ force: true });
});
cy.selectAction("Rename");
cy.get(explorer.editEntity).last().type(firstApiName, { force: true });
cy.get(explorer.editEntity)
.last()
.clear()
.type(firstApiName, { force: true });
cy.validateMessage(firstApiName);
agHelper.PressEnter();
entityExplorer.ActionContextMenuByEntityName({
Expand Down Expand Up @@ -98,7 +101,10 @@ describe("Entity Naming conflict test", { tags: ["@tag.IDE"] }, function () {
});
cy.selectAction("Rename");

cy.get(explorer.editEntity).last().type(firstApiName, { force: true });
cy.get(explorer.editEntity)
.last()
.clear()
.type(firstApiName, { force: true });
entityExplorer.ValidateDuplicateMessageToolTip(firstApiName);
cy.get("body").click(0, 0);
cy.wait(2000);
Comment thread
hetunandu marked this conversation as resolved.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ describe(
entityNameinLeftSidebar: "Tab2",
action: "Rename",
});
agHelper.TypeText(locators._entityNameEditing("Tab2"), tabname);
agHelper.TypeText(locators._entityNameEditing, tabname, { clear: true });
agHelper.Sleep(2000);
entityExplorer.ValidateDuplicateMessageToolTip(tabname);
cy.get(explorer.editEntity)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ describe(

gitSync.CreateGitBranch(tempBranch, true);
cy.wait(2000);
EditorNavigation.SelectEntityByName("Page1", EntityType.Page);
PageList.VerifyIsCurrentPage("Page1");
PageLeftPane.switchSegment(PagePaneSegment.JS);
// verify jsObject is not duplicated
PageLeftPane.assertPresence(jsObject);
Expand Down Expand Up @@ -232,8 +232,8 @@ describe(
cy.xpath("//input[@class='bp3-input' and @value='Success']").should(
"be.visible",
);
// switch to Page1 copy and verify jsObject data binding
EditorNavigation.SelectEntityByName("Page1", EntityType.Page);
// switch to Page1 and verify jsObject data binding
PageList.VerifyIsCurrentPage("Page1");
PageLeftPane.switchSegment(PagePaneSegment.JS);
// verify jsObject is not duplicated
PageLeftPane.assertPresence(jsObject);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ describe(
entityExplorer.RenameEntityFromExplorer(
"Page1",
pageName,
false,
true,
EntityItems.Page,
);
PageList.ClonePage(pageName);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ describe(
entityExplorer.RenameEntityFromExplorer(
"Page2",
"ParentPage1",
false,
true,
EntityItems.Page,
);
dataSources.NavigateToDSCreateNew();
Expand All @@ -101,7 +101,7 @@ describe(
entityExplorer.RenameEntityFromExplorer(
"Page2",
"ChildPage1",
false,
true,
EntityItems.Page,
);
dataSources.NavigateToDSCreateNew();
Expand Down Expand Up @@ -132,7 +132,7 @@ describe(
entityExplorer.RenameEntityFromExplorer(
"ParentPage1",
"ParentPageRenamed",
false,
true,
EntityItems.Page,
);
agHelper.RemoveUIElement(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,6 @@ describe("Canvas context Property Pane", { tags: ["@tag.IDE"] }, function () {

cy.get(".t--widget-imagewidget").eq(0).click();
//check if the entities are expanded
cy.get(`[data-guided-tour-id="explorer-entity-Image1"]`).should("exist");
cy.get(`[data-testid="t--entity-item-Image1"]`).should("exist");
});
});
Original file line number Diff line number Diff line change
Expand Up @@ -155,9 +155,14 @@ describe(
agHelper.Sleep();

// Assert context switching works when going back to canvas
EditorNavigation.SelectEntityByName("Page1", EntityType.Page);
cy.get(`div.t--entity-item[data-selected='true']`).should(
"have.length",
1,
);

cy.get(`div[data-testid='t--selected']`).should("have.length", 1);
EditorNavigation.SelectEntityByName("Text1", EntityType.Widget, {}, [
"Container1",
]);
cy.get(".t--property-pane-title").should("contain", "Text1");

// Go back to JS editor
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,10 @@ import {
installer,
draggableWidgets,
} from "../../../../support/Objects/ObjectsCore";
import {
import EditorNavigation, {
AppSidebar,
AppSidebarButton,
EntityType,
PageLeftPane,
PagePaneSegment,
} from "../../../../support/Pages/EditorNavigation";
Expand All @@ -32,6 +33,7 @@ describe(
AppSidebar.navigate(AppSidebarButton.Editor);
entityExplorer.DragDropWidgetNVerify(draggableWidgets.TABLE, 200, 200);
PageLeftPane.switchSegment(PagePaneSegment.UI);
EditorNavigation.SelectEntityByName("Table1", EntityType.Widget);
entityExplorer.RenameEntityFromExplorer("Table1", "jsonwebtoken");
AppSidebar.navigate(AppSidebarButton.Libraries);
installer.OpenInstaller();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,20 @@ describe("Fork application with Jsobjects", {}, function () {
for (let i = 1; i <= 11; i++) {
agHelper.GetNClick(locators._entityTestId(`JS${i}`));
agHelper.FailIfErrorToast("");
agHelper.AssertClassExists(locators._entityTestId(`JS${i}`), "active");
agHelper.AssertAttribute(
locators._entityTestId(`JS${i}`),
"data-selected",
"true",
);
}
for (let i = 12; i <= 17; i++) {
agHelper.GetNClick(locators._entityTestId(`J${i}`));
agHelper.FailIfErrorToast("");
agHelper.AssertClassExists(locators._entityTestId(`J${i}`), "active");
agHelper.AssertAttribute(
locators._entityTestId(`J${i}`),
"data-selected",
"true",
);
}

jsEditor.CreateJSObject('"MiddleName": "Test",\n', {
Expand All @@ -51,9 +59,17 @@ describe("Fork application with Jsobjects", {}, function () {
lineNumber: 5,
});
agHelper.GetNClick(locators._entityTestId("J16"));
agHelper.AssertClassExists(locators._entityTestId("J16"), "active");
agHelper.AssertAttribute(
locators._entityTestId("J16"),
"data-selected",
"true",
);
agHelper.GetNClick(locators._entityTestId("J17"));
agHelper.AssertClassExists(locators._entityTestId("J17"), "active");
agHelper.AssertAttribute(
locators._entityTestId("J17"),
"data-selected",
"true",
);
agHelper.GetNAssertContains(".CodeMirror-line ", '"MiddleName": "Test"');
});
});
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ describe("Slug URLs", { tags: ["@tag.AppUrl"] }, () => {
entityExplorer.RenameEntityFromExplorer(
"Page1",
"Renamed",
false,
true,
EntityItems.Page,
);
assertHelper.AssertNetworkStatus("updatePage");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,15 +24,16 @@ describe("Check Page Actions Menu", {}, function () {
agHelper.GetNClick(locators._contextMenuItem("Rename"));
agHelper.TypeText(propPane._placeholderName, `NewPage{enter}`, {
parseSpecialCharSeq: true,
clear: true,
});

PageList.ClonePage("NewPage");
PageList.HidePage("NewPage Copy");
PageList.ShowList();
agHelper.AssertAttribute(
locators._entityTestId("NewPage Copy"),
"disabled",
"disabled",
"data-disabled",
"true",
);
PageList.DeletePage("NewPage Copy");
PageList.assertAbsence("NewPage Copy");
Expand Down Expand Up @@ -84,6 +85,7 @@ describe("Check Page Actions Menu", {}, function () {
agHelper.GetNClick(locators._contextMenuItem("Rename"));
agHelper.TypeText(propPane._placeholderName, `Page2{enter}`, {
parseSpecialCharSeq: true,
clear: true,
});

PageList.ClonePage("Page2");
Expand All @@ -92,8 +94,8 @@ describe("Check Page Actions Menu", {}, function () {
PageList.ShowList();
agHelper.AssertAttribute(
locators._entityTestId("Page2 Copy"),
"disabled",
"disabled",
"data-disabled",
"true",
);
PageList.DeletePage("Page2 Copy");
PageList.assertAbsence("Page2 Copy");
Expand All @@ -106,15 +108,16 @@ describe("Check Page Actions Menu", {}, function () {
agHelper.GetNClick(locators._contextMenuItem("Rename"));
agHelper.TypeText(propPane._placeholderName, `HomePage{enter}`, {
parseSpecialCharSeq: true,
clear: true,
});

PageList.ClonePage("HomePage");
PageList.HidePage("HomePage Copy");
PageList.ShowList();
agHelper.AssertAttribute(
locators._entityTestId("HomePage Copy"),
"disabled",
"disabled",
"data-disabled",
"true",
);
PageList.DeletePage("HomePage Copy");
PageList.assertAbsence("HomePage Copy");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ describe(
entityExplorer.RenameEntityFromExplorer(
"Page1",
"Home",
false,
true,
entityItems.Page,
);
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -82,15 +82,30 @@ describe(
refactorInput.inputWidget.newName,
);
PageLeftPane.switchSegment(PagePaneSegment.Queries);

EditorNavigation.SelectEntityByName(
refactorInput.query.oldName,
EntityType.Query,
);
entityExplorer.RenameEntityFromExplorer(
refactorInput.query.oldName,
refactorInput.query.newName,
);

EditorNavigation.SelectEntityByName(
refactorInput.api.oldName,
EntityType.Api,
);
entityExplorer.RenameEntityFromExplorer(
refactorInput.api.oldName,
refactorInput.api.newName,
);

PageLeftPane.switchSegment(PagePaneSegment.JS);
EditorNavigation.SelectEntityByName(
refactorInput.jsObject.oldName,
EntityType.JSObject,
);
entityExplorer.RenameEntityFromExplorer(
refactorInput.jsObject.oldName,
refactorInput.jsObject.newName,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ describe(

it("3. Update 'setOptions' property - during onPage load", () => {
EditorNavigation.SelectEntityByName("JSObject1", EntityType.JSObject);
jsEditor.EnableDisableAsyncFuncSettings("myFun1", true, false); //for on page load execution
jsEditor.EnableDisableAsyncFuncSettings("myFun1", true); //for on page load execution
deployMode.DeployApp();
agHelper
.GetText(
Expand Down Expand Up @@ -268,7 +268,7 @@ describe(
])}, () => {showAlert('unable to run API')});
}
}`);
jsEditor.EnableDisableAsyncFuncSettings("myFunc1", true, false); //for on page load execution, since sync function is updated to async
jsEditor.EnableDisableAsyncFuncSettings("myFunc1", true); //for on page load execution, since sync function is updated to async
deployMode.DeployApp();
agHelper.WaitForCondition(
agHelper
Expand Down Expand Up @@ -313,7 +313,7 @@ describe(
Select3.setOptions(Select1.options.concat(Select2.options));
}
}`);
jsEditor.EnableDisableAsyncFuncSettings("myFunc1", true, false); //for on page load execution, since sync function is updated to async
jsEditor.EnableDisableAsyncFuncSettings("myFunc1", true); //for on page load execution, since sync function is updated to async
EditorNavigation.SelectEntityByName("Input1", EntityType.Widget);
propPane.UpdatePropertyFieldValue("Default value", "{{Select3.options}}");
deployMode.DeployApp();
Expand Down Expand Up @@ -355,7 +355,7 @@ describe(
setTimeout(() => {Select1.setOptions(localValue)}, 1000);
}
}`);
jsEditor.EnableDisableAsyncFuncSettings("myFun1", false, false); //for on page load execution
jsEditor.EnableDisableAsyncFuncSettings("myFun1", false); //for on page load execution
deployMode.DeployApp();
agHelper
.GetText(
Expand Down
Loading