test: Cypress - Bug 23822 Automation (#25784)
## Description - Added necessary assertions to cover https://github.com/appsmithorg/appsmith/issues/23822 ## Type of change - Automation ## How Has This Been Tested? - Cypress test runs ## Checklist: ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
This commit is contained in:
parent
73fdf61398
commit
da3fd27cc2
|
|
@ -37,7 +37,7 @@ describe("Import and validate older app (app created in older versions of Appsmi
|
||||||
homePage.AssertNCloseImport();
|
homePage.AssertNCloseImport();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("1. Validate merge status", () => {
|
it("1. Validate merge status + Bug23822", () => {
|
||||||
entityExplorer.AssertEntityPresenceInExplorer("ListingAndReviews");
|
entityExplorer.AssertEntityPresenceInExplorer("ListingAndReviews");
|
||||||
//Wait for the app to settle
|
//Wait for the app to settle
|
||||||
agHelper.Sleep(3000);
|
agHelper.Sleep(3000);
|
||||||
|
|
@ -49,11 +49,31 @@ describe("Import and validate older app (app created in older versions of Appsmi
|
||||||
agHelper.AssertElementVisible(gitSync._gitSyncModal);
|
agHelper.AssertElementVisible(gitSync._gitSyncModal);
|
||||||
|
|
||||||
//This is expected due to Canvas Splitting PR changes in v1.9.24
|
//This is expected due to Canvas Splitting PR changes in v1.9.24
|
||||||
|
agHelper.GetNAssertContains(
|
||||||
|
gitSync._gitStatusChanges,
|
||||||
|
/[0-9] page(|s) modified/,
|
||||||
|
);
|
||||||
agHelper.GetNAssertElementText(
|
agHelper.GetNAssertElementText(
|
||||||
gitSync._gitStatusChanges,
|
gitSync._gitStatusChanges,
|
||||||
"4 pages modified",
|
"Application settings modified",
|
||||||
"contain.text",
|
"not.contain.text",
|
||||||
);
|
);
|
||||||
|
agHelper.GetNAssertElementText(
|
||||||
|
gitSync._gitStatusChanges,
|
||||||
|
"Theme modified",
|
||||||
|
"not.contain.text",
|
||||||
|
);
|
||||||
|
agHelper.AssertContains(/[0-9] quer(y|ies) modified/, "not.exist");
|
||||||
|
|
||||||
|
// Commented out due to #25739 - to be fixed by dev later
|
||||||
|
// agHelper.GetNAssertElementText(
|
||||||
|
// gitSync._gitStatusChanges,
|
||||||
|
// "datasource modified",
|
||||||
|
// "not.contain.text",
|
||||||
|
// );
|
||||||
|
|
||||||
|
agHelper.AssertContains(/[0-9] JS Object(|s) modified/, "not.exist");
|
||||||
|
agHelper.AssertContains(/[0-9] librar(y|ies) modified/, "not.exist");
|
||||||
agHelper.GetNAssertElementText(
|
agHelper.GetNAssertElementText(
|
||||||
gitSync._gitStatusChanges,
|
gitSync._gitStatusChanges,
|
||||||
"Some of the changes above are due to an improved file structure designed to reduce merge conflicts. You can safely commit them to your repository.",
|
"Some of the changes above are due to an improved file structure designed to reduce merge conflicts. You can safely commit them to your repository.",
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user