fix(FirestoreProvider): Fix error with applying settings to firestore

Setting the 'timestampsInSnapshots' setting needs to be in a call to 'settings', not passed to the 'firestore' function.

Fixes #33

src/FirestoreProvider.js

@@ -6,12 +6,12 @@ export default class FirestoreProvider extends Component {
   static propTypes = {
     firebase: PropTypes.object.isRequired,
     children: PropTypes.node.isRequired,
-    useTimestampsInSnapshots: PropTypes.bool.isRequired
+    useTimestampsInSnapshots: PropTypes.bool.isRequired,
   };
 
   static defaultProps = {
-    useTimestampsInSnapshots: false
-  }
+    useTimestampsInSnapshots: false,
+  };
 
   static childContextTypes = {
     firestoreDatabase: PropTypes.object.isRequired,
@@ -21,10 +21,11 @@ export default class FirestoreProvider extends Component {
   constructor(props) {
     super(props);
 
-    const { firebase } = props;
+    const { firebase, useTimestampsInSnapshots } = props;
+    const settings = { timestampsInSnapshots: useTimestampsInSnapshots };
 
     this.state = {
-      firestoreDatabase: firebase.firestore({timestampsInSnapshots: this.props.useTimestampsInSnapshots}),
+      firestoreDatabase: firebase.firestore().settings(settings),
       firestoreCache: new FirestoreCache(),
     };
   }

src/__tests__/helpers/firestore-utils.js

@@ -60,7 +60,11 @@ function createBaseMocks(snapshot, options) {
   const documentMock = jest
     .fn()
     .mockReturnValue({ onSnapshot: onSnapshotMock });
-  const firestoreMock = { collection: collectionMock, doc: documentMock };
+  const firestoreMock = {
+    settings: jest.fn(),
+    collection: collectionMock,
+    doc: documentMock,
+  };
 
   return {
     firestoreMock,

src/__tests__/provider.adds-context.js

@@ -22,14 +22,15 @@ const createChild = () => {
 test('database and cache are added to child context', () => {
   const Child = createChild();
   const firestore = {};
+  firestore.settings = jest.fn().mockReturnValue(firestore);
   const firebase = {
     firestore: jest.fn().mockReturnValueOnce(firestore),
   };
 
   const component = mount(
     <FirestoreProvider firebase={firebase}>
       <Child />
-    </FirestoreProvider>
+    </FirestoreProvider>,
   );
   const child = component.find('Child');
 

src/__tests__/provider.firestore-settings.js

@@ -0,0 +1,33 @@
+import React from 'react';
+import { mount } from 'enzyme';
+import { FirestoreProvider } from '../';
+
+test('sets timestampsInSnapshots correctly', () => {
+  const firestore = {};
+  firestore.settings = jest.fn().mockReturnValue(firestore);
+  const firebase = {
+    firestore: jest.fn().mockReturnValue(firestore),
+  };
+
+  mount(
+    <FirestoreProvider firebase={firebase}>
+      <div>Test</div>
+    </FirestoreProvider>,
+  );
+
+  expect(firestore.settings).toHaveBeenCalledTimes(1);
+  expect(firestore.settings).toHaveBeenCalledWith({
+    timestampsInSnapshots: false,
+  });
+
+  mount(
+    <FirestoreProvider firebase={firebase} useTimestampsInSnapshots>
+      <div>Test</div>
+    </FirestoreProvider>,
+  );
+
+  expect(firestore.settings).toHaveBeenCalledTimes(2);
+  expect(firestore.settings).toHaveBeenCalledWith({
+    timestampsInSnapshots: true,
+  });
+});