moved performance tests under the metrics package and replaced the invalid JSON

string with \n with a valid one.

gson/src/test/java/com/google/gson/functional/StringTest.java

@@ -1,7 +1,6 @@
 package com.google.gson.functional;
 
 import com.google.gson.Gson;
-import com.google.gson.JsonParseException;
 
 import junit.framework.TestCase;
 
@@ -103,48 +102,4 @@ public class StringTest extends TestCase {
     String actual = gson.fromJson("[\"" + value + "\"]", String.class);
     assertEquals(value, actual);
   }
-
-  public void testReallyLongStringsDeserialization() throws Exception {
-    StringBuilder sb = new StringBuilder(8096);
-    sb.append("Once upon a time there was a really long string that caused a StackOverFlowError\n");
-    sb.append("and now it is fixed and instead throws a JsonParserException.....Yippie!!!\n");
-    sb.append("Wow....that is a really long string that is meant to be an exception stack trace, ");
-    sb.append("but is not :( \n\n\n\n\n\n.");
-    sb.append("lalalalala \n\n\n.");
-    sb.append("C'est la vie!!! \n\n\n\n\n");
-
-    for (int i = 0; i < 10; i++) {
-      sb.append(sb.toString());
-    }
-
-    while (true) {
-      try {
-        String stackTrace = sb.toString();
-        sb.append(stackTrace);
-        String json = "{\"message\":\"Error message.\","
-          + "\"stackTrace\":\"" + stackTrace + "\"}";
-        parseLongJson(json);
-      } catch (JsonParseException expected) {
-        break;
-      }
-    }
-  }
-  
-  private void parseLongJson(String json) throws JsonParseException {
-    ExceptionHolder target = gson.fromJson(json, ExceptionHolder.class);
-    assertTrue(target.message.contains("Error"));
-    assertTrue(target.stackTrace.contains("Yippie"));
-  }
-
-  private static class ExceptionHolder {
-    public final String message;
-    public final String stackTrace;
-    public ExceptionHolder() {
-      this("", "");
-    }
-    public ExceptionHolder(String message, String stackTrace) {
-      this.message = message;
-      this.stackTrace = stackTrace;
-    }
-  }
 }

gson/src/test/java/com/google/gson/metrics/PerformanceTest.java

@@ -0,0 +1,62 @@
+package com.google.gson.metrics;
+
+import com.google.gson.Gson;
+import com.google.gson.JsonParseException;
+
+import junit.framework.TestCase;
+
+/**
+ * Tests to measure performance for Gson. All tests in this file will be disabled in code. To run
+ * them remove disabled_ prefix from the tests and run them.
+ * 
+ * @author Inderjeet Singh
+ * @author Joel Leitch
+ */
+public class PerformanceTest extends TestCase {
+  private Gson gson;
+
+  @Override
+  protected void setUp() throws Exception {
+    super.setUp();
+    gson = new Gson();
+  }
+  
+  public void testDummy() {    
+    // This is here to prevent Junit for complaining when we disable all tests.
+  }
+
+  public void disabled_testStringDeserializationPerformance() {    
+    StringBuilder sb = new StringBuilder(8096);
+    sb.append("Error Yippie");
+
+    while (true) {
+      try {
+        String stackTrace = sb.toString();
+        sb.append(stackTrace);
+        String json = "{\"message\":\"Error message.\"," + "\"stackTrace\":\"" + stackTrace + "\"}";
+        parseLongJson(json);
+        System.out.println("Gson could handle a string of size: " + stackTrace.length());
+      } catch (JsonParseException expected) {
+        break;
+      }
+    }
+  }
+  
+  private void parseLongJson(String json) throws JsonParseException {
+    ExceptionHolder target = gson.fromJson(json, ExceptionHolder.class);
+    assertTrue(target.message.contains("Error"));
+    assertTrue(target.stackTrace.contains("Yippie"));
+  }
+
+  private static class ExceptionHolder {
+    public final String message;
+    public final String stackTrace;
+    public ExceptionHolder() {
+      this("", "");
+    }
+    public ExceptionHolder(String message, String stackTrace) {
+      this.message = message;
+      this.stackTrace = stackTrace;
+    }
+  }
+}