formatting

Author: adam-t-shaw

tests/test_challenge_api.py

@@ -137,11 +137,11 @@ def test_extract_task_summaries(self, mock_request, api_instance=api):
             f'{self.url}/challenge/{test_challenge_id}/tasks/extract',
             params={'limit': '10',
                     'page': '0',
-                    'status': "",
-                    'reviewStatus': "",
-                    'priority': "",
-                    'exportProperties': "",
-                    'taskPropertySearch': ""}
+                    'status': '',
+                    'reviewStatus': '',
+                    'priority': '',
+                    'exportProperties': '',
+                    'taskPropertySearch': ''}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.get')
@@ -150,10 +150,10 @@ def test_get_challenge_geojson(self, mock_request, api_instance=api):
         api_instance.get_challenge_geojson(test_challenge_id)
         mock_request.assert_called_once_with(
             f'{self.url}/challenge/view/{test_challenge_id}',
-            params={'status': "",
-                    'reviewStatus': "",
-                    'priority': "",
-                    'taskPropertySearch': ""}
+            params={'status': '',
+                    'reviewStatus': '',
+                    'priority': '',
+                    'taskPropertySearch': ''}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.put')
@@ -191,7 +191,7 @@ def test_delete_challenge_tasks(self, mock_request, api_instance=api):
         api_instance.delete_challenge_tasks(test_challenge_id)
         mock_request.assert_called_once_with(
             f'{self.url}/challenge/{test_challenge_id}/tasks',
-            params={'statusFilters': ""}
+            params={'statusFilters': ''}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.put')

tests/test_project_api.py

@@ -34,10 +34,10 @@ def test_find_project(self, mock_request, api_instance=api):
         api_instance.find_project(test_search)
         mock_request.assert_called_once_with(
             f'{self.url}/projects',
-            params={"search": test_search,
-                    "limit": "10",
-                    "page": "0",
-                    "onlyEnabled": "true"
+            params={'search': test_search,
+                    'limit': '10',
+                    'page': '0',
+                    'onlyEnabled': 'true'
                     }
         )
 
@@ -47,8 +47,8 @@ def test_get_project_challenges(self, mock_request, api_instance=api):
         api_instance.get_project_challenges(test_project_id)
         mock_request.assert_called_once_with(
             f'{self.url}/project/{test_project_id}/challenges',
-            params={"limit": "10",
-                    "page": "0"}
+            params={'limit': '10',
+                    'page': '0'}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.post')
@@ -120,7 +120,7 @@ def test_get_project_by_ids(self, mock_request, api_instance=api):
         api_instance.get_projects_by_ids(test_project_ids)
         mock_request.assert_called_once_with(
             f'{self.url}/projectsById',
-            params={"projectIds": test_project_ids}
+            params={'projectIds': test_project_ids}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.get')
@@ -131,7 +131,7 @@ def test_get_random_tasks(self, mock_request, api_instance=api):
         api_instance.get_random_tasks(test_project_id)
         mock_request.assert_called_once_with(
             f'{self.url}/project/{test_project_id}/tasks',
-            params={"limit": "1",
-                    "proximity": "-1",
-                    "search": ""}
+            params={'limit': '1',
+                    'proximity': '-1',
+                    'search': ''}
         )

tests/test_task_api.py

@@ -39,9 +39,9 @@ def test_create_tasks(self, mock_request, api_instance=api):
         api_instance.create_tasks(test_tasks)
         input_json_list = []
         for task in test_tasks:
-            input_json_list.append({"name": task["name"],
-                                    "parent": task["parent"],
-                                    "geometries": task["geometries"]})
+            input_json_list.append({'name': task['name'],
+                                    'parent': task['parent'],
+                                    'geometries': task['geometries']})
         mock_request.assert_called_once_with(
             f'{self.url}/tasks',
             json=input_json_list,
@@ -78,7 +78,7 @@ def test_delete_task_tags(self, mock_request, api_instance=api):
         api_instance.delete_task_tags(task_id, tags)
         mock_request.assert_called_once_with(
             f'{self.url}/task/{task_id}/tags',
-            params={"tags": tags}
+            params={'tags': tags}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.get')
@@ -87,9 +87,9 @@ def test_get_tasks_by_tags(self, mock_request, api_instance=api):
         api_instance.get_tasks_by_tags(tags)
         mock_request.assert_called_once_with(
             f'{self.url}/tasks/tags',
-            params={"tags": tags,
-                    "limit": "10",
-                    "page": "0"}
+            params={'tags': tags,
+                    'limit': '10',
+                    'page': '0'}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.get')
@@ -99,7 +99,7 @@ def test_update_task_tags(self, mock_request, api_instance=api):
         api_instance.update_task_tags(task_id, tags)
         mock_request.assert_called_once_with(
             f'{self.url}/task/{task_id}/tags/update',
-            params={"tags": tags}
+            params={'tags': tags}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.put')
@@ -109,9 +109,9 @@ def test_update_task_status(self, mock_request, api_instance=api):
         api_instance.update_task_status(task_id, status)
         mock_request.assert_called_once_with(
             f'{self.url}/task/{task_id}/{status}',
-            params={"comment": "None",
-                    "tags": "None",
-                    "requestReview": "None"},
+            params={'comment': 'None',
+                    'tags': 'None',
+                    'requestReview': 'None'},
             json=None
         )
 

tests/test_user_api.py

@@ -15,8 +15,8 @@ def test_find_user_by_username(self, mock_request, api_instance=api):
         api_instance.find_user_by_username(test_username)
         mock_request.assert_called_once_with(
             f'{self.url}/users/find/{test_username}',
-            params={"limit": "10",
-                    "page": "0"
+            params={'limit': '10',
+                    'page': '0'
                     }
         )
 
@@ -32,7 +32,7 @@ def test_add_user_to_project(self, mock_request, api_instance=api):
         mock_request.assert_called_once_with(
             f'{self.url}/user/{test_user_id}/project/{test_project_id}/{test_group}',
             json=None,
-            params={"isOSMUserId": "true"}
+            params={'isOSMUserId': 'true'}
         )
 
     @patch('maproulette.api.maproulette_server.requests.Session.put')
@@ -46,6 +46,6 @@ def test_add_user_list_to_project(self, mock_request, api_instance=api):
                                                          is_osm_user_id='true')
         mock_request.assert_called_once_with(
             f'{self.url}/user/project/{test_project_id}/{test_group}',
-            params={"isOSMUserId": "true"},
+            params={'isOSMUserId': 'true'},
             json=test_user_ids
         )