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

SYN-8479: synapse to use xdist for test splitting #4069

Merged
merged 10 commits into from
Jan 15, 2025

Conversation

MichaelSquires
Copy link
Contributor

No description provided.

@MichaelSquires MichaelSquires changed the base branch from master to dev_test_utils_isolatedasynciotestcase January 14, 2025 20:35
@MichaelSquires
Copy link
Contributor Author

MichaelSquires commented Jan 14, 2025

Note I removed the -s from pytest. Downside: messages to stdout won't be printed in the test output but they're not really useful anyway since they're all intertwined with running workers. Upside: we get percentage completion indicators in the output now, example:

[gw5] [ 15%] PASSED synapse/tests/test_lib_stormlib_env.py::StormLibEnvTest::test_stormlib_env 
synapse/tests/test_lib_stormlib_ethereum.py::EthereumTest::test_storm_ethereum 
[gw3] [ 15%] PASSED synapse/tests/test_lib_modelrev.py::ModelRevTest::test_modelrev_0_2_18 
synapse/tests/test_lib_modelrev.py::ModelRevTest::test_modelrev_0_2_19 
[gw0] [ 15%] PASSED synapse/tests/test_axon.py::AxonTest::test_axon_base 
synapse/tests/test_axon.py::AxonTest::test_axon_blob_v00_v01 
[gw0] [ 16%] PASSED synapse/tests/test_axon.py::AxonTest::test_axon_blob_v00_v01 
synapse/tests/test_axon.py::AxonTest::test_axon_http 
[gw7] [ 16%] PASSED synapse/tests/test_lib_layer.py::LayerTest::test_layer_abrv 
synapse/tests/test_lib_layer.py::LayerTest::test_layer_clone 
[gw5] [ 16%] PASSED synapse/tests/test_lib_stormlib_ethereum.py::EthereumTest::test_storm_ethereum 
synapse/tests/test_lib_stormlib_gen.py::StormLibGenTest::test_stormlib_gen 
[gw2] [ 16%] PASSED synapse/tests/test_lib_coro.py::CoroTest::test_lib_coro_forked 
synapse/tests/test_lib_coro.py::CoroTest::test_lib_coro_parserforked 
[gw2] [ 16%] PASSED synapse/tests/test_lib_coro.py::CoroTest::test_lib_coro_parserforked 

@MichaelSquires MichaelSquires marked this pull request as ready for review January 14, 2025 20:38
@MichaelSquires MichaelSquires force-pushed the blackout/SYN-8479/xdist branch from 29fa041 to 9aa3da6 Compare January 15, 2025 15:15
@MichaelSquires MichaelSquires changed the base branch from dev_test_utils_isolatedasynciotestcase to master January 15, 2025 15:15
@vEpiphyte vEpiphyte added this to the v2.194.0 milestone Jan 15, 2025
@vEpiphyte
Copy link
Contributor

shipit

@Cisphyx Cisphyx merged commit 2b4422c into master Jan 15, 2025
3 checks passed
@Cisphyx Cisphyx deleted the blackout/SYN-8479/xdist branch January 15, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants