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

Add support for randomizing test execution order #714

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jamestiotio
Copy link

This commit reintroduces the option to shuffle the test execution order into the test runner. This has been tested with the temp_sensor example project in Ceedling.

This fixes #2.

@jamestiotio
Copy link
Author

Hi @mvandervoord, I have resolved the unit test failures by fixing the test indexing issue for parameterized tests. Verified that all unit tests are passing locally.

This commit reintroduces the option to shuffle the test execution order
into the test runner. This has been tested with the temp_sensor example
project in Ceedling. Unit tests have also been successfully executed.

Signed-off-by: James Raphael Tiovalen <jamestiotio@gmail.com>
@jamestiotio
Copy link
Author

Hi @mvandervoord, I have improved the changes a little bit by:

  • Using || instead of or for the conditional statement to follow the Ruby coding style guide
  • Changing the implementation of shuffleTests to one with slightly fewer operations but with equivalent functionality
  • Adding some documentation for the 2 new parameters: :shuffle_tests and :rng_seed

Do kindly review the changes whenever you manage to find the time to do so. Thank you in advance!

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.

Random execution order of test cases (shuffle)
1 participant