Improve context reporting

If we fail at first step, we end up emitting an empty string, which
makes the output not nice. Quote the emitted string.

Change-Id: Ic54f7fd69a902de03205355b37fdc72d9979c051
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/data/yang-data-util/src/main/java/org/opendaylight/yangtools/yang/data/util/AbstractStringInstanceIdentifierCodec.java b/data/yang-data-util/src/main/java/org/opendaylight/yangtools/yang/data/util/AbstractStringInstanceIdentifierCodec.java
index b404a2d7287d214f1612322a9f0cb35fa2d6d046..d5a0672f6e877b7e5916ddee504fde7a47d3ba11 100644 (file)
--- a/data/yang-data-util/src/main/java/org/opendaylight/yangtools/yang/data/util/AbstractStringInstanceIdentifierCodec.java
+++ b/data/yang-data-util/src/main/java/org/opendaylight/yangtools/yang/data/util/AbstractStringInstanceIdentifierCodec.java
@@ -7,7 +7,6 @@
  */
 package org.opendaylight.yangtools.yang.data.util;
 
-import static com.google.common.base.Preconditions.checkArgument;
 import static java.util.Objects.requireNonNull;
 
 import com.google.common.annotations.Beta;
@@ -50,8 +49,10 @@ public abstract class AbstractStringInstanceIdentifierCodec extends AbstractName
         QNameModule lastModule = null;
         for (var arg : data.getPathArguments()) {
             current = current instanceof Composite composite ? composite.childByArg(arg) : null;
-            checkArgument(current != null, "Invalid input %s: schema for argument %s (after %s) not found", data, arg,
-                    sb);
+            if (current == null) {
+                throw new IllegalArgumentException(
+                    "Invalid input %s: schema for argument %s (after \"%s\") not found".formatted(data, arg, sb));
+            }
 
             if (current instanceof PathMixin) {
                 /*