From 2d198d7fe032c31400566d37d10bc8594e578907 Mon Sep 17 00:00:00 2001 From: Thierry Carrez Date: Thu, 15 Oct 2015 12:15:45 +0200 Subject: [PATCH] Switch to post-versioning This combines 2 commits in order to pass the gate. No longer mention next version on setup.cfg on stable branch. Remove oslo.policy implementation tests from keystone oslo.policy 0.12.0 contains a change to use requests to do the http check rather than urllib. This change caused keystone tests to fail because the keystone tests were mocking urllib, making assumptions about how oslo.policy is implemented. Keystone doesn't need to test internal features of oslo.policy, so these tests are removed. Closes-Bug: 1505374 (cherry picked from commit 6016d017004acaae288312b196ef07ea98e9962d) Change-Id: I75c44e88eb851a3b92a6d7d379d9a55e417703d6 --- keystone/tests/unit/test_policy.py | 24 ------------------------ setup.cfg | 1 - 2 files changed, 25 deletions(-) diff --git a/keystone/tests/unit/test_policy.py b/keystone/tests/unit/test_policy.py index b2f0e52564..686e2b70d1 100644 --- a/keystone/tests/unit/test_policy.py +++ b/keystone/tests/unit/test_policy.py @@ -16,10 +16,8 @@ import json import os -import mock from oslo_policy import policy as common_policy import six -from six.moves.urllib import request as urlrequest from testtools import matchers from keystone import exception @@ -118,28 +116,6 @@ class PolicyTestCase(BasePolicyTestCase): action = "example:allowed" rules.enforce(self.credentials, action, self.target) - def test_enforce_http_true(self): - - def fakeurlopen(url, post_data): - return six.StringIO("True") - - action = "example:get_http" - target = {} - with mock.patch.object(urlrequest, 'urlopen', fakeurlopen): - result = rules.enforce(self.credentials, action, target) - self.assertTrue(result) - - def test_enforce_http_false(self): - - def fakeurlopen(url, post_data): - return six.StringIO("False") - - action = "example:get_http" - target = {} - with mock.patch.object(urlrequest, 'urlopen', fakeurlopen): - self.assertRaises(exception.ForbiddenAction, rules.enforce, - self.credentials, action, target) - def test_templatized_enforcement(self): target_mine = {'project_id': 'fake'} target_not_mine = {'project_id': 'another'} diff --git a/setup.cfg b/setup.cfg index 8db7451b1e..7788537e62 100644 --- a/setup.cfg +++ b/setup.cfg @@ -1,6 +1,5 @@ [metadata] name = keystone -version = 8.0.0 summary = OpenStack Identity description-file = README.rst