Improved unit test logging:

Logging from the Application object in the Env is
redirected to the suite::log, with the severity level
set to error and above.

src/ripple/basics/Log.h

@@ -25,6 +25,7 @@
 #include <beast/utility/Journal.h>
 #include <boost/filesystem.hpp>
 #include <map>
+#include <memory>
 #include <mutex>
 #include <utility>
 
@@ -145,7 +146,9 @@ private:
     };
 
     std::mutex mutable mutex_;
-    std::map <std::string, Sink, beast::ci_less> sinks_;
+    std::map <std::string,
+        std::unique_ptr<beast::Journal::Sink>,
+            beast::ci_less> sinks_;
     beast::Journal::Severity level_;
     File file_;
     bool silent_ = false;
@@ -159,10 +162,10 @@ public:
     bool
     open (boost::filesystem::path const& pathToLogFile);
 
-    Sink&
+    beast::Journal::Sink&
     get (std::string const& name);
 
-    Sink&
+    beast::Journal::Sink&
     operator[] (std::string const& name);
 
     beast::Journal
@@ -195,6 +198,11 @@ public:
         silent_ = bSilent;
     }
 
+    virtual
+    std::unique_ptr<beast::Journal::Sink>
+    makeSink(std::string const& partition,
+        beast::Journal::Severity startingLevel);
+
 public:
     static
     LogSeverity

src/ripple/basics/impl/Log.cpp

@@ -119,17 +119,16 @@ Logs::open (boost::filesystem::path const& pathToLogFile)
     return file_.open(pathToLogFile);
 }
 
-Logs::Sink&
+beast::Journal::Sink&
 Logs::get (std::string const& name)
 {
     std::lock_guard <std::mutex> lock (mutex_);
-    auto const result (sinks_.emplace (std::piecewise_construct,
+    auto const result =
-        std::forward_as_tuple(name), std::forward_as_tuple (name,
+        sinks_.emplace(name, makeSink(name, level_));
-            level_, *this)));
+    return *result.first->second;
-    return result.first->second;
 }
 
-Logs::Sink&
+beast::Journal::Sink&
 Logs::operator[] (std::string const& name)
 {
     return get(name);
@@ -153,7 +152,7 @@ Logs::severity (beast::Journal::Severity level)
     std::lock_guard <std::mutex> lock (mutex_);
     level_ = level;
     for (auto& sink : sinks_)
-        sink.second.severity (level);
+        sink.second->severity (level);
 }
 
 std::vector<std::pair<std::string, std::string>>
@@ -164,7 +163,7 @@ Logs::partition_severities() const
     list.reserve (sinks_.size());
     for (auto const& e : sinks_)
         list.push_back(std::make_pair(e.first,
-            toString(fromSeverity(e.second.severity()))));
+            toString(fromSeverity(e.second->severity()))));
     return list;
 }
 
@@ -193,6 +192,14 @@ Logs::rotate()
     return "The log file could not be closed and reopened.";
 }
 
+std::unique_ptr<beast::Journal::Sink>
+Logs::makeSink(std::string const& name,
+    beast::Journal::Severity startingLevel)
+{
+    return std::make_unique<Sink>(
+        name, startingLevel, *this);
+}
+
 LogSeverity
 Logs::fromSeverity (beast::Journal::Severity level)
 {

src/ripple/test/jtx/Env.h

@@ -101,8 +101,10 @@ private:
         std::thread thread;
         std::unique_ptr<AbstractClient> client;
 
-        AppBundle (std::unique_ptr<Config> config);
+        AppBundle (beast::unit_test::suite& suite,
-        AppBundle (Application* app_);
+            std::unique_ptr<Config> config);
+        AppBundle (beast::unit_test::suite& suite,
+            Application* app_);
         ~AppBundle();
     };
 
@@ -139,11 +141,11 @@ public:
 
     // VFALCO Could wrap the suite::log in a Journal here
     template <class... Args>
-    Env (beast::unit_test::suite& test_,
+    Env (beast::unit_test::suite& suite_,
         std::unique_ptr<Config> config,
             Args&&... args)
-        : test (test_)
+        : test (suite_)
-        , bundle_ (std::move(config))
+        , bundle_ (suite_, std::move(config))
     {
         memoize(Account::master);
         Pathfinder::initPathTable();
@@ -151,9 +153,9 @@ public:
     }
 
     template <class... Args>
-    Env (beast::unit_test::suite& test_,
+    Env (beast::unit_test::suite& suite_,
             Args&&... args)
-        : Env(test_, []()
+        : Env(suite_, []()
             {
                 auto p = std::make_unique<Config>();
                 setupConfigForUnitTests(*p);

src/ripple/test/jtx/impl/Env.cpp

@@ -85,14 +85,69 @@ setupConfigForUnitTests (Config& cfg)
 
 namespace jtx {
 
-Env::AppBundle::AppBundle(Application* app_)
+class SuiteSink : public beast::Journal::Sink
+{
+    std::string partition_;
+    beast::unit_test::suite& suite_;
+
+public:
+    SuiteSink(std::string const& partition,
+            beast::unit_test::suite& suite)
+        : partition_(partition + " ")
+        , suite_ (suite)
+    {
+    }
+
+    void
+    write(beast::Journal::Severity level,
+        std::string const& text) override
+    {
+        std::string s;
+        switch(level)
+        {
+        case beast::Journal::kTrace:    s = "TRC:"; break;
+        case beast::Journal::kDebug:    s = "DBG:"; break;
+        case beast::Journal::kInfo:     s = "INF:"; break;
+        case beast::Journal::kWarning:  s = "WRN:"; break;
+        case beast::Journal::kError:    s = "ERR:"; break;
+        default:
+        case beast::Journal::kFatal:    s = "FTL:"; break;
+        }
+        suite_.log << s << partition_ << text;
+    }
+};
+
+class SuiteLogs : public Logs
+{
+    beast::unit_test::suite& suite_;
+
+public:
+    explicit
+    SuiteLogs(beast::unit_test::suite& suite)
+        : suite_(suite)
+    {
+    }
+
+    std::unique_ptr<beast::Journal::Sink>
+    makeSink(std::string const& partition,
+        beast::Journal::Severity startingLevel) override
+    {
+        return std::make_unique<SuiteSink>(partition, suite_);
+    }
+};
+
+//------------------------------------------------------------------------------
+
+Env::AppBundle::AppBundle(beast::unit_test::suite&,
+        Application* app_)
     : app(app_)
 {
 }
 
-Env::AppBundle::AppBundle(std::unique_ptr<Config> config)
+Env::AppBundle::AppBundle(beast::unit_test::suite& suite,
+    std::unique_ptr<Config> config)
 {
-    auto logs = std::make_unique<Logs>();
+    auto logs = std::make_unique<SuiteLogs>(suite);
     auto timeKeeper_ =
         std::make_unique<ManualTimeKeeper>();
     timeKeeper = timeKeeper_.get();
@@ -101,6 +156,7 @@ Env::AppBundle::AppBundle(std::unique_ptr<Config> config)
     owned = make_Application(std::move(config),
         std::move(logs), std::move(timeKeeper_));
     app = owned.get();
+    app->logs().severity(beast::Journal::kError);
     app->setup();
     timeKeeper->set(
         app->getLedgerMaster().getClosedLedger()->info().closeTime);