Flip the logic around to make the default code path clearer.

Author: coderanger

chef/api.py

@@ -203,11 +203,10 @@ def request(self, method, path, headers={}, data=None):
         except requests.ConnectionError as e:
             raise ChefServerError(e.message)
 
-        if response.ok:
-            return response
-
-        raise ChefServerError.from_error(response.reason, code=response.status_code)
+        if not response.ok:
+            raise ChefServerError.from_error(response.reason, code=response.status_code)
 
+        return response
 
     def api_request(self, method, path, headers={}, data=None):
         headers = dict((k.lower(), v) for k, v in six.iteritems(headers))