Verify revision string 84/110684/1
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 12 Mar 2024 21:37:39 +0000 (22:37 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Tue, 12 Mar 2024 21:40:28 +0000 (22:40 +0100)
yang.common.Revision has a guaranteed format, hence if toString() does
not match, it is not an IAE, but rather a VerifyException pointing out a
violation of our expectations.

Change-Id: Ieabc84f40df3c56bcc7690c2c277cd95583bf1de
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
binding/yang-binding/src/main/java/org/opendaylight/yangtools/yang/binding/contract/Naming.java

index 77261584747fe126dfc56fd49f6da2fd09fedb00..ccedb54c957da8f9179b97856604a8d37ee60537 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.yangtools.yang.binding.contract;
 
 import static com.google.common.base.Preconditions.checkArgument;
+import static com.google.common.base.Verify.verify;
 import static java.util.Objects.requireNonNull;
 
 import com.google.common.annotations.Beta;
@@ -207,7 +208,7 @@ public final class Naming {
             // Revision is in format 2017-10-26, we want the output to be 171026, which is a matter of picking the
             // right characters.
             final String rev = revision.toString();
-            checkArgument(rev.length() == 10, "Unsupported revision %s", rev);
+            verify(rev.length() == 10, "Revision.toString() resulted in unexpected '%s'", rev);
             builder.append("rev").append(rev, 2, 4).append(rev, 5, 7).append(rev.substring(8));
         } else {
             // No-revision packages are special