Skip to content
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

Push to PRD #248

Merged
merged 15 commits into from
Jan 22, 2025
Merged

Push to PRD #248

merged 15 commits into from
Jan 22, 2025

Conversation

bitbeckers
Copy link
Contributor

  • Filecoin RPC support including authenticated RPC endpoints
  • Fix bug in TakerBid event parsing. Both possible events are now handled correctly
  • Refactor evmClient and chainsauce files to use newly created RpcFactory. This established a single source of truth for chain configs and authentication flows
  • Support schema for creator updates on Sepolia
  • Support for Minter contract on filecoin mainnet

Updates the chainsauce and rpc builder to correctly use the FVM api key
to call on the calibration archive node. The max block range for
requests was reduces to meet the FVM limitations. Additionally, the
filecoin API key was added to constants.
Adds env var to github action so app boots on unit tests. Otherwise
assertExists fails.
Updates the evmClient file for readability and testability. To build the
evmClient and get rpc urls strategies are applies. Additionally, a test
suite was added to validate the implementation and increase test
coverage.
Implements parsing of two possible types of events. Adds test suite for
the event parser method
Increases values to match updates test suite
…not-handled-correctly

Takerbid event parsing
fix(import): missing js extension in import
Adds support for creator updates. The attestation documents files
uploaded on IPFS or links and allow for adding additional data to an
impact claim.
…ions

Add schema for creator updates (Sepolia)
Adds filecoin to the evmClient and chainsauce client constructor.
Updates seed file to index minter contract on filecoin mainnet
When adding another chain, this needed to happen in multiple files. This
is also true for setting a custom fetch client when we need to apss
bearer tokens.

This commit refactors the evmClient and chainsauce builder to rely on a
ChainFactory and RpcFactory to give both clients a consistent config
depending on the selected chain.

Test cases were added and failing tests were updated to respect the new
pattern
Adds support for the fallback provider pattern in viem. Tests updated
accordingly.
Copy link

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 27.2% (🎯 27%) 1680 / 6175
🟢 Statements 27.2% (🎯 27%) 1680 / 6175
🟢 Functions 70.9% (🎯 70%) 39 / 55
🟢 Branches 72.91% (🎯 72%) 105 / 144
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/clients/chainFactory.ts 97.43% 80% 100% 97.43% 37
src/clients/evmClient.ts 100% 100% 100% 100%
src/clients/rpcClientFactory.ts 100% 100% 100% 100%
src/parsing/parseTakerBidEvent.ts 100% 100% 100% 100%
src/utils/constants.ts 100% 100% 100% 100%
Generated in workflow #203 for commit 2ac3d9f by the Vitest Coverage Report Action

@bitbeckers bitbeckers merged commit 8389e4f into main Jan 22, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

1 participant