test(ids): make sure items in collections have an id

Make sure they're non-enumerable, etc pretty much like documents

Author: posva

test/collection.spec.js

@@ -2,6 +2,7 @@ import Vuefire from '../src'
 import {
   db,
   tick,
+  Key,
   Vue
 } from './helpers'
 
@@ -67,3 +68,19 @@ test('unbinds when the instance is destroyed', async () => {
     expect(vm.items).toEqual([])
   }).not.toThrow()
 })
+
+test('adds non-enumerable id', async () => {
+  const a = await collection.doc(new Key('u0'))
+  const b = await collection.doc(new Key('u1'))
+  await a.update({})
+  await b.update({})
+  expect(vm.items.length).toBe(2)
+  vm.items.forEach((item, i) => {
+    expect(Object.getOwnPropertyDescriptor(item, 'id')).toEqual({
+      configurable: false,
+      enumerable: false,
+      writable: false,
+      value: `u${i}`
+    })
+  })
+})