Skip to content

fix(frontend): align failing button-label test with updated UI text - #582

Merged
JuliusHenke merged 2 commits into
masterfrom
copilot/fix-frontend-build-failure
Oct 11, 2026
Merged

JuliusHenke merged 2 commits into
masterfrom
copilot/fix-frontend-build-failure

Conversation

Copilot AI commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

frontend_build / frontend_build was failing because a frontend test still asserted the old button label after the UI copy changed. This PR updates the test expectation to match the current label and removes the stale assertion mismatch causing the workflow failure.

  • Failure source

    • Test was coupled to outdated button text, so CI failed even though runtime behavior was correct.
  • Change made

    • Updated the affected frontend test to query/assert against the new button label.
    • Kept scope limited to test expectations tied to the renamed label.
  • Example

    // before
    expect(screen.getByRole('button', { name: 'Old label' })).toBeInTheDocument();
    
    // after
    expect(screen.getByRole('button', { name: 'New label' })).toBeInTheDocument();

Copilot AI changed the title [WIP] Fix failing GitHub Actions job frontend_build fix(frontend): align failing button-label test with updated UI text Oct 11, 2026
Copilot AI requested a review from JuliusHenke October 11, 2026 00:00
Co-authored-by: JuliusHenke <23460202+JuliusHenke@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit ab3bd3e):

https://tormap-36f44--pr582-copilot-fix-frontend-toxw0f1v.web.app

(expires Tue, 10 Nov 2026 10:25:09 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 0c1f71965de18e60b0e686b0d0356977fec090dc

@JuliusHenke
JuliusHenke marked this pull request as ready for review October 11, 2026 10:49
@JuliusHenke
JuliusHenke merged commit c6ec3de into master Oct 11, 2026
13 checks passed
@JuliusHenke
JuliusHenke deleted the copilot/fix-frontend-build-failure branch October 11, 2026 10:49
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