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

support multi txns testing #555

Merged
merged 3 commits into from
Oct 19, 2024
Merged

support multi txns testing #555

merged 3 commits into from
Oct 19, 2024

Conversation

yuunlimm
Copy link
Contributor

No description provided.

Copy link
Contributor Author

yuunlimm commented Oct 16, 2024

@yuunlimm yuunlimm mentioned this pull request Oct 16, 2024
@yuunlimm yuunlimm marked this pull request as ready for review October 16, 2024 23:37
@yuunlimm yuunlimm force-pushed the 10-16-add_sdk_tests branch from f80db7e to 0a38648 Compare October 17, 2024 01:36
@yuunlimm yuunlimm force-pushed the 10-16-support_multi_txns_testing branch from 411531a to 54b9e77 Compare October 17, 2024 01:36
@@ -5,7 +5,6 @@
"coin_type": "0x1::aptos_coin::AptosCoin",
"supply": "18544645433533405815",
"transaction_timestamp": "2022-10-13T01:46:57",
"transaction_epoch": 746,
"inserted_at": "2024-10-15T05:16:56.913064"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

weird. I thought these inserted_at fields were removed in a previous PR already

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this rebasing is tricky.. they are removed in the next stacked pr.

@yuunlimm yuunlimm requested a review from a team October 17, 2024 17:34
db_url: &str,
) -> (IndexerProcessorConfig, &'static str) {
let transaction_stream_config =
test_context.create_transaction_stream_config(txn_version, txn_count as u64);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -98,4 +116,54 @@ mod tests {
}
}
}

#[tokio::test(flavor = "multi_thread", worker_threads = 2)]
async fn testnet_events_processor_db_output_scenario_testing() {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yuunlimm yuunlimm force-pushed the 10-16-add_sdk_tests branch from 0a38648 to 006e9aa Compare October 17, 2024 21:13
@yuunlimm yuunlimm force-pushed the 10-16-support_multi_txns_testing branch from 54b9e77 to 597e4e2 Compare October 17, 2024 21:14
@yuunlimm yuunlimm force-pushed the 10-16-add_sdk_tests branch from 006e9aa to 7fa5c46 Compare October 17, 2024 21:17
@yuunlimm yuunlimm force-pushed the 10-16-support_multi_txns_testing branch from 597e4e2 to f8bb07c Compare October 17, 2024 21:17
@yuunlimm yuunlimm force-pushed the 10-16-add_sdk_tests branch from 7fa5c46 to 7648c32 Compare October 18, 2024 05:16
@yuunlimm yuunlimm force-pushed the 10-16-support_multi_txns_testing branch from f8bb07c to 3e81f8f Compare October 18, 2024 05:16
Base automatically changed from 10-16-add_sdk_tests to main October 18, 2024 05:42
@yuunlimm yuunlimm force-pushed the 10-16-support_multi_txns_testing branch from 3e81f8f to b2cad77 Compare October 18, 2024 06:52
@yuunlimm yuunlimm merged commit b347465 into main Oct 19, 2024
6 of 7 checks passed
@yuunlimm yuunlimm deleted the 10-16-support_multi_txns_testing branch October 19, 2024 05:15
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