Skip to content

Commit

Permalink
test: updated tests for checkboxgroup (#35945)
Browse files Browse the repository at this point in the history
RCA:
Test was failing after removing retry, the issue was it was clicking on
a checkbox which was not required.
the retry actually was bringing back the system to right state by
clicking again.

Solution:
We removed the step that was unnecessary in this case which resolved the
issue
EE: appsmithorg/appsmith-ee#4992

/ok-to-test tags="@tag.Sanity"

<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/10616697878>
> Commit: e599c6d
> <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=10616697878&attempt=1"
target="_blank">Cypress dashboard</a>.
> Tags: `@tag.Sanity`
> Spec:
> <hr>Thu, 29 Aug 2024 14:33:59 UTC
<!-- end of auto-generated comment: Cypress test results  -->

---------

Co-authored-by: karthik <[email protected]>
  • Loading branch information
NandanAnantharamu and karthik authored Aug 29, 2024
1 parent 83326ef commit 67de8c1
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,6 @@ describe(
it("3. handleSelectAllChange: unchecked", function () {
const selectAllSelector = formWidgetsPage.selectAllCheckboxControl;
const uncheckedOptionInputs = `${formWidgetsPage.checkboxGroupOptionInputs} input:not(:checked)`;
// Deselect all
cy.get(selectAllSelector).click();
// Should get 2 unchecked option inputs
cy.get(uncheckedOptionInputs).should("have.length", 2);
//handleSelectAllChange: checked", function () {
Expand Down

0 comments on commit 67de8c1

Please sign in to comment.