review comments

fabric8-analytics · Jul 2, 2019 · 08b0140 · 08b0140
1 parent 974f09f
commit 08b0140
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/rest_api.py b/src/rest_api.py
@@ -63,8 +63,8 @@ def sync_data():
     or latest version or both.
 
     valid input: {
-        "non_cve_sync": True / False,
-        "latest_version_sync": True / false,
+        "non_cve_sync": true/false,
+        "latest_version_sync": true/false,
         "cve_ecosystem": ['maven', 'pypi, 'npm']
     }
 

diff --git a/tests/test_rest_api.py b/tests/test_rest_api.py
@@ -160,7 +160,7 @@ def test_sync_data1(_mock1, client):
     url = api_route_for("sync-graph-data")
     inp = '''{
         "non_cve_sync": true,
-        "latest_version_sync": "true",
+        "latest_version_sync": true,
         "cve_ecosystem": ["npm"]
     }'''
     response = client.post(url,
@@ -169,17 +169,17 @@ def test_sync_data1(_mock1, client):
     assert response.status_code == 200
 
     inp = '''{
-        "non_cve_sync": "true",
-        "latest_version_sync": "true"
+        "non_cve_sync": true,
+        "latest_version_sync": true
     }'''
     response = client.post(url,
                            data=inp,
                            content_type='application/json')
     assert response.status_code == 400
 
     inp = '''{
-        "non_cve_sync": "true",
-        "latest_version_sync": "true",
+        "non_cve_sync": true,
+        "latest_version_sync": true,
         "cve_ecosystem": ["npm"]
     }'''
     response = client.post(url,