moved performance tests under the metrics package and replaced the invalid JSON
string with \n with a valid one.
diff --git a/gson/src/test/java/com/google/gson/functional/StringTest.java b/gson/src/test/java/com/google/gson/functional/StringTest.java
index ef8ad4b..165e3f4 100644
--- a/gson/src/test/java/com/google/gson/functional/StringTest.java
+++ b/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 @@
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;
- }
- }
}
diff --git a/gson/src/test/java/com/google/gson/metrics/PerformanceTest.java b/gson/src/test/java/com/google/gson/metrics/PerformanceTest.java
new file mode 100644
index 0000000..0696470
--- /dev/null
+++ b/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;
+ }
+ }
+}
