Skip to content

fix: update sidebar navigation to use "Runs" link instead of "Test Runs"#651

Open
empiricalrun[bot] wants to merge 2 commits intomainfrom
chat-session_g3u3q3twvLnV
Open

fix: update sidebar navigation to use "Runs" link instead of "Test Runs"#651
empiricalrun[bot] wants to merge 2 commits intomainfrom
chat-session_g3u3q3twvLnV

Conversation

@empiricalrun
Copy link
Copy Markdown
Contributor

@empiricalrun empiricalrun Bot commented Jan 22, 2026

Summary

This PR fixes the failing test "enable/disable environment and verify in test run trigger" in tests/environments.spec.ts.

Problem

The test was failing at line 65 because it could not find the "Test Runs" link in the sidebar:

TimeoutError: locator.click: Timeout 15000ms exceeded.
Call log:
  - waiting for getByRole('link', { name: 'Test Runs' })

Root Cause

The sidebar has been reorganized on the claude/slack-add-sidenav-grouping branch. "Test Runs" is now a section header (generic element), not a link. The actual navigation link to the test runs page is now called "Runs".

From the accessibility tree at time of failure:

- generic:
  - generic: Test Runs    # <-- Section header, NOT a link
  - generic:
    - link "Runs"         # <-- Actual link to /lorem-ipsum/test-runs
    - link "Analytics"
    - link "Issues"
    - link "Snoozes"

Fix

Updated both occurrences in tests/environments.spec.ts:

  • Line 65: Changed getByRole('link', { name: 'Test Runs' })getByRole('link', { name: 'Runs' })
  • Line 96: Changed getByRole('link', { name: 'Test Runs' })getByRole('link', { name: 'Runs' })

Test Results

✅ Test passed after the fix

Report URL: https://reports-r2.empirical.run/test-gen-chat-agent/1769069798595/index.html

Key steps executed successfully:

  • Step 3: locator.click(getByRole('link', { name: 'Runs' })) - 49ms
  • Step 12: locator.click(getByRole('link', { name: 'Runs' })) - 65ms

References

  • Failed Test Run: #45744
  • Diagnosis ID: YDOFUJ0yDIs7
  • Build: claude/slack-add-sidenav-grouping-03hOa

PR created from session #72263

Updated at 2026-01-22 08:17:50.903 UTC

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.

0 participants