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.

Sign up for GitHub

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

Jump to bottom

Solana ChainWriter #935

Open
wants to merge 28 commits into
base: develop
Choose a base branch
from
Open

Solana ChainWriter #935

wants to merge 28 commits into from
+4,410 −1,378

Conversation

silaslenihan
Copy link
Contributor

@silaslenihan silaslenihan commented Nov 20, 2024 •
edited by jira bot
Loading

Description

Related Tickets:
NONEVM-753
NONEVM-963
NONEVM-964

Requires Dependencies

Resolves Dependencies

@silaslenihan silaslenihan force-pushed the BCFR-995-Solana-ChainWriter branch from 8c8a858 to 845a9a6 Compare November 20, 2024 15:31
@silaslenihan silaslenihan force-pushed the BCFR-995-Solana-ChainWriter branch from 845a9a6 to 13cf316 Compare November 20, 2024 15:32
@silaslenihan silaslenihan force-pushed the BCFR-995-Solana-ChainWriter branch from 7332617 to 17379b5 Compare November 21, 2024 20:34
@silaslenihan silaslenihan force-pushed the BCFR-995-Solana-ChainWriter branch from 17379b5 to 93cba43 Compare November 21, 2024 20:39
silaslenihan and others added 21 commits December 24, 2024 09:06
@silaslenihan
Cleaned up exec config and added comments
2887537
@silaslenihan
Removed ValueSeeds and consolidated into a single Seeds array
c3b094f
@silaslenihan
Added decode location
d6eb801
@silaslenihan
Added commit report config example
9e56357
@silaslenihan
Slight changes to IDL and codec
c1c3c28
@silaslenihan
Updated ChainWriter implementation to reflect new design changes
ca98a68
@silaslenihan
Added codec implementation
018242f
@silaslenihan
updated CCIP example
9086215
@silaslenihan
Moved lookups logic to separate file
8726452
@silaslenihan
unit tests for lookups
2084871
@silaslenihan
Added utils to their own package
eb78592
@silaslenihan
Updated lookup tests and helpers
8a44f4f
@silaslenihan
Removed helpers_test
593c9f7
@silaslenihan
refactored ccip example
c82c48e
@silaslenihan
Completed chained lookup integration test
7914967
@amit-momin @silaslenihan
ChainWriter unit tests (#948)
ff65ec2 
* Added ChainWriter unit tests for GetFeeComponents and GetTransactionStatus

* Created SubmitTransaction tests

* Created SubmitTransaction tests

* Moved txm utils into own package and generated txm mock

* Updated chain writer tests to use txm mock

* Added GetAddresses unit test and fixed SubmitTransaction unit test

* Fixed linting and removed file read for IDL

* Fixed filter lookup table error case and fixed linting

* Added filter lookup table addresses unit tests

* Added new test case and fixed formatting issues

* Addressed golang lint suggestions

* Cleaned out unused dependency and fixed remaining golang lint errors

* Added derived lookup table indeces unit tests

---------

Co-authored-by: Silas Lenihan <[email protected]>
@silaslenihan
merged with develop
59392e8
@silaslenihan
Removed write_test and moved logic to contract_reader_interface test …
a7d0a2a 
…program
@silaslenihan
addressed feedback comments
dd3321a
@silaslenihan
solved conflicts between TXM changes and unit test changes
d656123
@silaslenihan
updated comments and slight tweaks
51de345
@silaslenihan silaslenihan force-pushed the BCFR-995-Solana-ChainWriter branch from b852c76 to 51de345 Compare December 24, 2024 14:07
@cl-sonarqube-production
Copy link

Quality Gate failed Quality Gate failed

Failed conditions
32.0% Coverage on New Code (required ≥ 75%)

See analysis details on SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amit-momin amit-momin amit-momin left review comments

@ilija42 ilija42 ilija42 left review comments

@Farber98 Farber98 Farber98 left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@silaslenihan @Farber98 @ilija42 @amit-momin @pablolagreca