Skip to content

Allow multiple test case arguments to testrunner#1755

Merged
amai2012 merged 1 commit into
cppcheck-opensource:masterfrom
rikardfalkeborn:testrunner-allow-multiple-testcases
Mar 26, 2019
Merged

Allow multiple test case arguments to testrunner#1755
amai2012 merged 1 commit into
cppcheck-opensource:masterfrom
rikardfalkeborn:testrunner-allow-multiple-testcases

Conversation

@rikardfalkeborn
Copy link
Copy Markdown
Contributor

Take some care to not run the same test case twice, even if running:

./testrunner TestClass TestClass::TestCase

@rikardfalkeborn
Copy link
Copy Markdown
Contributor Author

Take some care to not run the same test case twice, even if running:

./testrunner TestClass TestClass::TestCase

Not sure this is the wanted behaviour but it made sense to me when I tested it.

@IOBYTE
Copy link
Copy Markdown
Contributor

IOBYTE commented Mar 24, 2019

clang tidy lets you specify a class of checks and then remove specific checks.

Something like:

./testrunner TestClass -TestClass::TestCase

might make sense.

Take some care to not run the same test case twice, even if running:

    ./testrunner TestClass TestClass::TestCase
@rikardfalkeborn rikardfalkeborn force-pushed the testrunner-allow-multiple-testcases branch from 6a53b5d to d677155 Compare March 24, 2019 17:04
@rikardfalkeborn
Copy link
Copy Markdown
Contributor Author

clang tidy lets you specify a class of checks and then remove specific checks.

Something like:

./testrunner TestClass -TestClass::TestCase

might make sense.

Might make sense, but I don't really see a use for it (for me) at the moment, so I'm not too keen on implementing it now.

@amai2012
Copy link
Copy Markdown
Collaborator

Thanks

@amai2012 amai2012 merged commit 6b478c3 into cppcheck-opensource:master Mar 26, 2019
@rikardfalkeborn rikardfalkeborn deleted the testrunner-allow-multiple-testcases branch March 26, 2019 21: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.

3 participants