Skip to content

Comments

chore: e2e for secrets in argument editor#1838

Open
maxy-shpfy wants to merge 1 commit into02-19-chore_e2e_for_secrets_management_uifrom
02-20-chore_e2e_for_secrets_in_argument_editor
Open

chore: e2e for secrets in argument editor#1838
maxy-shpfy wants to merge 1 commit into02-19-chore_e2e_for_secrets_management_uifrom
02-20-chore_e2e_for_secrets_in_argument_editor

Conversation

@maxy-shpfy
Copy link
Collaborator

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

Description

Added comprehensive end-to-end tests for the secrets in component arguments feature. The test suite verifies the complete workflow from enabling the secrets beta flag, creating secrets, dropping components onto the canvas, assigning secrets to arguments, and preparing pipeline runs.

Enhanced test coverage by adding data-testid attributes to key UI components including the "Use Secret" button, argument input fields, and secret argument inputs to enable reliable test automation.

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 e2e test suite: npm run test:e2e -- secrets-in-arguments.spec.ts
  2. Verify that all test steps pass including:
    • Enabling secrets beta flag
    • Creating a test secret
    • Dropping the Github component onto canvas
    • Assigning the secret to the GITHUB_PAT argument
    • Filling remaining arguments with test data
    • Verifying Submit Run button becomes enabled
  3. Confirm test cleanup removes the created secret

Additional Comments

The test includes a custom Github component fixture that simulates a real-world use case requiring a secret (GitHub Personal Access Token). The test suite uses serial mode to ensure proper state management across test steps and includes proper cleanup to avoid test pollution.

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 01:10
@maxy-shpfy maxy-shpfy requested a review from a team as a code owner February 21, 2026 01:10
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