Skip to content

Conversation

@darwintree
Copy link
Contributor

@darwintree darwintree commented Sep 17, 2025

This change is Reviewable

@darwintree darwintree requested a review from Pana September 17, 2025 08:56
Copy link
Contributor

@ChenxingLi ChenxingLi left a comment

Choose a reason for hiding this comment

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

Before we accept this PR, we need some experimental data to ensure it actually helps improve test speed.

Reviewable status: 0 of 14 files reviewed, all discussions resolved (waiting on @Pana)

@darwintree
Copy link
Contributor Author

Before we accept this PR, we need some experimental data to ensure it actually helps improve test speed.

Reviewable status: 0 of 14 files reviewed, all discussions resolved (waiting on @Pana)

Here is part of the local test data (macOS, M2 PRO) for the result for the command pytest integration_tests/tests/execution_spec_tests -n 6 --dist loadscope:

before (in test mode): 829 passed, 904 warnings in 231.04s (0:03:51)
now (in dev mode): 829 passed, 904 warnings in 40.17s

And you can find the result in recent CI consoles:

Before(https://jenkins.conflux-chain.org.cn/job/conflux-rust/1359/console):

================ 1084 passed, 913 warnings in 327.92s (0:05:27) ================

This pr(https://jenkins.conflux-chain.org.cn/job/conflux-rust/1360/console):

================ 1084 passed, 913 warnings in 247.35s (0:04:07) ================

Turning a test containing lots of transaction sending would greatly accelerate it. But not all tests are suitable for dev mode so this pull request simply turns part of execution tests into dev mode

@darwintree darwintree force-pushed the dev-mode-for-integration-tests branch from 3f47095 to 4fe8cb1 Compare September 19, 2025 07:54
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