Title: add initial tests by juruen · Pull Request #14 · github/github-mcp-server · GitHub
Open Graph Title: add initial tests by juruen · Pull Request #14 · github/github-mcp-server
X Title: add initial tests by juruen · Pull Request #14 · github/github-mcp-server
Description: Context First of all, apologies for the large PR. I'm happy to split it into smaller PRs if needed. As we talked off-line, we need to start adding tests to the codebase. Although black-box tests will probably be easier to read and maintain, I wanted to make sure that we have some testing in place, even if it's not the best choice yet. This PR adds tests that are based in testing each one of the individual tools by mocking the GitHub client. We verify that: The tool schema is expected A successful use of the tool by checking that: The expected arguments are passed to the GitHub client The response is parsed and returned A limited number of permutations for those those tools that have optional args A failed use case of the tool We have added a few simple test helper functions to try to simplify the tests a bit.
Open Graph Description: Context First of all, apologies for the large PR. I'm happy to split it into smaller PRs if needed. As we talked off-line, we need to start adding tests to the codebase. Although black-box test...
X Description: Context First of all, apologies for the large PR. I'm happy to split it into smaller PRs if needed. As we talked off-line, we need to start adding tests to the codebase. Although black-box ...
Opengraph URL: https://github.com/github/github-mcp-server/pull/14
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:eedd0040-69e9-6575-1cc8-55561db04692 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | E5D2:3E511B:11C9ED9:186D155:6A61B4F2 |
| html-safe-nonce | 2fd2519b9b95c77c5696e6b998a5c264d25acd7aa156dada0f3cb0c93aea4404 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFNUQyOjNFNTExQjoxMUM5RUQ5OjE4NkQxNTU6NkE2MUI0RjIiLCJ2aXNpdG9yX2lkIjoiMzkwNTkzNTAyMjc0MzQ2NzI1MCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | de30eb41e19d15b1235fa1c607e6f33e6c1b5326256678453d12eb9589c1e97c |
| hovercard-subject-tag | pull_request:2397707141 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/github/github-mcp-server/pull/14/files |
| twitter:image | https://avatars.githubusercontent.com/u/11424?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/11424?s=400&v=4 |
| og:image:alt | Context First of all, apologies for the large PR. I'm happy to split it into smaller PRs if needed. As we talked off-line, we need to start adding tests to the codebase. Although black-box test... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 6f4633bcf01c1ad14b73fd07dd39ac31d61f3d3c2578ee08ec1792b7b351eeb9 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/github/github-mcp-server git https://github.com/github/github-mcp-server.git |
| octolytics-dimension-user_id | 9919 |
| octolytics-dimension-user_login | github |
| octolytics-dimension-repository_id | 942771284 |
| octolytics-dimension-repository_nwo | github/github-mcp-server |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 942771284 |
| octolytics-dimension-repository_network_root_nwo | github/github-mcp-server |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | ac296ae7f21856f1f92adbad22f870b6fbb4b907 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width