Skip to content

Query: Fixes HybridSearchQueryTests to account for backend changes in phrase search - #5317

Merged
microsoft-github-policy-service[bot] merged 1 commit into
masterfrom
users/ndeshpan/testFix
Jul 24, 2025
Merged

Query: Fixes HybridSearchQueryTests to account for backend changes in phrase search#5317
microsoft-github-policy-service[bot] merged 1 commit into
masterfrom
users/ndeshpan/testFix

Conversation

@neildsh

@neildsh neildsh commented Jul 24, 2025

Copy link
Copy Markdown
Contributor

Pull Request Template

Description

The most recent emulator release has backend fixes for phrase search. Earlier the search phrase "United States" would match any documents that had the token "United". This was recently fixed so that the backend returns only those documents that have the complete phrase "United States". This change modifies the test expectations accordingly.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@neildsh neildsh added the QUERY label Jul 24, 2025
@microsoft-github-policy-service
microsoft-github-policy-service Bot merged commit 8a0e8e5 into master Jul 24, 2025
28 checks passed
@microsoft-github-policy-service
microsoft-github-policy-service Bot deleted the users/ndeshpan/testFix branch July 24, 2025 23:50
Arooshi Avasthy (aavasthy) pushed a commit that referenced this pull request Aug 2, 2025
… phrase search (#5317)

# Pull Request Template

## Description

The most recent emulator release has backend fixes for phrase search.
Earlier the search phrase "United States" would match any documents that
had the token "United". This was recently fixed so that the backend
returns only those documents that have the complete phrase "United
States". This change modifies the test expectations accordingly.

## Type of change

- [x] Bug fix (non-breaking change which fixes an issue)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Enables automation to merge PRs QUERY

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants