Fixing start_at, end_at, equal_to queries to accept False values (#96)

* Fixing start_at, end_at, equal_to queries to accept False values

* Updating docs

Author: hiranya911

firebase_admin/db.py

@@ -442,10 +442,10 @@ def start_at(self, start):
           Query: The updated Query instance.
 
         Raises:
-          ValueError: If the value is empty or None.
+          ValueError: If the value is None.
         """
-        if not start:
-            raise ValueError('Start value must not be empty or None.')
+        if start is None:
+            raise ValueError('Start value must not be None.')
         self._params['startAt'] = json.dumps(start)
         return self
 
@@ -462,10 +462,10 @@ def end_at(self, end):
           Query: The updated Query instance.
 
         Raises:
-          ValueError: If the value is empty or None.
+          ValueError: If the value is None.
         """
-        if not end:
-            raise ValueError('End value must not be empty or None.')
+        if end is None:
+            raise ValueError('End value must not be None.')
         self._params['endAt'] = json.dumps(end)
         return self
 
@@ -481,10 +481,10 @@ def equal_to(self, value):
           Query: The updated Query instance.
 
         Raises:
-          ValueError: If the value is empty or None.
+          ValueError: If the value is None.
         """
-        if not value:
-            raise ValueError('Equal to value must not be empty or None.')
+        if value is None:
+            raise ValueError('Equal to value must not be None.')
         self._params['equalTo'] = json.dumps(value)
         return self
 

tests/test_db.py

@@ -694,16 +694,31 @@ def test_start_at_none(self):
         with pytest.raises(ValueError):
             query.start_at(None)
 
+    @pytest.mark.parametrize('arg', ['', 'foo', True, False, 0, 1, dict()])
+    def test_valid_start_at(self, arg):
+        query = self.ref.order_by_child('foo').start_at(arg)
+        assert query._querystr == 'orderBy="foo"&startAt={0}'.format(json.dumps(arg))
+
     def test_end_at_none(self):
         query = self.ref.order_by_child('foo')
         with pytest.raises(ValueError):
             query.end_at(None)
 
+    @pytest.mark.parametrize('arg', ['', 'foo', True, False, 0, 1, dict()])
+    def test_valid_end_at(self, arg):
+        query = self.ref.order_by_child('foo').end_at(arg)
+        assert query._querystr == 'endAt={0}&orderBy="foo"'.format(json.dumps(arg))
+
     def test_equal_to_none(self):
         query = self.ref.order_by_child('foo')
         with pytest.raises(ValueError):
             query.equal_to(None)
 
+    @pytest.mark.parametrize('arg', ['', 'foo', True, False, 0, 1, dict()])
+    def test_valid_equal_to(self, arg):
+        query = self.ref.order_by_child('foo').equal_to(arg)
+        assert query._querystr == 'equalTo={0}&orderBy="foo"'.format(json.dumps(arg))
+
     def test_range_query(self, initquery):
         query, order_by = initquery
         query.start_at(1)