Unit test logging fixes

[seelabs]

1. Check message queue before shutting down unit tests. Otherwise message may be lost.

2. Logging should be on if not quiet. The multiprocess unit test commit changed this behavior. This commit changes it back.

[mellery451]

works for me now - 👍

[ripplelabs-jenkins]

Jenkins Build Summary

Built from this commit

Built at 20171206 - 22:23:53

Test Results

Build Type	Result	Status
clang.debug.unity	970 cases, 0 failed, t: 370s	PASS ✅
coverage	970 cases, 0 failed, t: 589s	PASS ✅
gcc.debug.unity	970 cases, 0 failed, t: 420s	PASS ✅
clang.debug.nounity	968 cases, 0 failed, t: 713s	PASS ✅
gcc.debug.nounity	968 cases, 0 failed, t: 307s	PASS ✅
clang.release.unity	969 cases, 0 failed, t: 437s	PASS ✅
gcc.release.unity	969 cases, 0 failed, t: 457s	PASS ✅

[codecov-io]

Codecov Report

Merging #2293 into develop will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #2293      +/-   ##
===========================================
- Coverage    70.96%   70.95%   -0.02%     
===========================================
  Files          691      691              
  Lines        51663    51663              
===========================================
- Hits         36661    36655       -6     
- Misses       15002    15008       +6

Impacted Files	Coverage Δ
src/ripple/beast/clock/chrono_util.h	82.6% <0%> (-8.7%)	⬇️
src/ripple/app/ledger/PendingSaves.h	93.54% <0%> (-6.46%)	⬇️
src/ripple/json/impl/json_value.cpp	75% <0%> (-0.22%)	⬇️
src/ripple/app/main/Application.cpp	60.84% <0%> (-0.12%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 090d813...4a50d40. Read the comment docs.

[seelabs]

In 0.90.0-b3