Skip to content

feat(stylus): implement queryPriceFeed, priceFeedExists functions, clean tests #2849

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Jul 18, 2025

Conversation

ayushboss
Copy link
Contributor

Summary

This PR implements the queryPriceFeed and priceFeedExists functions. It also cleans up a lot of the testing logic, which was pretty messy to this point,

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

ayushboss and others added 11 commits July 10, 2025 13:18
…logic

- Change PriceFeedReturn from hex string to byte array ([u8; 32], U64, I32, I64, U64, I64, U64)
- Update parse_price_feed_updates_internal to return Vec<PriceFeedReturn> instead of Vec<([u8; 32], PriceFeedReturn)>
- Simplify parse_price_feed_updates_with_config by removing price_map and using direct indexing with find()
- Simplify update_price_feeds_internal to avoid dual variable iteration and directly index structs
- Update test data functions to use byte arrays from helper functions instead of hex strings
- Add helper functions for different feed IDs used in multiple updates tests

All tests pass successfully with these changes.

Co-Authored-By: [email protected] <[email protected]>
…oStorage

- Add price_id field as StorageFixedBytes<32> to PriceInfoStorage struct
- Rename PriceInfoStorage to PriceFeedStorage for consistency
- Update import statement and storage map declaration in lib.rs
- Update update_price_feeds_internal to populate price_id field when storing price data
- All tests pass with structural changes

Co-Authored-By: [email protected] <[email protected]>
…turn type and the overall price feed return that will be given from the query function
Copy link

vercel bot commented Jul 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 5:24pm
component-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 5:24pm
developer-hub ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 5:24pm
entropy-debugger ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 5:24pm
entropy-explorer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 5:24pm
insights ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 5:24pm
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 5:24pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 5:24pm

Copy link
Collaborator

@ali-behjati ali-behjati left a comment

Choose a reason for hiding this comment

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

Nice!

@ayushboss ayushboss merged commit c505951 into main Jul 18, 2025
9 checks passed
@ayushboss ayushboss deleted the pyth-stylus-final-functions branch July 18, 2025 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants