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

Fixes for #7419 - Support multiple calls to framework::init() allowing wrappers to support running tests using test tools in full systems #17

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

ja11sop
Copy link
Contributor

@ja11sop ja11sop commented Oct 18, 2014

Pull request for https://svn.boost.org/trac/boost/ticket/7419 allowing boost.test to be used in production systems in a test mode. In other words test can be configured using non-conflicting command-line options and be executed in arbitrary code. Please see the referenced ticket #7419 for full details which explains the use case. Our production code relies on this and the related patches/pull requests #15 and #16. As a set they enable powerful use of boost test by making the test tools available in non-test programs and allow sophisticated tools to be built off boost.test's output.

We've been using these patches for over 2 years painfully applying them to boost test for each release. They are small and unobtrusive and now that we're all comfy in git I think it's time to get them added so others can benefit and boost.test is no longer broken when used with boost.timer.

@ja11sop ja11sop changed the title Fixes for #7419 Fixes for #7419 - Support multiple calls to framework::init() allowing wrappers to support running tests using test tools in full systems Oct 18, 2014
@raffienficiaud
Copy link
Member

Please let me know if this is still of some interest for you. It would be good if you rebase on develop.

@ja11sop
Copy link
Contributor Author

ja11sop commented Jun 27, 2018

This is still of interest to me - I see most of the code relating to this has changed - I'll see what I can do

@raffienficiaud
Copy link
Member

I would suggest that we close this PR and start a new one, based on the latest develop. We have for instance a custom logger facility that may be used as is, and it is not clear to me what is needed from the init phase of the test setup. For instance the loggers may not need to be set up again at each iteration.

Let me know how you want to proceed

@ja11sop
Copy link
Contributor Author

ja11sop commented Oct 20, 2018

I'm looking at this now - it's still broken but the fix might be minor.

@ja11sop
Copy link
Contributor Author

ja11sop commented Oct 21, 2018

The current PR has no conflicts and supports the use case of calling init() on the framework more than once which we use to allow us to launch test suites as a special mode in production executables (sort of a self validating test run or runs). We need to be able to put the boost test command line parameters in a boost.test namespace so we don't conflict with actual application command line options.

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