delta/openstack/python-ceilometerclient.git/ceilometerclient/tests/unit/test_exc.py, branch 2.6.1 opendev.org: openstack/python-ceilometerclient.git Fix from_response method to process response from requests 2016-09-07T09:54:24+00:00 Andrey Kurilin akurilin@mirantis.com 2016-09-07T09:39:12+00:00 090e5ffc691c5d40e45ab0c20dc88032a0987b20 SessionClient uses requests library. It's response class doesn't have "status" property[1], so in case of any errors(with status code > 400), from_response method is called and raises AttributeError. Also, HTTPClient implementation uses requests lib by default[2](if 'http' argument was not transmitted), so from_response method will raise AttributeError too. [1] - http://docs.python-requests.org/en/master/api/#requests.Response.status_code [2] - https://github.com/openstack/python-ceilometerclient/blob/2.6.0/ceilometerclient/openstack/common/apiclient/client.py#L99-L100 Change-Id: Id8fb2f386e8101951716f30a6365c9aa15bd4b24 Closes-Bug: #1620974 (cherry picked from commit 1b1917ab9b56fe92235ce336d3df3e7a8c1c81d2) 
SessionClient uses requests library. It's response class doesn't have
"status" property[1], so in case of any errors(with status code > 400),
from_response method is called and raises AttributeError.

Also, HTTPClient implementation uses requests lib by default[2](if 'http'
argument was not transmitted), so from_response method will raise
AttributeError too.

[1] - http://docs.python-requests.org/en/master/api/#requests.Response.status_code
[2] - https://github.com/openstack/python-ceilometerclient/blob/2.6.0/ceilometerclient/openstack/common/apiclient/client.py#L99-L100

Change-Id: Id8fb2f386e8101951716f30a6365c9aa15bd4b24
Closes-Bug: #1620974
(cherry picked from commit 1b1917ab9b56fe92235ce336d3df3e7a8c1c81d2)
Move unit tests into their own subdirectory 2015-03-13T12:01:04+00:00 Chris Dent chdent@redhat.com 2015-03-12T13:31:06+00:00 d9d8cc614eccaffefdfd4aa4f536745f0aabc5e3 This is in preparation of moving functional tests currently in tempest in-tree. No surprises in this change: * move files * update the local imports Same number of tests run, all still pass. Change-Id: Ibfeac44421c5f36740b1377868f5ec469b2893f5 
This is in preparation of moving functional tests currently in
tempest in-tree.

No surprises in this change:

* move files
* update the local imports

Same number of tests run, all still pass.

Change-Id: Ibfeac44421c5f36740b1377868f5ec469b2893f5