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 the parallel_tests gem #350

Merged
merged 2 commits into from
May 23, 2017
Merged

Conversation

bastelfreak
Copy link
Member

this allows us to run spec tests in parallel. We already enabled this in
our configs in the past, but we didn't include this gem. It was a
dependency of puppetlabs_spec_helper. This got removed so we need to
install it explicitly. Changes tested at:
https://github.com/voxpupuli/puppet-zabbix/blob/master/Gemfile#L39
https://travis-ci.org/voxpupuli/puppet-zabbix/builds/235141105

this allows us to run spec tests in parallel. We already enabled this in
our configs in the past, but we didn't include this gem. It was a
dependency of puppetlabs_spec_helper. This got removed so we need to
install it explicitly. Changes tested at:
https://github.com/voxpupuli/puppet-zabbix/blob/master/Gemfile#L39
https://travis-ci.org/voxpupuli/puppet-zabbix/builds/235141105
@domcleal
Copy link
Contributor

Could you also change the rspec output format to dots in that case, in moduleroot/.rspec.erb or .rspec_parallel.erb if you want to only change the parallel mode?

It should make the output much easier to read than the documentation style when there's a failure, otherwise the output becomes interleaved between the parallel processes and it's very hard to find the error.

@domcleal domcleal merged commit 476b998 into voxpupuli:master May 23, 2017
@domcleal
Copy link
Contributor

Thanks @bastelfreak.

@bastelfreak bastelfreak deleted the parallel_tests branch May 23, 2017 10:06
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.

4 participants