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

Updated TestChainComponents to utilize ChainWriter #993

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

Updated TestChainComponents to utilize ChainWriter #993

wants to merge 25 commits into from
+122 −98

Conversation

silaslenihan
Copy link
Contributor

@silaslenihan silaslenihan commented Dec 27, 2024 •
edited
Loading

Description

Requires Dependencies

#935

Resolves Dependencies

@silaslenihan silaslenihan changed the base branch from develop to BCFR-995-Solana-ChainWriter December 27, 2024 17:04
@silaslenihan silaslenihan force-pushed the BCFR-995-Solana-ChainWriter branch 3 times, most recently from a7bfdec to f6302ac Compare December 27, 2024 17:28
@silaslenihan silaslenihan force-pushed the NONEVM-690-Interface-tests branch from 8602308 to f73928c Compare December 27, 2024 17:33
@silaslenihan silaslenihan force-pushed the NONEVM-690-Interface-tests branch from 8826631 to c2003a1 Compare December 27, 2024 19:21
@silaslenihan silaslenihan force-pushed the NONEVM-690-Interface-tests branch from c2003a1 to 3b1e0c7 Compare December 27, 2024 19:38
silaslenihan and others added 14 commits January 3, 2025 12:19
@silaslenihan
Added utils to their own package
4db5108
@silaslenihan
Updated lookup tests and helpers
b45d396
@silaslenihan
Removed helpers_test
f5a3174
@silaslenihan
refactored ccip example
c53ce3a
@silaslenihan
Completed chained lookup integration test
5a76e2c
@amit-momin @silaslenihan
ChainWriter unit tests (#948)
5b17978 
* 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
f6aa6d9
@silaslenihan
Removed write_test and moved logic to contract_reader_interface test …
0723ba6 
…program
@silaslenihan
addressed feedback comments
f9a3e7e
@silaslenihan
solved conflicts between TXM changes and unit test changes
fea1487
@silaslenihan
updated comments and slight tweaks
8ddccaf
@silaslenihan
Updated PDALookups Seeds field and fixed default accounts
cf148a9
@silaslenihan
Updated codec usage
bfa1f8d
@silaslenihan
updated comments and slight tweaks
52c4ef4
@silaslenihan silaslenihan force-pushed the NONEVM-690-Interface-tests branch from d06a8ee to d07e7f0 Compare January 3, 2025 17:27
@silaslenihan silaslenihan force-pushed the NONEVM-690-Interface-tests branch from d07e7f0 to 5b16318 Compare January 3, 2025 17:29
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@silaslenihan @amit-momin