Try to display multi-line messages with github problem matcher

build-scripts/build.sh

@@ -9,7 +9,7 @@ num_jobs=3
 function run_tests
 {
     # --min-duration shows timing lines for tests with a duration of at least that many seconds.
-    $WINE "$@" --min-duration 0.2 --use-colour yes --rng-seed time $EXTRA_TEST_OPTS
+    $WINE "$@" --min-duration 0.2 --use-colour yes --rng-seed time --error-format=github-problem-matcher $EXTRA_TEST_OPTS
 }
 
 # We might need binaries installed via pip, so ensure that our personal bin dir is on the PATH

build-scripts/requirements.sh

@@ -20,7 +20,6 @@ function just_json
 # (See https://github.com/actions/toolkit/blob/master/docs/problem-matchers.md)
 echo "::add-matcher::build-scripts/problem-matchers/catch2.json"
 echo "::add-matcher::build-scripts/problem-matchers/debugmsg.json"
-echo "::add-matcher::build-scripts/problem-matchers/json.json"
 
 if which travis_retry &>/dev/null
 then

data/json/achievements.json

@@ -3,7 +3,8 @@
     "id": "achievement_kill_zombie",
     "type": "achievement",
     "name": "One down, billions to go\u2026",
-    "requirements": [ { "event_statistic": "num_avatar_zombie_kills", "is": ">=", "target": 1 } ]
+    "requirements": [ { "event_statistic": "num_avatar_zombie_kills", "is": ">=", "target": 1 } ],
+    "foo": "bar"
   },
   {
     "id": "achievement_kill_cyborg",

src/cached_options.cpp

@@ -12,3 +12,4 @@ bool use_tiles;
 test_mode_spilling_action_t test_mode_spilling_action = test_mode_spilling_action_t::spill_all;
 bool direct3d_mode;
 bool pixel_minimap_option;
+error_log_format_t error_log_format = error_log_format_t::human_readable;

src/cached_options.h

@@ -27,4 +27,14 @@ extern test_mode_spilling_action_t test_mode_spilling_action;
 
 extern bool direct3d_mode;
 
+enum class error_log_format_t {
+    human_readable,
+    github_problem_matcher,
+};
+#ifndef CATA_IN_TOOL
+extern error_log_format_t error_log_format;
+#else
+constexpr error_log_format_t error_log_format = error_log_format_t::human_readable;
+#endif
+
 #endif // CATA_SRC_CACHED_OPTIONS_H

src/json.cpp

@@ -20,6 +20,7 @@
 #include "cached_options.h"
 #include "cata_utility.h"
 #include "debug.h"
+#include "output.h"
 #include "string_formatter.h"
 
 // JSON parsing and serialization tools for Cataclysm-DDA.
@@ -1671,14 +1672,28 @@ std::string JsonIn::line_number( int offset_modifier )
     }
     seek( pos );
     std::stringstream ret;
-    ret << name << ":" << line << ":" << offset;
+    switch( error_log_format ) {
+        case error_log_format_t::human_readable:
+            ret << name << ":" << line << ":" << offset;
+            break;
+        case error_log_format_t::github_problem_matcher:
+            ret << "file=" << name << ",line=" << line << ",col=" << offset;
+            break;
+    }
     return ret.str();
 }
 
 void JsonIn::error( const std::string &message, int offset )
 {
     std::ostringstream err;
-    err << "Json error: " << line_number( offset ) << ": " << message;
+    switch( error_log_format ) {
+        case error_log_format_t::human_readable:
+            err << "Json error: " << line_number( offset ) << ": " << message;
+            break;
+        case error_log_format_t::github_problem_matcher:
+            err << "::error " << line_number( offset ) << "::" << message;
+            break;
+    }
     // if we can't get more info from the stream don't try
     if( !stream->good() ) {
         throw JsonError( err.str() );
@@ -2061,8 +2076,20 @@ void JsonOut::null_member( const std::string &name )
     write_null();
 }
 
+static std::string replace_newlines( std::string str )
+{
+    switch( error_log_format ) {
+        case error_log_format_t::human_readable:
+            break;
+        case error_log_format_t::github_problem_matcher:
+            replace_substring( str, "\n", "%0A", true );
+            break;
+    }
+    return str;
+}
+
 JsonError::JsonError( const std::string &msg )
-    : std::runtime_error( msg )
+    : std::runtime_error( replace_newlines( msg ) )
 {
 }
 

tests/json_test.cpp

@@ -11,6 +11,8 @@
 #include <vector>
 
 #include "mutation.h"
+#include "cached_options.h"
+#include "cata_utility.h"
 #include "colony.h"
 #include "string_formatter.h"
 #include "type_id.h"
@@ -255,6 +257,8 @@ TEST_CASE( "translation_text_style_check", "[json][translation]" )
 {
     // this test case is mainly for checking the format of debug messages.
     // the text style check itself is tested in the lit test of clang-tidy.
+    restore_on_out_of_scope<error_log_format_t> restore_error_log_format( error_log_format );
+    error_log_format = error_log_format_t::human_readable;
 
     // string, ascii
     test_translation_text_style_check(
@@ -400,6 +404,9 @@ TEST_CASE( "translation_text_style_check", "[json][translation]" )
 
 TEST_CASE( "translation_text_style_check_error_recovery", "[json][translation]" )
 {
+    restore_on_out_of_scope<error_log_format_t> restore_error_log_format( error_log_format );
+    error_log_format = error_log_format_t::human_readable;
+
     SECTION( "string" ) {
         const std::string json =
             R"([)" "\n"
@@ -499,6 +506,9 @@ static void test_string_error_throws_matches( Matcher &&matcher, const std::stri
 
 TEST_CASE( "jsonin_get_string", "[json]" )
 {
+    restore_on_out_of_scope<error_log_format_t> restore_error_log_format( error_log_format );
+    error_log_format = error_log_format_t::human_readable;
+
     // read plain text
     test_get_string( "foo", R"("foo")" );
     // ignore starting spaces

tests/test_main.cpp

@@ -285,6 +285,16 @@ int main( int argc, const char *argv[] )
 
     std::string user_dir = extract_user_dir( arg_vec );
 
+    std::string error_fmt = extract_argument( arg_vec, "--error-format=" );
+    if( error_fmt == "github-problem-matcher" ) {
+        error_log_format = error_log_format_t::github_problem_matcher;
+    } else if( error_fmt == "human-readable" ) {
+        error_log_format = error_log_format_t::human_readable;
+    } else {
+        printf( "Unknown format %s", error_fmt.c_str() );
+        return EXIT_FAILURE;
+    }
+
     // Note: this must not be invoked before all DDA-specific flags are stripped from arg_vec!
     int result = session.applyCommandLine( arg_vec.size(), &arg_vec[0] );
     if( result != 0 || session.configData().showHelp ) {
@@ -294,6 +304,9 @@ int main( int argc, const char *argv[] )
         printf( "  -D, --drop-world             Don't save the world on test failure.\n" );
         printf( "  --option_overrides=n:v[,…]   Name-value pairs of game options for tests.\n" );
         printf( "                               (overrides config/options.json values)\n" );
+        printf( "  --error-format=<value>       Format of error messages. Possible values are:\n" );
+        printf( "                                   human-readable (default)\n" );
+        printf( "                                   github-problem-matcher\n" );
         return result;
     }