Skip to content

Commit

Permalink
Tweak some comments
Browse files Browse the repository at this point in the history
  • Loading branch information
JayH5 committed Sep 18, 2015
1 parent 9ee9830 commit a872615
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion consular/clients.py
Expand Up @@ -37,7 +37,8 @@ def _log_http_error(self, failure, url):
def request(self, method, path, endpoint=None, json_data=None, **kwargs):
"""
Perform a request. A number of basic defaults are set on the request
that make using a JSON API easier.
that make using a JSON API easier. These defaults can be overridden by
setting the parameters in the keyword args.
:param: method:
The HTTP method to use (example is `GET`).
Expand Down
4 changes: 2 additions & 2 deletions consular/tests/test_main.py
Expand Up @@ -559,8 +559,8 @@ def test_sync_apps(self):
def test_sync_apps_field_not_found(self):
"""
When syncing apps, and Marathon returns a JSON response with an
unexpected structure (the "apps" field is missing. A KeyError should
be raised.
unexpected structure (the "apps" field is missing). A KeyError should
be raised with a useful message.
"""
d = self.consular.sync_apps(purge=False)
marathon_request = yield self.requests.get()
Expand Down

0 comments on commit a872615

Please sign in to comment.