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 enable Batch/GetLatestValue tests #993

Merged
merged 36 commits into from
Feb 7, 2025
Merged

Updated TestChainComponents to enable Batch/GetLatestValue tests #993

merged 36 commits into from
Feb 7, 2025

Conversation

silaslenihan
Copy link
Contributor

@silaslenihan silaslenihan commented Dec 27, 2024 •
edited by amit-momin
Loading

core ref: 461d94af9431de5ee79447ab6972e3d2f64775fe

Description

Enables the chain-agnostic Batch/GetLatestValue interface tests below for Solana

  • ContractReaderGetLatestValueNoArgumentsAndPrimitiveReturnAsValuesDotValue
  • ContractReaderGetLatestValueNoArgumentsAndSliceReturnAsValueDotValue
  • ContractReaderGetLatestValueWithHeadData
  • ContractReaderGetLatestValueWithPrimitiveReturn
  • ContractReaderGetLatestValueNoArgumentsAndSliceReturn
  • ContractReaderBatchGetLatestValueNoArgumentsPrimitiveReturn
  • ContractReaderBatchGetLatestValueNoArgumentsWithSliceReturn
  • ContractReaderGetLatestValueFromMultipleContractsNamesSameFunction
  • ContractReaderBatchGetLatestValueMultipleContractNamesSameFunction
  • ContractReaderGetLatestValueWithModifiersUsingOwnMapstrctureOverrides
  • ContractReaderBatchGetLatestValueWithModifiersOwnMapstructureOverride

Requires Dependencies

#935

@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
@ilija42 ilija42 dismissed stale reviews from reductionista and themself via a3b7251 February 7, 2025 22:30
ilija42
ilija42 previously approved these changes Feb 7, 2025
View reviewed changes
@ilija42 ilija42 requested a review from reductionista February 7, 2025 22:33
reductionista
reductionista previously approved these changes Feb 7, 2025
View reviewed changes
@amit-momin amit-momin dismissed stale reviews from reductionista and ilija42 via 814f258 February 7, 2025 23:20
@ilija42 ilija42 requested a review from reductionista February 7, 2025 23:25
@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

@chris-de-leon-cll chris-de-leon-cll merged commit f1e0ecf into develop Feb 7, 2025
35 of 36 checks passed
@chris-de-leon-cll chris-de-leon-cll deleted the NONEVM-690-Interface-tests branch February 7, 2025 23:50
akuzni2 pushed a commit that referenced this pull request Feb 10, 2025
@silaslenihan @amit-momin
@ilija42 @akuzni2
Updated TestChainComponents to enable Batch/GetLatestValue tests (#993)
626fd97 
* Added Address searcher for decoded data

* Introduced new Solana config

* Completed iteration of ChainWriter config

* Created sample configuration for execute method

* Cleaned up exec config and added comments

* Added commit report config example

* Updated ChainWriter implementation to reflect new design changes

* Added codec implementation

* updated CCIP example

* unit tests for lookups

* Added utils to their own package

* Updated lookup tests and helpers

* Removed helpers_test

* refactored ccip example

* Completed chained lookup integration test

* ChainWriter unit tests  (#948)

* 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]>

* merged with develop

* Removed write_test and moved logic to contract_reader_interface test program

* addressed feedback comments

* solved conflicts between TXM changes and unit test changes

* updated comments and slight tweaks

* Updated PDALookups Seeds field and fixed default accounts

* Updated codec usage

* updated comments and slight tweaks

* Got ChainComponentsTests working with ChainWriter

* Updated interface tests to use unique PDA per test (#1027)

* Updated interface tests to initialize ContractReader and ContractWriter configs with a test index as a PDA seed

* Disabled tests that require a second program

* Updated contract reader test programs to enable interface tests

* Add Test Struct PDA

* Fixed interface tests for the latest ChainWriter changes

* Enabled test struct interface test

* Added store test struct method to secondary contract

* Enabled test struct batch interface test

* Enabled multi contract interface tests

* Removed failing Solana specific tests

* Removed Solana byte string modifier zero-value check

* Fixed rust linting

* Fixed Solana address modifier test

* Fixed rust linting

* Fixed interface tests on loop

---------

Co-authored-by: ilija <[email protected]>

* Disabled interface loop test

* Regenerated ts pnpm lock file with compatible version

* Regenerated example helloworld pnpm lock file with compatible version

* Fix merge conflicts

* Fixed interface tests after merge conflict

---------

Co-authored-by: amit-momin <[email protected]>
Co-authored-by: amit-momin <[email protected]>
Co-authored-by: ilija <[email protected]>
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

@reductionista reductionista reductionista approved these changes

@ilija42 ilija42 ilija42 approved these changes

@chris-de-leon-cll chris-de-leon-cll chris-de-leon-cll approved these changes

@EasterTheBunny EasterTheBunny Awaiting requested review from EasterTheBunny

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 @reductionista @ilija42 @amit-momin @chris-de-leon-cll