Skip to content
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

Jump to bottom

[Task]: Test cases for Anvil Sections and Zones #33739

Closed
marks0351 opened this issue May 27, 2024 · 0 comments · Fixed by #34063
Closed

[Task]: Test cases for Anvil Sections and Zones #33739

marks0351 opened this issue May 27, 2024 · 0 comments · Fixed by #34063
Assignees
@marks0351
Labels
Anvil Pod Issue related to Anvil project Task A simple Todo
Milestone
Anvil test covera...

Comments

@marks0351
Copy link
Contributor

marks0351 commented May 27, 2024 •
edited
Loading

Implement E2E/Integration tests for Anvil Sections and Zones

https://www.notion.so/appsmith/Zones-sections-dc8e35a8d2c549268d034e4a5f79352f
@marks0351 marks0351 added the Anvil Pod Issue related to Anvil project label May 27, 2024
@Nikhil-Nandagopal Nikhil-Nandagopal added the Task A simple Todo label May 27, 2024
@marks0351 marks0351 self-assigned this May 27, 2024
@github-actions github-actions bot mentioned this issue May 27, 2024
Closed
@marks0351 marks0351 added this to the Anvil test coverage - R1 milestone Jun 5, 2024
@marks0351 marks0351 mentioned this issue Jun 7, 2024
Merged
2 tasks
marks0351 added a commit that referenced this issue Jun 10, 2024
@marks0351
chore: Adding specs/tests for space distribution and copy paste sagas (…
99fa93d 
…#34063)

[![workerB](https://img.shields.io/endpoint?url=https%3A%2F%2Fworkerb.linearb.io%2Fv2%2Fbadge%2Fprivate%2FU2FsdGVkX1992iLVivcpoDwVtCrlTUPBIEmtU4nlPs%2Fcollaboration.svg%3FcacheSeconds%3D60)](https://workerb.linearb.io/v2/badge/collaboration-page?magicLinkId=mFGYFxI)
## Description
- Adding additional specs for space distribution and section deletion.
- Adding unit tests for anvil pasting.


Fixes #33739
_or_  
Fixes `Issue URL`
> [!WARNING]  
> _If no issue exists, please create an issue first, and check with the
maintainers if the issue is valid._

## Automation

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

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/9443154667>
> Commit: 4a77067
> Cypress dashboard url: <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=9443154667&attempt=1"
target="_blank">Click here!</a>

<!-- end of auto-generated comment: Cypress test results  -->






## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [ ] No


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **New Features**
- Added new test cases for verifying section removal and visual aspects
of background-less zones in the Anvil layout system.
- Introduced methods to handle mouse events for space distribution
within sections.
- Added mock data generation functionality for widgets, sections, zones,
and layouts.

- **Tests**
- Implemented tests for paste operations in the Anvil layout system,
including various mock functions and scenarios.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@coderabbitai coderabbitai bot mentioned this issue Jul 2, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@marks0351 marks0351

Labels
Anvil Pod Issue related to Anvil project Task A simple Todo
Projects
None yet
Milestone
Anvil test coverage - R1
Development

Successfully merging a pull request may close this issue.

chore: Adding specs/tests for space distribution and copy paste sagas appsmithorg/appsmith
2 participants
@Nikhil-Nandagopal @marks0351