From 71e13500fe69b9e159968bd8c34e4df8f62707ea Mon Sep 17 00:00:00 2001 From: Cyril Roelandt Date: Mon, 16 Dec 2013 15:00:59 +0100 Subject: [PATCH] Avoid meaningless comparison that leads to a TypeError The "requests" module tries to do this comparison in raise_for_status(): 400 <= self.status_code < 500 with "self.status_code" being "None". In Python2, it's a valid comparison, but in Python3, it triggers a TypeError. We fix it by giving a valid default value to "self.status_code". Change-Id: I9ea9ecfb698546a7f6526464aa528441bb7c1ccd --- keystoneclient/tests/utils.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/keystoneclient/tests/utils.py b/keystoneclient/tests/utils.py index a528a9eae..a4ec6e75b 100644 --- a/keystoneclient/tests/utils.py +++ b/keystoneclient/tests/utils.py @@ -109,7 +109,7 @@ class TestResponse(requests.Response): self._text = None super(TestResponse, self).__init__() if isinstance(data, dict): - self.status_code = data.get('status_code', None) + self.status_code = data.get('status_code', 200) headers = data.get('headers') if headers: self.headers.update(headers)