test(solana): migrate to local/offline containerized node #2187

mj-blockydevs · 2024-08-06T11:53:26Z

No description provided.

Add Dockerfile to run local solana validator for tests

unignore test test_solana_and_spl_balance_enable_spl_v2

Signed-off-by: Mariusz Jasuwienas <[email protected]>

…cal node Signed-off-by: Mariusz Jasuwienas <[email protected]>

This reverts commit 4277284

…provements-local

…cal node. Non-dockerised tests (out of solana - scope) excluded. Signed-off-by: Mariusz Jasuwienas <[email protected]>

mm2src/mm2_main/tests/docker_tests/docker_tests_common.rs

onur-ozkan · 2024-08-07T04:49:49Z

mm2src/mm2_main/tests/docker_tests/docker_tests_common.rs

+fn find_container_id_by_port(port: u16) -> Option<String> {
+    let output = Command::new("docker")
+        .args(["ps", "--filter", &format!("publish={}", port), "--format", "{{.ID}}"])
+        .output()
+        .expect("failed to execute docker ps");
+
+    let container_id = str::from_utf8(&output.stdout).ok()?.trim().to_string();
+    if container_id.is_empty() {
+        None
+    } else {
+        Some(container_id)
+    }
+}


Could you clarify why is this needed? Once the container is started we have DockerNode type that contains the container informations.

It technically is not. We could pass the DockerNode variable here.

onur-ozkan · 2024-08-07T04:52:29Z

mm2src/mm2_main/tests/docker_tests/solana_tests.rs

+#[test]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_keypair_from_secp_dockerized() { solana_keypair_from_secp(); }
+
+#[test]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_transaction_simulations_dockerized() { solana_transaction_simulations(); }
+
+#[test]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_transaction_zero_balance_dockerized() { solana_transaction_zero_balance(); }
+
+#[test]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_transaction_simulations_not_enough_for_fees_dockerized() {
+    solana_transaction_simulations_not_enough_for_fees();
+}
+
+#[test]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_transaction_simulations_max_dockerized() { solana_transaction_simulations_not_enough_for_fees(); }
+
+#[test]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_test_transactions_dockerized() { solana_test_transactions(); }
+
+// This test is just a unit test for brainstorming around tx_history for base_coin.
+#[test]
+#[ignore]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_test_tx_history_dockerized() { solana_test_transactions(); }
+
+#[test]
+fn solana_coin_send_and_refund_maker_payment_dockerized() { solana_coin_send_and_refund_maker_payment(); }
+
+#[test]
+fn solana_coin_send_and_spend_maker_payment_dockerized() { solana_coin_send_and_spend_maker_payment(); }


Is it possible to move those tests here instead of wrapping them with this functions?

Hello, @onur-ozkan I've moved the tests here, as you asked.

onur-ozkan · 2024-08-07T04:54:41Z

mm2src/mm2_main/tests/docker_tests/docker_tests_common.rs

@@ -113,7 +115,7 @@ pub const GETH_DOCKER_IMAGE_WITH_TAG: &str = "docker.io/ethereum/client-go:stabl
 pub const NUCLEUS_IMAGE: &str = "docker.io/komodoofficial/nucleusd";
 pub const ATOM_IMAGE: &str = "docker.io/komodoofficial/gaiad";
 pub const IBC_RELAYER_IMAGE: &str = "docker.io/komodoofficial/ibc-relayer";
-
+pub const SOLANA_CLUSTER_DOCKER_IMAGE: &str = "docker.io/0xbdj/solana-node-test";


We probably need to push this image under https://hub.docker.com/u/komodoofficial org.

Yes, that is a good idea. However, I'll complete this PR first to ensure that the expected final version is pushed there.

laruh

Thank you for your work!
I have one comment for now

laruh · 2024-08-07T07:56:17Z

mm2src/coins/solana/solana_common_tests.rs

may be move solana_tests and solana_common_tests files to /mm2_main/tests/mm2_tests?
It looks like there is too much test code in mm2src space

It's similar to this comment #2187 (comment)
They should be moved to mm2_main/tests/docker_tests/solana_tests.rs, spl_tests should also be moved to a new spl_tests.rs file in docker tests.

Ok, can you check it now, please @shamardy ?

shamardy

Thanks for the PR, we are moving in the right direction! Only one note from me for now as @onur-ozkan and @laruh covered my other note about moving tests.

shamardy · 2024-08-07T20:21:28Z

.docker/solana-accounts/mint.sh

+spl-token mint "$USDC_TOKEN" 100000
+spl-token mint "$ADEX_TOKEN" 100000
+# Iterate over each account file in the directory
+for ACCOUNT_FILE in "$ACCOUNTS_DIR"/*.json; do


I am fine with having multiple accounts for users created and funded while starting the docker node, but this will cause problems for you in the future while implementing HD wallet since there are multiple addresses/accounts per HD wallet and you will need to update the docker image for those tests. A better pattern is to have one distributor account and this sends coins/tokens to addresses at the start of each test similar to how it's done for ethereum and other coins that use dockerized nodes

komodo-defi-framework/mm2src/mm2_main/tests/docker_tests/eth_docker_tests.rs

Line 129 in 9b01339

pub fn fill_eth(to_addr: Address, amount: U256) {

komodo-defi-framework/mm2src/mm2_main/tests/docker_tests/eth_docker_tests.rs

Line 149 in 9b01339

fn fill_erc20(to_addr: Address, amount: U256) {

It's up to you if you want to handle this now or later, but handling it here is the right choice so we don't change the docker image again ref. #2187 (comment)

@shamardy, all the tests are now working correctly with SPL tokens minted to a single account (I've updated the mint.sh file as you suggested). I believe we can introduce the "filling" methods later, as they are primarily needed to cover the currently implemented test case scenarios.

…lated namespace Signed-off-by: Mariusz Jasuwienas <[email protected]>

Signed-off-by: Mariusz Jasuwienas <[email protected]>

laruh

Thank you for the fixes, next iteration

laruh · 2024-08-08T06:31:32Z

mm2src/mm2_main/tests/docker_tests/solana_tests.rs

+    let invalid_tx_details = block_on(
+        sol_coin
+            .withdraw(WithdrawRequest::new(
+                "SOL".to_string(),


It worth creating const SOL: &str = "SOL"; if there are string repetitions

laruh · 2024-08-08T06:32:58Z

mm2src/mm2_main/tests/docker_tests/solana_tests.rs

+#[test]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_transaction_simulations_not_enough_for_fees_dockerized() {
+    let passphrase = "non existent passphrase".to_string();


same for "fake passphrase" / "non existent passphrase" strings, you can create one const and reuse it

@laruh I've refactored the code to move all the repeated strings into constants. However, I'm concerned that this might have gone a bit too far (especially in test case scenarios). I feel it may have impacted readability in some areas. Let me know if you think we should adjust this or leave it as it is right now.

I'm concerned that this might have gone a bit too far

Yeah, some consts are used just once

It was basically enough to add consts for strings mentioned in review comments

In spl_tests.rs you leave one const USDC: &str = "USDC";

In solana_common_tests consts are fine, just it would be better to rename PASSPHRASE to SOL_PASSPHRASE to note that its for solana, same for ADDITIONAL_PASSPHRASE -> SOL_ADDITIONAL_PASSPHRASE

komodo-defi-framework/mm2src/mm2_main/tests/docker_tests/solana_common_tests.rs

Lines 19 to 20 in 00ecbb6

pub const PASSPHRASE: &str = "federal stay trigger hour exist success game vapor become comfort action phone bright ill target wild nasty crumble dune close rare fabric hen iron";

pub const ADDITIONAL_PASSPHRASE: &str =

@mj-blockydevs please when you answer in the review that something was fixed, provide commit link/links to this fix.
Like in this example #2169 (comment) and below.
This way makes review process a lot easier for all sides, especially when we have notes from different reviewers and various fix commits.

laruh · 2024-08-08T06:33:37Z

mm2src/mm2_main/tests/docker_tests/solana_tests.rs

+#[test]
+#[cfg(not(target_arch = "wasm32"))]
+fn solana_test_transactions_dockerized() {
+    let passphrase = "federal stay trigger hour exist success game vapor become comfort action phone bright ill target wild nasty crumble dune close rare fabric hen iron".to_string();


please also provide it as const

laruh · 2024-08-08T06:35:05Z

mm2src/mm2_main/tests/docker_tests/solana_tests.rs

+fn solana_coin_send_and_refund_maker_payment_dockerized() {
+    let passphrase = "federal stay trigger hour exist success game vapor become comfort action phone bright ill target wild nasty crumble dune close rare fabric hen iron".to_string();
+    let (_, coin) = solana_coin_for_test(passphrase, SolanaNet::Local);
+    let solana_program_id = "GCJUXKH4VeKzEtr9YgwaNWC3dJonFgsM3yMiBa64CZ8m";


"GCJUXKH4VeKzEtr9YgwaNWC3dJonFgsM3yMiBa64CZ8m" also can be used as const

laruh · 2024-08-08T07:06:55Z

mm2src/coins/solana.rs

 pub struct SolanaCoinImpl {
    ticker: String,
-    key_pair: SolKeypair,
+    pub key_pair: SolKeypair,
    client: RpcClient,
    decimals: u8,
-    my_address: String,
+    pub my_address: String,
    spl_tokens_infos: Arc<Mutex<HashMap<String, SplTokenInfo>>>,
    /// This spawner is used to spawn coin's related futures that should be aborted on coin deactivation
    /// and on [`MmArc::stop`].


Do you really need to make these fields (plus abortable) pub?
I could pass clippy warnings check making them private

also amount_to_ui_amount function is never used. I think it could be useful in the future, please add dead code annotation.

@laruh, yes, I was able to access the data using the existing methods, so there's no need to make them public.

laruh · 2024-08-08T07:13:58Z

mm2src/coins/solana.rs

+pub use solana_client;
+pub use solana_sdk;


There is no need to make these mods and consts pub

@laruh, I tried to access them from the Docker-related test crate, so I needed to expose them.

By the way, what kind of Clippy response is that? I can't seem to reproduce it. Could you provide some guidance on how to achieve that?

/komodo-defi-framework/: cargo clippy --all-targets --all-features -- -D warnings warning: /komodo-defi-framework/mm2src/mm2_bin_lib/Cargo.toml: file `/komodo-defi-framework/mm2src/mm2_bin_lib/src/mm2_bin.rs` found to be present in multiple build targets: * `bin` target `mm2` * `bin` target `kdf` Compiling common v0.1.0 (/komodo-defi-framework/mm2src/common) Compiling ser_error v0.1.0 (/komodo-defi-framework/mm2src/derives/ser_error) Checking shared_ref_counter v0.1.0 (/komodo-defi-framework/mm2src/common/shared_ref_counter) Checking primitives v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/primitives) Checking test_helpers v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/test_helpers) Compiling serialization_derive v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/serialization_derive) Compiling trezor v0.1.1 (/komodo-defi-framework/mm2src/trezor) Compiling enum_derives v0.1.0 (/komodo-defi-framework/mm2src/derives/enum_derives) Checking mm2_number v0.1.0 (/komodo-defi-framework/mm2src/mm2_number) Checking serialization v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/serialization) Compiling ser_error_derive v0.1.0 (/komodo-defi-framework/mm2src/derives/ser_error_derive) Compiling coins v0.1.0 (/komodo-defi-framework/mm2src/coins) Checking bitcrypto v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/crypto) Checking mm2_state_machine v0.1.0 (/komodo-defi-framework/mm2src/mm2_state_machine) Checking keys v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/keys) Compiling mm2_main v0.1.0 (/komodo-defi-framework/mm2src/mm2_main) Compiling mm2_bin_lib v2.1.0-beta (/komodo-defi-framework/mm2src/mm2_bin_lib) Checking chain v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/chain) Checking script v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/script) Checking spv_validation v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/spv_validation) Checking rpc v0.1.0 (/komodo-defi-framework/mm2src/mm2_bitcoin/rpc) Checking mm2_metamask v0.1.0 (/komodo-defi-framework/mm2src/mm2_metamask) Checking mm2_db v0.1.0 (/komodo-defi-framework/mm2src/mm2_db) Checking mm2_err_handle v0.1.0 (/komodo-defi-framework/mm2src/mm2_err_handle) Checking mm2_event_stream v0.1.0 (/komodo-defi-framework/mm2src/mm2_event_stream) Checking db_common v0.1.0 (/komodo-defi-framework/mm2src/db_common) Checking mm2_p2p v0.1.0 (/komodo-defi-framework/mm2src/mm2_p2p) Checking mm2_metrics v0.1.0 (/komodo-defi-framework/mm2src/mm2_metrics) Checking hw_common v0.1.0 (/komodo-defi-framework/mm2src/hw_common) Checking rpc_task v0.1.0 (/komodo-defi-framework/mm2src/rpc_task) Checking mm2_io v0.1.0 (/komodo-defi-framework/mm2src/mm2_io) Checking mm2_rpc v0.1.0 (/komodo-defi-framework/mm2src/mm2_rpc) Checking mm2_eth v0.1.0 (/komodo-defi-framework/mm2src/mm2_eth) Checking mm2_core v0.1.0 (/komodo-defi-framework/mm2src/mm2_core) Checking mm2_gui_storage v0.1.0 (/komodo-defi-framework/mm2src/mm2_gui_storage) Checking crypto v1.0.0 (/komodo-defi-framework/mm2src/crypto) Checking utxo_signer v0.1.0 (/komodo-defi-framework/mm2src/coins/utxo_signer) Checking mm2_net v0.1.0 (/komodo-defi-framework/mm2src/mm2_net) Checking mm2_git v0.1.0 (/komodo-defi-framework/mm2src/mm2_git) Checking mm2_test_helpers v0.1.0 (/komodo-defi-framework/mm2src/mm2_test_helpers) Checking coins_activation v0.1.0 (/komodo-defi-framework/mm2src/coins_activation) Finished dev [unoptimized + debuginfo] target(s) in 1m 01s warning: the following packages contain code that will be rejected by a future version of Rust: fs_extra v1.2.0, nom v6.1.2 note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 3`

By the way, what kind of Clippy response is that? I can't seem to reproduce it. Could you provide some guidance on how to achieve that?

@mj-blockydevs I was using cargo clippy --all-features -- -D warnings command.
It's not the first time that cargo clippy --all-features -- -D warnings shows a warning that wasn't caught by cargo clippy --all-targets --all-features -- -D warnings.

hmm, clippy also doesn't detect unused function for me now.

~/RustroverProjects/komodo-defi-framework git:[solana-docker-tests-improvements-local-review] cargo clippy --all-features -- -D warnings warning: /Users/sharon/RustroverProjects/komodo-defi-framework/mm2src/mm2_bin_lib/Cargo.toml: file `/Users/sharon/RustroverProjects/komodo-defi-framework/mm2src/mm2_bin_lib/src/mm2_bin.rs` found to be present in multiple build targets: * `bin` target `mm2` * `bin` target `kdf` Compiling mm2_bin_lib v2.1.0-beta (/Users/sharon/RustroverProjects/komodo-defi-framework/mm2src/mm2_bin_lib) Finished dev [unoptimized + debuginfo] target(s) in 2.03s warning: the following packages contain code that will be rejected by a future version of Rust: fs_extra v1.2.0, nom v6.1.2 note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 622`

Well, you can search the project to see that it's not used

Sometimes it happens with clippy so I re check usage manually by searching in project files and checking usage in ide

I recommend to add dead code annotation

Signed-off-by: Mariusz Jasuwienas <[email protected]>

* install libudev-dev build dep for docker-tests

…provements-local

shamardy

Next review iteration! I stopped review midway as some tests are removed and some are not copied right, will continue once this is done in a right way!

shamardy · 2024-08-10T00:28:15Z

.docker/solana-accounts/user1.json

I guess this user is not needed anymore

shamardy · 2024-08-10T00:28:23Z

.docker/solana-accounts/user2.json

Same for this user

shamardy · 2024-08-10T00:29:02Z

.docker/solana-accounts/token_m_a.json

And this file also

shamardy · 2024-08-10T00:42:45Z

mm2src/mm2_main/tests/docker_tests_main.rs

@@ -75,18 +78,20 @@ pub fn docker_tests_runner(tests: &[&TestDescAndFn]) {
        let qtum_node = qtum_docker_node(&docker, 9000);
        let for_slp_node = utxo_asset_docker_node(&docker, "FORSLP", 10000);
        let geth_node = geth_docker_node(&docker, "ETH", 8545);
+        let sol_node = sol_asset_docker_node(&docker, "SOLCLUSTER");


This should be the ticker SOL not SOLCLUSTER

shamardy · 2024-08-10T00:47:17Z

mm2src/mm2_main/tests/docker_tests/docker_tests_common.rs

 pub fn _solana_supplied_node() -> MarketMakerIt {
+    let adex_token_address = env::var("ADEX_TOKEN_ADDRESS").expect("ADEX_TOKEN_ADDRESS not set");
+    println!("adex_token_address: {}", &adex_token_address);


We don't use println!, we use log!

shamardy · 2024-08-10T00:55:14Z

mm2src/mm2_main/tests/docker_tests/solana_tests.rs

+    match error.into_inner() {
+        WithdrawError::NotSufficientBalance {
+            coin: _,
+            available,
+            required,
+        } => {
+            assert_eq!(available, 0.into());
+            assert_eq!(required, sol_required);
+        },
+        e => panic!("Unexpected err {:?}", e),
+    };


why change this

let balance = block_on(sol_coin.my_balance().compat()).unwrap().spendable; let (_, fees) = block_on(sol_coin.estimate_withdraw_fees()).unwrap(); let sol_required = lamports_to_sol(fees); assert_eq!(valid_tx_details.my_balance_change, sol_required.clone().neg()); assert_eq!(valid_tx_details.total_amount, balance); assert_eq!(valid_tx_details.spent_by_me, balance); assert_eq!(valid_tx_details.received_by_me, &balance - &sol_required); log!("{:?}", valid_tx_details);

From the original test? you just copied and pasted the previous test solana_transaction_simulations_not_enough_for_fees

shamardy · 2024-08-10T00:58:35Z

mm2src/mm2_main/tests/docker_tests/solana_tests.rs

+    let (_, sol_coin) = solana_coin_for_test(PASSPHRASE.to_owned(), SolanaNet::Local);
+    let valid_tx_details = block_on(
+        sol_coin
+            .withdraw(WithdrawRequest::new(
+                SOL.to_string(),
+                None,
+                sol_coin.my_address().unwrap(),
+                BigDecimal::try_from(0.0001).unwrap(),
+                false,
+                None,
+                None,
+            ))
+            .compat(),
+    )
+    .unwrap();
+    log!("{:?}", valid_tx_details);
+
+    let tx_str = hex::encode(&*valid_tx_details.tx.tx_hex().unwrap().0);
+    let res = block_on(sol_coin.send_raw_tx(&tx_str).compat()).unwrap();
+
+    let res2 = block_on(
+        sol_coin
+            .send_raw_tx_bytes(&valid_tx_details.tx.tx_hex().unwrap().0)
+            .compat(),
+    )
+    .unwrap();
+    assert_eq!(res, res2);


Same for this

let res = sol_coin .client .get_signatures_for_address(&sol_coin.key_pair.pubkey()) .unwrap(); let mut history = Vec::new(); for cur in res.iter() { let signature = Signature::from_str(cur.signature.clone().as_str()).unwrap(); let res = sol_coin .client .get_transaction(&signature, UiTransactionEncoding::JsonParsed) .unwrap(); log!("{}", serde_json::to_string(&res).unwrap()); let parsed = serde_json::to_value(&res).unwrap(); let tx_infos: SolanaConfirmedTransaction = serde_json::from_value(parsed).unwrap(); let mut txs = tx_infos.extract_solana_transactions(&sol_coin).unwrap(); history.append(&mut txs); } log!("{}", serde_json::to_string(&history).unwrap());

You just copied solana_test_transactions test.

shamardy · 2024-08-10T01:02:04Z

mm2src/coins/solana/solana_tests.rs

-#[test]
-#[cfg(not(target_arch = "wasm32"))]
-fn solana_my_balance() {
-    let passphrase = "federal stay trigger hour exist success game vapor become comfort action phone bright ill target wild nasty crumble dune close rare fabric hen iron".to_string();
-    let (_, sol_coin) = solana_coin_for_test(passphrase, SolanaNet::Testnet);
-    let res = block_on(sol_coin.my_balance().compat()).unwrap();
-    assert_ne!(res.spendable, BigDecimal::from(0));
-}
-
-#[test]
-#[cfg(not(target_arch = "wasm32"))]
-fn solana_block_height() {
-    let passphrase = "federal stay trigger hour exist success game vapor become comfort action phone bright ill target wild nasty crumble dune close rare fabric hen iron".to_string();
-    let (_, sol_coin) = solana_coin_for_test(passphrase, SolanaNet::Testnet);
-    let res = block_on(sol_coin.current_block().compat()).unwrap();
-    log!("block is : {}", res);
-    assert!(res > 0);
-}
-
-#[test]
-#[cfg(not(target_arch = "wasm32"))]
-fn solana_validate_address() {
-    let passphrase = "federal stay trigger hour exist success game vapor become comfort action phone bright ill target wild nasty crumble dune close rare fabric hen iron".to_string();
-    let (_, sol_coin) = solana_coin_for_test(passphrase, SolanaNet::Testnet);
-
-    // invalid len
-    let res = sol_coin.validate_address("invalidaddressobviously");
-    assert!(!res.is_valid);
-    let res = sol_coin.validate_address("GMtMFbuVgjDnzsBd3LLBfM4X8RyYcDGCM92tPq2PG6B2");
-    assert!(res.is_valid);
-
-    // Typo
-    let res = sol_coin.validate_address("Fr8fraJXAe1cFU81mF7NhHTrUzXjZAJkQE1gUQ11riH");
-    assert!(!res.is_valid);
-
-    // invalid len
-    let res = sol_coin.validate_address("r8fraJXAe1cFU81mF7NhHTrUzXjZAJkQE1gUQ11riHn");
-    assert!(!res.is_valid);
-}
-
-#[test]
-#[cfg(not(target_arch = "wasm32"))]
-fn test_sign_message() {
-    let passphrase = "spice describe gravity federal blast come thank unfair canal monkey style afraid".to_string();
-    let (_, sol_coin) = solana_coin_for_test(passphrase, SolanaNet::Testnet);
-    let signature = sol_coin.sign_message("test").unwrap();
-    assert_eq!(
-        signature,
-        "4dzKwEteN8nch76zPMEjPX19RsaQwGTxsbtfg2bwGTkGenLfrdm31zvn9GH5rvaJBwivp6ESXx1KYR672ngs3UfF"
-    );
-}


These tests are just removed and not moved to new files.

Signed-off-by: Mariusz Jasuwienas <[email protected]>

shamardy · 2024-10-10T09:47:52Z

ref. #1085 (comment) , the solana reference implementation has this latest changes here https://github.com/KomodoPlatform/komodo-defi-framework/commits/solana-ref-impl/

0xMMBD and others added 8 commits July 26, 2024 13:54

Add and fix Solana docker tests

73491ad

Add Dockerfile to run local solana validator for tests

fmt changes

47861f8

unignore test test_solana_and_spl_balance_enable_spl_v2

ignore tendermint tests for now

4277284

feat(solana-tests): moving to local node

5b52c69

Signed-off-by: Mariusz Jasuwienas <[email protected]>

feat(solana-tests): moving solana tests to be executed against the lo…

763f32b

…cal node Signed-off-by: Mariusz Jasuwienas <[email protected]>

Revert "ignore tendermint tests for now"

c79ae3c

This reverts commit 4277284

fixes after revert

efa9285

Merge remote-tracking branch 'origin/dev' into solana-docker-tests-im…

313283d

…provements-local

shamardy mentioned this pull request Aug 6, 2024

test(solana): migrate to local/offline containerized node #2168

Closed

feat(solana-tests): moving solana tests to be executed against the lo…

364a98f

…cal node. Non-dockerised tests (out of solana - scope) excluded. Signed-off-by: Mariusz Jasuwienas <[email protected]>

mj-blockydevs changed the title ~~DRAFT - testing pipelines~~ feat(solana-dockerised-tests) testing pipelines Aug 6, 2024

mj-blockydevs marked this pull request as ready for review August 6, 2024 13:56

shamardy changed the title ~~feat(solana-dockerised-tests) testing pipelines~~ test(solana): migrate to local/offline containerized node Aug 6, 2024

shamardy added the under review label Aug 6, 2024

shamardy requested review from shamardy and laruh August 7, 2024 01:10

onur-ozkan reviewed Aug 7, 2024

View reviewed changes

laruh reviewed Aug 7, 2024

View reviewed changes

shamardy reviewed Aug 7, 2024

View reviewed changes

mj-blockydevs added 4 commits August 7, 2024 23:00

feat(solana-tests): moving dockerized tests directly to the docker re…

2fb5cf4

…lated namespace Signed-off-by: Mariusz Jasuwienas <[email protected]>

feat(solana-tests): moving dockerized tests directly to the docker re…

8aa12b5

…lated namespace Signed-off-by: Mariusz Jasuwienas <[email protected]>

feat(solana-tests): removing comments from code

a81d5fd

Signed-off-by: Mariusz Jasuwienas <[email protected]>

feat(solana-tests): one account instead of multiple accounts

7f8fba1

Signed-off-by: Mariusz Jasuwienas <[email protected]>

laruh requested changes Aug 8, 2024

View reviewed changes

mj-blockydevs and others added 6 commits August 8, 2024 11:56

feat(solana-tests): fixing tests broken by previous commit

be15afa

Signed-off-by: Mariusz Jasuwienas <[email protected]>

feat(solana-tests): fixing tests broken by previous commit

220651d

Signed-off-by: Mariusz Jasuwienas <[email protected]>

feat(solana-tests): fixing tests broken by previous commit

824463e

Signed-off-by: Mariusz Jasuwienas <[email protected]>

feat(solana-tests): mint SPL to one account only

96d4855

Signed-off-by: Mariusz Jasuwienas <[email protected]>

feat(solana-tests): moving repeated strings to consts

f4f644a

Signed-off-by: Mariusz Jasuwienas <[email protected]>

enable solana feature for ci docker tests

30e8cbe

shamardy added 4 commits August 10, 2024 02:49

* add enable-solana flag to some parts

525625f

* install libudev-dev build dep for docker-tests

fix wasm compilation

19bbd93

Merge remote-tracking branch 'origin/dev' into solana-docker-tests-im…

6d5bc8e

…provements-local

make WithdrawRequest one implementation

00ecbb6

shamardy reviewed Aug 10, 2024

View reviewed changes

feat(solana-tests): bringing back few omitted tests

958de15

Signed-off-by: Mariusz Jasuwienas <[email protected]>

shamardy closed this Oct 10, 2024

	pub const PASSPHRASE: &str = "federal stay trigger hour exist success game vapor become comfort action phone bright ill target wild nasty crumble dune close rare fabric hen iron";
	pub const ADDITIONAL_PASSPHRASE: &str =

test(solana): migrate to local/offline containerized node #2187

test(solana): migrate to local/offline containerized node #2187

Conversation

mj-blockydevs commented Aug 6, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

mj-blockydevs Aug 7, 2024 • edited Loading

Choose a reason for hiding this comment

laruh left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shamardy Aug 7, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shamardy left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

laruh left a comment

Choose a reason for hiding this comment

laruh Aug 8, 2024 • edited Loading

Choose a reason for hiding this comment

laruh Aug 8, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

laruh Aug 10, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

laruh Aug 8, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shamardy left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shamardy commented Oct 10, 2024

mj-blockydevs Aug 7, 2024 •

edited

Loading

shamardy Aug 7, 2024 •

edited

Loading

laruh Aug 8, 2024 •

edited

Loading

laruh Aug 8, 2024 •

edited

Loading

laruh Aug 10, 2024 •

edited

Loading

laruh Aug 8, 2024 •

edited

Loading