Skip to content

Comments

chore: e2e test for sticky notes#1839

Open
maxy-shpfy wants to merge 1 commit into02-20-chore_e2e_for_secrets_in_argument_editorfrom
02-20-chore_e2e_test_for_sticky_notes
Open

chore: e2e test for sticky notes#1839
maxy-shpfy wants to merge 1 commit into02-20-chore_e2e_for_secrets_in_argument_editorfrom
02-20-chore_e2e_test_for_sticky_notes

Conversation

@maxy-shpfy
Copy link
Collaborator

@maxy-shpfy maxy-shpfy commented Feb 21, 2026

Description

Added comprehensive test identifiers (data-testid attributes) to React Flow components and implemented a complete end-to-end test suite for sticky note functionality. The test identifiers enable reliable automated testing of selection toolbar actions, stacking controls, color picker interactions, and sticky note sidebar items.

Related Issue and Pull requests

Type of Change

  • Bug fix
  • New feature
  • Improvement
  • Cleanup/Refactor
  • Breaking change
  • Documentation update

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

  1. Run the new sticky note e2e test suite: npm run test:e2e -- sticky-note.spec.ts
  2. Verify all selection toolbar buttons are accessible via test IDs
  3. Test stacking controls functionality with multiple sticky notes
  4. Confirm color picker interactions work correctly
  5. Validate YAML export includes sticky note annotations

Additional Comments

The new test suite covers the complete sticky note lifecycle including creation, editing, resizing, moving, color changes, stacking operations, duplication, and YAML serialization. All interactive elements now have proper test identifiers for reliable automation.

Copy link
Collaborator Author

maxy-shpfy commented Feb 21, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@maxy-shpfy maxy-shpfy marked this pull request as ready for review February 21, 2026 05:13
@maxy-shpfy maxy-shpfy requested a review from a team as a code owner February 21, 2026 05:13
@maxy-shpfy maxy-shpfy mentioned this pull request Feb 21, 2026
4 tasks
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.

1 participant