assert query params in search_test#97

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

juruen merged 4 commits intomainfromjuruen/search_test

Apr 4, 2025

Merged

assert query params in search_test#97

juruen merged 4 commits intomainfromjuruen/search_test

Apr 4, 2025

Conversation

Copy link

Collaborator

juruen commentedApr 4, 2025

Context

Follow-up PR to#96 to continue improving our test coverage. These changes assert that the request parameters and body are correctly handled in thesearch_test file.

assert query params

e041974

CopilotAI review requested due to automatic review settings

April 4, 2025 10:49

CopilotAI reviewed

Apr 4, 2025

View reviewed changes

Copy link

Contributor

CopilotAI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Pull Request Overview

This PR enhances test coverage in the search_test file by asserting that query parameters and the request body are correctly handled.

Updated tests for repository, code, and user search to use WithRequestMatchHandler with an expectQueryParams check.
Adjusted expected pagination parameters in the repository search tests.

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon.Learn more