Market + search tests.#555
Draft
ice-myles wants to merge 4 commits into
Draft
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds coverage around market-related endpoints (trading stats, OHLCV history, subscriptions, trade registration) and community-token search behavior, plus updates a shared test helper so token lookup strings include the contract address (matching production lookup composition).
Changes:
- Add
market_test.gocoveringGetTradingStats,GetOHLVCHistory,SubscribeTradingStats,SubscribeOHLVC, andregisterTrade. - Add
community_tokens_search_test.gocovering keyword search across username/display/ticker/contract address and pagination/case-insensitivity. - Update
helperInsertTestTokenlookup composition to includecontractAddressto support contract-address search.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| token-analytics/market_test.go | New end-to-end-ish tests for QuestDB-backed market stats, OHLCV, subscriptions, and trade registration. |
| token-analytics/community_tokens_search_test.go | New tests for community token keyword search behavior and pagination. |
| token-analytics/community_tokens_by_external_addresses_test.go | Adjust test token lookup fixture to include contract address for realistic search matching. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ice-myles
force-pushed
the
feature/market-and-search-tests
branch
from
f715ddf to
116c44b
Compare
ice-myles
force-pushed
the
feature/market-and-search-tests
branch
from
1ae03cd to
b7066f3
Compare
auto-merge was automatically disabled
February 11, 2026 10:27
Pull request was converted to draft
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.
No description provided.