Changeset 15864 in josm for trunk/src/org

Timestamp:

2020-02-16T15:46:03+01:00 (5 years ago)

Author:

simon04

Message:

UtilsTest: extend

Based on test coverage and mutation testing using Pitest.

File:

• trunk/src/org/openstreetmap/josm/tools/Utils.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/tools/Utils.java

@@ -1076 +1076 @@
      * @param maxLength maximum number of characters to keep (not including the "...")
      * @return the shortened string
+     * @throws IllegalArgumentException if maxLength is less than the length of "..."
      */
     public static String shortenString(String s, int maxLength) {
+        final String ellipses = "...";
+        CheckParameterUtil.ensureThat(maxLength >= ellipses.length(), "maxLength is shorter than " + ellipses.length());
         if (s != null && s.length() > maxLength) {
-            return s.substring(0, maxLength - 3) + "...";
+            return s.substring(0, maxLength - ellipses.length()) + ellipses;
         } else {
             return s;