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

[test] Data grid CRUD tests #3646

Merged
merged 11 commits into from
Jun 10, 2024
Merged

[test] Data grid CRUD tests #3646

merged 11 commits into from
Jun 10, 2024

Conversation

Janpot
Copy link
Member

@Janpot Janpot commented Jun 6, 2024 •
edited
Loading

Add tests for the CRUD operations in the data grid. For each of the operations, go through the full flow:

  • create/update/delete item
  • verify availability and disabled state of all possible actions
  • verify snackbar and subsequent filter + add fixes for mixed id types
  • Fix typing issues with forwardRef (just removing it for now)
  • Fix @toolpad/utils description and README

@Janpot Janpot added the test label Jun 6, 2024
@Janpot Janpot changed the title [test] Data grid tests [test] Data grid CRUD tests Jun 6, 2024
@Janpot Janpot marked this pull request as ready for review June 6, 2024 17:35
@Janpot Janpot requested a review from a team June 6, 2024 17:35
@Janpot Janpot enabled auto-merge (squash) June 10, 2024 11:43
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jun 10, 2024
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Jun 10, 2024
@Janpot Janpot merged commit 8321774 into mui:master Jun 10, 2024
13 checks passed
@Janpot Janpot deleted the data-grid-tests branch June 10, 2024 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apedroferreira apedroferreira apedroferreira approved these changes

Assignees
No one assigned
Labels
test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Janpot @apedroferreira