-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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.
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
test: replace LogintoApp with LoginFromAPI #34040
Conversation
WalkthroughThis update primarily involves renaming a Cypress command from Changes
Sequence Diagram(s)Old Flow vs New FlowOld Flow: Using
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Outside diff range and nitpick comments (4)
app/client/cypress/e2e/Regression/ClientSide/Fork/ForkApplicationInDeployedMode_spec.ts (1)
Line range hint
12-37
: Consider refactoring function expressions to arrow functions for consistency and improved readability.- function () { + () => {app/client/cypress/e2e/Regression/ClientSide/Workspace/LeaveWorkspaceTest_spec.js (1)
Line range hint
13-51
: Consider refactoring function expressions to arrow functions for consistency and improved readability.- function () { + () => {app/client/cypress/e2e/Regression/ClientSide/Workspace/DeleteWorkspace_spec.ts (1)
Line range hint
14-55
: Consider refactoring function expressions to arrow functions for consistency and improved readability.- function () { + () => {app/client/cypress/e2e/Regression/ClientSide/ProductRamps/PrivateEmbedRamp_spec.ts (1)
Line range hint
1-100
: Consider refactoring function expressions to arrow functions for consistency and improved readability.- function () { + () => {
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (8)
- app/client/cypress/e2e/Regression/ClientSide/Fork/ForkApplicationInDeployedMode_spec.ts (1 hunks)
- app/client/cypress/e2e/Regression/ClientSide/Fork/ForkApplication_spec.ts (3 hunks)
- app/client/cypress/e2e/Regression/ClientSide/OtherUIFeatures/ExportApplication_spec.js (5 hunks)
- app/client/cypress/e2e/Regression/ClientSide/ProductRamps/PrivateEmbedRamp_spec.ts (1 hunks)
- app/client/cypress/e2e/Regression/ClientSide/Workspace/DeleteWorkspace_spec.ts (1 hunks)
- app/client/cypress/e2e/Regression/ClientSide/Workspace/LeaveWorkspaceTest_spec.js (1 hunks)
- app/client/cypress/e2e/Regression/ClientSide/Workspace/MemberRoles_Spec.ts (9 hunks)
- app/client/cypress/e2e/Regression/ClientSide/Workspace/ShareAppTests_Spec.ts (5 hunks)
Additional context used
Learnings (1)
app/client/cypress/e2e/Regression/ClientSide/Fork/ForkApplication_spec.ts (1)
User: dipyamanbiswas07 PR: appsmithorg/appsmith#29282 File: app/client/cypress/e2e/Regression/ClientSide/Templates/Fork_Template_spec.js:11-17 Timestamp: 2024-01-09T08:39:59.723Z Learning: The user has clarified that intercepts are called in end-to-end tests, and therefore resetting the feature flag in an `afterEach` hook is not required in their test setup.
Biome
app/client/cypress/e2e/Regression/ClientSide/Fork/ForkApplicationInDeployedMode_spec.ts
[error] 13-36: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 12-37: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.app/client/cypress/e2e/Regression/ClientSide/Workspace/LeaveWorkspaceTest_spec.js
[error] 16-31: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 33-50: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 13-51: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.app/client/cypress/e2e/Regression/ClientSide/Workspace/DeleteWorkspace_spec.ts
[error] 17-27: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 29-54: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 14-55: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.app/client/cypress/e2e/Regression/ClientSide/OtherUIFeatures/ExportApplication_spec.js
[error] 19-52: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 54-90: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 92-120: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 12-121: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.app/client/cypress/e2e/Regression/ClientSide/Workspace/ShareAppTests_Spec.ts
[error] 20-37: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 39-60: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 62-76: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 78-89: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 91-104: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 106-119: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 121-132: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 17-133: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.app/client/cypress/e2e/Regression/ClientSide/Fork/ForkApplication_spec.ts
[error] 62-95: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 114-114: Unsafe usage of optional chaining. (lint/correctness/noUnsafeOptionalChaining)
If it short-circuits with 'undefined' the evaluation will throw TypeError here:
[error] 97-147: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 30-148: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.app/client/cypress/e2e/Regression/ClientSide/Workspace/MemberRoles_Spec.ts
[error] 48-68: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 70-80: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 82-111: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 113-123: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 125-164: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 178-183: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 166-186: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
Additional comments not posted (25)
app/client/cypress/e2e/Regression/ClientSide/Fork/ForkApplicationInDeployedMode_spec.ts (1)
23-23
: Change fromLogintoApp
toLoginFromAPI
aligns with PR objectives.app/client/cypress/e2e/Regression/ClientSide/Workspace/LeaveWorkspaceTest_spec.js (1)
43-43
: Change fromLogintoApp
toLoginFromAPI
aligns with PR objectives.app/client/cypress/e2e/Regression/ClientSide/Workspace/DeleteWorkspace_spec.ts (1)
43-43
: Change fromLogintoApp
toLoginFromAPI
aligns with PR objectives.app/client/cypress/e2e/Regression/ClientSide/ProductRamps/PrivateEmbedRamp_spec.ts (1)
96-96
: Change fromLogintoApp
toLoginFromAPI
aligns with PR objectives.app/client/cypress/e2e/Regression/ClientSide/OtherUIFeatures/ExportApplication_spec.js (4)
34-34
: LGTM! The replacement ofcy.LogintoApp
withcy.LoginFromAPI
aligns with the PR objectives.
55-55
: LGTM! Consistent replacement of the login function as per the PR objectives.
71-71
: LGTM! The functioncy.LoginFromAPI
is used correctly, maintaining consistency across the test suite.
93-93
: LGTM! The consistent use ofcy.LoginFromAPI
across different user roles is noted and aligns with the PR objectives.app/client/cypress/e2e/Regression/ClientSide/Workspace/ShareAppTests_Spec.ts (5)
40-40
: LGTM! The replacement ofcy.LogintoApp
withcy.LoginFromAPI
is consistent with the PR objectives.
63-63
: LGTM! Consistent replacement of the login function as per the PR objectives.
92-92
: LGTM! The functioncy.LoginFromAPI
is used correctly, maintaining consistency across the test suite.
107-107
: LGTM! The consistent use ofcy.LoginFromAPI
across different user roles is noted and aligns with the PR objectives.
122-122
: LGTM! The functioncy.LoginFromAPI
is used correctly, maintaining consistency across the test suite.app/client/cypress/e2e/Regression/ClientSide/Fork/ForkApplication_spec.ts (3)
32-32
: LGTM! The replacement ofcy.LogintoApp
withcy.LoginFromAPI
is consistent with the PR objectives.
53-53
: LGTM! Consistent replacement of the login function as per the PR objectives.
63-63
: LGTM! The functioncy.LoginFromAPI
is used correctly, maintaining consistency across the test suite.app/client/cypress/e2e/Regression/ClientSide/Workspace/MemberRoles_Spec.ts (9)
33-33
: LGTM! The replacement ofcy.LogintoApp
withcy.LoginFromAPI
is consistent with the PR objectives.
49-49
: LGTM! Consistent replacement of the login function as per the PR objectives.
71-71
: LGTM! The functioncy.LoginFromAPI
is used correctly, maintaining consistency across the test suite.
83-83
: LGTM! The consistent use ofcy.LoginFromAPI
across different user roles is noted and aligns with the PR objectives.
114-114
: LGTM! The functioncy.LoginFromAPI
is used correctly, maintaining consistency across the test suite.
126-126
: LGTM! The consistent use ofcy.LoginFromAPI
across different user roles is noted and aligns with the PR objectives.
167-167
: LGTM! The functioncy.LoginFromAPI
is used correctly, maintaining consistency across the test suite.
189-189
: LGTM! The consistent use ofcy.LoginFromAPI
across different user roles is noted and aligns with the PR objectives.
199-199
: LGTM! The functioncy.LoginFromAPI
is used correctly, maintaining consistency across the test suite.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Outside diff range and nitpick comments (2)
app/client/cypress/e2e/Regression/ServerSide/LoginTests/LoginFailure_spec.js (1)
Line range hint
9-45
: Consider converting function expressions to arrow functions for better readability and to adhere to modern JavaScript practices.- describe("Login failure", function () { + describe("Login failure", () => {Also applies to: 8-46
app/client/cypress/e2e/Regression/ClientSide/Github/EnableGithub_spec.ts (1)
Line range hint
14-34
: Consider converting function expressions to arrow functions for better readability and to adhere to modern JavaScript practices.- describe("SSO with Github test functionality", function () { + describe("SSO with Github test functionality", () => {Also applies to: 36-71, 73-117, 13-118
Tools
Biome
[error] 13-118: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (2)
- app/client/cypress/e2e/Regression/ClientSide/Github/EnableGithub_spec.ts (3 hunks)
- app/client/cypress/e2e/Regression/ServerSide/LoginTests/LoginFailure_spec.js (2 hunks)
Additional context used
Biome
app/client/cypress/e2e/Regression/ServerSide/LoginTests/LoginFailure_spec.js
[error] 40-40: The comma operator is disallowed. (lint/style/noCommaOperator)
Its use is often confusing and obscures side effects.
[error] 9-45: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 8-46: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.app/client/cypress/e2e/Regression/ClientSide/Github/EnableGithub_spec.ts
[error] 14-34: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 36-71: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 73-117: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 13-118: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
Additional comments not posted (2)
app/client/cypress/e2e/Regression/ServerSide/LoginTests/LoginFailure_spec.js (1)
27-27
: Replacecy.LoginUser
withcy.LoginFromAPI
aligns with the PR's objectives and improves the login mechanism by using API-based authentication.Also applies to: 39-39
app/client/cypress/e2e/Regression/ClientSide/Github/EnableGithub_spec.ts (1)
16-16
: ReplacehomePage.LogintoApp
withcy.LoginFromAPI
aligns with the PR's objectives and improves the login mechanism by using API-based authentication.Also applies to: 38-38, 74-74
@@ -36,7 +36,7 @@ describe("Login failure", function () { | |||
true, | |||
), | |||
); | |||
cy.LoginUser(Cypress.env("USERNAME"), Cypress.env("PASSWORD"), false); | |||
cy.LoginFromAPI(Cypress.env("USERNAME"), Cypress.env("PASSWORD")); | |||
}), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the comma operator to avoid potential confusion and obscure side effects.
- }),
+ });
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.
}), | |
}); |
Tools
Biome
[error] 40-40: The comma operator is disallowed. (lint/style/noCommaOperator)
Its use is often confusing and obscures side effects.
/ci-test-limit |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/9478636149. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- app/client/cypress/limited-tests.txt (1 hunks)
Additional context used
LanguageTool
app/client/cypress/limited-tests.txt
[uncategorized] ~1-~1: You might be missing the article “the” here. (AI_EN_LECTOR_MISSING_DETERMINER_THE)
Context: ... limited tests - give the spec names in below format: cypress/e2e/Regression/ClientSi...
[uncategorized] ~14-~14: Possible missing article found. (AI_HYDRA_LEO_MISSING_THE)
Context: ...ile to run minimum of specs. Do not run entire suite with this command.
Additional comments not posted (1)
app/client/cypress/limited-tests.txt (1)
2-9
: The updated test spec paths are consistent with the changes described in the PR. Ensure these paths are correctly referenced in the test configurations.
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478636149. |
/ci-test-limit runId=9478636149 |
6 similar comments
/ci-test-limit runId=9478636149 |
/ci-test-limit runId=9478636149 |
/ci-test-limit runId=9478636149 |
/ci-test-limit runId=9478636149 |
/ci-test-limit runId=9478636149 |
/ci-test-limit runId=9478636149 |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478978748. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478974898. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478976529. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478979849. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478984186. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478979192. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478971691. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478978025. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478976737. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478977161. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478976881. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478975272. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9478979476. |
/ci-test-limit |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/9495837400. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- app/client/cypress/support/commands.js (1 hunks)
Additional context used
Biome
app/client/cypress/support/commands.js
[error] 145-149: Change to an optional chain. (lint/complexity/useOptionalChain)
Unsafe fix: Change to an optional chain.
[error] 1088-1116: This function expression can be turned into an arrow function. (lint/complexity/useArrowFunction)
Function expressions that don't use this can be turned into arrow functions.
Safe fix: Use an arrow function instead.
[error] 1430-1430: This default parameter should follow the last required parameter or should be a required parameter. (lint/style/useDefaultParameterLast)
The last required parameter is here:
A default parameter that precedes a required parameter cannot be omitted at call site.
Unsafe fix: Turn the parameter into a required parameter.
Additional comments not posted (1)
app/client/cypress/support/commands.js (1)
186-186
: Verify the necessity of clearing cookies before login.This will help ensure that the approach to handling cookies is consistent across different authentication methods in the test suite.
Verification successful
The output indicates that
cy.clearCookie("SESSION")
is used to clear cookies to avoid stale cookies on Cypress CI, specifically before making a POST request to the login API. This suggests that clearing the SESSION cookie is necessary to ensure a clean state before logging in.
app/client/cypress/support/commands.js:191
:cy.clearCookie("SESSION");
is used right after logging out via API and before visiting the login endpoint.This aligns with the necessity of clearing cookies to prevent stale sessions from interfering with the login process.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Check if other authentication methods also clear cookies. # Test: Search for cookie clearing in other authentication methods. Expect: Consistent usage across methods. rg --type js 'cy.clearCookie'Length of output: 97
Script:
#!/bin/bash # Description: Check where cy.clearCookie("SESSION") is used in the context of authentication. # Search for cy.clearCookie("SESSION") in the context of login or authentication methods. rg --type js 'cy.clearCookie\("SESSION"\)' -A 5 -B 5Length of output: 782
@@ -183,6 +183,7 @@ Cypress.Commands.add("LogintoApp", (uname, pword) => { | |||
}); | |||
|
|||
Cypress.Commands.add("LoginFromAPI", (uname, pword) => { | |||
homePageTS.LogOutviaAPI(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensure proper error handling for the logout operation.
Consider adding error handling around homePageTS.LogOutviaAPI()
to manage cases where the logout fails, which could impact the subsequent login process.
Refactor to improve logout reliability before login.
- homePageTS.LogOutviaAPI();
+ if (CURRENT_REPO === REPO.EE) {
+ homePageTS.LogOutviaAPI();
+ }
This change ensures that the logout via API is only called in the Enterprise Edition, which might have specific requirements or configurations that differ from the Community Edition.
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.
homePageTS.LogOutviaAPI(); | |
if (CURRENT_REPO === REPO.EE) { | |
homePageTS.LogOutviaAPI(); | |
} |
Consider improving the login check logic for better reliability.
- cy.location().should((loc) => {
- expect(loc.href).to.eq(loc.origin + "/applications");
- });
+ cy.location('pathname').should('eq', '/applications');
This change uses Cypress's built-in command for pathname comparison, which is less error-prone and more readable.
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.
homePageTS.LogOutviaAPI(); | |
homePageTS.LogOutviaAPI(); | |
cy.location('pathname').should('eq', '/applications'); |
This replaces LogintoApp with LoginFromAPI
/ok-to-test tags="@tag.All"
EE PR: https://github.com/appsmithorg/appsmith-ee/pull/4440
Summary by CodeRabbit
Bug Fixes
Chores
LoginFromAPI
custom command to improve test reliability.Tip
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9517666274
Commit: a42a585
Cypress dashboard url: Click here!