Update get_groups() to use documented parameter to filter by name (#586)

Author: sujaygarlanka

HISTORY.rst

@@ -10,6 +10,7 @@ Next Release
 
 - Add metadata query functionality (`#574 <https://github.com/box/box-python-sdk/pull/574>`_)
 - Add folder lock functionality (`#581 <https://github.com/box/box-python-sdk/pull/581>`_)
+- Update `get_groups()` to use documented parameter to filter by name (`#586 <https://github.com/box/box-python-sdk/pull/586>`_)
 
 2.11.0 (2021-01-11)
 ++++++++

boxsdk/client/client.py

@@ -622,7 +622,7 @@ def get_groups(self, name=None, limit=None, offset=None, fields=None):
         url = self.get_url('groups')
         additional_params = {}
         if name:
-            additional_params['name'] = name
+            additional_params['filter_term'] = name
         return LimitOffsetBasedObjectCollection(
             url=url,
             session=self._session,

test/unit/client/test_client.py

@@ -584,7 +584,7 @@ def test_get_groups_return_the_correct_group_objects(
         assert group.name == str(expected_id)
         # pylint:disable=protected-access
         assert group._session == mock_box_session
-    mock_box_session.get.assert_called_once_with(expected_url, params={'offset': None, 'name': group_name})
+    mock_box_session.get.assert_called_once_with(expected_url, params={'offset': None, 'filter_term': group_name})
 
 
 def test_create_group_returns_the_correct_group_object(mock_client, mock_box_session, create_group_response):