That's a pretty notable review to accommodate the pecan change. In the
meantime, wouldn't something like this would get the py26 and py27 tests
passing?
index 9aced3f..4051fad 100644
--- a/climate/tests/api/test_root.py
+++ b/climate/tests/api/test_root.py
@@ -22,8 +22,7 @@ class TestRoot(api.APITest):
response = self.get_json('/',
expect_errors=True,
path_prefix='')
- self.assertEqual(response.status_int, 200)
- self.assertEqual(response.content_type, "text/html")
+ self.assertEqual(response.status_int, 204)
self.assertEqual(response.body, '')
On 07/02/14 09:08:35 PM, Fuente, Pablo A wrote:
> Blazar cores,
> Please review https://review.openstack.org/99389. We need this merged
> ASAP in order to get a +1 from Jenkins in our py26 and py27 jobs. Pecan
> new version returns a 204 instead 200 in one case (when the API returns
> and empty dictionary) and one test case is failing for this reason. This
> patch solve the bug as a side effect, because it returns the versions
> instead of an empty dictionary.
>
> Pablo.
> _______________________________________________
> OpenStack-dev mailing list
> [email protected]
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
--
Ryan Petrello
Senior Developer, DreamHost
[email protected]
_______________________________________________
OpenStack-dev mailing list
[email protected]
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
