Make compiler JSON formatter more resilient to unknown keywords (#1194)

Signed-off-by: Juan Cruz Viotti <[email protected]>

src/jsonschema/compile_json.cc

@@ -310,17 +310,14 @@ auto compiler_template_format_compare(const JSON::String &left,
                    {"absoluteKeywordLocation", 4},
                    {"relativeSchemaLocation", 5},
                    {"relativeInstanceLocation", 6},
-                   {"location", 7},
-                   {"report", 8},
-                   {"dynamic", 9},
-                   {"children", 10}};
+                   {"report", 7},
+                   {"dynamic", 8},
+                   {"children", 9}};
 
-  // We define and control all of these keywords, so if we are missing
-  // some here, then we did something wrong?
-  assert(rank.contains(left));
-  assert(rank.contains(right));
-
-  return rank.at(left) < rank.at(right);
+  constexpr std::uint64_t DEFAULT_RANK{999};
+  const auto left_rank{rank.contains(left) ? rank.at(left) : DEFAULT_RANK};
+  const auto right_rank{rank.contains(right) ? rank.at(right) : DEFAULT_RANK};
+  return left_rank < right_rank;
 }
 
 } // namespace sourcemeta::jsontoolkit

test/jsonschema/jsonschema_template_format_test.cc

@@ -9,6 +9,7 @@ TEST(JSONSchema_compiler_template_format, example_1) {
     "absoluteKeywordLocation": "#/properties",
     "category": "annotation",
     "report": true,
+    "unknown": 1,
     "dynamic": true,
     "type": "public",
     "relativeSchemaLocation": "",
@@ -36,6 +37,7 @@ TEST(JSONSchema_compiler_template_format, example_1) {
   "relativeSchemaLocation": "",
   "relativeInstanceLocation": "",
   "report": true,
-  "dynamic": true
+  "dynamic": true,
+  "unknown": 1
 })JSON");
 }