Skip to content

chore(table): add comprehensive testing infrastructure with Page Object Model#224

Merged
rgah2107 merged 4 commits intomainfrom
chore/add-table-tests
Oct 16, 2025
Merged

chore(table): add comprehensive testing infrastructure with Page Object Model#224
rgah2107 merged 4 commits intomainfrom
chore/add-table-tests

Conversation

@YasielCabrera
Copy link
Contributor

@YasielCabrera YasielCabrera commented Oct 15, 2025

Ticket

https://trello.com/c/7Q389yX6/939-create-default-editor-render-for-components-types-objectsettable

Description

This PR adds comprehensive testing infrastructure for the table module using a Page Object Model (POM) pattern. The implementation includes:

  • Page Object Model Infrastructure: Complete POM setup with base classes and specialized page objects for table interactions
  • Test Utilities: Custom render functions and mock data helpers for consistent test setup
  • Comprehensive Test Suites:
    • Basic rendering tests
    • Cell selection functionality
    • Row actions (edit, delete, custom actions)
    • Sorting behavior and state management
    • Row selection and multi-selection
  • Unified API: Single entry point through TablePage class for all table interactions
  • Mock Data System: Reusable mock data generation for consistent test scenarios

The POM pattern provides a clean, maintainable approach to testing complex table interactions while keeping tests readable and maintainable.

PR Author Checklist:

  • I have performed a self-review of my own code
  • I have performed a self-review of my own chromatic changes and everything is expected
  • I have removed any unnecessary console messages and alerts
  • I have removed any commented code
  • I have checked that there are no dummy or unnecessary comments
  • I have added new test cases (if it applies)
  • I have not introduced any linting issues or warnings

Screenshots (if apply)

Copy link
Contributor

@rgah2107 rgah2107 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rgah2107 rgah2107 merged commit 3087571 into main Oct 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants