Version v1.3.4 (prescottprue#96)

### Enhancements
* prescottprue#92, prescottprue#94 - Issue with query params containing numbers (thanks @biomorgoth)
* prescottprue#82, prescottprue#83 - dispatch parameter fixed in `unWatchEvent`
* prescottprue#89 - Separate sections added to docs for [`props.firebase`](http://react-redux-firebase.com/docs/api/props-firebase.html) and [`getFirebase`](http://react-redux-firebase.com/docs/api/get-firebase.html)

Author: prescottprue

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-redux-firebase",
-  "version": "1.3.3",
+  "version": "1.3.4",
   "description": "Redux integration for Firebase. Comes with a Higher Order Component for use with React.",
   "browser": "dist/react-redux-firebase.js",
   "main": "lib/index.js",

tests/unit/helpers.spec.js

@@ -282,8 +282,6 @@ describe('Helpers:', () => {
           { child: 'collaborators', root: rootName },
         ]
         // TODO: Test both children are populated
-        console.log('--------3', helpers.populatedDataToJS(exampleState, path, populates))
-        console.log('should have', exampleData.data[rootName])
         expect(helpers.populatedDataToJS(exampleState, `/${path}`, populates))
           .to
           .have

tests/unit/utils/actions.spec.js

@@ -5,7 +5,7 @@ import {
 const method = () => Promise.resolve()
 const failMethod = () => Promise.reject()
 const dispatch = () => {
-  console.log('dispatch called')
+  // console.log('dispatch called')
 }
 describe('Utils: Auth', () => {
   describe('wrapInDispatch', () => {

tests/unit/utils/query.spec.js

@@ -7,9 +7,47 @@ import {
   getQueryIdFromPath,
   applyParamsToQuery
 } from '../../../src/utils/query'
+const fakeFirebase = {
+  _: {
+    authUid: '123',
+    config: {
+      userProfile: 'users',
+      disableRedirectHandling: true,
+    },
+  },
+  database: () => ({
+    ref: () => ({
+      orderByValue: () => ({
+        on: () => ({ val: () => { some: 'obj' } }),
+        off: () => Promise.resolve({ val: () => { some: 'obj' }}),
+        once: () => Promise.resolve({ val: () => { some: 'obj' }})
+      }),
+      orderByPriority: () => ({
+        startAt: (startParam) => startParam,
+        toString: () => 'priority'
+      }),
+      orderByChild: (child) => ({
+        equalTo: (equalTo) => ({
+          child,
+          equalTo
+        }),
+        toString: () => child
+      }),
+      orderByKey: () => ({ }),
+      limitToFirst: () => ({ }),
+      limitToLast: () => ({ }),
+      equalTo: () => ({ }),
+      startAt: () => ({ }),
+      endAt: () => ({ }),
+    })
+  }),
+}
+let spy
+
 let createQueryFromParams = (queryParams) =>
-  applyParamsToQuery(queryParams, Firebase.database().ref())
+  applyParamsToQuery(queryParams, fakeFirebase.database().ref())
 const dispatch = () => {}
+let ref
 describe('Utils: Query', () => {
   describe('getWatchPath', () => {
     it('handles basic path', () => {
@@ -78,15 +116,92 @@ describe('Utils: Query', () => {
     it('orderByValue', () => {
       expect(createQueryFromParams(['orderByValue=uid'])).to.be.an.object
     })
-    it('orderByPriority', () => {
-      expect(createQueryFromParams(['orderByPriority=uid'])).to.be.an.object
+
+    describe('orderByPriority', () => {
+      it('handles single parameter', () => {
+        expect(createQueryFromParams(['orderByPriority']).toString())
+          .to.equal('priority')
+      })
+
+      describe('with startAt', () => {
+        it ('string containing number', () => {
+          const startAt = '123abc'
+          expect(createQueryFromParams(['orderByPriority', `startAt=${startAt}`]).toString())
+            .to.equal(startAt)
+        })
+      })
     })
+
     it('orderByKey', () => {
       expect(createQueryFromParams(['orderByKey'])).to.be.an.object
     })
-    it('orderByChild', () => {
-      expect(createQueryFromParams(['orderByChild=uid'])).to.be.an.object
+
+    describe('orderByChild', () => {
+      it('handles single parameter', () => {
+        const queryParams = createQueryFromParams(['orderByChild=uid'])
+        expect(queryParams.toString()).to.equal('uid')
+      })
+
+      describe('with equalTo', () => {
+        it('number', () => {
+          const child = 'emailAddress'
+          const equalTo = 1
+          const queryParams = createQueryFromParams([`orderByChild=${child}`, `equalTo=${equalTo}`])
+          expect(queryParams.child)
+            .to
+            .equal(child)
+          expect(queryParams.equalTo)
+            .to
+            .equal(equalTo)
+        })
+        it('boolean', () => {
+          const child = 'completed'
+          const equalTo = false
+          const queryParams = createQueryFromParams([`orderByChild=${child}`, `equalTo=${equalTo}`])
+          expect(queryParams.child)
+            .to
+            .equal(child)
+          expect(queryParams.equalTo)
+            .to
+            .equal(equalTo)
+        })
+        it('string containing a boolean', () => {
+          const child = 'emailAddress'
+          const equalTo = 'true'
+          const queryParams = createQueryFromParams([`orderByChild=${child}`, `equalTo=${equalTo}`])
+          expect(queryParams.child)
+            .to
+            .equal(child)
+          expect(queryParams.equalTo)
+            .to
+            .equal(true)
+        })
+        it('string containing null', () => {
+          const child = 'emailAddress'
+          const equalTo = 'null'
+          const queryParams = createQueryFromParams([`orderByChild=${child}`, `equalTo=${equalTo}`])
+          expect(queryParams.child)
+            .to
+            .equal(child)
+          expect(queryParams.equalTo)
+            .to
+            .equal(null)
+        })
+        it('string containing a number', () => {
+          const child = 'emailAddress'
+          const equalTo = '[email protected]'
+          const queryParams = createQueryFromParams([`orderByChild=${child}`, `equalTo=${equalTo}`])
+          expect(queryParams.child)
+            .to
+            .equal(child)
+          expect(queryParams.equalTo)
+            .to
+            .equal(equalTo)
+        })
+      })
+
     })
+
     it('limitToFirst', () => {
       expect(createQueryFromParams(['limitToFirst=1'])).to.be.an.object
     })