Fix query tests after schema migration to object-based filter syntax#141
Merged
hotlong merged 3 commits intoJan 25, 2026
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix issue in step 8 of the action run
Fix query tests after schema migration to object-based filter syntax
Jan 25, 2026
hotlong
marked this pull request as ready for review
January 25, 2026 05:17
hotlong
merged commit Jan 25, 2026
03a1654
into
copilot/fix-action-step-issue-another-one
6 checks passed
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the query tests to align with the new object-based filter DSL and the current QuerySchema (notably orderBy and FilterConditionSchema-based joins and havings), resolving the validation failures introduced by the schema migration.
Changes:
- Removed obsolete tests and imports for the now-absent
FilterOperatorandLogicOperatorenums. - Migrated all join
onclauses from array-based syntax toFilterConditionSchemaobject syntax, including support for$andand$fieldreferences. - Migrated all
havingclauses from array-based comparison tuples to object-based filter conditions and updated tests to useorderByinstead ofsortwhere applicable.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Query schema evolved from array-based to object-based filter syntax (MongoDB-style operators), but tests still used old array format causing 40+ validation failures.
Changes
Removed obsolete schema references
FilterOperatorandLogicOperator(removed from schema)Migrated join ON clauses (35 instances)
Migrated HAVING clauses (8 instances)
Fixed field name
sort→orderBy(schema property)Result: 105/105 query tests passing
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.