Renames StringSanitization.java to StringUtil.java.

Author: sebright2

BUILD

@@ -162,8 +162,8 @@ java_test(
 )
 
 java_test(
-    name = "StringSanitizationTest",
-    srcs = ["core/javatests/com/google/census/StringSanitizationTest.java"],
+    name = "StringUtilTest",
+    srcs = ["core/javatests/com/google/census/StringUtilTest.java"],
     deps = [
         ":census-core",
         ":census-core_native",

core/java/com/google/census/MetricName.java

@@ -19,10 +19,10 @@
  * MetricName's are {@link String}s with enforced restrictions.
  */
 public final class MetricName {
-  public static final int MAX_LENGTH = StringSanitization.MAX_LENGTH;
+  public static final int MAX_LENGTH = StringUtil.MAX_LENGTH;
 
   public MetricName(String name) {
-    this.name = StringSanitization.sanitize(name);
+    this.name = StringUtil.sanitize(name);
   }
 
   @Override

…om/google/census/StringSanitization.java …e/java/com/google/census/StringUtil.javacore/java/com/google/census/StringSanitization.java renamed to core/java/com/google/census/StringUtil.java core/java/com/google/census/StringSanitization.java renamed to core/java/com/google/census/StringUtil.java

@@ -14,9 +14,9 @@
 package com.google.census;
 
 /**
- * Utility methods for sanitizing tag keys, tag values, and metric names.
+ * Utility methods for working with tag keys, tag values, and metric names.
  */
-final class StringSanitization {
+final class StringUtil {
 
   static final int MAX_LENGTH = 255;
   static final char UNPRINTABLE_CHAR_SUBSTITUTE = '_';
@@ -50,7 +50,7 @@ private static boolean isPrintableChar(char c) {
   }
 
   //Visible for testing
-  StringSanitization() {
+  StringUtil() {
     throw new AssertionError();
   }
 }

core/java/com/google/census/TagKey.java

@@ -19,10 +19,10 @@
  * TagKey's are {@link String}s with enforced restrictions.
  */
 public final class TagKey {
-  public static final int MAX_LENGTH = StringSanitization.MAX_LENGTH;
+  public static final int MAX_LENGTH = StringUtil.MAX_LENGTH;
 
   public TagKey(String key) {
-    this.key = StringSanitization.sanitize(key);
+    this.key = StringUtil.sanitize(key);
   }
 
   @Override

core/java/com/google/census/TagValue.java

@@ -19,10 +19,10 @@
  * TagValue's are {@link String}s with enforced restrictions.
  */
 public final class TagValue {
-  public static final int MAX_LENGTH = StringSanitization.MAX_LENGTH;
+  public static final int MAX_LENGTH = StringUtil.MAX_LENGTH;
 
   public TagValue(String value) {
-    this.value = StringSanitization.sanitize(value);
+    this.value = StringUtil.sanitize(value);
   }
 
   @Override

core/javatests/com/google/census/MetricNameTest.java

@@ -39,8 +39,8 @@ public void testNameMaxLength() {
   @Test
   public void testNameBadChar() {
     assertThat(new MetricName("\2ab\3cd").toString())
-        .isEqualTo(StringSanitization.UNPRINTABLE_CHAR_SUBSTITUTE + "ab"
-                 + StringSanitization.UNPRINTABLE_CHAR_SUBSTITUTE + "cd");
+        .isEqualTo(StringUtil.UNPRINTABLE_CHAR_SUBSTITUTE + "ab"
+                 + StringUtil.UNPRINTABLE_CHAR_SUBSTITUTE + "cd");
   }
 
   @Test

…oogle/census/StringSanitizationTest.java …ts/com/google/census/StringUtilTest.javacore/javatests/com/google/census/StringSanitizationTest.java renamed to core/javatests/com/google/census/StringUtilTest.java core/javatests/com/google/census/StringSanitizationTest.java renamed to core/javatests/com/google/census/StringUtilTest.java

@@ -24,24 +24,24 @@
  * Tests for {@link StringUtil}.
  */
 @RunWith(JUnit4.class)
-public final class StringSanitizationTest {
+public final class StringUtilTest {
   @Test
   public void testMaxLength() {
-    char[] string = new char[StringSanitization.MAX_LENGTH];
-    char[] truncString = new char[StringSanitization.MAX_LENGTH + 10];
+    char[] string = new char[StringUtil.MAX_LENGTH];
+    char[] truncString = new char[StringUtil.MAX_LENGTH + 10];
     Arrays.fill(string, 'v');
     Arrays.fill(truncString, 'v');
-    assertThat(StringSanitization.sanitize(new String(truncString))).isEqualTo(new String(string));
+    assertThat(StringUtil.sanitize(new String(truncString))).isEqualTo(new String(string));
   }
 
   @Test
   public void testBadChar() {
     String string = "\2ab\3cd";
-    assertThat(StringSanitization.sanitize(string).toString()).isEqualTo("_ab_cd");
+    assertThat(StringUtil.sanitize(string).toString()).isEqualTo("_ab_cd");
   }
 
   @Test(expected=AssertionError.class)
   public void testConstructor() {
-    new StringSanitization();
+    new StringUtil();
   }
 }

core/javatests/com/google/census/TagKeyTest.java

@@ -41,8 +41,8 @@ public void testKeyMaxLength() {
   public void testKeyBadChar() {
     String key = "\2ab\3cd";
     assertThat(new TagKey(key).toString())
-        .isEqualTo(StringSanitization.UNPRINTABLE_CHAR_SUBSTITUTE + "ab"
-                 + StringSanitization.UNPRINTABLE_CHAR_SUBSTITUTE + "cd");
+        .isEqualTo(StringUtil.UNPRINTABLE_CHAR_SUBSTITUTE + "ab"
+                 + StringUtil.UNPRINTABLE_CHAR_SUBSTITUTE + "cd");
   }
 
   @Test

core/javatests/com/google/census/TagValueTest.java

@@ -40,8 +40,8 @@ public void testValueMaxLength() {
   public void testValueBadChar() {
     String value = "\2ab\3cd";
     assertThat(new TagValue(value).toString())
-        .isEqualTo(StringSanitization.UNPRINTABLE_CHAR_SUBSTITUTE + "ab"
-                 + StringSanitization.UNPRINTABLE_CHAR_SUBSTITUTE + "cd");
+        .isEqualTo(StringUtil.UNPRINTABLE_CHAR_SUBSTITUTE + "ab"
+                 + StringUtil.UNPRINTABLE_CHAR_SUBSTITUTE + "cd");
   }
 
   @Test