!!! BREAKING CHANGE !!! - Make TEST_SUITE work with blocks and add TEST_SUITE_BEGIN

[FSund]

I think making the following code valid would be a good idea

TEST_SUITE("math")
{
    TEST_CASE("") {} // part of the math test suite
}

and adding the macro/typedef TEST_SUITE_BEGIN, to match TEST_SUITE_END

TEST_SUITE_BEGIN("gui")

TEST_CASE("") {} // part of the gui test suite

TEST_SUITE_END();

[onqtam]

having _BEGIN in the macro name would have been the right choice indeed... but this would be a breaking change.

For allowing test suites to have bodies and test cases inside them - not sure how that could be implemented - but one thing is for sure - the TEST_CASE macro won't be usable in that context - there will have to be a separate macro - like TEST_SUITE_CASE for cases inside a test suite block.

I'll have to think about these things...

[onqtam]

Actually this is possible! - see UnitTest++

The only drawback is that it will be a breaking change but I don't think test suites are used that much given that the framework is not that known yet - will document it in the changelog as a breaking change.

[onqtam]

This is how test suites are used now (in the dev branch - not yet released):

TEST_CASE("") {} // not part of any test suite

TEST_SUITE("math") {
    TEST_CASE("") {} // part of the math test suite
    TEST_CASE("") {} // part of the math test suite
}

TEST_SUITE_BEGIN("utils");

TEST_CASE("") {} // part of the utils test suite

TEST_SUITE_END();

TEST_CASE("") {} // not part of any test suite

[FSund]

I don't mind the breaking change (which makes sense given that I'm the one who suggested this in the first place), and I don't think the string literal issue affects me.

Actually I'm not even using TEST_SUITE at the moment, so perhaps I'm not the best judge on this matter.

I still think the change to having a unified way of calling TEST_SUITE and TEST_CASE would be good though.

Nevermind :)

[onqtam]

I figured out how to circumvent the string literal issue - and even edited my comment from 2 hours ago.

This is already implemented in the dev branch! I too like the new unified way of calling TEST_SUITE and TEST_CASE - so thanks for the suggestion!

[FSund]

Sorry, I had the page open for a while before submitting my comment, so I didn't catch that.

Nice work!