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

Updated interface tests to use unique PDA per test #1027

Merged
merged 16 commits into from
Feb 7, 2025

Conversation

amit-momin
Copy link
Contributor

No description provided.

@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube

@amit-momin amit-momin merged commit 1fedbab into NONEVM-690-Interface-tests Feb 7, 2025
28 of 32 checks passed
@amit-momin amit-momin deleted the update-pda-usage branch February 7, 2025 19:34
chris-de-leon-cll pushed a commit that referenced this pull request Feb 7, 2025
* 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]>
chris-de-leon-cll pushed a commit that referenced this pull request Feb 8, 2025
* 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

* Implement multireader

* merge lint

* merge lint

* Format contracts

* Regenerated anchor-go

* Fix merge conflicts

* Fixed account initialization issue for interface tests

* Fixed interface tests after merge conflict

* Fix merge conflicts

* Fix merge conflicts

* Format CR test contract

* lint

* Regenerated anchor-go contracts

* Regenerated anchor-go

---------

Co-authored-by: Silas Lenihan <[email protected]>
Co-authored-by: amit-momin <[email protected]>
Co-authored-by: amit-momin <[email protected]>
akuzni2 pushed a commit that referenced this pull request Feb 10, 2025
* 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants