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

Refactor and reorganize riak_test_escript module #670

Merged

Conversation

kellymclaughlin
Copy link
Contributor

Refactor the riak_test_escript module to improve code clarity and
organization. This includes renaming functions to be more reflective
of their purpose and breaking large functions into smaller
functions.

Also add the ability to specify a comma-separated list of
tests with the -t option.

Refactor the riak_test_escript module to improve code clarity and
organization.  This includes renaming functions to be more reflective
of their purpose and breaking large functions into smaller
functions. Also add the ability to specify a comma-separated list of
tests with the -t option.
@kellymclaughlin kellymclaughlin mentioned this pull request Aug 4, 2014
15 tasks
@kellymclaughlin
Copy link
Contributor Author

This is part of the #667 refactoring effort

@bowrocker
Copy link
Contributor

This is great!

janderson::Jons-MacBook-Pro { ~/sandbox/riak_test_decouple/riak_test }->  ./riak_test -c rtee -t bucket_types,http_bucket_types,verify_build_cluster
Tests to run: [{verify_build_cluster,
                   [{id,-1},
                    {platform,<<"local">>},
                    {version,<<"riak_ee-2.0.0beta6-11-g5a161cf-develop">>},
                    {project,<<"riak_ee">>}]},
               {http_bucket_types,
                   [{id,-1},
                    {platform,<<"local">>},
                    {version,<<"riak_ee-2.0.0beta6-11-g5a161cf-develop">>},
                    {project,<<"riak_ee">>}]},
               {bucket_types,
                   [{id,-1},
                    {platform,<<"local">>},
                    {version,<<"riak_ee-2.0.0beta6-11-g5a161cf-develop">>},
                    {project,<<"riak_ee">>}]}]
----SNIP----
Test Results:
verify_build_cluster-bitcask: pass
http_bucket_types-bitcask   : pass
bucket_types-bitcask        : pass
---------------------------------------------
0 Tests Failed
3 Tests Passed
That's 100.0% for those keeping score

👍

@kellymclaughlin kellymclaughlin merged commit 2c8bc6c into feature/decouple-cluster-setup Aug 6, 2014
@kellymclaughlin kellymclaughlin deleted the riak-test-escript-refactor branch August 6, 2014 19:59
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