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

Patch: cmdline arguments, split tests into Tester.h #10

Open
GoogleCodeExporter opened this issue Mar 16, 2015 · 2 comments
Open

Patch: cmdline arguments, split tests into Tester.h #10

GoogleCodeExporter opened this issue Mar 16, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

Select tests to run from command line.

Move test definitions from main.cpp to Tester.cpp

Original issue reported on code.google.com by p...@mac.com on 2 Feb 2012 at 10:06

Attachments:

@GoogleCodeExporter
Copy link
Author

You're patch is far too complex and contains much more than just the addition 
of command line argument parsing. To have any hope of the maintainer committing 
your changes you'd need to submit small single purpose patches. I count at 
lease 5 in your patch (tidies like removing unused variables, breaking tests 
into individual functions, extracting common code from tests that use 
CombonationKeyTest, adding argument parsing, bulk code move of tests into 
Tester.h). Also, always adopt the existing coding style in your patches.

For those interested I attach a patch the the bulk code move reversed so that 
the OPs code changes are at least visible in a visual diff. The patch file is 
still too complex to eyeball directly. I may come back later and further 
subdivide this patch.

Original comment by shane.a....@gmail.com on 4 Feb 2012 at 1:31

Attachments:

@GoogleCodeExporter
Copy link
Author

Yeah, this change is too big to incorporate in one go. I might add pieces of it 
in, so I'll leave this bug open for now.

Original comment by tanj...@gmail.com on 11 May 2012 at 6:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant