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.

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

Behat tests for nested gridfield #389

Closed
GuySartorelli opened this issue May 15, 2024 · 1 comment
Closed

Behat tests for nested gridfield #389

GuySartorelli opened this issue May 15, 2024 · 1 comment

Comments

@GuySartorelli
Copy link
Collaborator

GuySartorelli commented May 15, 2024

#383 recently introduced a new nested gridfield component.

This is quite complex, so it should have some integration tests to ensure it works as expected going forward.

Acceptance criteria

  • Update the basic set up of Gridfiled extensions to include some behat test
  • Behat tests exist to cover the golden path for nested gridfield
    • Basic nested gridfield for hierarchical data, such as Group or SiteTree
    • Double nested gridfield for non-hierarchical relational data (use getNestedConfig())
    • Basic nested gridfield for non-relational data (return ArrayList)
  • Create a follow-up card for additional behat coverage for other components in this module

Possible specific things to test

  • Sorting main gridfield
  • filtering main gridfield
  • Drilling down on a specific record
  • Sort and filter the nested gridfield

Exclude

  • Covering the rest of the module with behat tests

PRs

Follow-up card

@GuySartorelli
Copy link
Collaborator Author

PRs merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants