[IMP] Add domain syntax for an OR query with a list

* Add logic to allow for an or query by using a list

Author: lasley

README.rst

@@ -64,6 +64,18 @@ Perform a search for a patient with the last name Smith
     for row in res:
         print row.fname
 
+Perform a search for a patient with the last name Smith or Jones
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+::
+
+    res = cp.search(
+        cp['Patient'],
+        {'lname': ['Smith', 'Jones']},
+    )
+    for row in res:
+        print row.fname
+
 Perform a patient search, but only return the ``mname`` column
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 

carepoint/db/carepoint.py

@@ -9,7 +9,7 @@
 
 from contextlib import contextmanager
 
-from sqlalchemy import bindparam
+from sqlalchemy import bindparam, or_
 from sqlalchemy.ext.declarative import declarative_base
 from sqlalchemy.inspection import inspect
 from sqlalchemy.orm import sessionmaker
@@ -209,6 +209,16 @@ def _unwrap_filters(self, model_obj, filters=None):
                         model_obj, col_name, _operator, _filter
                     ))
 
+            elif isinstance(col_filter, (list, tuple)):
+                query = []
+                for _filter in col_filter:
+                    query.append(
+                        self._create_criterion(
+                            model_obj, col_name, '==', _filter,
+                        ),
+                    )
+                new_filters.append(or_(*query))
+
             else:
                 new_filters.append(self._create_criterion(
                     model_obj, col_name, '==', col_filter

carepoint/tests/db/test_carepoint.py

@@ -550,6 +550,26 @@ def test_unwrap_filters_dict(self):
             self.carepoint._unwrap_filters(model_obj, filters)
             mk.assert_has_calls(expect, any_order=True)
 
+    @mock.patch('carepoint.db.carepoint.or_')
+    def test_unwrap_filters_list(self, or_):
+        """ It should create a proper or query for lists. """
+        model_obj = self.__get_model_obj()
+        col_expect = 'test_col'
+        query_expect = ['test_1', 'test_2']
+        filters = {
+            col_expect: query_expect
+        }
+        expect = [
+            mock.call(model_obj, col_expect, '==', query_expect[0]),
+            mock.call(model_obj, col_expect, '==', query_expect[1]),
+        ]
+        criterion_return = 'Criterion'
+        with mock.patch.object(self.carepoint, '_create_criterion') as mk:
+            mk.return_value = criterion_return
+            self.carepoint._unwrap_filters(model_obj, filters)
+            mk.assert_has_calls(expect, any_order=True)
+            or_.assert_called_once_with(criterion_return, criterion_return)
+
     def test_unwrap_filters_str(self):
         model_obj = self.__get_model_obj()
         col_expect = 'test_col'