ARCHIVE/xahaud
3
0
Fork 0
mirror of https://github.com/Xahau/xahaud.git synced 2025-12-06 17:27:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Unit test logging fixes (RIPD-1568):

Browse Source 
* Check message queue before shutting down unit tests
* Logging should be on if not quiet
This commit is contained in:
seelabs
2017-12-06 16:00:47 -05:00
committed by Nikolaos D. Bougalis
parent 44e5e8bccf
commit 6b56426719
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/test/unit_test/multi_runner.cpp
View File

@@ -363,7 +363,8 @@ multi_runner_parent::multi_runner_parent()
{ {
message_queue_thread_ = std::thread([this] { message_queue_thread_ = std::thread([this] {
std::vector<char> buf(1 << 20); std::vector<char> buf(1 << 20);
while (this->continue_message_queue_) while (this->continue_message_queue_ ||
this->message_queue_->get_num_msg())
{ {
// let children know the parent is still alive // let children know the parent is still alive
this->inc_keep_alive_count(); this->inc_keep_alive_count();
@@ -423,7 +424,7 @@ multi_runner_child::multi_runner_child(
: job_index_{checkout_job_index()} : job_index_{checkout_job_index()}
, num_jobs_{num_jobs} , num_jobs_{num_jobs}
, quiet_{quiet} , quiet_{quiet}
, print_log_{print_log} , print_log_{!quiet || print_log}
{ {
// incPort twice (2*jobIndex_) because some tests need two envs // incPort twice (2*jobIndex_) because some tests need two envs
for (std::size_t i = 0; i < 2 * job_index_; ++i) for (std::size_t i = 0; i < 2 * job_index_; ++i)