Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 19, 2026

Summary

Updates test code to use current dependency injection pattern after codebase refactoring changed from function-based to context-based dependency management.

Why

Tests were failing due to API mismatch after main branch merged significant architectural changes. The original panic fix remains intact, but test infrastructure needed updating.

What changed

  • Updated Test_repositoryResourceContentsHandler_NetworkError to use BaseDeps and ContextWithDeps instead of deprecated function injection
  • Fixed template getter tests to access tmpl.Template.URITemplate (reflects new ServerResourceTemplate structure)
  • Removed obsolete function parameters from test assertions
  • Added missing translations import

MCP impact

  • No tool or API changes

Security / limits

  • No security or limits impact

Tool renaming

  • I am not renaming tools as part of this PR

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: SamMorrowDrums <4811358+SamMorrowDrums@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix panic when fetching resources fails due to network error Update repository resource tests to work with refactored codebase Jan 19, 2026
Copilot AI requested a review from SamMorrowDrums January 19, 2026 22:56
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