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

Split system tests so that they can run in parallel #103

Merged
merged 4 commits into from
Jun 13, 2024

Conversation

linh2931
Copy link
Member

Change Description

Currently in CICD libtester tests, system_tests takes longest time (about 5 minutes)(https://github.com/AntelopeIO/spring/actions/runs/9468157289/job/26085190747#step:16:984). The next longest test eosio_system_finalizer_key_unit_test is about 1 minute. By splitting it into multiple suites such that each runs about 1 minute in parallel, it will reduce overall libtester running time by 4 minutes.

  • Change tests/CMakeList.txt to support multiple test suites in a single file
  • Split system_tests into multiple suites
  • Avoid produce unnecesary large number of blocks in eosio.system_tester.hpp

Resolved #102

Deployment Changes

  • Deployment Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@linh2931 linh2931 requested review from spoonincode and heifner June 13, 2024 18:00
@linh2931 linh2931 changed the title Run system tests in parallel Split system tests so that they can run in parallel Jun 13, 2024
tests/eosio.system_tester.hpp Outdated Show resolved Hide resolved
@linh2931 linh2931 merged commit efb7d81 into main Jun 13, 2024
1 check passed
@linh2931 linh2931 deleted the run_system_tests_in_parallel branch June 13, 2024 19:19
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.

Split eosio.system_tests.cpp so tests can run in parallel in CICD to reduce overall CICD time
3 participants