From 3fcc0de2bddef974dbe12c6037620bd15776e98b Mon Sep 17 00:00:00 2001 From: andrii Date: Mon, 24 Feb 2025 12:02:34 +0100 Subject: [PATCH 1/2] MTG-1283 clean up not working/not used test cases. --- nft_ingester/tests/api_tests.rs | 43 ++------------------------------- 1 file changed, 2 insertions(+), 41 deletions(-) diff --git a/nft_ingester/tests/api_tests.rs b/nft_ingester/tests/api_tests.rs index d6222257..c6eafdab 100644 --- a/nft_ingester/tests/api_tests.rs +++ b/nft_ingester/tests/api_tests.rs @@ -3913,7 +3913,7 @@ mod tests { assert_eq!(response["interface"], "V1_NFT".to_string()); } - /// Cover task changes MTG-947 + /// More test cases covered in the regular_nft_tests.rs/test_search_by_owner_with_show_zero_balance_false #[tokio::test(flavor = "multi_thread")] async fn test_determining_non_fungible_tokens() { let cnt = 100; @@ -4156,24 +4156,6 @@ mod tests { assert_eq!(res.items[0].clone().interface, Interface::V1NFT); assert_eq!(res.items[1].clone().interface, Interface::V1NFT); - let payload = SearchAssets { - limit: Some(1000), - page: Some(1), - owner_address: Some(owner.to_string()), - options: SearchAssetsOptions { - show_zero_balance: false, - show_unverified_collections: true, - ..Default::default() - }, - token_type: Some(TokenType::All), - ..Default::default() - }; - let res = api.search_assets(payload, mutexed_tasks.clone()).await.unwrap(); - let res: AssetList = serde_json::from_value(res).unwrap(); - - // todo FIX MTG-1283 assert value should be 1 - assert_eq!(res.items.len(), 2, "SearchAssets by token_type:All show_zero_balance: false"); - let payload = SearchAssets { limit: Some(1000), page: Some(1), @@ -4196,30 +4178,9 @@ mod tests { ); assert_eq!(res.items[0].clone().interface, Interface::V1NFT); assert_eq!(res.items[1].clone().interface, Interface::V1NFT); - - let payload = SearchAssets { - limit: Some(1000), - page: Some(1), - owner_address: Some(owner.to_string()), - options: SearchAssetsOptions { - show_zero_balance: false, - show_unverified_collections: true, - ..Default::default() - }, - token_type: Some(TokenType::NonFungible), - ..Default::default() - }; - let res = api.search_assets(payload, mutexed_tasks.clone()).await.unwrap(); - let res: AssetList = serde_json::from_value(res).unwrap(); - - // todo FIX MTG-1283 assert value should be 1 - assert_eq!( - res.items.len(), - 2, - "SearchAssets by token_type: NonFungible, show_zero_balance: false" - ); } + /// More test cases covered in the regular_nft_tests.rs/test_search_by_owner_with_show_zero_balance_false #[test] fn v1_payloads_are_parsed_correctly_from_v0_payloads() { // getAssetsByAuthority From f34f62bddaf7f8b4ee5c06c5c07e3ebe3bc4cd91 Mon Sep 17 00:00:00 2001 From: andrii Date: Mon, 24 Feb 2025 19:20:17 +0100 Subject: [PATCH 2/2] fix tests --- integration_tests/src/regular_nft_tests.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/integration_tests/src/regular_nft_tests.rs b/integration_tests/src/regular_nft_tests.rs index bbf0a29f..bfb6d722 100644 --- a/integration_tests/src/regular_nft_tests.rs +++ b/integration_tests/src/regular_nft_tests.rs @@ -1140,6 +1140,7 @@ async fn test_update_well_known_fungible_tokens() { } #[named] +#[serial] #[tokio::test] #[tracing_test::traced_test] async fn test_update_fungible_token_static_details_job() {