enh(tests): add unit test cases to concurrent group#34626
enh(tests): add unit test cases to concurrent group#34626
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
There was a problem hiding this comment.
Pull request overview
Updates the UnitTest case runner script to shift more unit tests into the concurrent ctest execution group, reducing reliance on manual binary invocations.
Changes:
- Shrinks the
ctest -Eexclusion list so more tests run underctest -j8. - Removes several manual test binary executions that were previously run outside of
ctest.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # Run ctest and capture output and return code | ||
| ctest_output="unit-test.log" | ||
| ctest -E "cunit_test|pcre*|example*|clientTest|connectOptionsTest|stmtTest|stmt2Test|tssTest|tmqTest|catalogTest|taoscTest" -j8 2>&1 | tee "$ctest_output" | ||
| ctest -E "cunit_test|pcre*|example*|clientTest|connectOptionsTest|tmqTest|taoscTest" -j8 2>&1 | tee "$ctest_output" |
There was a problem hiding this comment.
tmqTest is still excluded from the ctest run (-E), but the explicit build/bin/tmqTest invocation was removed below. This means tmqTest no longer runs at all from this script. Either remove tmqTest from the -E exclude list so it runs under ctest, or add back a dedicated build/bin/tmqTest execution after ctest (and keep it excluded).
https://project.feishu.cn/taosdata_td/job/detail/6841854247