Skip to content

Fix failing App.test.js after UI migration to OctoFit Tracker - #4

Merged
pmca31 merged 3 commits into
build-octofit-appfrom
copilot/sub-pr-2-again
Feb 11, 2026
Merged

Fix failing App.test.js after UI migration to OctoFit Tracker#4
pmca31 merged 3 commits into
build-octofit-appfrom
copilot/sub-pr-2-again

Conversation

Copilot AI commented Feb 11, 2026

Copy link
Copy Markdown
Contributor

The test file still asserted on the default CRA "learn react" link, causing npm test to fail after the App component was updated to render the OctoFit Tracker UI.

Changes

  • Replaced outdated "learn react" test with two tests matching current UI:
    • Verifies welcome heading renders
    • Verifies navbar presence using getByRole('navigation') for semantic specificity
test('renders OctoFit Tracker navbar', () => {
  render(<App />);
  const navbar = screen.getByRole('navigation');
  expect(navbar).toBeInTheDocument();
  expect(navbar).toHaveTextContent('OctoFit Tracker');
});

All tests now pass.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 11, 2026 19:20
Co-authored-by: pmca31 <6774003+pmca31@users.noreply.github.com>
Co-authored-by: pmca31 <6774003+pmca31@users.noreply.github.com>
Copilot AI changed the title [WIP] Update registration validation based on feedback Fix failing App.test.js after UI migration to OctoFit Tracker Feb 11, 2026
Copilot AI requested a review from pmca31 February 11, 2026 19:24
@pmca31
pmca31 marked this pull request as ready for review February 11, 2026 19:46
@pmca31
pmca31 merged commit ead17e1 into build-octofit-app Feb 11, 2026
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