Add test for immutable nodes. Update change log.

Author: oschwald

CHANGELOG.md

@@ -1,6 +1,15 @@
 CHANGELOG
 =========
 
+1.2.0 (2016-01-13)
+------------------
+
+* `JsonNode` containers returned by the `get(ip)` are now backed by
+  unmodifiable collections. Any mutation done to them will fail with an
+  `UnsupportedOperationException` exception. This allows safe caching of the
+  nodes to be done without doing a deep copy of the cached data. Pull request
+  by Viktor Szathmáry. GitHub #24.
+
 1.1.0 (2016-01-04)
 ------------------
 

src/test/java/com/maxmind/db/ReaderTest.java

@@ -1,32 +1,24 @@
 package com.maxmind.db;
 
-import static org.hamcrest.CoreMatchers.containsString;
-import static org.junit.Assert.assertArrayEquals;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
+import com.fasterxml.jackson.databind.JsonNode;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.databind.node.ArrayNode;
+import com.fasterxml.jackson.databind.node.ObjectNode;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.rules.ExpectedException;
 
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
 import java.math.BigInteger;
 import java.net.InetAddress;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Calendar;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.rules.ExpectedException;
+import java.util.*;
 
-import com.fasterxml.jackson.databind.JsonNode;
-import com.fasterxml.jackson.databind.ObjectMapper;
-import com.fasterxml.jackson.databind.node.ObjectNode;
+import static org.hamcrest.CoreMatchers.containsString;
+import static org.junit.Assert.*;
 
 public class ReaderTest {
     private final ObjectMapper om = new ObjectMapper();
@@ -243,6 +235,24 @@ private void testBrokenDataPointer(Reader reader) throws IOException {
         reader.get(InetAddress.getByName("1.1.1.16"));
     }
 
+    @Test
+    public void testObjectNodeMutation() throws IOException {
+        Reader reader = new Reader(getFile("MaxMind-DB-test-decoder.mmdb"));
+        ObjectNode record = (ObjectNode) reader.get(InetAddress.getByName("::1.1.1.0"));
+
+        thrown.expect(UnsupportedOperationException.class);
+        record.put("Test", "value");
+    }
+
+    @Test
+    public void testArrayNodeMutation() throws IOException {
+        Reader reader = new Reader(getFile("MaxMind-DB-test-decoder.mmdb"));
+        ObjectNode record = (ObjectNode) reader.get(InetAddress.getByName("::1.1.1.0"));
+
+        thrown.expect(UnsupportedOperationException.class);
+        ((ArrayNode) record.get("array")).add(1);
+    }
+
     @Test
     public void testClosedReaderThrowsException() throws IOException {
         Reader reader = new Reader(getFile("MaxMind-DB-test-decoder.mmdb"));