Aruha 473 check size of events

src/main/java/org/zalando/nakadi/domain/BatchFactory.java

@@ -13,10 +13,35 @@ public static List<BatchItem> from(final String events) {
         int brackets = 0;
         boolean insideQuote = false;
         boolean escaped = false;
-        if ((!events.startsWith("[")) || (!events.endsWith("]"))) {
-            throw new JSONException("Array must be surrounded with square brackets");
+        int start = 0;
+        final int length = events.length();
+        int end = length - 1;
+
+        while ((events.charAt(start) == ' '
+                || events.charAt(start) == '\t'
+                || events.charAt(start) == '\n'
+                || events.charAt(start) == '\r')
+                && start < end) {
+            start++;
+        }
+        while ((events.charAt(end) == ' '
+                || events.charAt(end) == '\t'
+                || events.charAt(end) == '\n'
+                || events.charAt(end) == '\r')
+                && end > start) {
+            end--;
+        }
+        if (!(events.charAt(start) == '[')) {
+            throw new JSONException(String.format("Unexpected character %s in position %d, expected '['",
+                    events.charAt(start), start));
         }
-        for (int i = 1; i < events.length() - 1; i++) {
+        start++;
+        if (!(events.charAt(end) == ']')) {
+            throw new JSONException(String.format("Unexpected character %s in position %d, expected ']'",
+                    events.charAt(end), end));
+        }
+
+        for (int i = start; i < end; i++) {
             if (!escaped && events.charAt(i) == '"') {
                 if (insideQuote) {
                     insideQuote = false;

src/test/java/org/zalando/nakadi/domain/BatchFactoryTest.java

@@ -99,6 +99,13 @@ public void testSpacesBetweenEvents() {
         assertEquals("{\"name\":\"MyOtherEvent\"}", batch.get(1).getEvent().toString());
     }
 
+    @Test
+    public void testEmptyCharactersAroundArray() {
+        final String events = "\t [{\"name\":\"MyEvent\"},{\"name\":\"MyOtherEvent\"}]\n\n";
+        final List<BatchItem> batch = BatchFactory.from(events);
+        assertEquals(2, batch.size());
+    }
+
     @Test
     public void testGarbageBetweenEvents() {
         final String events = "[{\"name\":\"MyEvent\"},atb#{\"name\":\"MyOtherEvent\"}]";