Skip to content

Conversation

@kyle-ssg
Copy link
Member

@kyle-ssg kyle-ssg commented Feb 3, 2026

Thanks for submitting a PR! Please check the boxes below:

  • I have read the Contributing Guide.
  • I have added information to docs/ if required so people know about the feature.
  • I have filled in the "Changes" section below.
  • I have filled in the "How did you test this code" section below.

Changes

  • Adds change request tests
  • Adds create feature settings tests (tags and archiving)
  • Adds identity override tab checking tests

Related #6562

How did you test this code?

  • Ran tests

@kyle-ssg kyle-ssg requested a review from a team as a code owner February 3, 2026 22:12
@kyle-ssg kyle-ssg requested review from talissoncosta and removed request for a team February 3, 2026 22:12
@vercel
Copy link

vercel bot commented Feb 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview Feb 4, 2026 0:17am
flagsmith-frontend-preview Ignored Ignored Preview Feb 4, 2026 0:17am
flagsmith-frontend-staging Ignored Ignored Preview Feb 4, 2026 0:17am

Request Review

@github-actions github-actions bot added front-end Issue related to the React Front End Dashboard chore labels Feb 3, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-e2e:pr-6652 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-api-test:pr-6652 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-api:pr-6652 Finished ✅ Results
ghcr.io/flagsmith/flagsmith:pr-6652 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-private-cloud:pr-6652 Finished ✅ Results

@kyle-ssg kyle-ssg marked this pull request as draft February 3, 2026 22:17
@kyle-ssg kyle-ssg removed the request for review from talissoncosta February 3, 2026 22:17
@github-actions github-actions bot added chore and removed chore labels Feb 3, 2026
- Add data-test support to TableFilter component
- Add data-test='table-filter-tags' to TableTagFilter
- Replace manual page.locator() calls with helpers (click, clickByText, closeModal)
- Add archived filter verification to flag tests
- Remove redundant scrollIntoViewIfNeeded calls

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Remove all waitForTimeout calls from change-request, project, and versioning tests
- Replace with proper wait helpers (waitForToast, waitForResponse, etc.)
- Add "Use Helpers, Not Raw Page Methods" section to E2E context
- Document all available wait and click helpers
- Emphasize that helpers include proper waiting, error handling, and scrolling

This makes tests more reliable and easier to maintain by using the established helper patterns.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@github-actions github-actions bot added chore and removed chore labels Feb 4, 2026
@github-actions github-actions bot added chore and removed chore labels Feb 4, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  42.1 seconds
commit  ad4088d
info  🖥️ Runner: depot-ubuntu-latest-16
🔄 Run: #14741 (attempt 1)
📋 Test Type: private-cloud

When archiving the only feature, the feature list becomes empty and filters
disappear. Create an additional non-archived feature to ensure filters remain
accessible when testing archived feature visibility.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@github-actions github-actions bot added chore and removed chore labels Feb 4, 2026
@github-actions github-actions bot added chore and removed chore labels Feb 4, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  10 passed

Details

stats  10 tests across 7 suites
duration  52.2 seconds
commit  f9f91a9
info  🖥️ Runner: depot-ubuntu-latest-16
🔄 Run: #14744 (attempt 1)
📋 Test Type: oss

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  10 passed

Details

stats  10 tests across 7 suites
duration  54.9 seconds
commit  f9f91a9
info  🖥️ Runner: depot-ubuntu-latest-arm-16
🔄 Run: #14744 (attempt 1)
📋 Test Type: oss

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  16 passed

Details

stats  16 tests across 13 suites
duration  36.3 seconds
commit  f9f91a9
info  🖥️ Runner: depot-ubuntu-latest-arm-16
🔄 Run: #14744 (attempt 1)
📋 Test Type: private-cloud

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  58.4 seconds
commit  f9f91a9
info  🖥️ Runner: depot-ubuntu-latest-16
🔄 Run: #14744 (attempt 1)
📋 Test Type: private-cloud

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

Labels

chore front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant