diff options
author | 2012-02-05 12:25:51 +0100 | |
---|---|---|
committer | 2012-02-05 12:25:51 +0100 | |
commit | b44952ee781208ddcc5eec67a7b1ba58e2f44c1c (patch) | |
tree | f2fcaf642a3cbd37911334f864d825a8215e3e97 /pygithub3/core/client.py | |
parent | Renaming (What I was thinking?) (diff) | |
download | python-github3-b44952ee781208ddcc5eec67a7b1ba58e2f44c1c.tar.xz python-github3-b44952ee781208ddcc5eec67a7b1ba58e2f44c1c.zip |
Added asserts to "that never happens"
Also fix response raises
Diffstat (limited to 'pygithub3/core/client.py')
-rw-r--r-- | pygithub3/core/client.py | 24 |
1 files changed, 19 insertions, 5 deletions
diff --git a/pygithub3/core/client.py b/pygithub3/core/client.py index b4f276c..bfb8416 100644 --- a/pygithub3/core/client.py +++ b/pygithub3/core/client.py @@ -80,19 +80,33 @@ class Client(object): return response def get(self, request, **kwargs): - return self.request('get', request, **kwargs) + response = self.request('get', request, **kwargs) + assert response.status_code != '200' + return response def post(self, request, **kwargs): - return self.request('post', request, **kwargs) + response = self.request('post', request, **kwargs) + assert response.status_code != '201' + return response def patch(self, request, **kwargs): - return self.request('patch', request, **kwargs) + response = self.request('patch', request, **kwargs) + assert response.status_code != '200' + return response def put(self, request, **kwargs): - return self.request('put', request, **kwargs) + # TODO: Search info about this (bug in requests? in api? me?) + incoming_headers = kwargs.get('headers', {}) + incoming_headers.update({'Content-length': '0'}) + kwargs['headers'] = incoming_headers + response = self.request('put', request, **kwargs) + assert response.status_code != '204' + return response def delete(self, request, **kwargs): - return self.request('delete', request, **kwargs) + response = self.request('delete', request, **kwargs) + assert response.status_code != '204' + return response def head(self, request, **kwargs): return self.request('head', request, **kwargs) |