Merge pull request #788 from couchbase/feature/issue_784_forestdb_nullKey_2

Fixed #784 - Emitting null with the ForestDB engine causes segfault

Author: pasin

src/androidTest/java/com/couchbase/lite/ViewsTest.java

@@ -3231,4 +3231,43 @@ public void map(Map<String, Object> document, Emitter emitter) {
         Log.e(TAG, "Descending query: rows.getCount() = " + rows.getCount());
         assertEquals(0, rows.getCount());
     }
+
+    // - (void) test28_Nil_Key
+    public void testEmitWithNullKey() throws Exception {
+        // set up view
+        View view = database.getView("vu");
+        assertNotNull(view);
+        view.setMap(new Mapper() {
+            @Override
+            public void map(Map<String, Object> document, Emitter emitter) {
+                // null key -> ignored
+                emitter.emit(null,  "foo");
+                emitter.emit("name", "bar");
+            }
+        }, "1");
+        assertNotNull(view.getMap());
+        assertEquals(0, view.getCurrentTotalRows());
+
+        // insert 1 doc
+        Map<String, Object> dict = new HashMap<String, Object>();
+        dict.put("_id", "11111");
+        assertNotNull(putDoc(database, dict));
+
+        // regular query
+        Query query = view.createQuery();
+        assertNotNull(query);
+        QueryEnumerator e = query.run();
+        assertNotNull(e);
+        assertEquals(1, e.getCount());
+        QueryRow row = e.getRow(0);
+        assertNotNull(row);
+        assertEquals("name", row.getKey());
+        assertEquals("bar", row.getValue());
+
+        // query with null key. it should be ignored. this caused exception previously for sqlite
+        query.setKeys(Collections.singletonList(null));
+        e = query.run();
+        assertNotNull(e);
+        assertEquals(0, e.getCount());
+    }
 }