Include field name when diagnosing field not found:

Closes #3263
XRPLF · Mar 5, 2020 · 053b6d9 · 053b6d9
1 parent f76a5a3
commit 053b6d9
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 15 deletions.
diff --git a/src/ripple/protocol/STObject.h b/src/ripple/protocol/STObject.h
@@ -41,7 +41,11 @@ namespace ripple {
 
 class STArray;
 
-//------------------------------------------------------------------------------
+inline void
+throwFieldNotFound(SField const& field)
+{
+    Throw<std::runtime_error>("Field not found: " + field.getName());
+}
 
 class STObject
     : public STBase
@@ -492,7 +496,7 @@ class STObject
         STBase* rf = getPField (field, true);
 
         if (! rf)
-            Throw<std::runtime_error> ("Field not found");
+            throwFieldNotFound(field);
 
         if (rf->getSType () == STI_NOTPRESENT)
             rf = makeFieldPresent (field);
@@ -552,7 +556,7 @@ class STObject
         const STBase* rf = peekAtPField (field);
 
         if (! rf)
-            Throw<std::runtime_error> ("Field not found");
+            throwFieldNotFound(field);
 
         SerializedTypeID id = rf->getSType ();
 
@@ -578,7 +582,7 @@ class STObject
         const STBase* rf = peekAtPField (field);
 
         if (! rf)
-            Throw<std::runtime_error> ("Field not found");
+            throwFieldNotFound(field);
 
         SerializedTypeID id = rf->getSType ();
 
@@ -602,7 +606,7 @@ class STObject
         STBase* rf = getPField (field, true);
 
         if (! rf)
-            Throw<std::runtime_error> ("Field not found");
+            throwFieldNotFound(field);
 
         if (rf->getSType () == STI_NOTPRESENT)
             rf = makeFieldPresent (field);
@@ -622,7 +626,7 @@ class STObject
         STBase* rf = getPField (field, true);
 
         if (! rf)
-            Throw<std::runtime_error> ("Field not found");
+            throwFieldNotFound(field);
 
         if (rf->getSType () == STI_NOTPRESENT)
             rf = makeFieldPresent (field);
@@ -642,7 +646,7 @@ class STObject
         STBase* rf = getPField (field, true);
 
         if (! rf)
-            Throw<std::runtime_error> ("Field not found");
+            throwFieldNotFound(field);
 
         if (rf->getSType () == STI_NOTPRESENT)
             rf = makeFieldPresent (field);

diff --git a/src/ripple/protocol/impl/STObject.cpp b/src/ripple/protocol/impl/STObject.cpp
@@ -343,7 +343,7 @@ const STBase& STObject::peekAtField (SField const& field) const
     int index = getFieldIndex (field);
 
     if (index == -1)
-        Throw<std::runtime_error> ("Field not found");
+        throwFieldNotFound(field);
 
     return peekAtIndex (index);
 }
@@ -353,7 +353,7 @@ STBase& STObject::getField (SField const& field)
     int index = getFieldIndex (field);
 
     if (index == -1)
-        Throw<std::runtime_error> ("Field not found");
+        throwFieldNotFound(field);
 
     return getIndex (index);
 }
@@ -453,7 +453,7 @@ STBase* STObject::makeFieldPresent (SField const& field)
     if (index == -1)
     {
         if (!isFree ())
-            Throw<std::runtime_error> ("Field not found");
+            throwFieldNotFound(field);
 
         return getPIndex (emplace_back(detail::nonPresentObject, field));
     }
@@ -473,7 +473,7 @@ void STObject::makeFieldAbsent (SField const& field)
     int index = getFieldIndex (field);
 
     if (index == -1)
-        Throw<std::runtime_error> ("Field not found");
+        throwFieldNotFound(field);
 
     const STBase& f = peekAtIndex (index);
 

diff --git a/src/test/protocol/STTx_test.cpp b/src/test/protocol/STTx_test.cpp
@@ -29,9 +29,22 @@
 #include <ripple/protocol/messages.h>
 
 #include <memory>
+#include <regex>
 
 namespace ripple {
 
+/**
+ * Return true if the string loosely matches the regex.
+ *
+ * Meant for testing human-readable strings that may change over time.
+ */
+inline bool
+matches(char const* string, char const* regex)
+{
+    return std::regex_search(
+        string, std::basic_regex<char>(regex, std::regex_constants::icase));
+}
+
 class STTx_test : public beast::unit_test::suite
 {
 public:
@@ -1283,8 +1296,7 @@ class STTx_test : public beast::unit_test::suite
                 }
                 catch (std::runtime_error const& ex)
                 {
-                    BEAST_EXPECT (
-                        std::strcmp (ex.what(), "Field not found") == 0);
+                    BEAST_EXPECT(matches(ex.what(), "field not found"));
                 }
             }
 
@@ -1348,8 +1360,7 @@ class STTx_test : public beast::unit_test::suite
                 }
                 catch (std::runtime_error const& ex)
                 {
-                    BEAST_EXPECT (
-                        std::strcmp (ex.what(), "Field not found") == 0);
+                    BEAST_EXPECT(matches(ex.what(), "field not found"));
                 }
             }