Merge pull request #500 from artoonie/clarify-rest-api

fix trailing slash, clarify rest API docs

visualizer/tests/testRestApiExampleCode.py

@@ -165,7 +165,8 @@ def test_update_visualization(self):
         # In our tests, we are not hitting the real server.
         # Your code will use https://www.rcvis.com as the live_server_url.
         # To modify existing data, supply the visId
-        url = self.live_server_url + "/api/bp/" + str(visId)
+        # Note: the trailing slash is mandatory. Without it, it will fail silently.
+        url = self.live_server_url + "/api/bp/" + str(visId) + "/"
 
         # API key should only be created as needed, and reused and kept secure
         apiKey = self.get_api_key()
@@ -177,11 +178,13 @@ def test_update_visualization(self):
 
         # Modify the data with a PATCH.
         # PUT is also supported, but not shown here.
-        data = {'dataSourceURL': 'https://www.example.com/a/different/url'}
+        differentUrl = 'https://www.example.com/a/different/url'
+        data = {'dataSourceURL': differentUrl}
 
-        # POST the data to create a visualization
-        response = requests.post(url, data=data, headers=headers, timeout=3)
+        # PATCH to update
+        response = requests.patch(url, data=data, headers=headers, timeout=3)
 
-        # The response is the same as when you created the data, sans any changes you just made
+        # Verify the change succeeded
         self.assertEqual(response.json()['slug'], 'favorite-ice-cream-flavors')
         self.assertEqual(response.json()['id'], visId)
+        self.assertEqual(response.json()['dataSourceURL'], differentUrl)